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
feat: add samples #44
feat: add samples #44
Conversation
Codecov Report
@@ Coverage Diff @@
## master #44 +/- ##
============================================
- Coverage 77.36% 77.32% -0.04%
- Complexity 1107 1108 +1
============================================
Files 73 73
Lines 5911 5915 +4
Branches 645 645
============================================
+ Hits 4573 4574 +1
- Misses 1012 1014 +2
- Partials 326 327 +1
Continue to review full report at Codecov.
|
|
||
public static void runExtractTableToJSON() { | ||
// TODO(developer): Replace these variables before running the sample. | ||
Table table = null; |
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.
Why is this null? We should be showing the user how to initialize this value.
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.
My initial thought is user should be able to pass in a Table object of their choice to extract into GCS as CSV - but I think what you said makes sense too. We can offer an example of what this Table object could be.
Fixes b/146347707, b/146452106