lukecwik commented on pull request #12202: URL: https://github.com/apache/beam/pull/12202#issuecomment-657646138
It seems like we are using ServiceProvider incorrectly since we are using ServiceProvider to load providers dynamically which allows `ParquetTableProvider.java` to live in a project containing (or directly depending on the project containing) `ParquetSchemaCapableIOProvider.java` and still outside of `sdks/java/extensions/sql` project. The interface that needs to be shared should be placed in a package that both `sdks/java/extensions/sql` and the IO packages can depend on. ---------------------------------------------------------------- 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: us...@infra.apache.org
