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.

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

ARROW-17199: [Java][FlightRPC] Clean up Flight SQL example server #13710

Merged
merged 1 commit into from
Aug 4, 2022

Conversation

lidavidm
Copy link
Member

  • Use ARROW-17004 for binding Arrow data to JDBC PreparedStatements
  • Properly return Flight errors for better error messages
  • Don't throw SQLException in the FlightSqlClient

@lidavidm
Copy link
Member Author

Depends on #13589

@github-actions
Copy link

@github-actions
Copy link

⚠️ Ticket has not been started in JIRA, please click 'Start Progress'.

@lidavidm lidavidm marked this pull request as ready for review July 26, 2022 17:00
@lidavidm
Copy link
Member Author

CC @jduo if you'd like to take a look here

@lidavidm lidavidm merged commit 916417d into apache:master Aug 4, 2022
@ursabot
Copy link

ursabot commented Aug 4, 2022

Benchmark runs are scheduled for baseline = 6d9a6a4 and contender = 916417d. 916417d is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Failed ⬇️0.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Finished ⬇️0.54% ⬆️0.0%] test-mac-arm
[Finished ⬇️0.82% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.43% ⬆️0.0%] ursa-thinkcentre-m75q
Buildkite builds:
[Failed] 916417da ec2-t3-xlarge-us-east-2
[Finished] 916417da test-mac-arm
[Finished] 916417da ursa-i9-9960x
[Finished] 916417da ursa-thinkcentre-m75q
[Failed] 6d9a6a4a ec2-t3-xlarge-us-east-2
[Finished] 6d9a6a4a test-mac-arm
[Finished] 6d9a6a4a ursa-i9-9960x
[Finished] 6d9a6a4a ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants