Skip to content
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

Jump to bottom

Utilitis/antlr4 parti ql #27

Open
wants to merge 26 commits into
base: map_datatype_support
Choose a base branch
from
Open

Utilitis/antlr4 parti ql #27

wants to merge 26 commits into from

Conversation

shoaibcldcvr
Copy link
Collaborator

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

  • Tests pass
  • Appropriate changes to README are included in PR

nikitajain1998 and others added 21 commits January 27, 2025 16:16
@nikitajain1998
updated initialization process (#4)
97cc8cf 
* updated initialization process

* updated readme file

* upgraded to latest go version (#3)

* spanner datatype changes

* Added DDL capability for tables

* Added scripts to test apis

* refactored

* updated readme file

* added support for spanner data type

* updated integration test setup

* updated config test

* resolved comments

* fixed comments

* refactored
@nikitajain1998
Fix/init process checks (#15)
388c1b1 
* upgraded to latest go version (#3)

* Added scripts to test apis (#7)

* Added scripts to test apis

* refactored

* fixed lint issues (#13)

* fixed lint-markdown issue (#14)
@nikitajain1998
Fix/init process checks (#16)
88459aa 
* upgraded to latest go version (#3)

* Added scripts to test apis (#7)

* Added scripts to test apis

* refactored

* fixed lint issues (#13)

* fixed lint-markdown issue (#14)

* fixed Circle Ci build issue

* updated go version
image update
7c02aeb
image update
f96cdb2
image update
4a4d2f2
image update
abe0ee6
image update
a151fb4
update INT64 to FLOAT64 for NS
cf05b9d
fix issue
11895e1
fix issue
5321c94
@nikitajain1998
fixed init process (#20)
34ca5b5
@nikitajain1998
Fix/new init process (#21)
54da44c 
* fixed init process

* fixed IT, UT & init

* refactored
@nikitajain1998
Fix/new init process (#22)
79ba5a1 
* fixed IT
@nikitajain1998 @shoaibcldcvr
Added scripts to test apis (#7)
75c78cb 
* Added scripts to test apis

* refactored
@nikitajain1998 @shoaibcldcvr
fixed lint issues (#13)
bfb88d2
@shoaibcldcvr
parser code with folder structure
6442443
@shoaibcldcvr
select change
5d92b49
@shoaibcldcvr
Merge branch 'map_datatype_support' into utilitis/antlr4_partiQL
df7d3ab
@shoaibcldcvr
utilitis/antlr4_partiQL
b42c0f9
@shoaibcldcvr
add utility functions using antlr4 to parse partiql queries
5bf2720
taherkl
taherkl requested changes Feb 12, 2025
View reviewed changes
translator/utils/translator_delete.go Outdated
}

// Optionally check if any clauses were added
fmt.Printf("Clauses collected: %v\n", deleteQueryMap.Clauses)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove fmt statements from everywhere

@taherkl taherkl marked this pull request as ready for review February 12, 2025 14:20
taherkl
taherkl requested changes Feb 12, 2025
View reviewed changes
translator/utils/model.go Outdated Show resolved Hide resolved
taherkl
taherkl requested changes Feb 12, 2025
View reviewed changes
samples/README.MD Outdated Show resolved Hide resolved
translator/utils/model.go Outdated Show resolved Hide resolved
translator/utils/model.go Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@taherkl taherkl taherkl requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shoaibcldcvr @taherkl @nikitajain1998