vinothchandar commented on a change in pull request #5125:
URL: https://github.com/apache/hudi/pull/5125#discussion_r839999705
##########
File path: hudi-sync/pom.xml
##########
@@ -35,5 +35,6 @@
<module>hudi-dla-sync</module>
<module>hudi-hive-sync</module>
<module>hudi-sync-common</module>
+ <module>hudi-bigquery-sync</module>
Review comment:
Thinking out aloud, what you are saying is vendor code (like BigQuery)
is in separate module but oss projects (like DataHub) can stay under hudi-sync?
That makes sense. We can also add lock providers there for Spanner etc. @vingov
You okay to move / rename the module?
Orthogonally, do we need to add bundles for hudi - aws and hudi-gcloud ?
--
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]
