Datepart(dw, datedts), datepart(weekday, datedts)

[snicol21]

The parser identifies dw and weekday as columns. I think these just need to be added to the list of sql key words to ignore

[j2jensen]

I'd be careful about just adding them to an ignore list, since they're not actually Reserved Keywords in T-SQL. For example, select 1 as weekday is a perfectly valid query. It's only in the context of a datepart(x, ...) call that these words have special meaning. And in that context, no other values are allowed.

[snicol21]

Yeah, you're totally right @j2jensen

[anderslyman]

@snicol21 is it identifying dw and weekday as columns when they are used in a datepart call?

[snicol21]

@anderslyman it looks like it still is identifying "dw" as a column when it's part of a DATEPART call.

Notice the "dw" columns it's parsing from this

[snicol21]

@anderslyman however, it looks like "year" works fine ...

[anderslyman]

Got it, I think I have handling specifically for datepart calls, I'll see if I can have it handle this as well

…

On Wed, May 30, 2018, 2:11 PM Spencer Nicol ***@***.***> wrote: @anderslyman <https://github.com/anderslyman> however, it looks like "year" works fine ... [image: image] <https://user-images.githubusercontent.com/15724638/40744954-5e7b5510-6413-11e8-8de7-a94deb233fc5.png> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABUR-YOxG1Qtd2VqnNmuqGdUShOva4prks5t3vz9gaJpZM4Sa8_h> .