Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/15659
@viirya: the pip install sanity check is done assuming it was installed
into an activated virtual env of just that Python version. You can see how to
use it in 'run-pip-tests-2'. If you have multipl
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15047
One testing technique we have used internally at Databricks (not for Spark)
is to use random data generator to generate a bunch of data, and run through
the reference implementation to get the results,
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/15276
@davies @rxin Should I send a PR to backport this to `branch-2.0` if you
don't have time?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/15807
[SPARK-18147][SQL] do not fail for very complex aggregator result type
## What changes were proposed in this pull request?
In `TypedAggregateExpression.evaluateExpression`, we may create
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15276
Please do! I was hoping you'd do 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 have this feature
enabl
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15668
**[Test build #68323 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68323/consoleFull)**
for PR 15668 at commit
[`6e58167`](https://github.com/apache/spark/commit/
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/15276
Oh, I see, wait for a while.
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15668
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/15668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68323/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15807
**[Test build #68331 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68331/consoleFull)**
for PR 15807 at commit
[`3917630`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15717
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
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15717
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68324/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15717
**[Test build #68324 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68324/consoleFull)**
for PR 15717 at commit
[`cba5bbd`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15807
Sorry I haven't noticed that https://github.com/apache/spark/pull/15693 was
merged. Then this PR becomes a cleanup, not a bug fix. But I'd like to keep the
regression test as it's from another JIR
Github user wzhfy closed the pull request at:
https://github.com/apache/spark/pull/15012
---
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 enab
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/15012
The issue is resolved after merging
https://github.com/apache/spark/pull/15024, so I'll close this pr.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15717
**[Test build #68327 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68327/consoleFull)**
for PR 15717 at commit
[`fda6d3a`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15717
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/15717
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68327/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15413
**[Test build #68332 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68332/consoleFull)**
for PR 15413 at commit
[`b2c2fa0`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15797
**[Test build #68329 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68329/consoleFull)**
for PR 15797 at commit
[`dddee47`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15797
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68329/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15797
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 WangTaoTheTonic commented on the issue:
https://github.com/apache/spark/pull/15803
thanks for the fix.
This patch parse the timestamp instead of the Date String returned. The
REST api still return the GMT time, which is insistent with UI showing.
I've goog
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/15808
[SPARK-17703][SQL][BACKPORT-2.0] Add unnamed version of addReferenceObj for
minor objects.
## What changes were proposed in this pull request?
This is a backport of #15276.
There a
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15797#discussion_r86946976
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala
---
@@ -591,25 +673,25 @@ abstract class ExternalCa
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/15276
I sent a new PR #15808. Thanks.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15808
**[Test build #68333 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68333/consoleFull)**
for PR 15808 at commit
[`140f929`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15717
**[Test build #68335 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68335/consoleFull)**
for PR 15717 at commit
[`0097809`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15807
**[Test build #68334 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68334/consoleFull)**
for PR 15807 at commit
[`0103bb4`](https://github.com/apache/spark/commit/0
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/15413#discussion_r86947971
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/GaussianMixture.scala ---
@@ -350,6 +458,145 @@ object GaussianMixture extends
DefaultParam
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/15413#discussion_r86948383
--- Diff: python/pyspark/ml/clustering.py ---
@@ -95,15 +95,10 @@ class GaussianMixture(JavaEstimator, HasFeaturesCol,
HasPredictionCol, HasMaxIte
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/15413#discussion_r86948993
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/clustering/GaussianMixtureSuite.scala
---
@@ -130,4 +230,29 @@ object GaussianMixtureSuite {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68326 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68326/consoleFull)**
for PR 15751 at commit
[`bae8db8`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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/15751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68326/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15805
**[Test build #68328 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68328/consoleFull)**
for PR 15805 at commit
[`0c18e27`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15805
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/15805
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68328/
Test PASSed.
---
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/15413
@sethah Yeah, I totally agree we can get around the issues I mentioned and
make ```mllib.GaussianMixture``` call into ```ml.GaussianMixture```. But I'd
prefer more to the current approach based o
Github user mozinrat commented on the issue:
https://github.com/apache/spark/pull/12162
@Earne is something relevant merged in spark 2.0.1, do we have FIFO
eviction policy?.
If yes how can I leverage it?
---
If your project is set up for it, you can reply to this email and have y
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15413
**[Test build #68332 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68332/consoleFull)**
for PR 15413 at commit
[`b2c2fa0`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15413
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68332/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15413
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/15797
**[Test build #68336 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68336/consoleFull)**
for PR 15797 at commit
[`f85bb27`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15807
**[Test build #68331 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68331/consoleFull)**
for PR 15807 at commit
[`3917630`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15807
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15769
**[Test build #68330 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68330/consoleFull)**
for PR 15769 at commit
[`9380f91`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15769
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68330/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15769
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 jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15484
ping @hvanhovell
---
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 scwf commented on the issue:
https://github.com/apache/spark/pull/15297
@YuhuWang2002
We should limit the use case for outer join:
For left outer join, such as A left join B, this implementation now can not
handle the case of skew of table B. That's because the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15807
**[Test build #68334 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68334/consoleFull)**
for PR 15807 at commit
[`0103bb4`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15807
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68334/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15807
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15808
**[Test build #68333 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68333/consoleFull)**
for PR 15808 at commit
[`140f929`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15808
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68333/
Test PASSed.
---
Github user windpiger commented on the issue:
https://github.com/apache/spark/pull/15803
UTC timestamp can translate to different zone's local time,which the user
see from the history ui.
It is reasonable to return UTC/GMT time to the user browser, and show it by
translate to the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15808
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/15717
**[Test build #68335 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68335/consoleFull)**
for PR 15717 at commit
[`0097809`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15717
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/15717
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68335/
Test FAILed.
---
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15745
Merging to master/2.1. Thanks!
---
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 a
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15745
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15668
LGTM. Merging to master/2.1. Thanks!
---
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
ena
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15668
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15668
@windpiger do you have a JIRA username? So I can credit you on the JIRA.
---
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 pr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15797
**[Test build #68336 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68336/consoleFull)**
for PR 15797 at commit
[`f85bb27`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15797
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/15797
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68336/
Test FAILed.
---
Github user windpiger commented on the issue:
https://github.com/apache/spark/pull/15668
@hvanhovell username is Song Jun
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15415
**[Test build #68337 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68337/consoleFull)**
for PR 15415 at commit
[`e5574be`](https://github.com/apache/spark/commit/e
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/12004
I had something tangible: the integration tests. It's clear those aren't
wanted. Now I'm proposing something more minimal, yet still tangible for anyone
trying to build spark such that it work
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/11105
@rxin: we could, although we would need to either break binary
comparability or have some dummy implementations for addImpl so 2.0
accumulators can override add and not provide addImpl.
---
If you
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15415
**[Test build #68337 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68337/consoleFull)**
for PR 15415 at commit
[`e5574be`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15415
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/15415
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68337/
Test PASSed.
---
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15722
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 wishes so
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15722
**[Test build #68338 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68338/consoleFull)**
for PR 15722 at commit
[`7a95697`](https://github.com/apache/spark/commit/7
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15798
I removed the entry in `NOTICE`. Interestingly we're supposed to preserve
the NOTICE file contents of projects that are redistributed and this comes from
Hive. However I suspect it is not required to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15798
**[Test build #68339 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68339/consoleFull)**
for PR 15798 at commit
[`34c55b8`](https://github.com/apache/spark/commit/3
Github user busbey commented on the issue:
https://github.com/apache/spark/pull/15798
That interpretation lines up with my understanding; we only need to
recreate the portions of NOTICE that are applicable.
---
If your project is set up for it, you can reply to this email and have yo
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/11105
Let me know which approach you prefer @rxin - initially this went with the
break binary compat approach then @squito suggested the sub-class approach, but
I'm fine for whichever approach we want to
Github user oza commented on the issue:
https://github.com/apache/spark/pull/15806
@brkyvz Thanks for taking a look. The problem I faced is: without HDFS
configuration, the [quick
example](https://spark.apache.org/docs/latest/structured-streaming-programming-guide.html#quick-example)
Github user oza commented on a diff in the pull request:
https://github.com/apache/spark/pull/15806#discussion_r86987111
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/StreamingQueryManager.scala
---
@@ -219,10 +219,11 @@ class StreamingQueryManager private[sql]
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/15480
@hvanhovell it'd be great if you can take a look at this, thanks!
---
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
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/15779#discussion_r86989031
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala ---
@@ -404,6 +406,13 @@ object LinearRegression extends
DefaultPara
Github user windpiger commented on the issue:
https://github.com/apache/spark/pull/15803
@ajbozarth could you help to review this? Thanks!
---
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
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15803
General comment: we should always render times with a timezone for clarity,
even if it's in a column header. This has been the source of so many problems
I've seen over the years.
---
If your proje
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15658
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/15658
Merged to master/2.1
---
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 oza commented on a diff in the pull request:
https://github.com/apache/spark/pull/15806#discussion_r86991525
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/StreamingQueryManager.scala
---
@@ -219,10 +219,11 @@ class StreamingQueryManager private[sql]
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/15806#discussion_r86991593
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/StreamingQueryManager.scala
---
@@ -219,10 +219,11 @@ class StreamingQueryManager private[sql
Github user oza commented on a diff in the pull request:
https://github.com/apache/spark/pull/15806#discussion_r86993123
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/StreamingQueryManager.scala
---
@@ -219,10 +219,11 @@ class StreamingQueryManager private[sql]
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15807
There is probably a bug of common subexpression elimination: we will
evalute all subexpressions at the very beginning, no matter the results of
subexpressions will be used or not. A counter exampl
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15783
Merged to master
---
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 i
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/15563
Jenkins, test 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 wis
Github user windpiger commented on the issue:
https://github.com/apache/spark/pull/15803
@srowen UTC timestamp(long) is the standard time, no matter it is
translated to which timezone, the long value is the same. But the readable date
string should be nicely adapter to the user's brow
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15783
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/15789
I'll merge if there are no further comments. To restate -- basically the
test helper function returns a path that is perfectly fine for reading test
data, and is flexible because it doesn't depend on
1 - 100 of 560 matches
Mail list logo