Skip to content

[ZEPPELIN-4885]. Add property to specify Module order of flink interpreter#3802

Closed
zjffdu wants to merge 1 commit intoapache:masterfrom
zjffdu:ZEPPELIN-4885
Closed

[ZEPPELIN-4885]. Add property to specify Module order of flink interpreter#3802
zjffdu wants to merge 1 commit intoapache:masterfrom
zjffdu:ZEPPELIN-4885

Conversation

@zjffdu
Copy link
Contributor

@zjffdu zjffdu commented Jun 16, 2020

What is this PR for?

It is a trivial PR which add property zeppelin.flink.module.enableHive to control whether load hive udf first when there's same udf name in both hive and flink.

What type of PR is it?

[Improvement]

Todos

  • - Task

What is the Jira issue?

How should this be tested?

  • CI pass, and manually tested.

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

Copy link
Contributor

@alexott alexott left a comment

Choose a reason for hiding this comment

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

lgtm

@asfgit asfgit closed this in 72c127f Jun 17, 2020
asfgit pushed a commit that referenced this pull request Jun 17, 2020
…reter

### What is this PR for?

It is a trivial PR which add property `zeppelin.flink.module.enableHive` to control whether load hive udf first when there's same udf name in both hive and flink.

### What type of PR is it?
[Improvement]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-4885

### How should this be tested?
* CI pass, and manually tested.

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: Jeff Zhang <zjffdu@apache.org>

Closes #3802 from zjffdu/ZEPPELIN-4885 and squashes the following commits:

4575458 [Jeff Zhang] [ZEPPELIN-4885]. Add property to specify Module order of flink interpreter

(cherry picked from commit 72c127f)
Signed-off-by: Jeff Zhang <zjffdu@apache.org>
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.

2 participants