-
Notifications
You must be signed in to change notification settings - Fork 4k
GH-35500: [C++][Go][Java][FlightRPC] Add support for result set expiration #36009
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
Merged
Changes from all commits
Commits
Show all changes
91 commits
Select commit
Hold shift + click to select a range
02085c1
GH-35500: [C++][Go][FlightRPC] Add support for result set expiration
kou 2635593
Use {}
kou e7fa0e0
Use std::nano::den
kou 01cede8
Reorder
kou 55e83ba
Use \deprecated
kou fc55763
Reorder
kou aca8f1e
Improve wording
kou eb87155
Reorder
kou 15a00b3
Add FlightSqlClient::CloseFlightInfo
kou 9a05526
Fix a typo
kou aa0fc17
Improve error message
kou 04aacb1
Add missing closed check
kou 7069008
Suppress warnings
kou 06007b5
Clear statuses for each GetFlightInfo
kou f241e47
Use FlightInfo instead of query
kou bd44381
Order
kou afe15ce
Order
kou af3634c
Don't use CancelFlightInfo yet
kou d56418e
Add Go implementation
kou 8932fe5
Format
kou ec97b0c
Add ResultStream::Drain()
kou 0cf45f7
Improve name in .proto
kou a37c041
Add a test for CancelFlightInfo
kou ddb41d3
Fix format
kou 11c7add
Add mock methods
kou 769f6c9
Fix static check
kou dd77974
CancelResult -> CancelStatus
kou c744447
Don't use any for CancelFlightInfo
kou 5d7f50a
Add Flight SQL server tests for built-in actions
kou 294d65d
Make CancelQUery deprecated
kou 9322427
Fix RefreshFlightEndpoint
kou 56fdbce
Add document
kou 39b8290
Fix wording
kou 346f4d7
Add more deprecated
kou d0acc07
Fix staticcheck
kou 28fe2b4
Move ReadUntilEOF from flightsql to flight
kou 3944a66
Add shortcuts
kou 871a1c1
Mock
kou 0dc1984
Implement Flight SQL server methods
kou b9ee3e6
Fix style
kou 2f99ac8
Improve comment
kou 8d9a872
Clean up
kou 69fe1cf
Add tests
kou 5786663
Add default CancelQuery implementation
kou d4f38a0
Add missing FlightEndpoint implementations
kou e8d0c78
Fix a typo
kou 1ca8e05
Use constructor style
kou 037b715
Use local variable
kou 246cd42
Fix expected
kou 5f633b5
Fix a typo
kou 69fc412
Change style
kou c4d765b
Specify std::nullopt explicitly
kou 53e069f
Add more std::nullopt
kou 187c417
Add missing cast
kou 0158ebc
Fix CI falures
kou edc86da
Fix CI failures
kou d647e06
Update expected
kou f69de19
Update test value
kou 99b141a
CancelFlightInfo: return only one result
kou d82fcac
Improve docstring
kou deeae0c
Simplify
kou 59607f3
Remove CancelQuery
kou 5c1fd0e
Fix wrong deprecated description of CancelQuery()
kou 4a0a6d6
Add Java support for proposal
lidavidm c9b69a1
Remove unused cancelQueryRequest
kou 7a7beef
Add fallback from cancelQuery to cancelFlightInfo
lidavidm 201ab9d
Implement CancelFlightInfo
kou c6cf2fc
Deprecate FlightSqlServerBase::CancelQuery()
kou e0295d1
Fix a typo
kou 73751ad
Remove needless import
kou fa8f7d4
Remove expired checks
kou 3e21487
Remove needless checks
kou 7d4ca10
Remove unused imports
kou 1f26633
Add support for CancelQuery() method
kou 05bea5f
Remove unused variables
kou 99d4618
Refresh -> Renew
kou 53152dc
Use RenewFlightInfoRequest
kou d0b44ae
Fix style
kou 54a37e7
Remove unused import
lidavidm b732b12
Java: refresh -> renew
lidavidm 111ecb9
Fix build error
kou e964103
Add CancelFlightInfoRequest
kou 95295c5
Remove CloseFlightInfo
kou 5acb080
Format
kou f0e3ace
Clean up
kou 61f3666
java: Add CancelFlightInfoRequest
kou 3affd5d
java: Remove CloseFlightInfo
kou 18c96ff
Remove unused function
kou b5ba62b
Add ticket content to error message
kou c2f2f05
Increase expiration time
kou 8c60daf
Follow name change
kou 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
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
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.
Uh oh!
There was an error while loading. Please reload this page.