Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13291#discussion_r64699243
--- Diff: python/pyspark/ml/clustering.py ---
@@ -64,6 +64,21 @@ class GaussianMixture(JavaEstimator, HasFeaturesCol,
HasPredictionCol, HasMaxIte
..
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13291#discussion_r64699018
--- Diff: python/pyspark/ml/clustering.py ---
@@ -227,15 +242,15 @@ class KMeans(JavaEstimator, HasFeaturesCol,
HasPredictionCol, HasMaxIter, HasTol
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13311
---
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 pull request:
https://github.com/apache/spark/pull/13311#issuecomment-221792095
Merging in master/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
Github user zhengruifeng commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221792040
@holdenk Thanks. I think you are right. I will revert `an one-xxx` to `a
one-xxx`.
---
If your project is set up for it, you can reply to this email and have your
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13318#issuecomment-221791831
FYI "This PR also change the loadFactor of BytesToBytesMap to 0.5 (it was
0.75)" this is a pretty low load factor.
---
If your project is set up for it, you can reply to
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/13314#issuecomment-221791580
@srowen willing to help with that too btw :)
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/13318#issuecomment-221791386
Can we add a unit test for this behavior?
---
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
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/13234#discussion_r64698507
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -249,10 +249,16 @@ private[spark] class TaskSchedulerImpl(
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221791122
Also, your change seems to have made a few odd changes "an one way" which
sounds odd, generally "a one way" is considered sounding "better" (I'm a bit
fuzzy on the exac
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13311#issuecomment-221791056
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221791117
**[Test build #59352 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59352/consoleFull)**
for PR 13317 at commit
[`230c801`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13311#issuecomment-221791057
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/13176#discussion_r64698276
--- Diff: docs/ml-features.md ---
@@ -145,9 +148,11 @@ for more details on the API.
passed to other algorithms like LDA.
During the fitting p
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13311#issuecomment-221790919
**[Test build #59343 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59343/consoleFull)**
for PR 13311 at commit
[`94d6e7b`](https://g
Github user zhengruifeng commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221790963
@holdenk Thanks. I have fixed this. and run `lint-java` to check java file.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13319#issuecomment-221790817
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13319#issuecomment-221790798
**[Test build #59349 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59349/consoleFull)**
for PR 13319 at commit
[`8d1958e`](https://g
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/13176#discussion_r64698136
--- Diff: docs/ml-features.md ---
@@ -53,7 +53,10 @@ collisions, where different raw features may become the
same term after hashing.
chance of collision
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13319#issuecomment-221790816
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 projec
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/13317#discussion_r64698017
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -105,7 +105,7 @@ private[spark] abstract class MapOutputTracker(conf:
SparkConf)
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221790462
So it seems that in a few places adding the extra character has pushed it
over the 100. You should probably run the linter explicitly if you have it
disabled by default
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13316#issuecomment-221790073
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13316#issuecomment-221790065
**[Test build #59348 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59348/consoleFull)**
for PR 13316 at commit
[`325a2ea`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13316#issuecomment-221790072
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221789832
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221789830
**[Test build #59351 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59351/consoleFull)**
for PR 13317 at commit
[`cff3aa8`](https://g
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13319#issuecomment-221789707
cc @cloud-fan @andrewor14
---
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 th
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221789833
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13319#discussion_r64697510
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala
---
@@ -178,12 +180,14 @@ class SparkSession private(
def udf: UDFRegistration
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/13318#issuecomment-221789570
cc @ericl
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13318#issuecomment-221789635
**[Test build #59350 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59350/consoleFull)**
for PR 13318 at commit
[`6d074f6`](https://gi
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/13319
[SPARK-15552][SQL] Remove unnecessary private[sql] methods in SparkSession
## What changes were proposed in this pull request?
SparkSession has a list of unnecessary private[sql] methods. These met
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13319#issuecomment-221789649
**[Test build #59349 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59349/consoleFull)**
for PR 13319 at commit
[`8d1958e`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13317#issuecomment-221789634
**[Test build #59351 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59351/consoleFull)**
for PR 13317 at commit
[`cff3aa8`](https://gi
GitHub user davies opened a pull request:
https://github.com/apache/spark/pull/13318
[SPARK-15391] [SQL] manage the temporary memory of timsort
## What changes were proposed in this pull request?
Currently, the memory for temporary buffer used by TimSort is always
allocated
GitHub user zhengruifeng opened a pull request:
https://github.com/apache/spark/pull/13317
[MINOR] Fix Typos
## What changes were proposed in this pull request?
`a` -> `an`
## How was this patch tested?
local build
You can merge this p
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/13216#discussion_r64697298
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/TypedFilterOptimizationSuite.scala
---
@@ -34,40 +35,47 @@ class TypedFilter
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13316#issuecomment-221788985
**[Test build #59348 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59348/consoleFull)**
for PR 13316 at commit
[`325a2ea`](https://gi
GitHub user holdenk opened a pull request:
https://github.com/apache/spark/pull/13316
[SPARK-15551][MINOR][DOCS][SQL] Replace groupBy with groupByKey in
KeyValueGroupedDataset Scaladoc
## What changes were proposed in this pull request?
Replace groupBy with groupByKey in Ke
Github user xwu0226 commented on the pull request:
https://github.com/apache/spark/pull/13088#issuecomment-221788192
@rxin @andrewor14 @cloud-fan Please help review! Thanks!
---
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 wangmiao1981 commented on the pull request:
https://github.com/apache/spark/pull/13284#issuecomment-221786544
@shivaram I will create a JIRA soon. Thursday and Friday, I will be on
travel to NYC. Will do it on Saturday.
---
If your project is set up for it, you can reply
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12836#issuecomment-221786353
**[Test build #59347 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59347/consoleFull)**
for PR 12836 at commit
[`9cacd4d`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13283#issuecomment-221786340
**[Test build #59346 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59346/consoleFull)**
for PR 13283 at commit
[`b9e12f8`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12836#issuecomment-221785096
**[Test build #59345 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59345/consoleFull)**
for PR 12836 at commit
[`0928740`](https://gi
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13300#discussion_r64694941
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVRelation.scala
---
@@ -142,6 +145,75 @@ object CSVRelation extends Loggi
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/13165#discussion_r64694857
--- Diff: R/pkg/inst/tests/testthat/test_includeJAR.R ---
@@ -21,10 +21,13 @@ runScript <- function() {
sparkTestJarPath <- "R/lib/SparkR/test_support/
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13300#discussion_r64694834
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -42,16 +42,23 @@ private[csv] object CSVInferSc
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-221784338
on a related note, @squito can you in the future leave a msg indicating the
branch a pr was merged once you merge it? There have been cases that lead to
race conditions in
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/8559#issuecomment-221784058
This is pretty cool!
---
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 fea
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13234#issuecomment-221783307
**[Test build #59344 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59344/consoleFull)**
for PR 13234 at commit
[`8f2534b`](https://gi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8559
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13310#issuecomment-221780558
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13310#issuecomment-221780560
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13310#issuecomment-221780470
**[Test build #59333 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59333/consoleFull)**
for PR 13310 at commit
[`f40a898`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13290#issuecomment-221777604
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13290#issuecomment-221777607
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13290#issuecomment-221777507
**[Test build #59334 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59334/consoleFull)**
for PR 13290 at commit
[`127024d`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13311#issuecomment-221777001
**[Test build #59343 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59343/consoleFull)**
for PR 13311 at commit
[`94d6e7b`](https://gi
Github user WeichenXu123 closed the pull request at:
https://github.com/apache/spark/pull/13313
---
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
Github user ueshin commented on the pull request:
https://github.com/apache/spark/pull/13235#issuecomment-221776141
It looks like #12351 is the same issue about whole stage codegen with
`splitExpressions`.
---
If your project is set up for it, you can reply to this email and have you
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13283#issuecomment-221775767
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13283#issuecomment-221775766
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13283#issuecomment-221775683
**[Test build #59332 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59332/consoleFull)**
for PR 13283 at commit
[`76f4f80`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221775528
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221775529
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221775481
**[Test build #59342 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59342/consoleFull)**
for PR 13308 at commit
[`cbd5163`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-221774354
**[Test build #59341 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59341/consoleFull)**
for PR 9192 at commit
[`f67095e`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-221774407
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-221774409
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221774053
**[Test build #59342 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59342/consoleFull)**
for PR 13308 at commit
[`cbd5163`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13296#issuecomment-221773158
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13296#issuecomment-221773157
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13296#issuecomment-221773071
**[Test build #59329 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59329/consoleFull)**
for PR 13296 at commit
[`367e3b8`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221773031
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221773032
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221772977
**[Test build #59340 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59340/consoleFull)**
for PR 13308 at commit
[`88319c0`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13165#issuecomment-221772828
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 projec
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-221772896
Thanks for the update. LGTM. Will merge after Jenkins passes.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13165#issuecomment-221772829
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13165#issuecomment-221772778
**[Test build #59339 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59339/consoleFull)**
for PR 13165 at commit
[`0482ebb`](https://g
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13284
---
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 shivaram commented on the pull request:
https://github.com/apache/spark/pull/13284#issuecomment-221772558
Yeah thats a good idea @wangmiao1981 can you open a JIRA to not mask
`startsWith` and `endsWith` by updating our generics ?
LGTM - Merging this to master and
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-221772591
**[Test build #59341 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59341/consoleFull)**
for PR 9192 at commit
[`f67095e`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221772521
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221772519
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221772475
**[Test build #59338 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59338/consoleFull)**
for PR 13308 at commit
[`07806de`](https://g
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/13284#issuecomment-221772080
looks fine - I think we should really try to make startsWith and endsWith
work though, but that could be a follow up.
---
If your project is set up for it, you can
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221771654
**[Test build #59340 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59340/consoleFull)**
for PR 13308 at commit
[`88319c0`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13308#issuecomment-221771180
**[Test build #59338 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59338/consoleFull)**
for PR 13308 at commit
[`07806de`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13165#issuecomment-221771183
**[Test build #59339 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59339/consoleFull)**
for PR 13165 at commit
[`0482ebb`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/13283#issuecomment-221770911
**Update**: The latest code changes contains
- For JDBC format, we added an extra checking in the rule
`ResolveRelations` of `Analyzer`. Without the PR, Spark wi
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13165#issuecomment-221770814
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 thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13165#issuecomment-221770536
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13165#issuecomment-221770524
**[Test build #59336 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59336/consoleFull)**
for PR 13165 at commit
[`0482ebb`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13165#issuecomment-221770538
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-221770384
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-221770383
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9192#issuecomment-221770380
**[Test build #59337 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59337/consoleFull)**
for PR 9192 at commit
[`90641a7`](https://git
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/13290#discussion_r64688437
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1448,6 +1450,37 @@ class Analyzer(
}
1 - 100 of 693 matches
Mail list logo