You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/10/26 13:03:19 UTC

[GitHub] [pulsar] eolivelli opened a new pull request #8379: Issue 8473: testManagedLedgerBookieClientStats fails very often on CI

eolivelli opened a new pull request #8379:
URL: https://github.com/apache/pulsar/pull/8379


   Fixes #8473
   
   ### Modifications
   Parse explicitly Prometheus "+Inf" and "-Inf"
   
   ### Verifying this change
   
   This change is a trivial rework / code cleanup without any test coverage.
   
   


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

[GitHub] [pulsar] merlimat commented on pull request #8379: Issue 8473: testManagedLedgerBookieClientStats fails very often on CI

Posted by GitBox <gi...@apache.org>.
merlimat commented on pull request #8379:
URL: https://github.com/apache/pulsar/pull/8379#issuecomment-716734233


   @eolivelli 
   
   There's a compilation error: 
   
   ```PrometheusMetricsTest.java:[584,24] variable value is already defined in method parseMetrics(java.lang.String)```


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

[GitHub] [pulsar] eolivelli commented on pull request #8379: Issue 8473: testManagedLedgerBookieClientStats fails very often on CI

Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #8379:
URL: https://github.com/apache/pulsar/pull/8379#issuecomment-716531884


   The problem was introduced probably with #6814
   @codelipenghui @hangc0276 PTAL


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

[GitHub] [pulsar] merlimat merged pull request #8379: Issue 8473: testManagedLedgerBookieClientStats fails very often on CI

Posted by GitBox <gi...@apache.org>.
merlimat merged pull request #8379:
URL: https://github.com/apache/pulsar/pull/8379


   


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