fornwall opened a new pull request, #4510:
URL: https://github.com/apache/arrow-adbc/pull/4510

   `InfoCode` was a closed set of the 11 standard codes, so requests for 
XDBC-range (`[500`, 1_000`)`) or vendor-specific (`>= 10_000`) info codes could 
not be represented: the FFI exporter errored on unknown values before the 
wrapped Rust driver was called, and the JavaScript client dropped them before 
the driver manager forwarded the request to the loaded driver.
   
   A driver could still emit such codes when asked for everything (`info_codes` 
== `None`), so a vendor code would appear in "fetch all" results but could not 
be requested explicitly.
   
   Add an `InfoCode::Other(u32)` catch-all variant (mirroring the existing 
`Other` variants on `OptionDatabase`/`OptionConnection`/`OptionStatement`) so 
codes survive the u32 round-trip, and pass requested codes through unfiltered 
in both the FFI exporter and the JavaScript client, leaving "ignore 
unrecognized codes" to the driver as adbc.h states:
   
   > Drivers/vendors will ignore requests for unrecognized codes
   > (the row will be omitted from the result)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to