[GitHub] zeppelin issue #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-29 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1430 Reopening the pull request as the build failed --- 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

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-29 Thread vensant
GitHub user vensant reopened a pull request: https://github.com/apache/zeppelin/pull/1430 [ZEPPELIN-868] Notebook import fails when notebook is large ### What is this PR for? A bug fix: Added validation in the note import dialog box to check for the uploaded json file size

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-29 Thread vensant
Github user vensant closed the pull request at: https://github.com/apache/zeppelin/pull/1430 --- 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

[GitHub] zeppelin issue #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-27 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1430 Reopening the pull request as the build failed --- 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

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-27 Thread vensant
Github user vensant closed the pull request at: https://github.com/apache/zeppelin/pull/1430 --- 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

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-27 Thread vensant
GitHub user vensant reopened a pull request: https://github.com/apache/zeppelin/pull/1430 [ZEPPELIN-868] Notebook import fails when notebook is large ### What is this PR for? A bug fix: Added validation in the note import dialog box to check for the uploaded json file size

[GitHub] zeppelin issue #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-27 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1430 @corneadoug instead of validating the error in server side, I have just changed the code in the client side, that compares the input file size with the maximum limit of file size that is already

[GitHub] zeppelin issue #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-22 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1430 @r-kamath thanks for the inputs! --- 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

[GitHub] zeppelin issue #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-22 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1430 @corneadoug once the file is uploaded, it tries to process the json but because the file exceeds the max limit it tries to close the websocket event. so I am trying to broadcast and catch

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-16 Thread vensant
Github user vensant closed the pull request at: https://github.com/apache/zeppelin/pull/1430 --- 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

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-16 Thread vensant
GitHub user vensant reopened a pull request: https://github.com/apache/zeppelin/pull/1430 [ZEPPELIN-868] Notebook import fails when notebook is large ### What is this PR for? A bug fix: Added validation in the note import dialog box to check for the uploaded json file size

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-16 Thread vensant
Github user vensant closed the pull request at: https://github.com/apache/zeppelin/pull/1430 --- 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

[GitHub] zeppelin issue #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-16 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1430 Closed and reopened the pull request in order to re-trigger the Travis CI build. Build failed due to test failures Again --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-16 Thread vensant
GitHub user vensant reopened a pull request: https://github.com/apache/zeppelin/pull/1430 [ZEPPELIN-868] Notebook import fails when notebook is large ### What is this PR for? A bug fix: Added validation in the note import dialog box to check for the uploaded json file size

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-16 Thread vensant
Github user vensant closed the pull request at: https://github.com/apache/zeppelin/pull/1430 --- 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

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-16 Thread vensant
GitHub user vensant reopened a pull request: https://github.com/apache/zeppelin/pull/1430 [ZEPPELIN-868] Notebook import fails when notebook is large ### What is this PR for? A bug fix: Added validation in the note import dialog box to check for the uploaded json file size

[GitHub] zeppelin issue #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

2016-09-16 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1430 Closed and reopened the pull request in order to re-trigger the Travis CI build. Build failed due to test failures --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

2016-09-15 Thread vensant
GitHub user vensant opened a pull request: https://github.com/apache/zeppelin/pull/1430 [ZEPPELIN-868] Notebook import fails when notebook is large ### What is this PR for? A bug fix: Added validation in the note import dialog box to check for the uploaded json file size

[GitHub] zeppelin issue #1362: [ZEPPELIN-1178] Tooltip: Show chart type when hovering...

2016-08-26 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1362 Code changes done as per the review comment. --- 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

[GitHub] zeppelin issue #1362: [ZEPPELIN-1178] Tooltip: Show chart type when hovering...

2016-08-25 Thread vensant
Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1362 Hi all, I will change the tool tip names as suggested. Thank you all Regards, Venkat On 26 Aug 2016 7:57 am, "Lee moon soo" <notificati...@gith

[GitHub] zeppelin pull request #1362: [ZEPPELIN-1178] Tooltip: Show chart type when h...

2016-08-25 Thread vensant
GitHub user vensant opened a pull request: https://github.com/apache/zeppelin/pull/1362 [ZEPPELIN-1178] Tooltip: Show chart type when hovering over chart icon ### What is this PR for? A usability improvement: Added tooltips for the chart icons in Zeppelin paragraphs, showing