TheNeuralBit commented on pull request #10487: URL: https://github.com/apache/beam/pull/10487#issuecomment-656920061
Hey @milantracy sorry for the churn on this... I didn't realize this was ready to go and I went ahead and merged #12090 which is causing the conflicts. That PR moves the majority of the logic for PubsubJsonTableProvider next to the GCP IO module. Some of it was just straight file renames (e.g. PubsubMessageToRow) which shouldn't be too tough to resolve but PubsubIOJsonTable and PubsubJsonTableProvider are trickier, as we added a new interface that will be amenable for use in cross-language (see s.apache.org/beam-schema-io for details on this) If you don't have bandwidth to resovle the conflicts, I could probably help out after next week. ---------------------------------------------------------------- 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]
