Skip to content

Conversation

@davies
Copy link
Contributor

@davies davies commented Nov 9, 2015

The DataFrame APIs that takes a SQL expression always use SQLParser, then the HiveFunctionRegistry will called outside of Hive state, cause NPE if there is not a active Session State for current thread (in PySpark).

cc @rxin @yhuai

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a note. In HiveQLDialect, when we parse a query, we use sqlContext.executionHive.withHiveState. Right now, create function is still broken because the metadata of the function is in metadata hive. However, it is the work of another PR.

@yhuai
Copy link
Contributor

yhuai commented Nov 9, 2015

LGTM pending jenkins.

@SparkQA
Copy link

SparkQA commented Nov 9, 2015

Test build #45427 has finished for PR 9576 at commit db74784.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Nov 10, 2015

Test build #2023 has finished for PR 9576 at commit db74784.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@davies
Copy link
Contributor Author

davies commented Nov 10, 2015

Merged into master and 1.6.

@asfgit asfgit closed this in d6cd3a1 Nov 10, 2015
asfgit pushed a commit that referenced this pull request Nov 10, 2015
The DataFrame APIs that takes a SQL expression always use SQLParser, then the HiveFunctionRegistry will called outside of Hive state, cause NPE if there is not a active Session State for current thread (in PySpark).

cc rxin yhuai

Author: Davies Liu <[email protected]>

Closes #9576 from davies/hive_udf.

(cherry picked from commit d6cd3a1)
Signed-off-by: Davies Liu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants