david-streamlio commented on a change in pull request #2865: Refactored JCloud 
Tiered Storage
URL: https://github.com/apache/pulsar/pull/2865#discussion_r229770461
 
 

 ##########
 File path: 
tiered-storage/jcloud/src/test/java/org/apache/bookkeeper/mledger/offload/jcloud/BlobStoreTestBase.java
 ##########
 @@ -26,9 +28,12 @@
 import org.testng.annotations.AfterMethod;
 import org.testng.annotations.BeforeMethod;
 
+import lombok.extern.slf4j.Slf4j;
+
+@Slf4j
 
 Review comment:
   Ivan, 
   
   I definitely appreciate the amount of effort required to review the 
additional code changes that aren't directly related to the refactoring. My 
intent was to avoid introducing a "check-style bomb" that would create 100s of 
violations when someone attempted to build the entire code based and validate 
its style. I chose to err on the side of caution and NOT submit code that I 
knew was bad. 
   
   I can back out these changes if you prefer.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to