Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11053 )
Change subject: IMPALA-7203. Support UDFs in LocalCatalog ...................................................................... IMPALA-7203. Support UDFs in LocalCatalog This adds support to LocalCatalog to load persistent UDFs (both Java and native) from the HMS. Transient UDFs are not supported, since, without a central component to store them between restarts, there isn't any clear path to doing so. The various UDF tests do not yet pass reliably with this change since so many of them rely on the transient Java UDF functionality. This is a legacy feature that isn't commonly used today, and we may not be able to support it in LocalCatalog. For now, I tested manually that I could create, drop, and run some UDFs and that they show up in 'show functions'. Change-Id: I6130d07b9c641525382a618a9f8da048c7ae75ed Reviewed-on: http://gerrit.cloudera.org:8080/11053 Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Reviewed-by: Todd Lipcon <t...@apache.org> --- M fe/src/main/java/org/apache/impala/analysis/FunctionCallExpr.java M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java M fe/src/main/java/org/apache/impala/catalog/Db.java M fe/src/main/java/org/apache/impala/catalog/local/DirectMetaProvider.java M fe/src/main/java/org/apache/impala/catalog/local/LocalDb.java M fe/src/main/java/org/apache/impala/catalog/local/MetaProvider.java M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java A fe/src/main/java/org/apache/impala/util/FunctionUtils.java M fe/src/main/java/org/apache/impala/util/PatternMatcher.java 9 files changed, 539 insertions(+), 255 deletions(-) Approvals: Impala Public Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11053 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6130d07b9c641525382a618a9f8da048c7ae75ed Gerrit-Change-Number: 11053 Gerrit-PatchSet: 6 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tianyi Wang <tw...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>
