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

update ndc-sdk #14

Merged
merged 41 commits into from
Sep 20, 2024
Merged

update ndc-sdk #14

merged 41 commits into from
Sep 20, 2024

Conversation

pranshi06
Copy link
Collaborator

What

How

@pranshi06
rename main crate as ndc-bigquery
1e1e14a
@pranshi06
wip: update query engine crate
1d0e9ef
@pranshi06
update configuration, connector and cli crates
eadff3c
@pranshi06
add SQL for cli update command
ef11caf
@pranshi06
create tables info for cli config updation
eab9cd5
@pranshi06
create tables info for cli config updation
62b31b4
@pranshi06
cli update command
a87b7dc
@pranshi06
update service key value
e728d9c
@pranshi06
format
4937d01
@pranshi06
format
05daa4f
@pranshi06
update sql to take database name from the user
ba2ab82
@pranshi06
send empty row instead of null value when no element is present in th…
72268a0 
…e query result
@pranshi06
wip: fix ndc-tests
00e8f5e
@pranshi06
update ndc-spec version
3c9b2f6
@pranshi06
take project_id and dataset_id as input
97c313a
@pranshi06
format
9922302
@pranshi06
remove unused imports/variables
2a2cc92
@pranshi06
lint fix
4a38020
@pranshi06
wip: lint fix
d2f44dd
@pranshi06
wip: lint fix
9dd4f0e
@pranshi06
lint fix
9dfe0dd
@pranshi06
cli binary generation
a09e358
@pranshi06
wip: cleanup
ddb2ff0
@pranshi06
collect all env var as part of connection_settings
b78d729
@pranshi06
wip: fix tests: query tests, ndc-tests
7f4e31b
@pranshi06
empty dataset config generation fix
e448f7b
@pranshi06
wip: fix tests
9324f6d
@pranshi06
update config gen query to take dataset name
e59519b
@pranshi06
translation tests
bb80198
@pranshi06
remove debug statements
1a04039
@pranshi06 pranshi06 marked this pull request as ready for review September 19, 2024 19:58
@pranshi06 pranshi06 merged commit 31ae50f into main Sep 20, 2024
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@codingkarthik codingkarthik codingkarthik approved these 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
@pranshi06 @codingkarthik @danieljharvey