MaxGekk commented on a change in pull request #27366:
URL: https://github.com/apache/spark/pull/27366#discussion_r451582334
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonFilters.scala
##
@@ -0,0 +1,133 @@
+/*
+ * Licensed to the Apache
SparkQA commented on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-655549681
**[Test build #125372 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125372/testReport)**
for PR 29034 at commit
HyukjinKwon commented on pull request #29040:
URL: https://github.com/apache/spark/pull/29040#issuecomment-655549870
I filed a JIRA for that at SPARK-32231
This is an automated message from the Apache Git Service.
To respond
AmplabJenkins commented on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-655549139
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
AmplabJenkins removed a comment on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-655549139
This is an automated message from the Apache Git Service.
To respond to the message, please log on
MaxGekk commented on a change in pull request #27366:
URL: https://github.com/apache/spark/pull/27366#discussion_r451581234
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/StructFilters.scala
##
@@ -0,0 +1,161 @@
+/*
+ * Licensed to the Apache Software
SparkQA commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655548572
**[Test build #125371 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125371/testReport)**
for PR 29039 at commit
HyukjinKwon commented on pull request #29040:
URL: https://github.com/apache/spark/pull/29040#issuecomment-655548695
cc @dongjoon-hyun.
BTW, I will use Hadoop 3 profile in AppVeyor later in a separate PR.
This is an
MaxGekk commented on a change in pull request #27366:
URL: https://github.com/apache/spark/pull/27366#discussion_r451578687
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/StructFilters.scala
##
@@ -0,0 +1,161 @@
+/*
+ * Licensed to the Apache Software
AmplabJenkins removed a comment on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-655545966
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
maropu commented on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-655547083
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please
AmplabJenkins removed a comment on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-655545955
Merged build finished. Test FAILed.
This is an automated message from the Apache Git Service.
To
AmplabJenkins commented on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-655545955
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
AmplabJenkins removed a comment on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655544926
This is an automated message from the Apache Git Service.
To respond to the message, please log on
AmplabJenkins removed a comment on pull request #28993:
URL: https://github.com/apache/spark/pull/28993#issuecomment-655544789
This is an automated message from the Apache Git Service.
To respond to the message, please log on
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655544813
This is an automated message from the Apache Git Service.
To respond to the message, please log on
SparkQA commented on pull request #28993:
URL: https://github.com/apache/spark/pull/28993#issuecomment-655545021
**[Test build #125369 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125369/testReport)**
for PR 28993 at commit
HeartSaVioR commented on pull request #29017:
URL: https://github.com/apache/spark/pull/29017#issuecomment-655544712
I hope we do the migration iteratively - we all know it became a huge pain
to see the green light of the build (rarely possible according to the Jenkins
PRB build trends),
AmplabJenkins commented on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655544926
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
AmplabJenkins removed a comment on pull request #28998:
URL: https://github.com/apache/spark/pull/28998#issuecomment-655543957
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
AmplabJenkins commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655544813
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
SparkQA removed a comment on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-655426277
**[Test build #125345 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125345/testReport)**
for PR 29034 at commit
AmplabJenkins commented on pull request #28993:
URL: https://github.com/apache/spark/pull/28993#issuecomment-655544789
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
SparkQA commented on pull request #29034:
URL: https://github.com/apache/spark/pull/29034#issuecomment-65552
**[Test build #125345 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125345/testReport)**
for PR 29034 at commit
SparkQA commented on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655544417
**[Test build #125370 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125370/testReport)**
for PR 28683 at commit
HeartSaVioR commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655544209
retest this, please
This is an automated message from the Apache Git Service.
To respond to the message,
AmplabJenkins removed a comment on pull request #28998:
URL: https://github.com/apache/spark/pull/28998#issuecomment-655543947
Merged build finished. Test FAILed.
This is an automated message from the Apache Git Service.
To
AmplabJenkins commented on pull request #28998:
URL: https://github.com/apache/spark/pull/28998#issuecomment-655543947
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
cloud-fan commented on pull request #28993:
URL: https://github.com/apache/spark/pull/28993#issuecomment-655542118
retest this please
This is an automated message from the Apache Git Service.
To respond to the message,
cloud-fan commented on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655541763
retest this please
This is an automated message from the Apache Git Service.
To respond to the message,
AmplabJenkins removed a comment on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655540149
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
AmplabJenkins commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655540609
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
AmplabJenkins removed a comment on pull request #29041:
URL: https://github.com/apache/spark/pull/29041#issuecomment-655540433
This is an automated message from the Apache Git Service.
To respond to the message, please log on
AmplabJenkins removed a comment on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655540135
Merged build finished. Test FAILed.
This is an automated message from the Apache Git Service.
To
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655540609
This is an automated message from the Apache Git Service.
To respond to the message, please log on
SparkQA commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655540008
**[Test build #125368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125368/testReport)**
for PR 29039 at commit
AmplabJenkins commented on pull request #29041:
URL: https://github.com/apache/spark/pull/29041#issuecomment-655540433
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
AmplabJenkins commented on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655540135
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
SparkQA removed a comment on pull request #28998:
URL: https://github.com/apache/spark/pull/28998#issuecomment-655344891
**[Test build #125327 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125327/testReport)**
for PR 28998 at commit
SparkQA commented on pull request #29041:
URL: https://github.com/apache/spark/pull/29041#issuecomment-655540157
**[Test build #125367 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125367/testReport)**
for PR 29041 at commit
HyukjinKwon commented on pull request #28859:
URL: https://github.com/apache/spark/pull/28859#issuecomment-655539393
Honestly, I do merge too time to time e.g.) when there are some reasons or
when the Jenkins build itself is completely orthogonal such as fixing AppVeyor
CI issue .. but
SparkQA commented on pull request #28998:
URL: https://github.com/apache/spark/pull/28998#issuecomment-655539732
**[Test build #125327 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125327/testReport)**
for PR 28998 at commit
SparkQA removed a comment on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655472327
**[Test build #125356 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125356/testReport)**
for PR 28683 at commit
SparkQA commented on pull request #28683:
URL: https://github.com/apache/spark/pull/28683#issuecomment-655537963
**[Test build #125356 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125356/testReport)**
for PR 28683 at commit
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655536916
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655536902
Merged build finished. Test FAILed.
This is an automated message from the Apache Git Service.
To
AmplabJenkins commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655536902
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HyukjinKwon opened a new pull request #29041:
URL: https://github.com/apache/spark/pull/29041
### What changes were proposed in this pull request?
This PR proposes to partially reverts the simple string in `NullType` at
https://github.com/apache/spark/pull/28833:
SparkQA removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655500419
**[Test build #125363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125363/testReport)**
for PR 29039 at commit
SparkQA commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655535127
**[Test build #125363 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125363/testReport)**
for PR 29039 at commit
tgravescs commented on a change in pull request #28972:
URL: https://github.com/apache/spark/pull/28972#discussion_r451547527
##
File path:
core/src/test/scala/org/apache/spark/resource/ResourceProfileSuite.scala
##
@@ -256,4 +266,29 @@ class ResourceProfileSuite extends
HeartSaVioR commented on pull request #28859:
URL: https://github.com/apache/spark/pull/28859#issuecomment-655530835
Oh OK. Thanks for noticing. I hope we'd be a bit flexible and pragmatic, but
I also sincerely respect the rule and policy. I'll follow it in next time.
AmplabJenkins removed a comment on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655522163
This is an automated message from the Apache Git Service.
To respond to the message, please log on
AmplabJenkins commented on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655522163
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HyukjinKwon commented on pull request #28859:
URL: https://github.com/apache/spark/pull/28859#issuecomment-655522336
Hey @HeartSaVioR, let's don't merge PRs when tests are being failed.
This is an automated message from the
SparkQA commented on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655521983
**[Test build #125366 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125366/testReport)**
for PR 29019 at commit
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655521039
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655521030
Merged build finished. Test FAILed.
This is an automated message from the Apache Git Service.
To
MaxGekk commented on a change in pull request #27366:
URL: https://github.com/apache/spark/pull/27366#discussion_r451545802
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonFilters.scala
##
@@ -0,0 +1,133 @@
+/*
+ * Licensed to the Apache
AmplabJenkins commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655521030
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
SparkQA commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655520302
**[Test build #125365 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125365/testReport)**
for PR 29039 at commit
SparkQA removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655504940
**[Test build #125365 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125365/testReport)**
for PR 29039 at commit
HyukjinKwon commented on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655517986
retest this please
This is an automated message from the Apache Git Service.
To respond to the message,
AmplabJenkins removed a comment on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655516134
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
AmplabJenkins commented on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655516113
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
AmplabJenkins removed a comment on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655516113
Merged build finished. Test FAILed.
This is an automated message from the Apache Git Service.
To
SparkQA removed a comment on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655472334
**[Test build #125354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125354/testReport)**
for PR 29019 at commit
SparkQA commented on pull request #29019:
URL: https://github.com/apache/spark/pull/29019#issuecomment-655512819
**[Test build #125354 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125354/testReport)**
for PR 29019 at commit
AmplabJenkins removed a comment on pull request #29002:
URL: https://github.com/apache/spark/pull/29002#issuecomment-655510817
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
AmplabJenkins removed a comment on pull request #29002:
URL: https://github.com/apache/spark/pull/29002#issuecomment-655510803
Merged build finished. Test FAILed.
This is an automated message from the Apache Git Service.
To
AmplabJenkins commented on pull request #29002:
URL: https://github.com/apache/spark/pull/29002#issuecomment-655510803
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HeartSaVioR commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655508577
retest this, please
This is an automated message from the Apache Git Service.
To respond to the
Ngone51 commented on a change in pull request #29028:
URL: https://github.com/apache/spark/pull/29028#discussion_r451528174
##
File path: core/src/main/scala/org/apache/spark/rdd/RDD.scala
##
@@ -1509,22 +1509,26 @@ abstract class RDD[T: ClassTag](
* @return an array of
SparkQA commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655504940
**[Test build #125365 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125365/testReport)**
for PR 29039 at commit
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655504789
This is an automated message from the Apache Git Service.
To respond to the message, please log on
SparkQA commented on pull request #29002:
URL: https://github.com/apache/spark/pull/29002#issuecomment-655505147
**[Test build #125316 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125316/testReport)**
for PR 29002 at commit
SparkQA removed a comment on pull request #29002:
URL: https://github.com/apache/spark/pull/29002#issuecomment-655337239
**[Test build #125316 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125316/testReport)**
for PR 29002 at commit
AmplabJenkins commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655504789
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HeartSaVioR commented on pull request #28859:
URL: https://github.com/apache/spark/pull/28859#issuecomment-655504143
Thanks @zhli1142015 for the contribution! Merged into master, branch-3.0,
branch-2.4 respectively.
This is
HeartSaVioR closed pull request #28859:
URL: https://github.com/apache/spark/pull/28859
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
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655502223
This is an automated message from the Apache Git Service.
To respond to the message, please log on
AmplabJenkins commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655502223
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
maropu commented on pull request #27983:
URL: https://github.com/apache/spark/pull/27983#issuecomment-655501312
Could you update the PR description, too?
This is an automated message from the Apache Git Service.
To respond
AmplabJenkins removed a comment on pull request #29040:
URL: https://github.com/apache/spark/pull/29040#issuecomment-655500970
This is an automated message from the Apache Git Service.
To respond to the message, please log on
zhli1142015 commented on pull request #28859:
URL: https://github.com/apache/spark/pull/28859#issuecomment-655501284
Thank you for your helping @HeartSaVioR . cc @gengliangwang , is it ok to
merge this PR, or I need to run test again?
AmplabJenkins commented on pull request #29040:
URL: https://github.com/apache/spark/pull/29040#issuecomment-655500970
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HeartSaVioR commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655500750
retest this, please
This is an automated message from the Apache Git Service.
To respond to the message,
SparkQA commented on pull request #29040:
URL: https://github.com/apache/spark/pull/29040#issuecomment-655500364
**[Test build #125362 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125362/testReport)**
for PR 29040 at commit
SparkQA commented on pull request #29038:
URL: https://github.com/apache/spark/pull/29038#issuecomment-655500319
**[Test build #125364 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125364/testReport)**
for PR 29038 at commit
SparkQA commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655500419
**[Test build #125363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125363/testReport)**
for PR 29039 at commit
HyukjinKwon opened a new pull request #29040:
URL: https://github.com/apache/spark/pull/29040
### What changes were proposed in this pull request?
https://ci.appveyor.com/project/ApacheSoftwareFoundation/spark/builds/33976604
AppVeyor build seems broken after we change the
HeartSaVioR commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655497736
retest this, please
This is an automated message from the Apache Git Service.
To respond to the message,
AmplabJenkins removed a comment on pull request #29038:
URL: https://github.com/apache/spark/pull/29038#issuecomment-655497174
This is an automated message from the Apache Git Service.
To respond to the message, please log on
AmplabJenkins removed a comment on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655497233
This is an automated message from the Apache Git Service.
To respond to the message, please log on
AmplabJenkins commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655497233
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HeartSaVioR commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655497019
Rationalization: the failures always come from classloader issue;
```
sbt.ForkMain$ForkError: org.mockito.exceptions.base.MockitoException:
ClassCastException
AmplabJenkins commented on pull request #29038:
URL: https://github.com/apache/spark/pull/29038#issuecomment-655497174
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
SparkQA commented on pull request #29039:
URL: https://github.com/apache/spark/pull/29039#issuecomment-655496732
**[Test build #125361 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125361/testReport)**
for PR 29039 at commit
maropu commented on a change in pull request #27983:
URL: https://github.com/apache/spark/pull/27983#discussion_r451514820
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/script/ScriptTransformbase.scala
##
@@ -0,0 +1,167 @@
+/*
+ * Licensed to the Apache
HyukjinKwon commented on pull request #29038:
URL: https://github.com/apache/spark/pull/29038#issuecomment-655494623
The tests failure in AppVeyor is orthogonal. It still prints fine the
package information:
```
[00:01:52] Package Version
[00:01:52] arrow
801 - 900 of 1423 matches
Mail list logo