You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@helix.apache.org by GitBox <gi...@apache.org> on 2020/10/14 02:37:21 UTC

[GitHub] [helix] kaisun2000 commented on pull request #1462: Disable TestRoutingTableProviderPeriodicRefresh #1250

kaisun2000 commented on pull request #1462:
URL: https://github.com/apache/helix/pull/1462#issuecomment-708118225


   > I'm pretty sure this is not fixing #1250. Please don't describe the PR in this confusing way.
   > 
   > Also, please mention this issue number in the code comment so we have a tight connection between TODOs and issues.
   Good point, changed the description to disable instead of fix. Also added the tracking issue number as todo.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org