You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/07/04 21:11:24 UTC

[GitHub] [incubator-druid] leventov opened a new issue #7360: Break up SeekableStreamIndexTaskRunner.runInternal()

leventov opened a new issue #7360: Break up SeekableStreamIndexTaskRunner.runInternal()
URL: https://github.com/apache/incubator-druid/issues/7360
 
 
   This method has 550+ lines of code. Its cyclomatic complexity is 74 (the reasonable limit of cyclomatic complexity for methods is considered 14).
   
   @jihoonson @gianm probably you are the only persons who understand this method. It would probably take days for anybody else to understand it to the level to be able to make any change in it with confidence.

----------------------------------------------------------------
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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org