You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2022/03/31 20:40:38 UTC

[GitHub] [hudi] vinothchandar commented on a change in pull request #5125: [HUDI-3357] MVP implementation of BigQuerySyncTool

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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org