Properly handle the case where there are more than 10 streams #14304
Conversation
Previously log file parser only used the first character of the stream id when converting from a string to a number. This change makes sure to get all the digits.
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_8_1_X. It involves the following packages: FWCore/Concurrency @cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
|
please test |
|
+1 |
cmsbuild
added
core-approved
fully-signed
and removed
core-pending
pending-signatures
labels
|
The tests are being triggered in jenkins. |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar |
|
@VinInn This fixes the problem where only 10 streams were being reported. Now it reports all 24 streams from your job. |
|
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Previously log file parser only used the first character of the stream id when converting from a string to a number. This change makes sure to get all the digits.