apache / gravitino

World's most powerful open data catalog for building a high-performance, geo-distributed and federated metadata lake.
https://datastrato.ai/docs/
Apache License 2.0
681 stars 208 forks source link

[#4040] Update Rat check to ignore Datastrato headers #4044

Closed justinmclean closed 4 days ago

justinmclean commented 4 days ago

What changes were proposed in this pull request?

No longer accept Datastrato headers.

Why are the changes needed?

See above.

Fix: #4040

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Note that until all file headers have changed the Rat test will not pass. I made it a separate PR for easy review.

justinmclean commented 4 days ago

This is failing for a valid reason as someone merged a file without an ASF header.