Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19472
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82634/
Test FAILed.
---
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/18924
@akopich LGTM. and do you have time to create a PR to resolve random seed
not working issue mentioned by @hhbyyh ? Thanks!
---
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19472
**[Test build #82636 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82636/testReport)**
for PR 19472 at commit
[`0738359`](https://github.com/apache/spark/commit/07
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18979
I don't have strong opinion against this. Incorrect size is an issue but I
can't think a better solution for now...
---
-
To unsu
Github user a10y commented on a diff in the pull request:
https://github.com/apache/spark/pull/18931#discussion_r144025706
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -175,6 +175,25 @@ trait CodegenSupport extends SparkPlan {
GitHub user ala opened a pull request:
https://github.com/apache/spark/pull/19473
[SPARK-22251] Metric 'aggregate time' is incorrect when codegen is off
## What changes were proposed in this pull request?
Adding the code for setting 'aggregate time' metric to non-codegen pat
Github user susanxhuynh commented on the issue:
https://github.com/apache/spark/pull/19437
@skonto I haven't tested with TLS; I'll work on that in the next couple of
days.
---
-
To unsubscribe, e-mail: reviews-unsub
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19473
**[Test build #82637 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82637/testReport)**
for PR 19473 at commit
[`470b54f`](https://github.com/apache/spark/commit/47
Github user susanxhuynh commented on the issue:
https://github.com/apache/spark/pull/19437
@vanzin Would you mind reviewing this PR? A followup to ArtRand's secrets
PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user zivanfi commented on the issue:
https://github.com/apache/spark/pull/19250
@attilajeges has just found a problem with the behavior specified in the
requirements:
* Partitions of a table can use different file formats.
* As a result, a single table can have data file
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19464#discussion_r144031167
--- Diff: core/src/test/scala/org/apache/spark/FileSuite.scala ---
@@ -510,4 +510,16 @@ class FileSuite extends SparkFunSuite with
LocalSparkContext {
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19464#discussion_r144030646
--- Diff: core/src/main/scala/org/apache/spark/rdd/HadoopRDD.scala ---
@@ -196,7 +196,10 @@ class HadoopRDD[K, V](
// add the credentials here a
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19464#discussion_r144031728
--- Diff: core/src/test/scala/org/apache/spark/FileSuite.scala ---
@@ -510,4 +510,16 @@ class FileSuite extends SparkFunSuite with
LocalSparkContext {
Github user ala commented on the issue:
https://github.com/apache/spark/pull/19473
@hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19464#discussion_r144029852
--- Diff: docs/configuration.md ---
@@ -1211,6 +1211,14 @@ Apart from these, the following properties are also
available, and may be useful
dat
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19316#discussion_r144034339
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -388,7 +388,13 @@ private[spark] class MemoryStore(
// p
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19464#discussion_r144030461
--- Diff: core/src/main/scala/org/apache/spark/rdd/HadoopRDD.scala ---
@@ -196,7 +196,10 @@ class HadoopRDD[K, V](
// add the credentials here a
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19456
Could you elaborate on the scenario that you should need to make these
settings configurable?
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18386#discussion_r144037286
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileFormatWriter.scala
---
@@ -111,9 +111,18 @@ object FileFormatWriter exte
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19077#discussion_r144037771
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeArrayWriter.java
---
@@ -57,7 +57,7 @@ public void initial
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19077#discussion_r144037069
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -48,6 +49,15 @@ public long size() {
}
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19077#discussion_r144037194
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -48,6 +49,15 @@ public long size() {
}
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19077#discussion_r144038007
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -116,9 +116,10 @@ private [sql] object
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/19474
[SPARK-22252][SQL] FileFormatWriter should respect the input query schema
## What changes were proposed in this pull request?
In https://github.com/apache/spark/pull/18064, we allowed `Ru
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19474
**[Test build #82638 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82638/testReport)**
for PR 19474 at commit
[`3b1174f`](https://github.com/apache/spark/commit/3b
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/19473
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.ap
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/19473
Could you add [SQL] to the title? That makes it easier for others to scan
PRs.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user brad-kaiser commented on the issue:
https://github.com/apache/spark/pull/19041
Thanks @vanzin I will work on these comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addition
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/18979
@viirya : the new data writer API will allow for a broader set of stats to
be propagated back from workers. When you are working with the object stores,
an useful stat to get back is throttle
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/19468
@skonto, there was some discussion about this on the SPIP. We see them as
separate and independent issues with the pluggable API being a long term goal.
It would involve a working group of people dr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19474
**[Test build #82639 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82639/testReport)**
for PR 19474 at commit
[`0667ac8`](https://github.com/apache/spark/commit/06
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19474
cc @gatorsmile @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19474
For a simple command `Seq(1 -> "a").toDF("i",
"j").write.parquet("/tmp/qwe")`, the UI before this PR:
https://user-images.githubusercontent.com/3182036/31452520-bc74bb44-aee1-11e7-8721-23492585
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/19337#discussion_r144060858
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala ---
@@ -224,6 +224,24 @@ private[clustering] trait LDAParams extends Params
with HasFe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19474
**[Test build #82640 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82640/testReport)**
for PR 19474 at commit
[`9d4c7a2`](https://github.com/apache/spark/commit/9d
Github user PerilousApricot commented on the issue:
https://github.com/apache/spark/pull/18098
Is there any documentation for this feature? How would I expose my topology
to mesos/spark?
---
-
To unsubscribe, e-mail
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144065041
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetCommitterSuite.scala
---
@@ -0,0 +1,149 @@
+/*
+
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19472
**[Test build #82636 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82636/testReport)**
for PR 19472 at commit
[`0738359`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19472
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82636/
Test FAILed.
---
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144065074
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,13 @@ class ParquetFi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19472
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144065810
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,13 @@ class ParquetFi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18664
**[Test build #82641 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82641/testReport)**
for PR 18664 at commit
[`d1617fd`](https://github.com/apache/spark/commit/d1
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/18664
Thanks @ueshin , I agree it is better to convert the timezone to Python
system local first and then localize to make tz-naive in case the Python system
local tz is different that `DateTimeUtils.
Github user blyncsy-david-lewis commented on the issue:
https://github.com/apache/spark/pull/19456
I have a multiuser application where I use the userId as the name of the
scheduling pool so that users are balanced equally by spark and within a user's
workload I can set the scheduling
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19472
**[Test build #82642 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82642/testReport)**
for PR 19472 at commit
[`a814eb3`](https://github.com/apache/spark/commit/a8
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19466
> here with this change all the resources should be fetched from local
driver
That's a good point. You should download resources just to add them to the
driver's classpath, but executors can
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19448
**[Test build #82643 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82643/testReport)**
for PR 19448 at commit
[`d634f9e`](https://github.com/apache/spark/commit/d6
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19471
+1 for this change. BTW, wow, there are lots of test case failures: 81
failures.
---
-
To unsubscribe, e-mail: reviews-uns
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18460
Please let me know if there is something to do more~ Thank you always,
@gatorsmile .
---
-
To unsubscribe, e-mail: reviews
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19473
**[Test build #82637 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82637/testReport)**
for PR 19473 at commit
[`470b54f`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19473
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82637/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19473
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18460#discussion_r144075681
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -100,6 +101,17 @@ object TypeCoercion {
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/18460#discussion_r144078834
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -100,6 +101,17 @@ object TypeCoercion {
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/18805
The
[code](https://github.com/luben/zstd-jni/blob/master/src/main/java/com/github/luben/zstd/util/Native.java)
overwrites the original exception message that might shed some light on what's
going on
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/18805
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18805
**[Test build #82644 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82644/testReport)**
for PR 18805 at commit
[`029a753`](https://github.com/apache/spark/commit/02
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19429
+1 for more detailed documentation (we should steer away from `inferSchema`)
---
-
To unsubscribe, e-mail: reviews-unsubscr..
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/18460#discussion_r144082145
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -100,6 +101,17 @@ object TypeCoercion {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144082508
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,10 @@ class ParquetFileF
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/19467
cc @tdas
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/19468
> iron out the kinks
A large chunk of the difficulty in identifying and ironing out kinks in
such a project is the difficulty of writing adequate tests of the scheduler
code. I'd expect
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19474
**[Test build #82638 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82638/testReport)**
for PR 19474 at commit
[`3b1174f`](https://github.com/apache/spark/commit/3
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144086925
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetCommitterSuite.scala
---
@@ -0,0 +1,152 @@
+/*
+ * L
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19474
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19474
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82638/
Test PASSed.
---
Github user swaapnika-guntaka commented on the issue:
https://github.com/apache/spark/pull/30
I see the Java EOF Exception when I run python packaged jar(using JDK 8)
using Spark-2.2
I'm trying to run this using the below command.
`time bash -x $SPARK_HOME/bin/spark-submit --dr
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144088592
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,10 @@ class ParquetFileForma
Github user akopich commented on the issue:
https://github.com/apache/spark/pull/18924
@WeichenXu123, yes sure. But can this wait until this PR is merged?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apac
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18979#discussion_r144094437
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/BasicWriteStatsTracker.scala
---
@@ -57,7 +60,14 @@ class BasicWriteTaskSta
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19474
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82639/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19474
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18386#discussion_r144097084
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileFormatWriter.scala
---
@@ -111,9 +111,18 @@ object FileFormatWriter ext
Github user sathiyapk commented on the issue:
https://github.com/apache/spark/pull/19451
@gengliangwang Ready for a next review :)
> put case ... in a new line
Are your sure? I thought according to the coding style, while calling on a
partial function if there is only
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19474
**[Test build #82639 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82639/testReport)**
for PR 19474 at commit
[`0667ac8`](https://github.com/apache/spark/commit/0
Github user loneknightpy closed the pull request at:
https://github.com/apache/spark/pull/19466
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.o
Github user loneknightpy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19466#discussion_r144100747
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -366,6 +366,16 @@ object SparkSubmit extends CommandLineUtils with
Loggi
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/19269#discussion_r144097061
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/DataSourceV2Writer.java
---
@@ -0,0 +1,88 @@
+/*
+ * Licensed to the Apache
Github user joseph-torres commented on the issue:
https://github.com/apache/spark/pull/19465
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19448
**[Test build #82643 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82643/testReport)**
for PR 19448 at commit
[`d634f9e`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19448
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82643/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19448
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19451
**[Test build #82645 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82645/testReport)**
for PR 19451 at commit
[`5ca1e88`](https://github.com/apache/spark/commit/5c
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144105439
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,10 @@ class ParquetFileF
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19424#discussion_r144109443
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceReaderHolder.scala
---
@@ -0,0 +1,64 @@
+/*
+ * Licens
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18664
**[Test build #82641 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82641/testReport)**
for PR 18664 at commit
[`d1617fd`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19474
**[Test build #82640 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82640/testReport)**
for PR 19474 at commit
[`9d4c7a2`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18664
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19474
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82640/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18664
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82641/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19474
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19424#discussion_r144111358
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceReaderHolder.scala
---
@@ -0,0 +1,64 @@
+/*
+ * Licens
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19472
**[Test build #82642 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82642/testReport)**
for PR 19472 at commit
[`a814eb3`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19472
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19472
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82642/
Test FAILed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18979
LGTM except a minor comment.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19470
Could you create test cases with the different schemas between files and
hive metastore.
---
-
To unsubscribe, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19270
**[Test build #82646 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82646/testReport)**
for PR 19270 at commit
[`f649e42`](https://github.com/apache/spark/commit/f6
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144119701
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,10 @@ class ParquetFileForma
101 - 200 of 396 matches
Mail list logo