You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "lidavidm (via GitHub)" <gi...@apache.org> on 2023/04/01 22:31:48 UTC

[GitHub] [arrow-adbc] lidavidm commented on issue #479: [Java] driver/flight-sql: Add Dremio validation suite

lidavidm commented on issue #479:
URL: https://github.com/apache/arrow-adbc/issues/479#issuecomment-1493144735

   Hmm, possibly. I think a lot of that code is over-architected. The Flight SQL driver in Go implements enough authentication to work with Dremio without going that far, and it's fairly straightforward: if given a username/password, authenticate, then use the authorization header in the response:
   
   https://github.com/apache/arrow-adbc/blob/639ca716801c995c7532a924a4809e0acd00ad85/go/adbc/driver/flightsql/flightsql_adbc.go#L556-L569
   
   Also, if you are intending to go all the way to getting this in CI, also take a look at https://github.com/apache/arrow-adbc/issues/558 and consider borrowing the docker-compose setup from my closed PR here: https://github.com/ibis-project/ibis/pull/5475


-- 
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-unsubscribe@arrow.apache.org

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