You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/02/14 12:36:55 UTC

[GitHub] [spark] xuanyuanking opened a new pull request #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

xuanyuanking opened a new pull request #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581
 
 
   ### What changes were proposed in this pull request?
   Follow-up work for #25600. In this PR, we move `sql/dynamicpruning` to `sql/catalyst/dynamicpruning`.
   
   ### Why are the changes needed?
   Fix the unexpected public APIs in 3.0.0 #27560.
   
   ### Does this PR introduce any user-facing change?
   No.
   
   ### How was this patch tested?
   Existing UT.
   

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586275568
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23184/
   Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586864081
 
 
   **[Test build #118541 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118541/testReport)** for PR 27581 at commit [`9329ce1`](https://github.com/apache/spark/commit/9329ce1dcbd42bd88fa2cdd3d78b2f7372317f43).
    * This patch **fails due to an unknown error code, -9**.
    * This patch merges cleanly.
    * This patch adds no public classes.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] cloud-fan commented on a change in pull request #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
cloud-fan commented on a change in pull request #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#discussion_r379970896
 
 

 ##########
 File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst/dynamicpruning/CleanupDynamicPruningFilters.scala
 ##########
 @@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.spark.sql.dynamicpruning
+package org.apache.spark.sql.catalyst.dynamicpruning
 
 Review comment:
   let's move it to execution? it's also where AQE landed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586837681
 
 
   **[Test build #118541 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118541/testReport)** for PR 27581 at commit [`9329ce1`](https://github.com/apache/spark/commit/9329ce1dcbd42bd88fa2cdd3d78b2f7372317f43).

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586864186
 
 
   Merged build finished. Test FAILed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] xuanyuanking commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
xuanyuanking commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586906203
 
 
   retest this please

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-587040215
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118564/
   Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586909178
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586909178
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-587040215
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118564/
   Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586838074
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586275565
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] xuanyuanking commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
xuanyuanking commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-587239458
 
 
   Thanks for the review.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] HyukjinKwon commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586847733
 
 
   Nice!

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586909183
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23319/
   Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586838077
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23296/
   Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] cloud-fan closed pull request #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
cloud-fan closed pull request #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581
 
 
   

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586382379
 
 
   Merged build finished. Test FAILed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586381659
 
 
   **[Test build #118427 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118427/testReport)** for PR 27581 at commit [`ec7ba69`](https://github.com/apache/spark/commit/ec7ba69b4bf96128d018e35cb8c972bb0cd94154).
    * This patch **fails PySpark pip packaging tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586275568
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23184/
   Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-587040197
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] xuanyuanking commented on a change in pull request #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
xuanyuanking commented on a change in pull request #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#discussion_r379995671
 
 

 ##########
 File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst/dynamicpruning/CleanupDynamicPruningFilters.scala
 ##########
 @@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.spark.sql.dynamicpruning
+package org.apache.spark.sql.catalyst.dynamicpruning
 
 Review comment:
   Copy 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:
users@infra.apache.org


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586864186
 
 
   Merged build finished. Test FAILed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586908655
 
 
   **[Test build #118564 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118564/testReport)** for PR 27581 at commit [`9329ce1`](https://github.com/apache/spark/commit/9329ce1dcbd42bd88fa2cdd3d78b2f7372317f43).

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] cloud-fan commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
cloud-fan commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586871418
 
 
   retest it please

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586838077
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23296/
   Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586908655
 
 
   **[Test build #118564 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118564/testReport)** for PR 27581 at commit [`9329ce1`](https://github.com/apache/spark/commit/9329ce1dcbd42bd88fa2cdd3d78b2f7372317f43).

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586909183
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23319/
   Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586275565
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586864196
 
 
   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118541/
   Test FAILed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586382386
 
 
   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118427/
   Test FAILed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586837681
 
 
   **[Test build #118541 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118541/testReport)** for PR 27581 at commit [`9329ce1`](https://github.com/apache/spark/commit/9329ce1dcbd42bd88fa2cdd3d78b2f7372317f43).

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586274963
 
 
   **[Test build #118427 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118427/testReport)** for PR 27581 at commit [`ec7ba69`](https://github.com/apache/spark/commit/ec7ba69b4bf96128d018e35cb8c972bb0cd94154).

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-587039238
 
 
   **[Test build #118564 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118564/testReport)** for PR 27581 at commit [`9329ce1`](https://github.com/apache/spark/commit/9329ce1dcbd42bd88fa2cdd3d78b2f7372317f43).
    * This patch passes all tests.
    * This patch merges cleanly.
    * This patch adds no public classes.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586382386
 
 
   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118427/
   Test FAILed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586838074
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586274963
 
 
   **[Test build #118427 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118427/testReport)** for PR 27581 at commit [`ec7ba69`](https://github.com/apache/spark/commit/ec7ba69b4bf96128d018e35cb8c972bb0cd94154).

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-587040197
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586864196
 
 
   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118541/
   Test FAILed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] xuanyuanking commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
xuanyuanking commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586270656
 
 
   cc @cloud-fan @maryannxue 

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/catalyst/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-586382379
 
 
   Merged build finished. Test FAILed.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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


[GitHub] [spark] cloud-fan commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning

Posted by GitBox <gi...@apache.org>.
cloud-fan commented on issue #27581: [SPARK-11150][SQL][FOLLOWUP] Move sql/dynamicpruning to sql/execution/dynamicpruning
URL: https://github.com/apache/spark/pull/27581#issuecomment-587095707
 
 
   thanks, merging to master/3.0!

----------------------------------------------------------------
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


With regards,
Apache Git Services

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