Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Flight SQL Producer #41

Open
wants to merge 77 commits into
base: master
Choose a base branch
from
Open

Flight SQL Producer #41

wants to merge 77 commits into from

Conversation

rafael-telles
Copy link

No description provided.

rafael-telles and others added 30 commits November 1, 2021 17:01
@rafael-telles @vfraga
Refactor DremioFlightProducer to delegate between FlightSql and legac…
172cc4b 
…y producers
@rafael-telles @vfraga
Remove TODOs regarding to FlightSqlUtils
e06cea6
@rafael-telles @vfraga
Refactor DremioFlightProducer so that it implements FlightSqlProducer…
e2e6c01 
…, remova DremioFlightSqlProducer
@rafael-telles @vfraga
Fix minor style issues
83571ad
@rafael-telles @vfraga
Implement CommandPreparedStatementQuery
7501d2d
@rafael-telles @vfraga
Fix minor style issues
eced748
@rafael-telles @vfraga
Fix minor style issues (2)
ed0dd13
@rafael-telles @vfraga
Avoid start imports
ca43291
@rafael-telles @vfraga
Implement CommandGetCatalogs
a0a2d3f
@rafael-telles @vfraga
Remove unused imports from TestFLightSqlServer*.java
cbdc9f1
@rafael-telles @vfraga
Add missing JavaDoc
32c2698
@rafael-telles @vfraga
Add missing JavaDoc
d0d0128
@rafael-telles @vfraga
Refactor CancellableUserResponseHandler and its previous subclasses
b2089e0
@rafael-telles @vfraga
Add missing JavaDocs
c22e651
@rafael-telles @vfraga
Update TestCancellableUserResponseHandler
232a737
@rafael-telles @vfraga
Fix missing CallOptions on AbstractTestFlightSqlServer
8f34588
@jcralmeida @vfraga
Implement FlightSqlProducer and adapt preparedStatement to run with b…
dc9f98e 
…oth mode (Legacy and FlightSql)
@jcralmeida @vfraga
Abstract test to query with FlightSql and Flight
8f867dc
@rafael-telles @vfraga
Refactor DremioFlightProducer to delegate between FlightSql and Legac…
d6a9735 
…y producers
@jcralmeida @vfraga
Implement getTables on FlightSqlProducer
ebf2b81
@jcralmeida @vfraga
Create a TableResponseHandlerClass
6746adc
@jcralmeida @vfraga
Create method runTables
b5952af
@jcralmeida @vfraga
Add tests for the getTables methods
55ba5eb
@jcralmeida @vfraga
Remove duplicate methods
c2c6ab2
@jcralmeida @vfraga
Change tableTable filter on tests
4c60a18
@jcralmeida @vfraga
Change interval from counter at completed method in GetTablesResponse…
2b99d9c 
…Handler
@jcralmeida @vfraga
Move tests for AbstractTestFlightSqlServer
524c811
@jcralmeida @vfraga
Remove GetTablesResponseHandler class and move logic to FlightWorkMan…
ba70123 
…ager
@jcralmeida @vfraga
Refactor call to command getTables
6a42808
@jcralmeida @vfraga
Add missing java doc to runGetTables
133a235
jcralmeida and others added 29 commits November 1, 2021 17:01
@jcralmeida @vfraga
Remove UNKNOWN_TABLE_TYPE and UNRECOGNIZED from getTableTypes
fa27beb
@jcralmeida @vfraga
Refactor the way of removing the undesired table types from enum
79801cc
@jcralmeida @vfraga
Change the tableTypes list to a hard-coded list
f7d1000
@jcralmeida @vfraga
Change || to && in the predicate
4b9d5ad
@jcralmeida @vfraga
Remove unused imports
15b27fd
@rafael-telles @vfraga
Implement Statement support on DremioFlightProducer
53dca49
@rafael-telles @vfraga
Refactor DremioFlightProducer tests to be able to run query tests wit…
f12c9cd 
…h both statement and preparedstatement
@rafael-telles @vfraga
Add 'final' modifier to various variables
febd440
@rafael-telles @vfraga
Add missing ticket parameter on DremioFlightProducer#getStreamStatement
20c5a72
@rafael-telles @vfraga
Extract getFlightClientWrapper().getSqlClient() as a member field on …
e8d0ef5 
…unit tests
@jcralmeida @vfraga
Refactor retrieve of schemas on flight-sql operations
b0ed364
@jcralmeida @vfraga
Remove extra empty line
9e9cc09
@jcralmeida @vfraga
Remove argument from getStreamStatement
2206097
@jcralmeida @vfraga
Add missing java doc to createPreparedStatement method
dfcfbe1
@rafael-telles @vfraga
nit: Remove extra blank line on AbstractTestFlightServer
beb71a3
@rafael-telles @vfraga
Fix CheckStyle issues
ef9961e
@jcralmeida @vfraga
Add more tests for catalog methods at flight sql server
b247382
@jcralmeida @vfraga
Fix checkstyle
ec99350
@jcralmeida @vfraga
Add flightStream to try-with-resources for consistency
af81b94
@rafael-telles @vfraga
Fix build of Arrow Flight service
c94acc0
@jcralmeida @vfraga
Check if the includeSchema is true and use the correct Schema on getT…
2ddb853 
…ables
@jcralmeida @vfraga
Enhance logic from getTables to deal with tables schemas
24dc84c
@jcralmeida @vfraga
Create a test check if the schemas are coming when includeSchema is s…
6add5f9 
…et in getTables
@jcralmeida @vfraga
Refactor duplicated code on getTables execution
ccbdf08
@jcralmeida @vfraga
Refactor getArrowType methods
0e0babb
@jcralmeida @vfraga
Refactor the method runGetTables to reduce its size
4b497cd
@vfraga
Added Schema serialization to PreparedStatement
22d989d
@vfraga
NIT static default_ipc
b5907de
@vfraga
Update Arrow Flight Module with Arrow 7.0.0
cc8bd8a
@vfraga vfraga force-pushed the flight-sql-producer branch from 872535d to cc8bd8a Compare November 1, 2021 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rafael-telles @jcralmeida @vfraga