You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zeppelin.apache.org by "jongyoul (via GitHub)" <gi...@apache.org> on 2023/06/14 01:00:37 UTC

[GitHub] [zeppelin] jongyoul commented on pull request #4598: [ZEPPELIN-5896]feat:add sql debug feature

jongyoul commented on PR #4598:
URL: https://github.com/apache/zeppelin/pull/4598#issuecomment-1590269638

   @zhugezifang Hello, sorry for the late reply. I glance at your PR and suggest you make a child interpreter like `jdbc.debug` instead of adding a menu. The debug paragraph is currently only for JDBC, but the menu would be exposed for all kinds of paragraphs. `zeppelin-interpreter` package and `NotebookServer` class affect all of interpreters. If you want to make an interpreter-specific feature, I recommend adding a new sub-interpreter. WDYT?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org