Skip to content

[ZEPPELIN-4750]. Don't display temporary tables in show tables of flink interpreter#3731

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

[ZEPPELIN-4750]. Don't display temporary tables in show tables of flink interpreter#3731
zjffdu wants to merge 1 commit intoapache:masterfrom
zjffdu:ZEPPELIN-4750

Conversation

@zjffdu
Copy link
Contributor

@zjffdu zjffdu commented Apr 10, 2020

What is this PR for?

This PR would filter the temporary tables in show tables, only show the permanent tables. Otherwise it would confuse users if they see lots of temporary tables created by zeppelin.

What type of PR is it?

[ Improvement ]

Todos

  • - Task

What is the Jira issue?

How should this be tested?

  • CI pass

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 07a7f93 Apr 11, 2020
asfgit pushed a commit that referenced this pull request Apr 11, 2020
…nk interpreter

### What is this PR for?

This PR would filter the temporary tables in show tables, only show the permanent tables. Otherwise it would confuse users if they see lots of temporary tables created by zeppelin.

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

### Todos
* [ ] - Task

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

### How should this be tested?
* CI pass

### 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 #3731 from zjffdu/ZEPPELIN-4750 and squashes the following commits:

85fe8b1 [Jeff Zhang] [ZEPPELIN-4750]. Don't display temporary tables in show tables of flink interpreter

(cherry picked from commit 07a7f93)
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