[GitHub] zeppelin issue #2902: ZEPPELIN-3140. Fixed Checkstyle issues in zeppelin-ser...

2018-04-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2902 Yes, there seemed to be an inconsistency between apache wis and apache github. It's resolved when I merge one more PR into wis. ---

[GitHub] zeppelin issue #2902: ZEPPELIN-3140. Fixed Checkstyle issues in zeppelin-ser...

2018-04-05 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2902 weird.. I thought I merged it and resolved the bug yesterday. ---

[GitHub] zeppelin issue #2902: ZEPPELIN-3140. Fixed Checkstyle issues in zeppelin-ser...

2018-04-05 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2902 Let's merge it ---

[GitHub] zeppelin issue #2902: ZEPPELIN-3140. Fixed Checkstyle issues in zeppelin-ser...

2018-04-03 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2902 merging if no more comment ---

[GitHub] zeppelin issue #2902: ZEPPELIN-3140. Fixed Checkstyle issues in zeppelin-ser...

2018-03-31 Thread HorizonNet
Github user HorizonNet commented on the issue: https://github.com/apache/zeppelin/pull/2902 @felixcheung Fixed the review comment. Let's wait for the build results. ---