[GitHub] zeppelin issue #2532: [ZEPPELIN-2850]

2017-08-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2532 Thanks for @imnotkind addressing BOM. Tested and LGTM. CI failure is not related to this change. Merge to master if no further comment. --- If your project is set up for it, you

[GitHub] zeppelin issue #2532: [ZEPPELIN-2850]

2017-08-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2532 Thanks @imnotkind for great improvement. Tested this PR and it works well on large data. However, it removes BOM and multi-byte char does not displayed correctly when open