Re: [PR] Be able to load segments on Peons (druid)

["maytasm (via GitHub)" <gi...@apache.org>]

maytasm commented on PR #14239:
URL: https://github.com/apache/druid/pull/14239#issuecomment-2111177145

   This change broke our ingestion job...
   we have the following in the task ingestionspec context map:
     "context": {
       "druid.indexer.fork.property.druid.processing.buffer.sizeBytes": 100000000
     }
   This was working fine in 26.0.0 but after upgrading all our ingestion tasks failed.
   Workaround is to double quote the integer values and make them String:
     "context": {
       "druid.indexer.fork.property.druid.processing.buffer.sizeBytes": "100000000"
     }
   Broken by this PR as StringUtils.format takes in Object as argument but the new method addSystemProperty added in this above PR has String type in the method definition


-- 
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: commits-unsubscribe@druid.apache.org

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


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