Aggarwal-Raghav commented on code in PR #6313: URL: https://github.com/apache/hive/pull/6313#discussion_r3119457182
########## accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/TestAccumuloDefaultIndexScanner.java: ########## @@ -7,7 +7,7 @@ * "License"); you may not use this file except in compliance * with the License. You may obtain a copy of the License at * <p> Review Comment: Thanks for the PR @okumin , quick question, few license header has `<p>` compared to other files is that ok? I raised https://github.com/apache/tez/pull/488/ and used `maven spotless plugin` to fix the license header where java-header contains the correct asf license. ``` <java> ... <licenseHeader> <file>path-to-file/java-header</file> </licenseHeader> </java> ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
