[SAMZA-1414] SamzaContainer log statements incorrectly expect containerId is a number

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.14.0
• Labels: newbie
• Assignee: Unassigned
• Reporter: Jake Maes
• Created: 05/Sep/17 15:00
• Updated: 19/Sep/17 21:44
• Resolved: 19/Sep/17 21:44

Description

The line numbers wont match up with the latest, but here's a stack trace.

java.util.IllegalFormatConversionException: d != java.lang.String
    at java.util.Formatter$FormatSpecifier.failConversion(Formatter.java:4302)
    at java.util.Formatter$FormatSpecifier.printInteger(Formatter.java:2793)
    at java.util.Formatter$FormatSpecifier.print(Formatter.java:2747)
    at java.util.Formatter.format(Formatter.java:2520)
    at java.util.Formatter.format(Formatter.java:2455)
    at java.lang.String.format(String.java:2940)
    at scala.collection.immutable.StringLike$class.format(StringLike.scala:266)
    at scala.collection.immutable.StringOps.format(StringOps.scala:31)
    at org.apache.samza.container.SamzaContainer$$anonfun$startLocalityManager$4.apply(SamzaContainer.scala:857)
    at org.apache.samza.container.SamzaContainer$$anonfun$startLocalityManager$4.apply(SamzaContainer.scala:857)
    at org.apache.samza.util.Logging$class.warn(Logging.scala:74)
    at org.apache.samza.container.SamzaContainer.warn(SamzaContainer.scala:640)
    at org.apache.samza.container.SamzaContainer.startLocalityManager(SamzaContainer.scala:856)
    at org.apache.samza.container.SamzaContainer.run(SamzaContainer.scala:684)
    at org.apache.samza.runtime.LocalContainerRunner.run(LocalContainerRunner.java:104)
    at org.apache.samza.runtime.LocalContainerRunner.main(LocalContainerRunner.java:149)

Looks like the log statements need to expect containerId as a string.

Issue Links

• links to: GitHub Pull Request #298