-
Notifications
You must be signed in to change notification settings - Fork 125
ENH: Convert read_gbq() function to use google-cloud-python #25
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
Merged
tswast
merged 43 commits into
googleapis:master
from
jasonqng:read-gbq-google-cloud-library
Dec 20, 2017
Merged
Changes from 42 commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
83b66c2
Add new from_gbq() method and tweak requirements
438ff59
Remove unecessary type conversion
03a9548
Remove old read_gbq, rename from_gbq to read_gbq, and add verbose que…
ae7c0e9
Handle new iterator return type from fetch_data
jasonqng d2d01bc
Remove old read_gbq
jasonqng 97410dd
Change requirement to google-cloud-bigquery and pin to 0.25.0
jasonqng a0847f8
Pass any private key to client for auth
jasonqng 12bacd8
Pin google-cloud-bigquery to 0.25.0 in setup.py
jasonqng f50bcca
Catch renaming of results to result in 0.26.0
jasonqng cd17b8d
Pin requirements to include 0.26.0
jasonqng 48ac795
Remove legacy read_gbq code and add initial user auth flow
jasonqng c4a2c36
Use GbqConnector for credentials checking
jasonqng 04373eb
Dealing with tests part 1
jasonqng dcf014a
Fix formatting to make linter happy plus remove obj conversion in test
jasonqng 72253aa
Rewrite query_parameters test with documentation
jasonqng 751f39b
Rewrite sync and async job flow, handle timeouts, add BadRequest erro…
jasonqng d83fd22
Remove two configuration tests which are no longer relevant
jasonqng e597a76
Rewrite tests part 2 and can specify return type
jasonqng 347bca2
Pin bigquery to 0.26.0 and more linting
jasonqng fc6134a
Remove uuid from requirements
jasonqng 61db191
Clean up/revert some of the documentation
jasonqng 28c0ae7
Unpin in testing requirements and expand config error message
jasonqng cd75cda
Include 0.27 in setup.py install_requires
jasonqng 6264177
Initial run_query function for modularizing read_gbq
jasonqng 208e39f
Move schema handling and creating the df to separate functions, updat…
jasonqng 2cd32c7
Linting
jasonqng 4468197
Update schema test
jasonqng 6a6fc2d
Fix read_gbq docstring for Sphinx
jasonqng 9ab0de3
Remove timeout_ms paramater from read_gbq, it should be set in config…
jasonqng e203ba4
Add more documentation
jasonqng 92f3943
Moved read_query to GbqConnector, update credentials and client gener…
jasonqng 3cc153a
Move sizeof_fmt back into GbqConnector
jasonqng 91f900e
Fix sizeof_fmt test and move create_df to top-level function
jasonqng 4d967e6
Add import error message
jasonqng 476d72b
Revert test_query_response_bytes test to original
jasonqng 6da0ef2
Convert rest of methods to use google-cloud-bigquery
tswast cd551bb
Indentation
jasonqng a6865e0
Ignore mode property when comparing schemas.
tswast c636783
Document new dependency on google-cloud-bigquery.
tswast 362e9db
Merge branch 'master' into read-gbq-google-cloud-library
tswast e959571
Document dependencies for previous verions.
tswast 6448abb
Remove print statement used for debugging.
tswast 26d6431
Add deps and StreamingInsertError to changelog.
tswast File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,5 @@ | ||
google-api-python-client | ||
google-auth | ||
google-auth-httplib2 | ||
google-auth-oauthlib | ||
PyCrypto | ||
python-gflags | ||
mock | ||
google-cloud-bigquery |
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,4 @@ | ||
google-api-python-client==1.6.0 | ||
google-auth==1.0.0 | ||
google-auth-httplib2==0.0.1 | ||
google-auth-oauthlib==0.0.1 | ||
mock | ||
google-cloud-bigquery==0.28.0 |
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,4 @@ | ||
google-api-python-client | ||
google-auth | ||
google-auth-httplib2 | ||
google-auth-oauthlib | ||
mock | ||
google-cloud-bigquery |
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,4 @@ | ||
google-api-python-client | ||
google-auth | ||
google-auth-httplib2 | ||
google-auth-oauthlib | ||
mock | ||
google-cloud-bigquery |
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
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.
needs a more comprehensive note here. show what you used to import / depend on, and what it should be now.
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.
you can link to install.rst as well