Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/833#issuecomment-216084962
Looks great, thank you for taking care.
I have one more question: in case of no auth configured, can somebody
please help me to understand what is the performance implication or price
per-API call to `/serach/`, that is added by this changeset?
We want search to be as fast as possible in a common case, so just want to
make sure that we keep an eye on it. Would that be reasonable to add [something
like
this](https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/search/LuceneSearch.java#L285)
to the query API code and at least manually compare results once (post here),
to make sure preformance does not degraded, or see if any optimization are
needed for case when no auth is configuted?
Please let me know what do you think!
---
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.
---
