EdColeman commented on PR #4122: URL: https://github.com/apache/accumulo/pull/4122#issuecomment-1904973428
One quirk of adding the test files where they currently are, the license check shows a warning because it does not know how to check a *.rf file. (The build completes though) Unsure if a general exclusion for all rfiles, or something more targeted to the file / directory location would be appropiate. @ctubbsii do you have a suggestion or preference? Another curiosity is that the licence check inserted the ASL into the export distcp.txt file, but it does not seem to matter to the import code. The maven warnings: ``` [INFO] --- license:4.3:format (license-headers) @ accumulo-test --- [INFO] Updating license headers... [WARNING] Unknown file extension: .../test/src/main/resources/v2_import_test/data/A0000008.rf [WARNING] Unknown file extension: .../test/src/main/resources/v2_import_test/data/A000000a.rf [WARNING] Unknown file extension: .../test/src/main/resources/v2_import_test/data/A000000b.rf [WARNING] Unknown file extension: .../test/src/main/resources/v2_import_test/data/A0000009.rf [WARNING] Unable to find a comment style definition for some files. You may want to add a custom mapping for the relevant file extensions. ``` -- 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]
