[GitHub] spark pull request: [SPARK-10359] Enumerate dependencies in a file...

2015-12-24 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10461#issuecomment-167161664 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

[GitHub] spark pull request: [SPARK-12522] [SQL] [MINOR] Add the missing th...

2015-12-24 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/10471 [SPARK-12522] [SQL] [MINOR] Add the missing the document strings for the SQL configuration Fixing the missing the document for the configuration. We can see the missing messages "TODO" when

[GitHub] spark pull request: [SPARK-12521][SQL][WIP] JDBCRelation does not ...

2015-12-24 Thread xguo27
GitHub user xguo27 opened a pull request: https://github.com/apache/spark/pull/10473 [SPARK-12521][SQL][WIP] JDBCRelation does not honor lowerBound/upperBound JDBCRelation is not bounding the rows when lowerBound/upperBound are given. This change honors the bounds given. You can

[GitHub] spark pull request: [SPARK-8676][SQL]Lazy start event logger in sq...

2015-12-24 Thread WangTaoTheTonic
Github user WangTaoTheTonic closed the pull request at: https://github.com/apache/spark/pull/8048 --- 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

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167162791 **[Test build #48315 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48315/consoleFull)** for PR 10388 at commit

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167162799 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167162798 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

[GitHub] spark pull request: [SPARK-10359] Enumerate dependencies in a file...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10461#issuecomment-167167484 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

[GitHub] spark pull request: [SPARK-10359] Enumerate dependencies in a file...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10461#issuecomment-167167489 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12504][SQL] Masking credentials in the ...

2015-12-24 Thread sureshthalamati
Github user sureshthalamati commented on the pull request: https://github.com/apache/spark/pull/10452#issuecomment-167167658 Thank you very much for reviewing the patch Michael , Sean. @Michael : I agree with you long unnecessary strings in the plan output is annoying. If

[GitHub] spark pull request: [SPARK-10359] Enumerate dependencies in a file...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10461#issuecomment-167167270 **[Test build #48316 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48316/consoleFull)** for PR 10461 at commit

[GitHub] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10451#discussion_r48432615 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -153,6 +153,15 @@ object SetOperationPushDown

[GitHub] spark pull request: [SPARK-9716] [ML] BinaryClassificationEvaluato...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10472#issuecomment-167166797 **[Test build #48318 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48318/consoleFull)** for PR 10472 at commit

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167180493 **[Test build #48319 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48319/consoleFull)** for PR 10388 at commit

[GitHub] spark pull request: [SPARK-10263] [ML] Add @Since annotation to ml...

2015-12-24 Thread taishi-oss
Github user taishi-oss commented on the pull request: https://github.com/apache/spark/pull/8935#issuecomment-167196430 I modified some mistakes that was pointed out in #8534. @mengxr Could you take a look? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/10451#discussion_r48432925 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -153,6 +153,15 @@ object SetOperationPushDown

[GitHub] spark pull request: [SPARK-12522] [SQL] [MINOR] Add the missing th...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10471#issuecomment-167165557 **[Test build #48317 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48317/consoleFull)** for PR 10471 at commit

[GitHub] spark pull request: [SPARK-12522] [SQL] [MINOR] Add the missing th...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10471#issuecomment-167172286 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12522] [SQL] [MINOR] Add the missing th...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10471#issuecomment-167172267 **[Test build #48317 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48317/consoleFull)** for PR 10471 at commit

[GitHub] spark pull request: [SPARK-12522] [SQL] [MINOR] Add the missing th...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10471#issuecomment-167172285 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

[GitHub] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10451#issuecomment-167203391 **[Test build #48320 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48320/consoleFull)** for PR 10451 at commit

[GitHub] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10451#issuecomment-167203394 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10451#issuecomment-167203393 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

[GitHub] spark pull request: [SPARK-12316] Wait a minutes to avoid cycle ca...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10475#issuecomment-167204565 **[Test build #48321 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48321/consoleFull)** for PR 10475 at commit

[GitHub] spark pull request: [SPARK-10359] Enumerate dependencies in a file...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10461#issuecomment-167162372 **[Test build #48316 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48316/consoleFull)** for PR 10461 at commit

[GitHub] spark pull request: [SPARK-9716] [ML] BinaryClassificationEvaluato...

2015-12-24 Thread BenFradet
GitHub user BenFradet opened a pull request: https://github.com/apache/spark/pull/10472 [SPARK-9716] [ML] BinaryClassificationEvaluator should accept Double prediction column This PR aims to allow the prediction column of `BinaryClassificationEvaluator` to be of double type. You

[GitHub] spark pull request: [SPARK-5095][MESOS] Support capping cores and ...

2015-12-24 Thread rnowling
Github user rnowling commented on the pull request: https://github.com/apache/spark/pull/4027#issuecomment-167166741 It's really unfortunate that this patch was closed without merging. I disagree with @andrewor14 and others that it exposes to much to the average user. It's much

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167160783 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167160782 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

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167160752 **[Test build #48314 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48314/consoleFull)** for PR 10456 at commit

[GitHub] spark pull request: [SPARK-9716] [ML] BinaryClassificationEvaluato...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10472#issuecomment-167169467 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9716] [ML] BinaryClassificationEvaluato...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10472#issuecomment-167169466 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

[GitHub] spark pull request: [SPARK-9716] [ML] BinaryClassificationEvaluato...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10472#issuecomment-167169451 **[Test build #48318 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48318/consoleFull)** for PR 10472 at commit

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

2015-12-24 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10330#issuecomment-167185431 @cloud-fan any other comments? --- 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

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167202451 > I have committed the proposed change which moves setName after the map. I've tested locally and now the name field is valid. > > does it make sense ?

[GitHub] spark pull request: [SPARK-12316] Wait a minutes to avoid cycle ca...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10475#issuecomment-167205800 **[Test build #48321 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48321/consoleFull)** for PR 10475 at commit

[GitHub] spark pull request: [SPARK-12316] Wait a minutes to avoid cycle ca...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10475#issuecomment-167205989 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

[GitHub] spark pull request: [SPARK-12316] Wait a minutes to avoid cycle ca...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10475#issuecomment-167205992 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12436][SQL] Now InferSchema should retu...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10474#issuecomment-167198456 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

[GitHub] spark pull request: [SPARK-8676][SQL]Lazy start event logger in sq...

2015-12-24 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on the pull request: https://github.com/apache/spark/pull/8048#issuecomment-167198843 No prob. will close this. --- 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] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10451#issuecomment-167203168 **[Test build #48320 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48320/consoleFull)** for PR 10451 at commit

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread kevinyu98
Github user kevinyu98 commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167179672 I delete the test cases from DataFrameNaFunctionsSuite.scala. I checked the previous failure, not sure why it is failed. I worked when I run the local test on my

[GitHub] spark pull request: [SPARK-12521][SQL][WIP] JDBCRelation does not ...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10473#issuecomment-167190846 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

[GitHub] spark pull request: [SPARK-12521][SQL][WIP] JDBCRelation does not ...

2015-12-24 Thread xguo27
Github user xguo27 commented on the pull request: https://github.com/apache/spark/pull/10473#issuecomment-167190872 Marking it [WIP] to invite discussion here. : ) As I suspect the original code includes infinity on both smaller than side and greater than side for a reason. --- If

[GitHub] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10451#issuecomment-167202389 @cloud-fan Just added a quick fix by introducing a stop flag to avoid adding extra `Limit` in the subsequent iteration, but I am not sure if my way is good.

[GitHub] spark pull request: [SPARK-12316] Wait a minutes to avoid cycle ca...

2015-12-24 Thread SaintBacchus
GitHub user SaintBacchus opened a pull request: https://github.com/apache/spark/pull/10475 [SPARK-12316] Wait a minutes to avoid cycle calling. When application end, AM will clean the staging dir. But if the driver trigger to update the delegation token, it will can't find the

[GitHub] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10451#issuecomment-167205352 **[Test build #48322 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48322/consoleFull)** for PR 10451 at commit

[GitHub] spark pull request: [SPARK-10359] Enumerate dependencies in a file...

2015-12-24 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10461#discussion_r48426589 --- Diff: dev/deps/spark-deps-hadoop-2.4 --- @@ -0,0 +1,185 @@ +JavaEWAH-0.3.2.jar --- End diff -- IMO, there's very little overhead to

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167189998 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167189967 **[Test build #48319 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48319/consoleFull)** for PR 10388 at commit

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167189997 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

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

2015-12-24 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10330#discussion_r48432747 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala --- @@ -200,6 +204,10 @@ case class TakeOrderedAndProject(

[GitHub] spark pull request: [SPARK-12436][SQL] Now InferSchema should retu...

2015-12-24 Thread naveenminchu
GitHub user naveenminchu opened a pull request: https://github.com/apache/spark/pull/10474 [SPARK-12436][SQL] Now InferSchema should return NullType and ArrayType(NullType) a.) Currently I have fix for InferSchema to return NullType and ArrayType(NullType) b.) @rxin Can you

[GitHub] spark pull request: [SPARK-12396][Core]Once driver connect to a ma...

2015-12-24 Thread echoTomei
Github user echoTomei closed the pull request at: https://github.com/apache/spark/pull/10407 --- 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

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167150757 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

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167150758 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167150742 **[Test build #48313 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48313/consoleFull)** for PR 10456 at commit

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread wyaron
Github user wyaron commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167151653 Thanks for the quick CR ! Apparently, the new unit test has just failed for sc.wholeTextFiles(...) :-) My guess is that it was broken in commit

[GitHub] spark pull request: [SPARK-12231][SQL]create a combineFilters' pro...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10388#issuecomment-167152963 **[Test build #48315 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48315/consoleFull)** for PR 10388 at commit

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167153204 **[Test build #48314 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48314/consoleFull)** for PR 10456 at commit

[GitHub] spark pull request: [SPARK-12505] [SQL] Pushdown a Limit on top of...

2015-12-24 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10454#issuecomment-167138258 The new fix contains two parts: 1) Since the full outer join will remove the duplicates, we are unable to add the extra `limit` to both sides. As long as we

[GitHub] spark pull request: [SPARK-12515][SQL][DOC] minor doc update for r...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10465#issuecomment-167139802 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

[GitHub] spark pull request: [SPARK-12515][SQL][DOC] minor doc update for r...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10465#issuecomment-167139803 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12515][SQL][DOC] minor doc update for r...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10465#issuecomment-167139745 **[Test build #48311 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48311/consoleFull)** for PR 10465 at commit

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167141253 **[Test build #48313 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48313/consoleFull)** for PR 10456 at commit

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread sarutak
Github user sarutak commented on a diff in the pull request: https://github.com/apache/spark/pull/10456#discussion_r48422742 --- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala --- @@ -274,6 +274,31 @@ class SparkContextSuite extends SparkFunSuite with

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread sarutak
Github user sarutak commented on a diff in the pull request: https://github.com/apache/spark/pull/10456#discussion_r48422480 --- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala --- @@ -274,6 +274,31 @@ class SparkContextSuite extends SparkFunSuite with

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167141665 **[Test build #48312 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48312/consoleFull)** for PR 10456 at commit

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167141687 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10456#issuecomment-167141686 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

[GitHub] spark pull request: [SPARK-12517] add default RDD name for one cre...

2015-12-24 Thread sarutak
Github user sarutak commented on a diff in the pull request: https://github.com/apache/spark/pull/10456#discussion_r48422596 --- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala --- @@ -274,6 +274,31 @@ class SparkContextSuite extends SparkFunSuite with

[GitHub] spark pull request: [SPARK-12409][SPARK-12387][SPARK-12391][SQL] S...

2015-12-24 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/10468#discussion_r48402545 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala --- @@ -186,13 +187,19 @@ private[sql] object JDBCRDD

[GitHub] spark pull request: [SPARK-10427][SQL]Respect -S option for spark-...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8915#issuecomment-167068117 **[Test build #48295 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48295/consoleFull)** for PR 8915 at commit

[GitHub] spark pull request: [SPARK-10427][SQL]Respect -S option for spark-...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8915#issuecomment-167068160 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

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10330#issuecomment-167069840 **[Test build #48296 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48296/consoleFull)** for PR 10330 at commit

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10330#issuecomment-167069917 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

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10330#issuecomment-167069919 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12409][SPARK-12387][SPARK-12391][SQL] S...

2015-12-24 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/10468#discussion_r48404669 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCSuite.scala --- @@ -186,8 +187,26 @@ class JDBCSuite extends SparkFunSuite

[GitHub] spark pull request: [SPARK-12515][SQL][DOC] minor doc update for r...

2015-12-24 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10465#discussion_r48405245 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala --- @@ -159,8 +159,8 @@ class DataFrameReader private[sql](sqlContext:

[GitHub] spark pull request: [SPARK-12505] [SQL] Pushdown a Limit on top of...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10454#issuecomment-167077729 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12409][SPARK-12387][SPARK-12391][SQL] S...

2015-12-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10468#discussion_r48402449 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala --- @@ -186,13 +187,19 @@ private[sql] object JDBCRDD extends

[GitHub] spark pull request: [SPARK-8641][SPARK-12455][SQL] Native Spark Wi...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10402#issuecomment-167070921 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

[GitHub] spark pull request: [SPARK-8641][SPARK-12455][SQL] Native Spark Wi...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10402#issuecomment-167070923 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-8641][SPARK-12455][SQL] Native Spark Wi...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10402#issuecomment-167070865 **[Test build #48294 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48294/consoleFull)** for PR 10402 at commit

[GitHub] spark pull request: [SPARK-12504][SQL] Masking credentials in the ...

2015-12-24 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10452#discussion_r48405336 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRelation.scala --- @@ -19,6 +19,7 @@ package

[GitHub] spark pull request: [SPARK-12504][SQL] Masking credentials in the ...

2015-12-24 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10452#discussion_r48405368 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/jdbc/DB2Dialect.scala --- @@ -29,4 +29,8 @@ private object DB2Dialect extends JdbcDialect {

[GitHub] spark pull request: [SPARK-11939] [ML] [PySpark] PySpark support m...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10469#issuecomment-167077459 **[Test build #48300 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48300/consoleFull)** for PR 10469 at commit

[GitHub] spark pull request: [SPARK-12409][SPARK-12387][SPARK-12391][SQL] R...

2015-12-24 Thread viirya
GitHub user viirya opened a pull request: https://github.com/apache/spark/pull/10470 [SPARK-12409][SPARK-12387][SPARK-12391][SQL] Refactor filter pushdown for JDBCRDD and add few filters This patch refactors the filter pushdown for JDBCRDD and also adds few filters. Added

[GitHub] spark pull request: [SPARK-12409][SPARK-12387][SPARK-12391][SQL] R...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10470#issuecomment-167078592 **[Test build #48302 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48302/consoleFull)** for PR 10470 at commit

[GitHub] spark pull request: [SPARK-12503] [SQL] Pushing Limit Through Unio...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10451#issuecomment-167067050 **[Test build #48299 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48299/consoleFull)** for PR 10451 at commit

[GitHub] spark pull request: [SPARK-11394][SQL] Throw IllegalArgumentExcept...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9350#issuecomment-167069188 **[Test build #48292 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48292/consoleFull)** for PR 9350 at commit

[GitHub] spark pull request: [SPARK-12476][SQL] Implement JdbcRelation#unha...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10427#issuecomment-167071998 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

[GitHub] spark pull request: [SPARK-11394][SQL] Throw IllegalArgumentExcept...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9350#issuecomment-167076426 **[Test build #2254 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2254/consoleFull)** for PR 9350 at commit

[GitHub] spark pull request: [SPARK-12504][SQL] Masking credentials in the ...

2015-12-24 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10452#discussion_r48405397 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/jdbc/JdbcDialects.scala --- @@ -108,6 +108,19 @@ abstract class JdbcDialect extends Serializable {

[GitHub] spark pull request: [SPARK-12504][SQL] Masking credentials in the ...

2015-12-24 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10452#discussion_r48405431 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCSuite.scala --- @@ -517,4 +519,65 @@ class JDBCSuite extends SparkFunSuite

[GitHub] spark pull request: [SPARK-12505] [SQL] Pushdown a Limit on top of...

2015-12-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10454#issuecomment-167077728 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

[GitHub] spark pull request: [SPARK-12505] [SQL] Pushdown a Limit on top of...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10454#issuecomment-167077669 **[Test build #48298 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48298/consoleFull)** for PR 10454 at commit

[GitHub] spark pull request: [SPARK-12505] [SQL] Pushdown a Limit on top of...

2015-12-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10454#issuecomment-167066005 **[Test build #48298 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48298/consoleFull)** for PR 10454 at commit

[GitHub] spark pull request: [SPARK-12515][SQL][DOC] minor doc update for r...

2015-12-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10465#discussion_r48402502 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala --- @@ -159,8 +159,8 @@ class DataFrameReader private[sql](sqlContext:

[GitHub] spark pull request: [SPARK-7995][SPARK-6280][Core]Remove AkkaRpcEn...

2015-12-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10459#discussion_r48402964 --- Diff: project/MimaExcludes.scala --- @@ -171,7 +171,19 @@ object MimaExcludes { ) ++ // SPARK-11314: YARN backend moved to yarn

  1   2   >