Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/19259
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maver1ck commented on the issue:
https://github.com/apache/spark/pull/19249
@ueshin
I think that for Maptype this is not a solution because every key / value
of MapType is the same type so we need conversion for all entries or for nothing
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17819
@original-brownbear Thanks for letting me know.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maver1ck commented on the issue:
https://github.com/apache/spark/pull/19234
I'm asking because such a code is 2x faster than my solution
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user maver1ck commented on the issue:
https://github.com/apache/spark/pull/19234
Any idea why we're not using
`datetime.datetime.fromtimestamp(ts / 10.)` ?
There is a comment about overflow. But if it exists ?
---
Github user original-brownbear commented on the issue:
https://github.com/apache/spark/pull/17819
@viirya re `HiveExternalCatalogVersionsSuite`, jup it is
https://github.com/apache/spark/commit/dbb824125d4d31166d9a47c330f8d51f5d159515#commitcomment-24354358
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12646
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19259
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19259
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81866/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12646
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81865/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19135
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12646
**[Test build #81865 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81865/testReport)**
for PR 12646 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19135
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81868/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19135
**[Test build #81868 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81868/testReport)**
for PR 19135 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17819
**[Test build #81869 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81869/testReport)**
for PR 17819 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16578#discussion_r139346525
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetReadSupport.scala
---
@@ -63,9 +74,22 @@ private[parquet] class
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17819
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17819
`HiveExternalCatalogVersionsSuite` seems flaky?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17819
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81867/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17819
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17819
**[Test build #81867 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81867/testReport)**
for PR 17819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19135
**[Test build #81868 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81868/testReport)**
for PR 19135 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19135#discussion_r139343473
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
@@ -385,4 +385,17 @@ package object config {
.checkValue(v =>
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19135
LGTM except one minor comment
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19135
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19135#discussion_r139343176
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
@@ -385,4 +385,17 @@ package object config {
.checkValue(v =>
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19211#discussion_r139342594
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -65,53 +60,76 @@ private[spark] class LiveListenerBus(conf:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19211#discussion_r139342432
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -65,53 +60,76 @@ private[spark] class LiveListenerBus(conf:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19211#discussion_r139342382
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -65,53 +60,76 @@ private[spark] class LiveListenerBus(conf:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18317
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/12646
LGTM pending Jenkins
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19211#discussion_r139341982
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -65,53 +60,76 @@ private[spark] class LiveListenerBus(conf:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19211#discussion_r139341925
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -65,53 +60,76 @@ private[spark] class LiveListenerBus(conf:
401 - 433 of 433 matches
Mail list logo