[GitHub] [ozone] adoroszlai commented on a diff in pull request #3889: HDDS-7416. Unify license headers for hdds-annotation-processing

[GitBox <gi...@apache.org>]

adoroszlai commented on code in PR #3889:
URL: https://github.com/apache/ozone/pull/3889#discussion_r1006160046


##########
hadoop-hdds/annotations/src/main/java/org/apache/ozone/annotations/RequestFeatureValidatorProcessor.java:
##########
@@ -1,13 +1,13 @@
 /*
  * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with this
- * work for additional information regarding copyright ownership.  The ASF

Review Comment:
   I think we should keep double space between sentences to minimize future changes: almost all files have double space.
   
   ```
   $ rg -Fl 'agreements. See' | wc -l                                                                                                                                        
   24
   
   $ rg -Fl 'agreements.  See' | wc -l
   3886
   ```
   
   Similarly:
   
   ```
   $ rg -Fl 'ownership. The' | wc -l
   24
   
   $ rg -Fl 'ownership.  The' | wc -l
   3485
   ```



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

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org