[GitHub] zeppelin issue #1182: ZEPPELIN-1170 Handsontable fails to display data on se...

2016-07-14 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1182 Tested, 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 this feature enabled and wishes so,

[GitHub] zeppelin issue #1182: ZEPPELIN-1170 Handsontable fails to display data on se...

2016-07-13 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1182 @corneadoug from what you say, it's still not clear what's the plan with this one. I do not think it was rebased, where `rebased = git rebase` - that way all the commits would show up AFTER

[GitHub] zeppelin issue #1182: ZEPPELIN-1170 Handsontable fails to display data on se...

2016-07-13 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1182 So what is the consensus, shall we actually *rebase* this one on master and merge, or just close, as soon as #1171 is merged? --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin issue #1182: ZEPPELIN-1170 Handsontable fails to display data on se...

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1182 @Leemoonsoo Yes, I agree, we should merge #1171. --- 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