To add a couple things to Gail's good points:

Unless there's a very good reason you're not doing it (say, nearly all the data is changed every day), I'd...

On the 2 d.p., I'm assuming 'two decimal places' is the intent.

For that, you'll just need to do an appropriate CAST/CONVERT to a DECIMAL with a scale of...

First, some housekeeping 🙂

I see you updated your question and script instead of posting a reply. I'd recommend not doing that for a couple reasons.

1) it...

Unless there's more to what you're running than you're showing us, that error shouldn't occur with that query.

You WOULD get an error saying that item_name is invalid in...

Small hint: Between the DECLARE of @cnt and your check to see if it's >0, what all do you do to that variable? 🙂

Cheers!

That is a CLR table-valued function, so you'd need to look at the assembly it references.

See https://docs.microsoft.com/en-us/sql/relational-databases/clr-integration-database-objects-user-defined-functions/clr-table-valued-functions for a little bit of explanation about this.

Cheers!

The query you posted as your answer does not do what you verbally described as your requirement.

It will only return rows where AdmFormData.FieldName is either 'NAFTA' or NULL.

No, BETWEEN @FromDate and @ToDate is equivalent to >=@FromDate and <=@ToDate.

You'll want to do >=@FromDate and <@ToDate with the variables set as suggested.

The nice thing about...

Just to pile on, using the DDL you provided, both queries show 24 rows for the VeritVRU tool  on April 30 when I run them.

If you're missing rows,...