8000 Cesql v1 fixes by Cali0707 · Pull Request #641 · cloudevents/sdk-java · GitHub
[go: up one dir, main page]
Skip to content

Cesql v1 fixes #641

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Jun 21, 2024
Merged

Cesql v1 fixes #641

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9b8c9df
feat: add exception factory for cesql exceptions
Cali0707 Jun 19, 2024
1c895dd
feat: extend EvaluationResult to be usable internally as well
Cali0707 Jun 19, 2024
762b665
feat: expressions use results instead of a thrower interface
Cali0707 Jun 19, 2024
b25cb7d
feat: functinos use results instead of a thrower interface
Cali0707 Jun 19, 2024
5c78074
fix: parser handles not equals correctly, does not eagerly evaluate w…
Cali0707 Jun 19, 2024
5886481
feat: update interfaces for cesql runtime and context
Cali0707 Jun 19, 2024
48fd18e
feat: update runtime classes for CESQL
Cali0707 Jun 19, 2024
d1c3584
fix: parser handles integer literals properly
Cali0707 Jun 19, 2024
d88a3cf
test: updated test files to test v1 spec
Cali0707 Jun 19, 2024
dd7fee5
Apply suggestions from code review
Cali0707 Jun 19, 2024
cfcff87
cleanup: wrap -> wrapExceptions
Cali0707 Jun 19, 2024
09bc8ba
cleanup: shouldFail -> shouldThrow
Cali0707 Jun 19, 2024
6022507
fix(test): TCK test suite fails properly
Cali0707 Jun 20, 2024
23b82ee
Merge branch 'main' into cesql-v1-fixes
Cali0707 Jun 20, 2024
61bf082
fix: removed missing link from pom.xml
Cali0707 Jun 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: parser handles integer literals properly 
Signed-off-by: Calum Murray <cmurray@redhat.com>
  • Loading branch information
@Cali0707
Cali0707 committed Jun 19, 2024
commit d1c3584cad9be7e211874b0315d4752d50e36e34
2 changes: 1 addition & 1 deletion sql/src/main/antlr4/imports/CESQLLexer.g4
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ FALSE: 'FALSE';

DQUOTED_STRING_LITERAL: DQUOTA_STRING;
SQUOTED_STRING_LITERAL: SQUOTA_STRING;
INTEGER_LITERAL: INT_DIGIT+;
INTEGER_LITERAL: ('+' | '-')? INT_DIGIT+;

// Identifiers

Expand Down