-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add param for syncing sheets from shared drives (Copy of PR #60) #89
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
savan-chovatiya
approved these changes
Aug 31, 2022
RushiT0122
approved these changes
Aug 31, 2022
dmosorast
approved these changes
Aug 31, 2022
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.
Looks good to me, considering the testing performed on #69
bryantgray
approved these changes
Aug 31, 2022
Merged
RushiT0122
pushed a commit
that referenced
this pull request
Sep 1, 2022
* TDL-19742 Run discover mode if catalog is not given in sync mode (#85) * TDL-19742 Run discover mode if catalog is not given in sync mode * Add call count assertion and remove unused import Co-authored-by: Jay Tilala <jay.tilala@CDSYS.LOCAL> * TDL-14450: fix inconsistency of empty cell for boolean (#84) * handled empty value scenario for the booleans * Added unit test for null value in boolean * Removed workaround from the integration test Co-authored-by: harshpatel4crest <harsh.patel4@crestdatasys.com> * TDL-14449: Syncs do not parse datetime values in number datatyped columns as strings (#83) * updated the code to store datetime values as string in numberType column * updated the test cases and handled boolean in numberType * updated code for handling numberType values * updated code to handle number format and int exponential values * added comments for transforming number format data * updated datatype integration test * resolved comments * resolved comments * updated comments * updated comments * handled duplicate code in transform numberType values * added comments * added logger and removed redundant arg * updated unittest and code * updated comment * updated bug id test case * TDL-19029: Add support for data collection from the shared drives (#80) * TDL-17517 - Add missing tap-tester cases (#65) * Added missing test cases * Updated standard bookmark test name * Added back bookmark for full_table stream * Updated comment * Removed bookmark test case. * TDL-17698: Dict based to class based refactoring (#66) * refactored code to class based * resolve unittest failure * updated the code to write state after syncing records for file metadata stream * added code change to return if file is not changed * added code change to write file metadata bookmark at the end of the sync * added function comments * created a function to get path with query params * updated code according to pylint * resolved unittest failure * TDL-19029: Add support for data collection from the shared drives * change streams.py * update README for supportsAllDrives * remove invalid datatype exception * include false as string or bool * add unitest * updated error message * change default value * update comments * update parameter name snake case * solved circleci error * update comments in unittest * Add exception for the invalid value of the supports_all_drives * optimize code as per collin suggestion * Addressed andy's comments * rename import sync name * rename import sync name * updated unittest Co-authored-by: Prijen Khokhani <88327452+prijendev@users.noreply.github.com> Co-authored-by: Harsh <80324346+harshpatel4crest@users.noreply.github.com> Co-authored-by: prijendev <prijen.khokhani@crestdatasys.com> Co-authored-by: Jay Tilala <jay.tilala@CDSYS.LOCAL> Co-authored-by: harshpatel4crest <harsh.patel4@crestdatasys.com> * added param for shared drive for all syncs (#89) Co-authored-by: jtilala <104966482+jtilala@users.noreply.github.com> Co-authored-by: Jay Tilala <jay.tilala@CDSYS.LOCAL> Co-authored-by: savan-chovatiya <80703490+savan-chovatiya@users.noreply.github.com> Co-authored-by: Prijen Khokhani <88327452+prijendev@users.noreply.github.com> Co-authored-by: prijendev <prijen.khokhani@crestdatasys.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.
Description of change
Add param for syncing sheets from shared drives
"supportsAllDrives": True
for syncing sheets from shared drives.Manual QA steps
Risks
Rollback steps