-
Notifications
You must be signed in to change notification settings - Fork 610
Enhance new JDBC driver #765
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
Conversation
|
@zhicwu let me know if I can help. |
Thank you @enqueue as always. Could you help with below items?
|
Created #769 for accidentally closed this one :p |
Ad 1: In some gradle projects test fixtures are used. Perhaps a similar concept can be used in this maven project. Ad 2: This will require some manual testing, e.g checking. how DBeaver or similar tools behave. We have added unit tests over the last couple of years, but of course they won't spot everything. Ad 3: I don't know enough about benchmarking, so I won't be any help there. Sorry. |
Cool. Will you be able to spend some time to take care of this one?
Yes, I've been tried DBeaver and SQuirreL SQL. Actually I also tried DataGrip earlier but had to stop due to license expiration ;) Are you interested in code review? Any suggestion is greatly appreciated.
No problem. I think we can start a simple program and then add it into benchmarks. Let me know if there's any specific item you're interested. I created a pre-release |
This is derived from PR #747 focusing on below items: