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

Fix DremioProducer and Include SqlInfoProvider #46

Open
wants to merge 88 commits into
base: flight-sql-producer
Choose a base branch
from
Open

Fix DremioProducer and Include SqlInfoProvider #46

wants to merge 88 commits into from

Conversation

vfraga
Copy link

@vfraga vfraga commented Oct 25, 2021

No description provided.

rafael-telles and others added 30 commits August 12, 2021 17:59
@rafael-telles
Refactor DremioFlightProducer to delegate between FlightSql and legac…
3791911 
…y producers
@rafael-telles
Remove TODOs regarding to FlightSqlUtils
a221f0d
@rafael-telles
Refactor DremioFlightProducer so that it implements FlightSqlProducer…
ea6a462 
…, remova DremioFlightSqlProducer
@rafael-telles
Fix minor style issues
6498b2c
@rafael-telles
Implement CommandPreparedStatementQuery
310a079
@rafael-telles
Fix minor style issues
fb74615
@rafael-telles
Implement CommandGetCatalogs
fdb9e11
@rafael-telles
Fix minor style issues (2)
b4b11a0
@rafael-telles
Avoid start imports
7b9b205
@rafael-telles
Remove unused imports from TestFLightSqlServer*.java
55b37fc
@rafael-telles
Merge pull request #31 from s1mbi0se/update-prepared-statement
1a6be3e 
Implement FlightSqlProducer - PreparedStatement query
@rafael-telles
Add missing JavaDoc
c679c3a
@rafael-telles
Add missing JavaDoc
6644a61
@rafael-telles
Refactor CancellableUserResponseHandler and its previous subclasses
defb39a
@rafael-telles
Add missing JavaDocs
d748158
@rafael-telles
Update TestCancellableUserResponseHandler
cfac6b1
@rafael-telles
Merge pull request #32 from s1mbi0se/implement-flightsqlproducer-getC…
ff553df 
…atalogs

Implement FlightSqlProducer - CommandGetCatalogs
@jcralmeida
Implement FlightSqlProducer and adapt preparedStatement to run with b…
a607347 
…oth mode (Legacy and FlightSql)
@jcralmeida
Abstract test to query with FlightSql and Flight
5ccde13
@rafael-telles @jcralmeida
Refactor DremioFlightProducer to delegate between FlightSql and Legac…
723747e 
…y producers
@jcralmeida
Implement getTables on FlightSqlProducer
4069b0b
@jcralmeida
Create a TableResponseHandlerClass
e652e0b
@jcralmeida
Create method runTables
2b9ddf0
@jcralmeida
Add tests for the getTables methods
dad927b
@jcralmeida
Remove duplicate methods
8ff972d
@jcralmeida
Change tableTable filter on tests
8dd80f6
@jcralmeida
Change interval from counter at completed method in GetTablesResponse…
58a4e58 
…Handler
@jcralmeida
Move tests for AbstractTestFlightSqlServer
fb4fe31
@jcralmeida
Remove GetTablesResponseHandler class and move logic to FlightWorkMan…
771738a 
…ager
@jcralmeida
Refactor call to command getTables
3c2cb34
rafael-telles and others added 26 commits August 23, 2021 15:45
@rafael-telles
Add missing ticket parameter on DremioFlightProducer#getStreamStatement
1f84af6
@rafael-telles
Extract getFlightClientWrapper().getSqlClient() as a member field on …
dac65a2 
…unit tests
@jcralmeida
Refactor retrieve of schemas on flight-sql operations
a44e7e4
@jcralmeida
Remove extra empty line
4cf3f37
@jcralmeida
Remove argument from getStreamStatement
86adb66
@jcralmeida
Add missing java doc to createPreparedStatement method
c5381d0
@rafael-telles
nit: Remove extra blank line on AbstractTestFlightServer
fedba9a
@rafael-telles
Fix CheckStyle issues
e8d4df8
@jcralmeida
Add more tests for catalog methods at flight sql server
85047b9
@jcralmeida
Fix checkstyle
ee349c3
@rafael-telles
Merge pull request #38 from s1mbi0se/implement-flightsqlproducer-stat…
a7eae14 
…ement

Implement Statement support on DremioFlightProducer
@jcralmeida
Add flightStream to try-with-resources for consistency
ba53781
@jcralmeida
Merge pull request #39 from s1mbi0se/increase-flight-sql-tests
44bc595 
Add more tests for catalog methods at flight sql server
@rafael-telles
Fix build of Arrow Flight service
b6e4093
@jcralmeida
Check if the includeSchema is true and use the correct Schema on getT…
1c05d48 
…ables
@jcralmeida
Enhance logic from getTables to deal with tables schemas
bb8e0e1
@jcralmeida
Create a test check if the schemas are coming when includeSchema is s…
793536f 
…et in getTables
@jcralmeida
Refactor duplicated code on getTables execution
26446e1
@jcralmeida
Refactor getArrowType methods
05159d1
@jcralmeida
Refactor the method runGetTables to reduce its size
8b597db
@jcralmeida
Merge pull request #42 from s1mbi0se/include-schema-get-tables
b61c324 
Return schema when using getTables from FlightSQL
@vfraga
Added Schema serialization to PreparedStatement
d5b6cdb
@vfraga
NIT static default_ipc
4ab27c1
@rafael-telles
Merge pull request #45 from s1mbi0se/fix-datagrip-bug
872535d 
Added Schema serialization to PreparedStatement
@vfraga
Fix Schemas and SqlInfo
44e83a9
@vfraga
Apply changes from base FlightSqlProducer
cb5e1eb
@vfraga vfraga added the enhancement New feature or request label Oct 25, 2021
@vfraga vfraga self-assigned this Oct 25, 2021
@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

@vfraga vfraga

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

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