[GitHub] spark pull request: [SPARK-6379][SQL] Support an implicit conversi...

2015-03-21 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/5061#issuecomment-84455446 Once you have updated, can you also update the PR description, as this becomes the commit message. Thanks! --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-6379][SQL] Support an implicit conversi...

2015-03-21 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5061#discussion_r26897050 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -212,6 +212,22 @@ class SQLContext(@transient val sparkContext: SparkContext)

[GitHub] spark pull request: [SPARK-6379][SQL] Support an implicit conversi...

2015-03-18 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/5061#issuecomment-82888774 @rxin fixed, and plz check 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

[GitHub] spark pull request: [SPARK-6379][SQL] Support an implicit conversi...

2015-03-17 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/5061#issuecomment-82214695 You mean df.select($id, callUDF($value))? --- 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] spark pull request: [SPARK-6379][SQL] Support an implicit conversi...

2015-03-17 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5061#issuecomment-82607897 I meant something like ```scala df.select($id, callUDF(simpleUDF, $value)) ``` --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-6379][SQL] Support an implicit conversi...

2015-03-17 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/5061#issuecomment-82673585 Ok, I'll try to refine my codes. --- 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-6379][SQL] Support an implicit conversi...

2015-03-16 Thread maropu
GitHub user maropu opened a pull request: https://github.com/apache/spark/pull/5061 [SPARK-6379][SQL] Support an implicit conversion from udfname to an UDF defined in SQLContext This is useful for using pre-defined UDFs in SQLContext; val df = Seq((id1, 1), (id2, 4), (id3,

[GitHub] spark pull request: [SPARK-6379][SQL] Support an implicit conversi...

2015-03-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5061#issuecomment-82109406 Can't we make this explicit by using callUDF? --- 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-6379][SQL] Support an implicit conversi...

2015-03-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5061#issuecomment-82108605 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