GitHub user ravicodder opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/859
[ZEPPELIN-599]notebook search should search paragraph title
### What is this PR for?
Allow notebook search to search paragraph title too.
### What type of PR is it?
Github user ravicodder commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-215164810
Ready for review.
---
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 no
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-215941636
Thanks @ravicodder for the improvement.
Tried and works well.
If it is not too difficult, it might be better to have a simple testcase
for se
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-216086558
ð for adding a new test-case for the search API on a new behavoir
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user ravicodder commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-216158267
@Leemoonsoo @bzz Thanks for review , Fixed indentation and added test.
@bzz thanks for informing , will take care in future PR's
---
If your project
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-216866674
Looks good to me!
---
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
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-217057001
@ravicodder thank you for prompt responce!
Looks great to me, modulo style nitpicks above. Let's merge as soon as
those are addressed!
---
If your proj
Github user ravicodder commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-217127237
@bzz Done required changes. Please have a look.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-217782293
LGTM
---
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 thi
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-218070339
Merging if no more discussion.
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/859
---
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 fea
11 matches
Mail list logo