Github user vgmartinez commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/361#issuecomment-169696454
  
    Hi @jongyoul,
    I think it is not necessary to add another interpreter, I think you can 
improve the interpreter hive to add the jars to the classpath and use the hive 
interpreter for all jdbc. What do you think?
    
    I tried your implementation in this 
[branch](https://github.com/vgmartinez/incubator-zeppelin/tree/jdbc_interpreter)
 and work for:
    - postgres
    - mysql
    - mariadb
    - redshift
    
    I do not see useful to have another interpreter where the code is almost 
the same as the hive.


---
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.
---

Reply via email to