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/03/04 21:47:28 UTC
[GitHub] [spark] maryannxue opened a new pull request #27798:
[SPARK-31045][SQL] Add config for AQE logging level
maryannxue opened a new pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798
### What changes were proposed in this pull request?
This PR adds an internal config for changing the logging level of adaptive execution query plan evolvement.
### Why are the changes needed?
To make AQE debugging easier.
### Does this PR introduce any user-facing change?
No.
### How was this patch tested?
Added 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] cloud-fan commented on a change in pull request #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
cloud-fan commented on a change in pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#discussion_r388370367
##########
File path: sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala
##########
@@ -729,5 +731,51 @@ class AdaptiveQueryExecSuite
s"${SQLConf.ADAPTIVE_EXECUTION_ENABLED.key} is" +
s" enabled but is not supported for")))
}
+
+ test("test log level") {
+
Review comment:
remove blank line
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594872751
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595518364
**[Test build #119416 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119416/testReport)** for PR 27798 at commit [`3151b01`](https://github.com/apache/spark/commit/3151b019ff1025a41334eca78464d4baa5eb8c9c).
* 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 commented on issue #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595518959
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595084374
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595383127
**[Test build #119403 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119403/testReport)** for PR 27798 at commit [`7e999fa`](https://github.com/apache/spark/commit/7e999fa7592f5172fba63237e539c883ab62bea4).
* This patch passes all tests.
* This patch **does not merge 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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595084374
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] AmplabJenkins commented on issue #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594872751
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595383958
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594871922
**[Test build #119340 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119340/testReport)** for PR 27798 at commit [`e93825d`](https://github.com/apache/spark/commit/e93825d0b1fe7cbc09d3efc6b7ae7aabb834fc48).
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594871922
**[Test build #119340 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119340/testReport)** for PR 27798 at commit [`e93825d`](https://github.com/apache/spark/commit/e93825d0b1fe7cbc09d3efc6b7ae7aabb834fc48).
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595084383
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119362/
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 removed a comment on issue #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595518966
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119416/
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595300167
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24140/
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595029056
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24099/
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595300167
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24140/
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] Ngone51 commented on a change in pull request #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
Ngone51 commented on a change in pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#discussion_r388040216
##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -394,6 +394,18 @@ object SQLConf {
.booleanConf
.createWithDefault(false)
+ val ADAPTIVE_EXECUTION_LOG_LEVEL = buildConf("spark.sql.adaptive.logLevel")
Review comment:
It's still based on global log level?
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594872767
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24077/
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595029056
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24099/
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594872767
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24077/
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594990128
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594989556
**[Test build #119340 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119340/testReport)** for PR 27798 at commit [`e93825d`](https://github.com/apache/spark/commit/e93825d0b1fe7cbc09d3efc6b7ae7aabb834fc48).
* 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] cloud-fan commented on a change in pull request #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
cloud-fan commented on a change in pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#discussion_r388233784
##########
File path: sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/AdaptiveSparkPlanExec.scala
##########
@@ -67,6 +67,8 @@ case class AdaptiveSparkPlanExec(
@transient private val lock = new Object()
+ @transient private val logLevel = conf.adaptiveExecutionLogLevel
Review comment:
nit:
```
@transient private val log: String => Unit = conf.adaptiveExecutionLogLevel match {
case "TRACE" => logTrace
...
}
```
to avoid checking the log level everytime.
----------------------------------------------------------------
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] HeartSaVioR commented on issue #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
HeartSaVioR commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595605960
I just fixed it via #27826 coincidentally - please take a look.
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
cloud-fan commented on a change in pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#discussion_r388231574
##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -394,6 +394,18 @@ object SQLConf {
.booleanConf
.createWithDefault(false)
+ val ADAPTIVE_EXECUTION_LOG_LEVEL = buildConf("spark.sql.adaptive.logLevel")
+ .internal()
+ .doc("Configures the log level for adaptive execution logging. The value can be 'trace', " +
+ "'debug', 'info', 'warn', or 'error'. The default log level is 'debug'.")
+ .version("3.0.0")
+ .stringConf
+ .transform(_.toUpperCase(Locale.ROOT))
+ .checkValue(logLevel => Set("TRACE", "DEBUG", "INFO", "WARN", "ERROR").contains(logLevel),
Review comment:
nit: `checkValues(Set("TRACE", "DEBUG", "INFO", "WARN", "ERROR"))`
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
cloud-fan commented on a change in pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#discussion_r388370006
##########
File path: sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala
##########
@@ -729,5 +731,51 @@ class AdaptiveQueryExecSuite
s"${SQLConf.ADAPTIVE_EXECUTION_ENABLED.key} is" +
s" enabled but is not supported for")))
}
+
+ test("test log level") {
+
+ def verifyLog(expectedLevel: Level): Unit = {
+ val logAppender = new LogAppender("adaptive execution")
+ withLogAppender(
+ logAppender,
Review comment:
2 space indentation
----------------------------------------------------------------
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] dongjoon-hyun commented on a change in pull request #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on a change in pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#discussion_r387968844
##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -394,6 +394,18 @@ object SQLConf {
.booleanConf
.createWithDefault(false)
+ val ADAPTIVE_EXECUTION_LOG_LEVEL = buildConf("spark.sql.adaptive.logLevel")
+ .internal()
+ .doc("Configures the log level for adaptive execution logging. The value can be 'trace', " +
+ "'debug', 'info', 'warn', or 'error'. The default log level is 'trace'.")
Review comment:
`The default log level is 'trace'` -> `The default log level is 'debug'` because the default is `debug` at line 407?
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595300151
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595409044
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24151/
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595383958
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595409044
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24151/
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595084151
**[Test build #119362 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119362/testReport)** for PR 27798 at commit [`68dd879`](https://github.com/apache/spark/commit/68dd879fb7b0ebf9b7e63f779bfa2ee4bb8d26d5).
* 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] AmplabJenkins commented on issue #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595084383
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119362/
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595409034
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 removed a comment on issue #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595299459
**[Test build #119403 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119403/testReport)** for PR 27798 at commit [`7e999fa`](https://github.com/apache/spark/commit/7e999fa7592f5172fba63237e539c883ab62bea4).
----------------------------------------------------------------
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594990128
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] dongjoon-hyun commented on a change in pull request #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on a change in pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#discussion_r388713212
##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -405,6 +405,16 @@ object SQLConf {
.booleanConf
.createWithDefault(false)
+ val ADAPTIVE_EXECUTION_LOG_LEVEL = buildConf("spark.sql.adaptive.logLevel")
+ .internal()
+ .doc("Configures the log level for adaptive execution logging of plan changes. The value " +
+ "can be 'trace', 'debug', 'info', 'warn', or 'error'. The default log level is 'debug'.")
+ .version("3.0.0")
Review comment:
Hi, @cloud-fan . This seems to break `branch-3.0`.
```
[error] /home/jenkins/workspace/spark-branch-3.0-test-sbt-hadoop-2.7-hive-2.3/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala:385: value version is not a member of org.apache.spark.internal.config.ConfigBuilder
[error] possible cause: maybe a semicolon is missing before `value version'?
[error] .version("3.0.0")
[error] ^
[error] one error found
```
----------------------------------------------------------------
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
cloud-fan commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595585238
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
[GitHub] [spark] cloud-fan closed pull request #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
cloud-fan closed pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798
----------------------------------------------------------------
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595383964
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119403/
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594990137
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119340/
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595409034
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595028788
**[Test build #119362 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119362/testReport)** for PR 27798 at commit [`68dd879`](https://github.com/apache/spark/commit/68dd879fb7b0ebf9b7e63f779bfa2ee4bb8d26d5).
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
cloud-fan commented on a change in pull request #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#discussion_r388233094
##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -394,6 +394,18 @@ object SQLConf {
.booleanConf
.createWithDefault(false)
+ val ADAPTIVE_EXECUTION_LOG_LEVEL = buildConf("spark.sql.adaptive.logLevel")
+ .internal()
+ .doc("Configures the log level for adaptive execution logging. The value can be 'trace', " +
Review comment:
nit: `for adaptive execution logging of plan changes.` as it doesn't affect the logging in adaptive execution optimization rules.
----------------------------------------------------------------
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595029051
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595518966
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119416/
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595028788
**[Test build #119362 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119362/testReport)** for PR 27798 at commit [`68dd879`](https://github.com/apache/spark/commit/68dd879fb7b0ebf9b7e63f779bfa2ee4bb8d26d5).
----------------------------------------------------------------
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 #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595300151
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595518959
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595412367
**[Test build #119416 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119416/testReport)** for PR 27798 at commit [`3151b01`](https://github.com/apache/spark/commit/3151b019ff1025a41334eca78464d4baa5eb8c9c).
----------------------------------------------------------------
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] dongjoon-hyun commented on issue #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595600294
Could you make a quick hot fix?
----------------------------------------------------------------
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 #27798: [SPARK-31045][SQL] Add
config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595299459
**[Test build #119403 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119403/testReport)** for PR 27798 at commit [`7e999fa`](https://github.com/apache/spark/commit/7e999fa7592f5172fba63237e539c883ab62bea4).
----------------------------------------------------------------
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] maryannxue commented on issue #27798: [SPARK-31045][SQL]
Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
maryannxue commented on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594871138
cc @gatorsmile @cloud-fan
----------------------------------------------------------------
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595383964
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119403/
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-594990137
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119340/
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 #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595029051
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 removed a comment on issue #27798:
[SPARK-31045][SQL] Add config for AQE logging level
Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27798: [SPARK-31045][SQL] Add config for AQE logging level
URL: https://github.com/apache/spark/pull/27798#issuecomment-595412367
**[Test build #119416 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119416/testReport)** for PR 27798 at commit [`3151b01`](https://github.com/apache/spark/commit/3151b019ff1025a41334eca78464d4baa5eb8c9c).
----------------------------------------------------------------
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