Github user tzolov commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/181#issuecomment-129305944
@felixcheung, thanks for the suggestion. i've thought of ZEPPELIN-188 as
well. Indeed the dirty page event is a good trigger but i am worried of the
performance impact. Perhaps it is neglectable, also we can improve the set
logic to SET-ONLY-IF-DIFFERENT.
But let us first merge the current fix. This will resolve ZEPPELIN-141 and
fix the broken autocompletion. Then we can generalize the solution (the way you
have suggested) in the context of ZEPPELIN-188. What do you think?
I agree about the extensible interpreters. IMHO this will require extending
the Interpreter interface, may be extend the websocket protocol with an
additional command and figuring out how to enable the ACE features in the
front-end. For example now only few ACE modes are defined in the bower.json. If
you need more (like SH or PYTHON for example) one have to add them to
bower.json and re-build. Maybe we can (pre)enable large set of the ACE modes
upfront.
Anyway i think this work deserves its one JIRA ticket (e.g. this would be
the 3rd step)?
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
