forked from Sunbird-Obsrv/obsrv-api-service
-
Notifications
You must be signed in to change notification settings - Fork 9
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.
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
Sanketika-obsrv/issue-tracker#180 feat: API changes to support hudi changes in obsrv #160
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…efault type column and fixed sql query api
…ith underscore in table name
…able checks in hudi schema
…able checks in hudi schema
ravismula
approved these changes
May 23, 2024
anandp504
approved these changes
May 24, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved 👍
manjudr
added a commit
that referenced
this pull request
May 27, 2024
* default config for dataset_config (#99) * Sanketika-Obsrv/issue-tracker#14: feat: validate sql query (#100) * Issue #304 datasource configuration changes to support Hudi schema. (#111) * Sanketika-obsrv/issue-tracker#304: modified mandatory fields for datasource create request (#112) * Configure the vuln scan for obsrv-api-service (#117) * Revert "Sanketika-obsrv/issue-tracker#304: modified mandatory fields for data…" (#121) This reverts commit 0789f3f. * Revert "Issue #304 datasource configuration changes to support Hudi schema. (…" (#122) This reverts commit ef918f6. * Sanketika-Obsrv/issue-tracker#182: added event validation service against schema for particular dataset * Sanketika-Obsrv/issue-tracker#182: added table name from config * Sanketika-Obsrv/issue-tracker#182: added validation status in response handler * Sanketika-Obsrv/issue-tracker#180 feat: API changes to support hudi changes in obsrv (#160) * Sanketika-obsrv/issue-tracker#304: modified name for lakehouse spec * #0000 feat: added type column * #273 feat: API changes to support lakehouse queries * Sanketika-obsrv/issue-tracker#273 feat: modified datasource apis to default type column and fixed sql query api * Sanketika-obsrv/issue-tracker#273 feat: updated postman collection and swagger doc * Sanketika-obsrv/issue-tracker#273 feat: updated testcases for latest changes * Sanketika-obsrv/issue-tracker#273 feat: removed unnecessary log statements * Sanketika-obsrv/issue-tracker#273 feat: removed unnecessary flatmapping and await * Sanketika-obsrv/issue-tracker#273 feat: replaced special characters with underscore in table name * Sanketika-Obsrv/issue-tracker#180 fix: removed unused constants and table checks in hudi schema * Sanketika-Obsrv/issue-tracker#180 fix: removed unused constants and table checks in hudi schema * Release 1.0.6-GA merge fix (#164) * Release 1.0.4-GA (#105) * default config for dataset_config (#99) * Sanketika-Obsrv/issue-tracker#14: feat: validate sql query (#100) --------- Co-authored-by: harishkumar gangula <harish@sanketika.in> Co-authored-by: Shreyas Bhaktharam <121869503+shreyasb22@users.noreply.github.com> * Release 1.0.5-GA (#147) --------- Co-authored-by: Manjunath Davanam <manjunath@sanketika.in> Co-authored-by: Shreyas Bhaktharam <121869503+shreyasb22@users.noreply.github.com> Co-authored-by: Ravi Mula <ravismula@users.noreply.github.com> --------- Co-authored-by: harishkumar gangula <harish@sanketika.in> Co-authored-by: Shreyas Bhaktharam <121869503+shreyasb22@users.noreply.github.com> Co-authored-by: GayathriSrividya <gayathrirajavarapu7@gmail.com> Co-authored-by: divyagovindaiah <110388603+divyagovindaiah@users.noreply.github.com> Co-authored-by: yashashk <yashashk@sanketika.in> Co-authored-by: Manjunath Davanam <manjunath@sanketika.in>
manjudr
added a commit
that referenced
this pull request
May 27, 2024
* default config for dataset_config (#99) * Sanketika-Obsrv/issue-tracker#14: feat: validate sql query (#100) * Issue #304 datasource configuration changes to support Hudi schema. (#111) * Sanketika-obsrv/issue-tracker#304: modified mandatory fields for datasource create request (#112) * Configure the vuln scan for obsrv-api-service (#117) * Revert "Sanketika-obsrv/issue-tracker#304: modified mandatory fields for data…" (#121) This reverts commit 0789f3f. * Revert "Issue #304 datasource configuration changes to support Hudi schema. (…" (#122) This reverts commit ef918f6. * Sanketika-Obsrv/issue-tracker#182: added event validation service against schema for particular dataset * Sanketika-Obsrv/issue-tracker#182: added table name from config * Sanketika-Obsrv/issue-tracker#182: added validation status in response handler * Sanketika-Obsrv/issue-tracker#180 feat: API changes to support hudi changes in obsrv (#160) * Sanketika-obsrv/issue-tracker#304: modified name for lakehouse spec * #0000 feat: added type column * #273 feat: API changes to support lakehouse queries * Sanketika-obsrv/issue-tracker#273 feat: modified datasource apis to default type column and fixed sql query api * Sanketika-obsrv/issue-tracker#273 feat: updated postman collection and swagger doc * Sanketika-obsrv/issue-tracker#273 feat: updated testcases for latest changes * Sanketika-obsrv/issue-tracker#273 feat: removed unnecessary log statements * Sanketika-obsrv/issue-tracker#273 feat: removed unnecessary flatmapping and await * Sanketika-obsrv/issue-tracker#273 feat: replaced special characters with underscore in table name * Sanketika-Obsrv/issue-tracker#180 fix: removed unused constants and table checks in hudi schema * Sanketika-Obsrv/issue-tracker#180 fix: removed unused constants and table checks in hudi schema * Release 1.0.6-GA merge fix (#164) * Release 1.0.4-GA (#105) * default config for dataset_config (#99) * Sanketika-Obsrv/issue-tracker#14: feat: validate sql query (#100) --------- Co-authored-by: harishkumar gangula <harish@sanketika.in> Co-authored-by: Shreyas Bhaktharam <121869503+shreyasb22@users.noreply.github.com> * Release 1.0.5-GA (#147) --------- Co-authored-by: Manjunath Davanam <manjunath@sanketika.in> Co-authored-by: Shreyas Bhaktharam <121869503+shreyasb22@users.noreply.github.com> Co-authored-by: Ravi Mula <ravismula@users.noreply.github.com> * Sanketika-Obsrv/issue-tracker#224: updated the alpine package versions (#165) --------- Co-authored-by: harishkumar gangula <harish@sanketika.in> Co-authored-by: Shreyas Bhaktharam <121869503+shreyasb22@users.noreply.github.com> Co-authored-by: GayathriSrividya <gayathrirajavarapu7@gmail.com> Co-authored-by: divyagovindaiah <110388603+divyagovindaiah@users.noreply.github.com> Co-authored-by: yashashk <yashashk@sanketika.in> Co-authored-by: Ravi Mula <ravismula@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR includes:
Changes in the obsrv APIs to support the Hudi schema.
Updated Datasource CRUD APIs
Updated Query APIs
Added a sample request to the Postman collection of OBSRV APIs.
Type of change
How Has This Been Tested?
Checklist: