Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20454
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20454
**[Test build #86914 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86914/testReport)**
for PR 20454 at commit
Github user ssaavedra commented on the issue:
https://github.com/apache/spark/pull/20383
Yes, sorry for the misunderstanding I was also probably too eager with
this. However, if the changes I'm stating up there don't work, I am not sure
what I'm missing now. I'll take a further look
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20471#discussion_r165288359
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/MutableColumnarRow.java
---
@@ -162,8 +162,9 @@ public ColumnarArray
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20455
**[Test build #86923 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86923/testReport)**
for PR 20455 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20468
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20455
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/472/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20455
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20455
@ueshin Yes, missing it. Thanks. I'll add it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20400
Yup, we could also string as a column but I was thinking of matching the
signature with the Scala one for now, just for consistency ..
On 1 Feb 2018 5:24 pm, "Liang-Chi Hsieh"
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20455
We also need to add `if (isNullAt(rowId)) return null;` to
`WritableColumnVector.getMap()`?
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20471
**[Test build #86922 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86922/testReport)**
for PR 20471 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20471
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/471/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20471
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20471
cc @cloud-fan @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/20471
[SPARK-23280][SQL][FOLLOWUP] Enable `MutableColumnarRow.getMap()`.
## What changes were proposed in this pull request?
This is a followup pr of #20450.
We should've enabled
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20400#discussion_r165284328
--- Diff: python/pyspark/sql/window.py ---
@@ -208,20 +236,27 @@ def rangeBetween(self, start, end):
and "5" means the five off after the
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20400#discussion_r165284238
--- Diff: python/pyspark/sql/window.py ---
@@ -120,20 +122,46 @@ def rangeBetween(start, end):
and "5" means the five off after the current row.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19219
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19219
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86911/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19219
**[Test build #86911 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86911/testReport)**
for PR 19219 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20400
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86919/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20470
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20400
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20400
**[Test build #86919 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86919/testReport)**
for PR 20400 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20470
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user gerashegalov opened a pull request:
https://github.com/apache/spark/pull/20470
[SPARK-23296][YARN] Include stacktrace in YARN-app diagnostic
## What changes were proposed in this pull request?
Include stacktrace in the diagnostics message upon abnormal
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20450
@ueshin Ok. No problem. :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20450
@viirya Thanks! but I'm working on it. I'll do it soon.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20450
I found that we don't enable `getMap` API in `MutableColumnarRow` in this
change, should we do it? If so, I can make a small follow-up PR for it.
---
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/18136
I just came across this expression and I have a few concerns:
1. A row will not get the same UUID assigned when a task fails. This might
cause some really weird problems when the UUID
601 - 631 of 631 matches
Mail list logo