[GitHub] [incubator-dolphinscheduler] CalvinKirs commented on pull request #3315: add feature : query top n success process instance order by running duration

2020-07-29 Thread GitBox 


CalvinKirs commented on pull request #3315:
URL: 
https://github.com/apache/incubator-dolphinscheduler/pull/3315#issuecomment-665623595


   
   
   
   
   In fact, the replacement of the CRLF string has been completed here, and I 
don't understand why sonar still reports this error.
   On 07/29/2020 19:46，Yichao Yang wrote：
   
   Good job,
   I think this sonar checker can be removed. @dailidong , Please help to 
confirm.
   
   
   —
   You are receiving this because you commented.
   Reply to this email directly, view it on GitHub, or unsubscribe.



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.

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

[GitHub] [incubator-dolphinscheduler] CalvinKirs commented on pull request #3315: add feature : query top n success process instance order by running duration

2020-07-26 Thread GitBox 


CalvinKirs commented on pull request #3315:
URL: 
https://github.com/apache/incubator-dolphinscheduler/pull/3315#issuecomment-664103800


   Hi, have you encountered any problems? You can add me to WeChat to 
communicate. My WeChat is Kris_Evil



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.

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