[GitHub] zeppelin issue #1814: [ZEPPELIN-1869] changed the API response to generate t...

2017-01-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1814 LGTM. Will merge if there's 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 project does not have this

[GitHub] zeppelin issue #1814: [ZEPPELIN-1869] changed the API response to generate t...

2017-01-07 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1814 Fixed done. CI is green :) --- 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

[GitHub] zeppelin issue #1814: [ZEPPELIN-1869] changed the API response to generate t...

2017-01-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1814 @Leemoonsoo Ah! sorry, i will fix! --- 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

[GitHub] zeppelin issue #1814: [ZEPPELIN-1869] changed the API response to generate t...

2017-01-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1814 @cloverhearts I think you need update related unittests, too. ``` Results : Failed tests: InterpreterRestApiTest.testAddDeleteRepository:352 Test create method: