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/11/19 04:54:33 UTC

[GitHub] [incubator-druid] suneet-amp opened a new issue #8904: Replace StringUtils#format in CachingLocalSegmentAllocator#getSequenceName

suneet-amp opened a new issue #8904: Replace StringUtils#format in CachingLocalSegmentAllocator#getSequenceName
URL: https://github.com/apache/incubator-druid/issues/8904
 
 
   StringUtils#format is slow because it uses pattern matching under the covers to replace the format string with the variables.
   
   Since this is called in a loop from InputSourceProcessor#process we should see a perf gain by switching this to using a StringBuilder or String addition instead.

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