You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by mebelousov <gi...@git.apache.org> on 2018/05/16 11:37:43 UTC
[GitHub] zeppelin pull request #2970: ZEPPELIN-3379 Refactoring of JDBC autocomplete.
GitHub user mebelousov opened a pull request:
https://github.com/apache/zeppelin/pull/2970
ZEPPELIN-3379 Refactoring of JDBC autocomplete.
### What is this PR for?
Current implementation of autocomplete allows only completion in `schema.table.column` scenario. Such scenario exclude tables for default schema.
The PR fixes the scenario with tables in default schema and alias processing.
Also I want to add feature to complete columns for used tables in statement from blank cursor.
The autocomplete was tested for Greenplum 4.3 (PostgreSQL), Oracle 11g, MySQL (Hive Metastore) and Hive.
### What type of PR is it?
Improvement
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3379
### How should this be tested?
* Manually
* Tests pass
### Screenshots (if appropriate)
![mysql](https://user-images.githubusercontent.com/9324163/40114378-61d0fd7c-5915-11e8-9fc4-199bfe5e913a.gif)
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/TinkoffCreditSystems/zeppelin ZEPPELIN-3379
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2970.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2970
----
commit f01f3fd63690cc86a8ed344d37d79bc83f209849
Author: mebelousov <me...@...>
Date: 2018-05-16T11:26:00Z
ZEPPELIN-3379 Refactoring of JDBC autocomplete.
----
---
[GitHub] zeppelin pull request #2970: ZEPPELIN-3379 Refactoring of JDBC autocomplete.
Posted by mebelousov <gi...@git.apache.org>.
Github user mebelousov closed the pull request at:
https://github.com/apache/zeppelin/pull/2970
---