[GitHub] [zeppelin] Reamer commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-09-19 Thread GitBox


Reamer commented on PR #4321:
URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1250660568

   I suspect the checkstyle plugin. This plugin produces a lot of the Maven 
warnings at the beginning.
   I've been wanting to fix this for a long time, but haven't gotten around to 
it yet.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] jongyoul commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-09-19 Thread GitBox


jongyoul commented on PR #4321:
URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1250823146

   >I suspect the checkstyle plugin. This plugin produces a lot of the Maven 
warnings at the beginning.
   I've been wanting to fix this for a long time, but haven't gotten around to 
it yet.
   
   Agreed. We should have better idea to enforce to keep it. I'll think of it. 
:-)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] huage1994 commented on pull request #4392: [ZEPPELIN-5694] Support apache kyuubi

2022-09-19 Thread GitBox


huage1994 commented on PR #4392:
URL: https://github.com/apache/zeppelin/pull/4392#issuecomment-1251031730

   > Sorry for the late response @huage1994 , PR LGTM, trigger the CI again and 
wait for its result before merge
   
   Thanks!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] huage1994 commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-09-19 Thread GitBox


huage1994 commented on PR #4321:
URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1251100264

   @zjffdu @Reamer @jongyoul , really thanks for  helping out~ 
   I'm going to test your guesses and continue to address this issue
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org