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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to