robinyqiu commented on pull request #12202: URL: https://github.com/apache/beam/pull/12202#issuecomment-659577530
To further clarify the previous problem: It is not a bug introduced by this PR. Instead this PR unveil this existing problem that Kenn and Luke explained above (wrong usage of `TableProvider` SPI). The workaround I suggested changes the code path to the previous state, such that it will not need the unavailable module at run time (only constructor and `getTableType()` are called), which unblocks your current work but does not solve the problem itself. @sclukas77 Could you please create a JIRA issue to track the `TableProvider` problem, and assign it to yourself, or @TheNeuralBit ? I think fixing that will require quite some refactoring (you will need to move `TableProvider` and all other classes it depends on, directly and indirectly, to sdk:java:core). I think it make sense for that to be in a separate PR and not block this one. ---------------------------------------------------------------- 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
