-
Notifications
You must be signed in to change notification settings - Fork 85
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
Java-API implementation #1650
Java-API implementation #1650
Conversation
Skipped functions that require prepared statements. Need to test after KuzuQueryResult is implemented.
Changed binding logic for PreparedStatement Note: the c++ implementation of FlatTuple also returns one Value at a time
Need to review use of generics
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #1650 +/- ##
==========================================
- Coverage 91.93% 91.49% -0.45%
==========================================
Files 685 727 +42
Lines 24640 26386 +1746
==========================================
+ Hits 22654 24142 +1488
- Misses 1986 2244 +258 ☔ View full report in Codecov by Sentry. |
Tests added |
Please remove commented-out code. |
All commented code removde |
Let's still implement the |
All changes done |
tempDir.toFile().deleteOnExit(); | ||
|
||
BufferedReader reader; | ||
db = new KuzuDatabase("java_api_test_db", 0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You added a temporary directory but did use it for the tests.
No description provided.