HHH-18808 Fix the bug of wrong logging output when keyword is used as identifier in HqlParser.g4

[NathanQingyangXu]

https://hibernate.atlassian.net/browse/HHH-18808

Not sure why not move the keyword list into HqlLexer.g4, which would simplify a lot and I think we could still overwrite HqlLexer class. But here is a quick and effective bug fix.

[gavinking]

I think we should simply remove that log message. It's not useful, IMO.

[NathanQingyangXu]

I think we should simply remove that log message. It's not useful, IMO.

If so, we could end up with lots of simplification to the ANTLR grammer, code or even perf.

[gavinking]

I think we should simply remove that log message. It's not useful, IMO.

If so, we could end up with lots of simplification to the ANTLR grammer, code or even perf.

WDYM?

[gavinking]

Other than that it looks good.

[NathanQingyangXu]

I think we should simply remove that log message. It's not useful, IMO.

If so, we could end up with lots of simplification to the ANTLR grammer, code or even perf.

WDYM?

nvm. I misunderstood. I'd thought we could move the keyword list to lexer file then I found it doesn't work.

[gavinking]

@NathanQingyangXu Please squash the commits and then we are good to go. Thanks!