amirhzd commented on issue #1203: URL: https://github.com/apache/arrow-java/issues/1203#issuecomment-4876960662
Thank you for the questions — and apologies for the confusion. I'm fairly new to open-source contribution, and I leaned on an AI assistant to write this up, which made it read like a PR when I'd only meant to report the problem. Let me clarify honestly: (1) The change is real — I did apply it locally and the driver's test suite passed, and it fixed a live Looker→Dremio connection for me. If I understood you correctly, I'd rather let CI verify it properly than have you take my word for it. (2) Totally fair point. I actually opened a feature request on Looker's portal over two months ago, but it's still sitting in "Awaiting Review," so a fix from their side doesn't look imminent. In principle, the malformed URL is arguably a Looker (or Dremio-integration) concern rather than something the standard Arrow driver should absorb — I genuinely don't know where it best belongs, and that's your call. That said, I'd like to make the case for accepting a driver-side workaround: since the upstream fix is blocked and this is likely to trip up anyone connecting from Looker (or similar systems that don't emit the standard :// form). And also tolerating the :direct= format lowers the barrier for more people to adopt the driver. But I'll defer to your judgment on the trade-off. -- 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]
