David Li
David Li
I've been pushing various tweaks to the spec while implementing it in C++. One thing I will need to change: transaction IDs should be supplied when creating a prepared statement,...
There's now implementations in C++ and Java. TODOs: - Integration tests in C++ - https://issues.apache.org/jira/browse/ARROW-17254 - Ensure integration tests work across languages - Fix C++ API/Proto comments (methods should accept...
Hmm, Windows builds fail because of a similar issue to https://github.com/apache/arrow/pull/13434 - Protobuf and DLLs don't interact well, since you can't get `protoc` to insert the `dllimport`/`dllexport` declarations correctly. The...
The Protobuf definitions deserve more scrutiny; for the code, I'm just looking for general opinions.
I need to figure out the AppVeyor failure here, but this should generally be ready.
@jduo any further comments on the spec (or implementation) here?
I will fix ARROW-17254/ARROW-17420 and rebase this, but otherwise I think this is ready
ARROW-17420 was merged, rebased on top of PR for ARROW-17254 (#13898) and updated
@pitrou were you planning on taking a second look or were you just passing by? :slightly_smiling_face: @zeroshade if you had any comments on the definitions before implementation, they would be...
No worries, just want to make sure!