[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread aa8y
Github user aa8y commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163522315 @JoshRosen I've created a new pull request, #10247 with a fresh new commit which should merge. @srowen I would really appreciate it if you could get Jenkins to test it. Pl

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread aa8y
Github user aa8y closed the pull request at: https://github.com/apache/spark/pull/10028 --- 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 enabl

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163468427 Looks like this PR is now broken by a bad merge with master. Would you mind closing for now and re-opening once things are fixed? Thanks! --- If your project is set

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163385128 Not sure but the error is: ``` [error] /home/jenkins/workspace/SparkPullRequestBuilder/sql/catalyst/src/test/scala/org/apache/spark/sql/RowTest.scala:83:0:

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread aa8y
Github user aa8y commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163384083 ```aallamsetty@aallamsetty-lap ~/c/spark> ./dev/run-tests Can't install SparkR as R is was not found in PATH [info] Using build tool sbt with Hadoop profile hadoop2.

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread aa8y
Github user aa8y commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163372735 I remember it passed the style tests last time I checked. I'll run them again. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163359366 **[Test build #47439 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47439/consoleFull)** for PR 10028 at commit [`e47c912`](https://g

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163359376 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163359373 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] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163358939 **[Test build #47439 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47439/consoleFull)** for PR 10028 at commit [`e47c912`](https://gi

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163355759 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 t

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-12-09 Thread aa8y
Github user aa8y commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-163349969 @rxin You ran a Jenkins build for my previous pull request for this issue. Could you please do the same for this build request? --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-11-30 Thread aa8y
Github user aa8y commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-160761838 Anything else you guys need for 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-11962] Add optional getAs functions to ...

2015-11-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10028#issuecomment-160392284 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 p

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-11-29 Thread aa8y
GitHub user aa8y opened a pull request: https://github.com/apache/spark/pull/10028 [SPARK-11962] Add optional getAs functions to Row getAsOpt[T] functions have been added to Row and GeneicRowWithSchema to get the values present in a row object optionally. Corresponding tests have al

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-11-28 Thread aa8y
Github user aa8y commented on the pull request: https://github.com/apache/spark/pull/10027#issuecomment-16038 I just found out this would fail style checks. --- 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 proj

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-11-28 Thread aa8y
Github user aa8y closed the pull request at: https://github.com/apache/spark/pull/10027 --- 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 enabl

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-11-28 Thread aa8y
Github user aa8y commented on the pull request: https://github.com/apache/spark/pull/10027#issuecomment-160388606 I am still learning to work with Git. Looks like I have a couple commits which are not mine in there. However a compare did not show any code other than mine, so I filed a

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-11-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10027#issuecomment-160388604 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 p

[GitHub] spark pull request: [SPARK-11962] Add optional getAs functions to ...

2015-11-28 Thread aa8y
GitHub user aa8y opened a pull request: https://github.com/apache/spark/pull/10027 [SPARK-11962] Add optional getAs functions to Row `getAsOpt[T]` functions have been added to `Row` and `GeneicRowWithSchema` to get the values present in a row object optionally. Corresponding tests h