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

   ## Problem
   So we need to accommodate fields higher than 128 bytes in some cases. The Go 
Library is proven to support arbitrary lengths but this Rust layer artificially 
constrains values. This is a problem because some tokens go well beyond 128-1 
bytes (-1 for the null terminator).
   
   This bug was encountered while trying to use `Database::set_option` and 
`Database::get_option` for a string higher than 128 bytes in length. 
`Database::set_option` doesn't have this restriction curiously.
   
   ## Impact
   
   Without this function, we cannot complete native oauth in Snowflake.
   
   ## Solution
   To always allocate 1024 truly would be overkill. So I'm splitting the 
difference with 1024 bytes.
   
   I'm stabbing a bit in the dark here but I think the type to override for 
will be `c_char`.
   
   If you know a more efficient workaround, please, let's do that, not this :D 
   
   


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to