These files are automatically detected as binary files on unix and thus automatically ignored by the RAT plugin, but not on Windows. Most Flink devs run unix and don't run into this problem, and thus don't define a proper exclusion.
This happens regularly.

It would be great if you could open a PR to add the missing exclusions.

On 09.07.2018 21:47, NEKRASSOV, ALEXEI wrote:
https://flink.apache.org/contribute-code.html recommends running 'mvn clean 
verify', but when I do (on master), the build fails:

...
[INFO] Parsing exclusions from C:\usr\git\flink\.gitignore
...
[INFO] Rat check: Summary over all files. Unapproved: 2, unknown: 2, generated: 
0, approved: 8064 licenses.
...
[INFO] flink .............................................. FAILURE [ 20.726 s]

Files with unapproved licenses:

   flink-runtime/src/test/resources/heap_keyed_statebackend_1_2.snapshot
   flink-runtime/src/test/resources/heap_keyed_statebackend_1_5_map.snapshot

Do we need to update .gitignore? For these two files only, or for some new 
pattern?

Thanks,
Alex


Reply via email to