yaooqinn commented on pull request #30055: URL: https://github.com/apache/spark/pull/30055#issuecomment-709707525
> On the other hand, do we plan to use Hive's TCLIService.thrift all the time? hi, @wangyum, I believe that when you changed the IDL of `TCLIService.thrift`, you have to make your `hive-jdbc` artifact for the APIs that you added. Similarly, if we are not going to use `TCLIService.thrift` anymore in the future, we have to make our own JDBC driver module, and the `hive-jdbc` from Hive may be not able to use anymore. If we really need to extend something of spark's own in the future. A more legitimate way is to use `thrift Include` to inherit `TCLIService.thrift` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
