| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
@lidavidm I have drafted something; could you help me determine if I am on the right track? Currently, all the tests are being modified in-place, and I plan to refine them further. Do you think there are additional tests I should include? |
Sorry, something went wrong.
There was a problem hiding this comment.
Looks reasonable to me.
Sorry, something went wrong.
|
Thanks for your implementation! Our project is being blocked by this. What help I can provide so this pr can be merged soon? |
Sorry, something went wrong.
|
@lewiszlw It's ready for review now. |
Sorry, something went wrong.
|
I build jdbc driver package from your branch and tested it in our project, but got error Unsupported vector type: org.apache.arrow.driver.jdbc.shaded.org.apache.arrow.vector.ViewVarBinaryVector. @ViggoC |
Sorry, something went wrong.
|
could you share the stack? @lewiszlw |
Sorry, something went wrong.
Our stack is rust built server based on Datafusion query engine, and expose flight sql api. Our clients are Java stack. I used dbevaer software to verify your branch. |
Sorry, something went wrong.
|
@lewiszlw |
Sorry, something went wrong.
|
I sent a pr to your branch. I'm testing it. Note that I can only test binary view type. @ViggoC |
Sorry, something went wrong.
|
@lewiszlw Given that you have a real usage environment, it would be great if you are willing to make the fixes. |
Sorry, something went wrong.
Sorry, something went wrong.
There was a problem hiding this comment.
I tested this pr in our project.
Sorry, something went wrong.
|
JNI failures are unrelated. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
What's Changed
Flight SQL JDBC related StringView components implementation.
Closes #110.