[GitHub] [spark] dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested schema pruning to all V2 sources

2019-12-10 Thread GitBox
dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested 
schema pruning to all V2 sources
URL: https://github.com/apache/spark/pull/26751#issuecomment-564097372
 
 
   Hi, @aokolnychyi . #26830 is merged to support this PR.
   Could you rebase this PR?


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested schema pruning to all V2 sources

2019-12-04 Thread GitBox
dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested 
schema pruning to all V2 sources
URL: https://github.com/apache/spark/pull/26751#issuecomment-561958942
 
 
   Hi, @rdblue and @cloud-fan . Could you give some directional advice for the 
following?
   - https://github.com/apache/spark/pull/26751#discussion_r354102149


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested schema pruning to all V2 sources

2019-12-04 Thread GitBox
dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested 
schema pruning to all V2 sources
URL: https://github.com/apache/spark/pull/26751#issuecomment-561834508
 
 
   Thank you for summarizing, @aokolnychyi .


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested schema pruning to all V2 sources

2019-12-04 Thread GitBox
dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested 
schema pruning to all V2 sources
URL: https://github.com/apache/spark/pull/26751#issuecomment-561800059
 
 
   That's true, but I wanted to know what breaks the UT at the initial commit.
   > I know the review usually doesn't start before that.


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested schema pruning to all V2 sources

2019-12-04 Thread GitBox
dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested 
schema pruning to all V2 sources
URL: https://github.com/apache/spark/pull/26751#issuecomment-561705072
 
 
   Hi, @aokolnychyi .
   Is there a merge-conflict? If you do `force-pushed`, it hides the last 
change from the reviewers . In other words, the reviews need to restart from 
the bottom . 


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested schema pruning to all V2 sources

2019-12-03 Thread GitBox
dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested 
schema pruning to all V2 sources
URL: https://github.com/apache/spark/pull/26751#issuecomment-561391894
 
 
   The 5 failures look relevant. Could you fix the UT failure first?
   https://user-images.githubusercontent.com/9700541/70096281-93ce1880-15db-11ea-8c93-8e6780207fb8.png";>
   


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested schema pruning to all V2 sources

2019-12-03 Thread GitBox
dongjoon-hyun commented on issue #26751: [SPARK-30107][SQL] Expose nested 
schema pruning to all V2 sources
URL: https://github.com/apache/spark/pull/26751#issuecomment-561391422
 
 
   Thank you for pinging me, @aokolnychyi !


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org