Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15594
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r84566581
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,343 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r84566224
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/BitSamplingSuite.scala ---
@@ -0,0 +1,100 @@
+/*
+ * Licensed to the Apache Software Fou
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r84566504
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,343 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r84566599
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -245,50 +246,50 @@ private[ml] abstract class LSHModel[T <: LSHModel[T]]
extend
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r84565977
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/BitSampling.scala ---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r84566821
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,343 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15398
**[Test build #67366 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67366/consoleFull)**
for PR 15398 at commit
[`85c16e6`](https://github.com/apache/spark/commit/8
Github user jodersky commented on the issue:
https://github.com/apache/spark/pull/15398
I went ahead and changed the parsing function to throw an AnalysisException
in case the input escaper sequence is invalid. Feel free to revert (or just
tell me and I'll do it) if we should stick to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15595
**[Test build #67367 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67367/consoleFull)**
for PR 15595 at commit
[`4a79782`](https://github.com/apache/spark/commit/4
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15581
yea of course, I'll do it soon
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled an
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15592
**[Test build #67361 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67361/consoleFull)**
for PR 15592 at commit
[`96079fe`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15592
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67361/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15592
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15539#discussion_r84568025
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ListingFileCatalog.scala
---
@@ -38,14 +38,16 @@ class ListingFileCatalog(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15394
**[Test build #67363 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67363/consoleFull)**
for PR 15394 at commit
[`28f8c2f`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15394
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15394
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67363/
Test FAILed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15539
LGTM, next maybe we can refactor the `PartitionAwareFileCatalog` and make
it use the new global cache better. I'm going to merge it to unblock other
works. thanks!
---
If your project is set up
Github user zhengruifeng commented on a diff in the pull request:
https://github.com/apache/spark/pull/15585#discussion_r84568230
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/evaluation/RankingMetricsSuite.scala
---
@@ -65,5 +65,4 @@ class RankingMetricsSuite extends Spar
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15539
I had some network problems, I'll ask @yhuai to merge it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15377
**[Test build #67368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67368/consoleFull)**
for PR 15377 at commit
[`af5a95b`](https://github.com/apache/spark/commit/a
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15575
`CoalesceExec` can also have `SinglePartition` as `outputPartitioning` too.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your pro
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15585
**[Test build #67369 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67369/consoleFull)**
for PR 15585 at commit
[`5b1bc5f`](https://github.com/apache/spark/commit/5
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15575
But `ExpandExec`'s `GroupExpressions` could generate data with arbitrary
partitioning, so even it doesn't move rows across input partitions, doesn't it
still can change child' output partition?
---
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/15587
@rxin How about this way: modify the parameter as
`Option[PartitionerCoalescer with Serializable]`, so that if user forget to
mark the implementation class as serializable, compiling time will
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/15516
@felixcheung
Remove the unrelated jar file.
and about the `String` look up table, here seems there are not the mapping
relationship between these String constant, so that the code I thi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15516
**[Test build #67370 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67370/consoleFull)**
for PR 15516 at commit
[`5af4a07`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15595
**[Test build #67364 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67364/consoleFull)**
for PR 15595 at commit
[`c62f84f`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15595
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15595
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67364/
Test PASSed.
---
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/15596
[SQL] Remove shuffle codes in CollectLimitExec
## What changes were proposed in this pull request?
`CollectLimitExec` now shuffles its child's data in `doExecute`. As we have
`EnsureRequire
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15596
cc @rxin @yhuai May you take a quick look to see if this direction is ok
for you? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15596
**[Test build #67371 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67371/consoleFull)**
for PR 15596 at commit
[`3d24f79`](https://github.com/apache/spark/commit/3
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15596#discussion_r84569163
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/BufferedRowIterator.java
---
@@ -39,7 +39,7 @@
protected int partitionIndex = -1;
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15596#discussion_r84569198
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/limit.scala ---
@@ -83,9 +83,8 @@ trait BaseLimitExec extends UnaryExecNode with
CodegenSupp
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15596
BTW, we can see if there is an exchange added for `CollectLimit` from
physical plan, e.g.,
CollectLimit 1
+- Exchange SinglePartition
+- *LocalLimit 1
+- *H
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15515
**[Test build #67365 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67365/consoleFull)**
for PR 15515 at commit
[`f903243`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15515
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15515
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67365/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15516
**[Test build #67370 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67370/consoleFull)**
for PR 15516 at commit
[`5af4a07`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15516
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67370/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15516
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15398
**[Test build #67366 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67366/consoleFull)**
for PR 15398 at commit
[`85c16e6`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15398
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67366/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15398
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15585
**[Test build #67372 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67372/consoleFull)**
for PR 15585 at commit
[`9abd9ae`](https://github.com/apache/spark/commit/9
Github user frreiss commented on a diff in the pull request:
https://github.com/apache/spark/pull/14553#discussion_r84569335
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala
---
@@ -337,17 +343,27 @@ class StreamExecution(
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15585
**[Test build #67369 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67369/consoleFull)**
for PR 15585 at commit
[`5b1bc5f`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15585
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67369/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15585
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15398
@jodersky I think the current approach is fine if we have documented it
clear.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15595
**[Test build #67367 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67367/consoleFull)**
for PR 15595 at commit
[`4a79782`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15595
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15595
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67367/
Test PASSed.
---
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/15573
@gatorsmile
> Could you explain the pattern? For example, How many do such columns
exist in your query? How large is your table? Are your query complex? Or just a
simple SELECT query?
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/15573#discussion_r84569857
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapData.scala
---
@@ -35,6 +39,23 @@ object ArrayBasedMapData {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15573
**[Test build #67373 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67373/consoleFull)**
for PR 15573 at commit
[`4864672`](https://github.com/apache/spark/commit/4
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/15573
>> Could you do a quick check for all the callers who are using
I have updated all places except three :
- JacksonParser : can be abstracted out but the API will be too specific to
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570152
--- Diff:
core/src/test/scala/org/apache/spark/serializer/UnsafeKryoSerializerSuite.scala
---
@@ -0,0 +1,28 @@
+/*
+ * Licensed to the Apache Softwar
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15596
**[Test build #67371 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67371/consoleFull)**
for PR 15596 at commit
[`3d24f79`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15596
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15596
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67371/
Test FAILed.
---
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15394
jenkins retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15394
**[Test build #67374 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67374/consoleFull)**
for PR 15394 at commit
[`28f8c2f`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15377
**[Test build #67368 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67368/consoleFull)**
for PR 15377 at commit
[`af5a95b`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15377
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15377
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67368/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15587
**[Test build #67375 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67375/consoleFull)**
for PR 15587 at commit
[`0411839`](https://github.com/apache/spark/commit/0
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15573
High level change LGTM.
@gatorsmile can you do the final review & merge?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15586
Merging in master / branch-2.0.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and w
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15590
BTW overall this patch is really well written!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this featur
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15586
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15178
hm sorry for the delay. This is pretty complicated and we need to compare
the two implementations.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15587
Is this an API breaking change?
How about we just add a require check in CoalesceRDD to verify the passed
in PartitionCoalescer is serializable?
---
If your project is set up for it, you c
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15178
@rxin ok. no problem. Please let me know if anything I need to explain more
in details.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user techaddict commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570678
--- Diff:
core/src/test/scala/org/apache/spark/serializer/UnsafeKryoSerializerSuite.scala
---
@@ -0,0 +1,28 @@
+/*
+ * Licensed to the Apache Sof
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570704
--- Diff:
core/src/test/scala/org/apache/spark/serializer/UnsafeKryoSerializerSuite.scala
---
@@ -0,0 +1,28 @@
+/*
+ * Licensed to the Apache Software
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570727
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -78,8 +79,14 @@ class KryoSerializer(conf: SparkConf)
.filter(!_.
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570736
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -186,9 +193,15 @@ class KryoSerializer(conf: SparkConf)
private[spark
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570741
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -248,8 +266,9 @@ class KryoDeserializationStream(
}
}
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570738
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -219,9 +232,14 @@ class KryoSerializationStream(
private[spark]
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12913
**[Test build #67376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67376/consoleFull)**
for PR 12913 at commit
[`ed2c3a1`](https://github.com/apache/spark/commit/e
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570758
--- Diff:
core/src/test/scala/org/apache/spark/serializer/KryoBenchmark.scala ---
@@ -0,0 +1,133 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570771
--- Diff:
core/src/test/scala/org/apache/spark/serializer/KryoBenchmark.scala ---
@@ -0,0 +1,133 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12913#discussion_r84570784
--- Diff:
core/src/test/scala/org/apache/spark/serializer/KryoBenchmark.scala ---
@@ -0,0 +1,133 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12913
**[Test build #67377 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67377/consoleFull)**
for PR 12913 at commit
[`3977ccb`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15394
**[Test build #67374 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67374/consoleFull)**
for PR 15394 at commit
[`28f8c2f`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15394
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67374/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15394
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15585
**[Test build #67372 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67372/consoleFull)**
for PR 15585 at commit
[`9abd9ae`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15585
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67372/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15585
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/15566
> This PR makes partition spec case preserving
Sorry if there has already been a discussion on this. Why would any user
want this ? Its great that the engine can preserve cases but its j
Github user techaddict commented on the issue:
https://github.com/apache/spark/pull/12913
@rxin can you review again, all comments addressed ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12913
**[Test build #67378 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67378/consoleFull)**
for PR 12913 at commit
[`cdf535e`](https://github.com/apache/spark/commit/c
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/15597
[SPARK-18063][SQL] Failed to infer constraints over multiple aliases
## What changes were proposed in this pull request?
The `UnaryNode.getAliasedConstraints` function fails to replace
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15597
**[Test build #67379 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67379/consoleFull)**
for PR 15597 at commit
[`2fe728f`](https://github.com/apache/spark/commit/2
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/12913
LGTM pending Jenkins.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15573
**[Test build #67373 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67373/consoleFull)**
for PR 15573 at commit
[`4864672`](https://github.com/apache/spark/commit/
301 - 400 of 402 matches
Mail list logo