fix: parser Removed KEY from nonReserved in SqlBase.g4

[sukalpomitra]

Description

This fixes https://github.com/confluentinc/ksql/issues/6896. Removed KEY from nonReserved in SqlBase.g4 under ksqldb-parser

[ghost]

It looks like @sukalpomitra hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence. Wikipedia

You can read and sign our full Contributor License Agreement here.

Once you've signed reply with [clabot:check] to prove it.

Appreciation of efforts,

clabot

[sukalpomitra]

[clabot:check]

[ghost]

@confluentinc It looks like @sukalpomitra just signed our Contributor License Agreement. :+1:

Always at your service,

clabot

[sukalpomitra]

hi @mjsax @agavra please help me. The error in jenkins build is nnot reproducible on my branch. How can I fix this test failure?

[agavra]

hello @sukalpomitra thanks for the PR! You need to build locally for it to verify, do a mvn clean install -pl :ksqldb-parser and you should be able to reproduce the issue.

Perhaps more importantly, however, this isn't a backwards compatible change (which is why the tests are failing). We'll need to be thoughtful about how we make this change so as not to break existing applications.

[sukalpomitra]

hi @agavra - All tests are passing

[agavra]

hello @sukalpomitra - tests are not passing in all modules (you can see the Jenkins CI build). I suspect the root cause is because of what I mentioned, that this change is not necessarily backwards compatible.

[sukalpomitra]

@agavra i am unable to run tests. Have a look at this issue - https://forum.confluent.io/t/ksqldb-compilation-is-failing/1267

[cla-assistant[bot]]

All committers have signed the CLA.

[vvcephei]

Closing stale PR. I'm sorry that we didn't get this done. Please resubmit as a new PR if you still want to get it in.