[GitHub] spark pull request: [SPARK-5426][SQL] Add SparkSQL Java API helper...

2015-02-03 Thread kul
Github user kul commented on the pull request:

https://github.com/apache/spark/pull/4243#issuecomment-72786838
  
@marmbrus Thanks for review!
Rebased against master and sqashed in a new commit renaming 
`schemaRDDOperations` to now more aptly called `dataFrameRDDOperations`.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-5551][SQL] Create type alias for Schema...

2015-02-02 Thread kul
Github user kul commented on the pull request:

https://github.com/apache/spark/pull/4327#issuecomment-72607021
  
since SchemaRDD in 1.3.0 was java compatible a few days back. Would it make 
sense to alias `JavaSchemaRDD` as well?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-5551][SQL] Create type alias for Schema...

2015-02-02 Thread kul
Github user kul commented on the pull request:

https://github.com/apache/spark/pull/4327#issuecomment-72607251
  
Got it! 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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-5426][SQL] Add SparkSQL Java API helper...

2015-01-28 Thread kul
GitHub user kul opened a pull request:

https://github.com/apache/spark/pull/4243

[SPARK-5426][SQL] Add SparkSQL Java API helper methods.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/kul/spark master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/4243.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #4243


commit b51b656f5397d36b7bb5e446e271c1714b662046
Author: kul kuldeep.b...@gmail.com
Date:   2015-01-28T08:47:56Z

[SPARK-5426][SQL] Add SparkSQL Java API helper methods.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-5426][SQL] Add SparkSQL Java API helper...

2015-01-28 Thread kul
Github user kul commented on the pull request:

https://github.com/apache/spark/pull/4243#issuecomment-71807950
  
Looking into it further, seems like even in Scala one will have to do with 
`.rdd` for normal spark operations as functions like filter etc are being 
overwritten for the query dsl. So `.toJavaRDD` is consistent with the scala 
counterpart `.rdd`.
But functions which are not being overwitten e.g. `map` will work like 
normal spark operations which is a bit inconsistent though.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org