cloud-fan commented on pull request #28726:
URL: https://github.com/apache/spark/pull/28726#issuecomment-641316304
thanks, merging to master!
This is an automated message from the Apache Git Service.
To respond to the message
AmplabJenkins removed a comment on pull request #28619:
URL: https://github.com/apache/spark/pull/28619#issuecomment-641023126
This is an automated message from the Apache Git Service.
To respond to the message, please log on
HyukjinKwon commented on a change in pull request #28754:
URL: https://github.com/apache/spark/pull/28754#discussion_r437128608
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Average.scala
##
@@ -40,10 +40,17 @@ case class Average
SparkQA removed a comment on pull request #28708:
URL: https://github.com/apache/spark/pull/28708#issuecomment-640763276
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
AmplabJenkins commented on pull request #28751:
URL: https://github.com/apache/spark/pull/28751#issuecomment-640557597
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
SparkQA commented on pull request #28258:
URL: https://github.com/apache/spark/pull/28258#issuecomment-640632086
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
AmplabJenkins commented on pull request #28760:
URL: https://github.com/apache/spark/pull/28760#issuecomment-640938193
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
maropu commented on a change in pull request #28751:
URL: https://github.com/apache/spark/pull/28751#discussion_r437361561
##
File path:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/SharedThriftServer.scala
##
@@ -53,11 +55,21 @@ trait SharedThri
SparkQA commented on pull request #28764:
URL: https://github.com/apache/spark/pull/28764#issuecomment-641066479
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
SparkQA commented on pull request #28768:
URL: https://github.com/apache/spark/pull/28768#issuecomment-641206321
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
SparkQA removed a comment on pull request #28757:
URL: https://github.com/apache/spark/pull/28757#issuecomment-640842850
**[Test build #123646 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123646/testReport)**
for PR 28757 at commit
[`37f5ba1`](https://gi
AmplabJenkins commented on pull request #28708:
URL: https://github.com/apache/spark/pull/28708#issuecomment-640797313
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
srowen commented on a change in pull request #28750:
URL: https://github.com/apache/spark/pull/28750#discussion_r436668197
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/StringUtils.scala
##
@@ -122,8 +122,8 @@ object StringUtils extends Logging {
fornaix opened a new pull request #28755:
URL: https://github.com/apache/spark/pull/28755
### What changes were proposed in this pull request?
This pr is to support ISO 8601 format for intervals.
- P[n]Y[n]M[n]DT[n]H[n]M[n]S
- P[n]W
- P[]-[MM]-[DD]T[hh]:[mm]:[ss]
F
zhli1142015 opened a new pull request #28769:
URL: https://github.com/apache/spark/pull/28769
### What changes were proposed in this pull request?
Close LevelDBIterator when LevelDB.close() is called.
### Why are the changes needed?
This pull request would preven
HeartSaVioR commented on a change in pull request #28391:
URL: https://github.com/apache/spark/pull/28391#discussion_r437051581
##
File path:
sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamingDeduplicationSuite.scala
##
@@ -270,7 +271,12 @@ class StreamingDedupli
attilapiros edited a comment on pull request #28708:
URL: https://github.com/apache/spark/pull/28708#issuecomment-641294356
Hi @holdenk!
I have opened a new PR for your branch:
https://github.com/holdenk/spark/pull/6
I know this seems to be a lot of change but it is really ju
viirya commented on pull request #28704:
URL: https://github.com/apache/spark/pull/28704#issuecomment-640955747
@srowen @huaxingao any more comments for this change that I need to address?
This is an automated message from th
SparkQA removed a comment on pull request #28593:
URL: https://github.com/apache/spark/pull/28593#issuecomment-640524296
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
SparkQA commented on pull request #28758:
URL: https://github.com/apache/spark/pull/28758#issuecomment-640875286
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
SparkQA removed a comment on pull request #28756:
URL: https://github.com/apache/spark/pull/28756#issuecomment-640700418
**[Test build #123639 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123639/testReport)**
for PR 28756 at commit
[`be3d752`](https://gi
SparkQA commented on pull request #28761:
URL: https://github.com/apache/spark/pull/28761#issuecomment-640955046
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
tgravescs commented on pull request #27598:
URL: https://github.com/apache/spark/pull/27598#issuecomment-640882029
thanks @shanyu merged to master
This is an automated message from the Apache Git Service.
To respond to the me
HeartSaVioR commented on pull request #24996:
URL: https://github.com/apache/spark/pull/24996#issuecomment-640921998
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
sarutak edited a comment on pull request #28585:
URL: https://github.com/apache/spark/pull/28585#issuecomment-640153448
The errors were reproduced in PR builder and they are fixed by excluding the
old `bytebuddy` so I believe it's safe to merge.
What do you think, @dongjoon-hyun, @Hyukji
SparkQA removed a comment on pull request #28619:
URL: https://github.com/apache/spark/pull/28619#issuecomment-641022846
**[Test build #123668 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123668/testReport)**
for PR 28619 at commit
[`61f850d`](https://gi
gengliangwang commented on a change in pull request #28760:
URL: https://github.com/apache/spark/pull/28760#discussion_r437060140
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
##
@@ -570,7 +570,7 @@ case class DataSource(
sarutak commented on a change in pull request #28748:
URL: https://github.com/apache/spark/pull/28748#discussion_r436694872
##
File path:
streaming/src/test/scala/org/apache/spark/streaming/UISeleniumSuite.scala
##
@@ -125,24 +125,37 @@ class UISeleniumSuite
// Chec
AmplabJenkins removed a comment on pull request #28754:
URL: https://github.com/apache/spark/pull/28754#issuecomment-640556879
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
therealJacobWu commented on a change in pull request #28731:
URL: https://github.com/apache/spark/pull/28731#discussion_r436981238
##
File path: bin/beeline
##
@@ -28,5 +28,7 @@ if [ -z "${SPARK_HOME}" ]; then
source "$(dirname "$0")"/find-spark-home
fi
+. "${SPARK_HOME}"
maropu commented on pull request #28755:
URL: https://github.com/apache/spark/pull/28755#issuecomment-640949585
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
HeartSaVioR commented on pull request #28391:
URL: https://github.com/apache/spark/pull/28391#issuecomment-640935752
cc. @tdas @zsxwing @jose-torres
This is an automated message from the Apache Git Service.
To respond to the
SparkQA removed a comment on pull request #28752:
URL: https://github.com/apache/spark/pull/28752#issuecomment-640462659
**[Test build #123624 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123624/testReport)**
for PR 28752 at commit
[`acd9d54`](https://gi
AmplabJenkins commented on pull request #27246:
URL: https://github.com/apache/spark/pull/27246#issuecomment-641313089
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
holdenk commented on pull request #28708:
URL: https://github.com/apache/spark/pull/28708#issuecomment-640736229
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
viirya commented on pull request #28761:
URL: https://github.com/apache/spark/pull/28761#issuecomment-641073844
cc @dbtsai @dongjoon-hyun @cloud-fan @maropu
This is an automated message from the Apache Git Service.
To respon
SparkQA removed a comment on pull request #28650:
URL: https://github.com/apache/spark/pull/28650#issuecomment-641066478
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
HyukjinKwon opened a new pull request #28762:
URL: https://github.com/apache/spark/pull/28762
### What changes were proposed in this pull request?
This PR proposes to fix wordings in the Python UDF exception error message
from:
From:
An exception was thrown from Python
AmplabJenkins commented on pull request #28767:
URL: https://github.com/apache/spark/pull/28767#issuecomment-641163291
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
AmplabJenkins removed a comment on pull request #27986:
URL: https://github.com/apache/spark/pull/27986#issuecomment-640592619
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 #28737:
URL: https://github.com/apache/spark/pull/28737#issuecomment-640689295
This is an automated message from the Apache Git Service.
To respond to the message, please log on
maropu commented on pull request #18323:
URL: https://github.com/apache/spark/pull/18323#issuecomment-640955951
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
SparkQA commented on pull request #28708:
URL: https://github.com/apache/spark/pull/28708#issuecomment-640763276
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
AmplabJenkins removed a comment on pull request #27518:
URL: https://github.com/apache/spark/pull/27518#issuecomment-640643335
This is an automated message from the Apache Git Service.
To respond to the message, please log on
yaooqinn commented on a change in pull request #28751:
URL: https://github.com/apache/spark/pull/28751#discussion_r436649612
##
File path:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/SharedThriftServer.scala
##
@@ -74,18 +86,31 @@ trait SharedTh
cloud-fan commented on a change in pull request #27986:
URL: https://github.com/apache/spark/pull/27986#discussion_r436656608
##
File path:
sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala
##
@@ -1021,4 +1021,20 @@ class AdaptiveQuer
AmplabJenkins removed a comment on pull request #28751:
URL: https://github.com/apache/spark/pull/28751#issuecomment-640557597
This is an automated message from the Apache Git Service.
To respond to the message, please log on
AmplabJenkins commented on pull request #28752:
URL: https://github.com/apache/spark/pull/28752#issuecomment-640590456
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
AmplabJenkins commented on pull request #28761:
URL: https://github.com/apache/spark/pull/28761#issuecomment-640955464
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
maropu commented on pull request #28768:
URL: https://github.com/apache/spark/pull/28768#issuecomment-641202844
ok to test
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 #28645:
URL: https://github.com/apache/spark/pull/28645#issuecomment-640584004
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
SparkQA commented on pull request #28422:
URL: https://github.com/apache/spark/pull/28422#issuecomment-640538971
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
SparkQA removed a comment on pull request #28762:
URL: https://github.com/apache/spark/pull/28762#issuecomment-640964944
**[Test build #123663 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123663/testReport)**
for PR 28762 at commit
[`9eb69c0`](https://gi
cloud-fan commented on pull request #25715:
URL: https://github.com/apache/spark/pull/25715#issuecomment-640741243
I think breaking changes like this should add an item in the migration guide.
This is an automated message fro
prakharjain09 commented on a change in pull request #28619:
URL: https://github.com/apache/spark/pull/28619#discussion_r437133100
##
File path:
core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala
##
@@ -1892,6 +1892,112 @@ class TaskSetManagerSuite
te
maropu commented on a change in pull request #27983:
URL: https://github.com/apache/spark/pull/27983#discussion_r437476009
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/script/ScriptTransformationExec.scala
##
@@ -0,0 +1,239 @@
+/*
+ * Licensed to the Ap
turboFei commented on pull request #28765:
URL: https://github.com/apache/spark/pull/28765#issuecomment-641093704
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
SparkQA removed a comment on pull request #28764:
URL: https://github.com/apache/spark/pull/28764#issuecomment-641073304
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
AmplabJenkins removed a comment on pull request #28708:
URL: https://github.com/apache/spark/pull/28708#issuecomment-640797313
This is an automated message from the Apache Git Service.
To respond to the message, please log on
turboFei opened a new pull request #28765:
URL: https://github.com/apache/spark/pull/28765
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How wa
SaurabhChawla100 commented on pull request #27636:
URL: https://github.com/apache/spark/pull/27636#issuecomment-641018544
> @tgravescs @SaurabhChawla100 .. I am wondering if you guys are still
pushing for this PR to get into Spark master ? Are you also thinking of
targeting to Spark 3.0 br
SparkQA removed a comment on pull request #28258:
URL: https://github.com/apache/spark/pull/28258#issuecomment-640632086
**[Test build #123637 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123637/testReport)**
for PR 28258 at commit
[`0e152f4`](https://gi
AmplabJenkins removed a comment on pull request #28593:
URL: https://github.com/apache/spark/pull/28593#issuecomment-640697045
This is an automated message from the Apache Git Service.
To respond to the message, please log on
zsxwing commented on pull request #28744:
URL: https://github.com/apache/spark/pull/28744#issuecomment-640687413
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
SparkQA removed a comment on pull request #27518:
URL: https://github.com/apache/spark/pull/27518#issuecomment-640641799
**[Test build #123638 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123638/testReport)**
for PR 27518 at commit
[`af55a08`](https://gi
cloud-fan commented on a change in pull request #28650:
URL: https://github.com/apache/spark/pull/28650#discussion_r437231685
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
##
@@ -56,6 +54,26 @@ trait TimeZoneAware
skambha closed pull request #27629:
URL: https://github.com/apache/spark/pull/27629
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
AmplabJenkins commented on pull request #28258:
URL: https://github.com/apache/spark/pull/28258#issuecomment-640633493
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
iRakson commented on pull request #28748:
URL: https://github.com/apache/spark/pull/28748#issuecomment-640578773
cc @srowen I forgot to cc you
This is an automated message from the Apache Git Service.
To respond to the messag
asfgit closed pull request #27598:
URL: https://github.com/apache/spark/pull/27598
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 t
maropu commented on pull request #27019:
URL: https://github.com/apache/spark/pull/27019#issuecomment-641077009
retest this please
This is an automated message from the Apache Git Service.
To respond to the message, please lo
MaxGekk opened a new pull request #28757:
URL: https://github.com/apache/spark/pull/28757
### What changes were proposed in this pull request?
Add benchmarks for `HiveResult.hiveResultString()/toHiveString()` to measure
throughput of `toHiveString` for the date/timestamp types:
- java
AmplabJenkins removed a comment on pull request #28733:
URL: https://github.com/apache/spark/pull/28733#issuecomment-640550996
This is an automated message from the Apache Git Service.
To respond to the message, please log on
HeartSaVioR commented on pull request #28763:
URL: https://github.com/apache/spark/pull/28763#issuecomment-641002505
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
zsxwing commented on a change in pull request #28758:
URL: https://github.com/apache/spark/pull/28758#discussion_r436988149
##
File path: core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala
##
@@ -485,29 +483,6 @@ class JsonProtocolSuite extends SparkFunSuite {
AmplabJenkins removed a comment on pull request #28761:
URL: https://github.com/apache/spark/pull/28761#issuecomment-640955464
This is an automated message from the Apache Git Service.
To respond to the message, please log on
SparkQA removed a comment on pull request #28759:
URL: https://github.com/apache/spark/pull/28759#issuecomment-640930710
**[Test build #123651 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123651/testReport)**
for PR 28759 at commit
[`e300be5`](https://gi
AmplabJenkins commented on pull request #28758:
URL: https://github.com/apache/spark/pull/28758#issuecomment-640875960
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
AmplabJenkins commented on pull request #28756:
URL: https://github.com/apache/spark/pull/28756#issuecomment-640701041
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
AmplabJenkins commented on pull request #27518:
URL: https://github.com/apache/spark/pull/27518#issuecomment-640643335
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
AmplabJenkins commented on pull request #28619:
URL: https://github.com/apache/spark/pull/28619#issuecomment-641023126
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
baohe-zhang edited a comment on pull request #28412:
URL: https://github.com/apache/spark/pull/28412#issuecomment-641024447
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
sathyaprakashg commented on pull request #28703:
URL: https://github.com/apache/spark/pull/28703#issuecomment-641075311
@bart-samwel @cloud-fan @maropu
In case of subtract use case, If left expression is string and right
expression is timestamp, then there is no ambiguity. Timestamp
maropu commented on pull request #17953:
URL: https://github.com/apache/spark/pull/17953#issuecomment-641072690
@amit-hitachi I think we don't have any plan for this work. But, you could
revive this discussion in the corresponding jira side.
---
AmplabJenkins removed a comment on pull request #28645:
URL: https://github.com/apache/spark/pull/28645#issuecomment-640584830
This is an automated message from the Apache Git Service.
To respond to the message, please log on
cloud-fan commented on pull request #28765:
URL: https://github.com/apache/spark/pull/28765#issuecomment-641156270
ok to test
This is an automated message from the Apache Git Service.
To respond to the message, please log on
SparkQA removed a comment on pull request #28755:
URL: https://github.com/apache/spark/pull/28755#issuecomment-640950793
**[Test build #123658 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/123658/testReport)**
for PR 28755 at commit
[`4bfac69`](https://gi
AmplabJenkins commented on pull request #28757:
URL: https://github.com/apache/spark/pull/28757#issuecomment-640844245
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
cloud-fan commented on pull request #28734:
URL: https://github.com/apache/spark/pull/28734#issuecomment-641319075
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
HeartSaVioR commented on a change in pull request #28412:
URL: https://github.com/apache/spark/pull/28412#discussion_r437043628
##
File path: core/src/main/scala/org/apache/spark/deploy/history/HybridStore.scala
##
@@ -0,0 +1,205 @@
+/*
+ * Licensed to the Apache Software Found
AmplabJenkins commented on pull request #28764:
URL: https://github.com/apache/spark/pull/28764#issuecomment-641067170
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
SparkQA commented on pull request #28759:
URL: https://github.com/apache/spark/pull/28759#issuecomment-640930710
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
AmplabJenkins removed a comment on pull request #28766:
URL: https://github.com/apache/spark/pull/28766#issuecomment-641146441
This is an automated message from the Apache Git Service.
To respond to the message, please log on
zhengruifeng commented on a change in pull request #28704:
URL: https://github.com/apache/spark/pull/28704#discussion_r437107792
##
File path: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala
##
@@ -248,6 +248,19 @@ object MLUtils extends Logging {
}.toArray
AngersZh commented on a change in pull request #28733:
URL: https://github.com/apache/spark/pull/28733#discussion_r437277043
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
##
@@ -198,6 +199,88 @@ trait PredicateHelper {
maropu commented on a change in pull request #28726:
URL: https://github.com/apache/spark/pull/28726#discussion_r437178386
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala
##
@@ -85,6 +85,7 @@ trait NamedExpression exte
AmplabJenkins commented on pull request #28745:
URL: https://github.com/apache/spark/pull/28745#issuecomment-641290863
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
HeartSaVioR commented on pull request #28422:
URL: https://github.com/apache/spark/pull/28422#issuecomment-641097576
I agree the new addition of the similar option feels tricky.
Maybe you've already indicated there're some cases `maxFileAge` has to be
ignored which means Spark is ne
AmplabJenkins commented on pull request #28762:
URL: https://github.com/apache/spark/pull/28762#issuecomment-640965214
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
SparkQA commented on pull request #28766:
URL: https://github.com/apache/spark/pull/28766#issuecomment-641145651
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
601 - 700 of 729 matches
Mail list logo