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 2021/04/13 22:01:26 UTC

[GitHub] [pulsar] merlimat commented on pull request #10217: Adding more permits debug statements to better diagnose permit issues

merlimat commented on pull request #10217:
URL: https://github.com/apache/pulsar/pull/10217#issuecomment-819081766


   @devinbost There are few checkstyle violations reported in the CI jobs: 
   
   ```
   Error:  src/main/java/org/apache/pulsar/broker/service/PulsarCommandSenderImpl.java:[254] (sizes) LineLength: Line is longer than 120 characters (found 130).
   5726
   Error:  src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java:[189,119] (whitespace) OperatorWrap: '+' should be on a new line.
   5727
   Error:  src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java:[212,104] (whitespace) OperatorWrap: '+' should be on a new line.
   5728
   Error:  src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java:[496,98] (whitespace) OperatorWrap: '+' should be on a new line.
   5729
   Error:  src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java:[529,17] (whitespace) WhitespaceAround: 'if' is not followed by whitespace.
   5730
   Error:  src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java:[530,106] (whitespace) OperatorWrap: '+' should be on a new line.
   5731
   Error:  src/main/java/org/apache/pulsar/broker/service/Consumer.java:[239,21] (whitespace) WhitespaceAround: 'if' is not followed by whitespace.
   5732
   Error:  src/main/java/org/apache/pulsar/broker/service/Consumer.java:[240,109] (whitespace) OperatorWrap: '+' should be on a new line.
   5733
   Error:  src/main/java/org/apache/pulsar/broker/service/Consumer.java:[257,9] (whitespace) WhitespaceAround: 'if' is not followed by whitespace.
   5734
   Error:  src/main/java/org/apache/pulsar/broker/service/Consumer.java:[258,114] (whitespace) OperatorWrap: '+' should be on a new line.
   5735
   Error:  src/main/java/org/apache/pulsar/broker/service/Consumer.java:[531,117] (whitespace) OperatorWrap: '+' should be on a new line.
   ```


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