CurtHagenlocher commented on PR #2667: URL: https://github.com/apache/arrow-adbc/pull/2667#issuecomment-2773925103
My overall feedback before going into any individual details is that I don't think this change is correct. It might be true that there are Spark services such as Databricks or HDInsight (does that even still exist) which are always managed and for which these options are not worthwhile, but it's also true that anyone can stand up a Spark instance for which they might need to set these properties. It does not make sense to limit this driver in ways that won't work for that case. -- 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]
