Skip to content

Conversation

@mdayakar
Copy link
Contributor

HIVE-29329: Negative value in DEFAULT constraint throwing ParseException.

What changes were proposed in this pull request?

Changed grammar files to support + and - signs for the default numeric values and changed the code to handle the sign.

Why are the changes needed?

Changes required to support + and - symbols for the default numeric values.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Using q file tests.
mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=default_constraint_negative_value.q -pl itests/qtest -Pitests

@sonarqubecloud
Copy link

@kasakrisz kasakrisz merged commit 87a503f into apache:master Nov 25, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants