[GitHub] [hudi] bschell commented on pull request #1922: [HUDI-1152] Add option to skip syncing Hudi metadata columns

2020-08-06 Thread GitBox


bschell commented on pull request #1922:
URL: https://github.com/apache/hudi/pull/1922#issuecomment-670038375


   @vinothchandar thanks for the detailed explanation! When I was considering 
this feature I was only considering the feedback that the extra columns were 
confusing for end users who have no knowledge of the backend systems. It makes 
sense you already would not have included them if they were not necessary. I'll 
close this.



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




[GitHub] [hudi] bschell commented on pull request #1922: [HUDI-1152] Add option to skip syncing Hudi metadata columns

2020-08-05 Thread GitBox


bschell commented on pull request #1922:
URL: https://github.com/apache/hudi/pull/1922#issuecomment-669697321


   @vinothchandar These metadata columns are required for incremental queries? 
That would make sense but I did not realize that, if this is not possible we 
can close this. The purpose was to hide the extra columns for end users who 
might be confused or not have understanding of the hudi columns when querying. 



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