[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2015-04-14 Thread DanielMe
Github user DanielMe commented on the pull request: https://github.com/apache/spark/pull/1063#issuecomment-92678825 Okay, thanks for the clarification. Initially, I had naively assumed that the functionality you added was just a layer above the Hive API hence it was a bit confusing

[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2015-04-13 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/1063#issuecomment-92457478 The biggest reason for the divergence is this API is much lighter weight (you can define functions in a single line, inline with the rest of your program). We can

[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2015-04-13 Thread DanielMe
Github user DanielMe commented on the pull request: https://github.com/apache/spark/pull/1063#issuecomment-92301932 Excuse my naive question, however, it seems that this does not use the regular Hive UDF API, right? (Like when I would run `hiveContext.sql(CREATE TEMPORARY FUNCTION

[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2014-08-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1063#issuecomment-50975589 QA tests have started for PR 1063. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17781/consoleFull ---

[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2014-08-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1063#issuecomment-50975896 QA results for PR 1063:br- This patch PASSES unit tests.br- This patch merges cleanlybr- This patch adds the following public classes (experimental):brtrait

[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2014-08-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1063#issuecomment-50976747 QA results for PR 1063:br- This patch FAILED unit tests.br- This patch merges cleanlybr- This patch adds the following public classes (experimental):brtrait

[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2014-08-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1063#issuecomment-50977296 QA results for PR 1063:br- This patch PASSES unit tests.br- This patch merges cleanlybr- This patch adds the following public classes (experimental):brtrait

[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2014-08-02 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/1063#issuecomment-50977690 Thanks for looking this over! I've merged to master and 1.1. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-2097][SQL] UDF Support

2014-08-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/1063 --- 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