- Notifications
You must be signed in to change notification settings - Fork126
Normalise Execution Response (clean backend interfaces)#587
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes fromall commits
Commits
Show all changes
49 commits Select commitHold shift + click to select a range
138c2ae [squash from exec-sea] bring over execution phase changes
varun-edachali-dbx3e3ab94 remove excess test
varun-edachali-dbx4a78165 add docstring
varun-edachali-dbx0dac4aa remvoe exec func in sea backend
varun-edachali-dbx1b794c7 remove excess files
varun-edachali-dbxda5a6fe remove excess models
varun-edachali-dbx686ade4 remove excess sea backend tests
varun-edachali-dbx31e6c83 cleanup
varun-edachali-dbx69ea238 re-introduce get_schema_desc
varun-edachali-dbx66d7517 remove SeaResultSet
varun-edachali-dbx71feef9 clean imports and attributes
varun-edachali-dbxae9862f pass CommandId to ExecResp
varun-edachali-dbxd8aa69e remove changes in types
varun-edachali-dbxdb139bc add back essential types (ExecResponse, from_sea_state)
varun-edachali-dbxb977b12 fix fetch types
varun-edachali-dbxda615c0 excess imports
varun-edachali-dbx0da04a6 reduce diff by maintaining logs
varun-edachali-dbxea9d456 fix int test types
varun-edachali-dbxd97463b move guid_to_hex_id import to utils
varun-edachali-dbx139e246 reduce diff in guid utils import
varun-edachali-dbxe3ee4e4 move arrow_schema_bytes back into ExecuteResult
varun-edachali-dbxf448a8f maintain log
varun-edachali-dbx82ca1ee remove un-necessary assignment
varun-edachali-dbxe96a078 remove un-necessary tuple response
varun-edachali-dbx27158b1 remove un-ncessary verbose mocking
varun-edachali-dbxd3200c4 move Queue construction to ResultSert
varun-edachali-dbx8a014f0 move description to List[Tuple]
varun-edachali-dbx39c41ab frmatting (black)
varun-edachali-dbx2cd04df reduce diff (remove explicit tuple conversion)
varun-edachali-dbx067a019 remove has_more_rows from ExecuteResponse
varun-edachali-dbx48c83e0 remove un-necessary has_more_rows aclc
varun-edachali-dbx281a9e9 default has_more_rows to True
varun-edachali-dbx192901d return has_more_rows from ExecResponse conversion during GetRespMetadata
varun-edachali-dbx55f5c45 remove unnecessary replacement
varun-edachali-dbxedc36b5 better mocked backend naming
varun-edachali-dbx81280e7 remove has_more_rows test in ExecuteResponse
varun-edachali-dbxc1d3be2 introduce replacement of original has_more_rows read test
varun-edachali-dbx5ee4136 call correct method in test_use_arrow_schema
varun-edachali-dbxb881ab0 call correct method in test_fall_back_to_hive_schema
varun-edachali-dbx53bf715 re-introduce result response read test
varun-edachali-dbx45a32be simplify test
varun-edachali-dbxe3fe299 remove excess fetch_results mocks
varun-edachali-dbxe8038d3 more minimal changes to thrift_backend tests
varun-edachali-dbx2f6ec19 move back to old table types
varun-edachali-dbx73bc282 remove outdated arrow_schema_bytes return
varun-edachali-dbx7c483f2 remove duplicate import
varun-edachali-dbx8cbeb08 rephrase model docstrings to explicitly denote that they are represen…
varun-edachali-dbx36b9cfb has_more_rows -> is_direct_results
varun-edachali-dbxc04d583 switch docstring format to align with Connection class
varun-edachali-dbxFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
2 changes: 0 additions & 2 deletionssrc/databricks/sql/backend/databricks_client.py
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
4 changes: 2 additions & 2 deletionssrc/databricks/sql/backend/sea/models/requests.py
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
2 changes: 1 addition & 1 deletionsrc/databricks/sql/backend/sea/models/responses.py
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
2 changes: 1 addition & 1 deletionsrc/databricks/sql/backend/sea/utils/http_client.py
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
153 changes: 88 additions & 65 deletionssrc/databricks/sql/backend/thrift_backend.py
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.
Uh oh!
There was an error while loading.Please reload this page.
Oops, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
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.