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 2020/07/15 21:05:58 UTC

[GitHub] [hudi] umehrot2 commented on pull request #1722: [HUDI-69] Support Spark Datasource for MOR table

umehrot2 commented on pull request #1722:
URL: https://github.com/apache/hudi/pull/1722#issuecomment-659010020


   > @vinothchandar I agree we should use @umehrot2 RDD approach.
   > 
   > > So you can also in parallel just proceed?
   > 
   > Yes, I will change this PR in parallel.
   > 
   > > Do you just want the two util classes landed? SparkUtils/TablePathUtils?
   > 
   > Yes, @umehrot2 do you mind making a separate PR for these two classes?
   
   @vinothchandar @garyli1019 thank you guys for reviewing the PR. Will look at the comments and start addressing. As for the `HudiSparkUtils/TablePathUtils` let me create a separate PR so that you can re-use those pieces for your PR and are unblocked. Meanwhile you can copy the code over to unblock coding from your side while I work on the PR. Thanks!


----------------------------------------------------------------
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:
users@infra.apache.org