Github user tzolov commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/361#issuecomment-172505545
Thanks for the quick response @vgmartinez ,
The SqlCompleter (removed by the last commit) is meant to be 100% JDBC API
compliant and hopefully easy to reuse in a generic jdbc interpreter.
IMO though it is unacceptable to remove a working interpreters until the
JDBC interpreter is completed. Some Zeppelin users (like me) use the master to
build Zeppelin and now suddenly some essential (for us) functionality is gone.
I think that as a percussion it has to be confirmed that the new JDBC
driver can support an existing (jdbc based) interpreter before the later is
removed.
I suggest returning the PSQL interpreter code ASAP. Let me know if you need
help.
In the meantime we can continue working on the generic JDBC. I will help
porting the Completer functionality and missing tests.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
