tuhaihe opened a new pull request, #160: URL: https://github.com/apache/cloudberry-pxf/pull/160
## What Replace four directory-wide Apache RAT excludes with explicit per-file lists, in the existing "legacy Greenplum PXF files" block: - `server/pxf-api/src/main/java/.../api/function/**` (4 files) - `server/pxf-service/src/main/java/.../service/controller/**` (8 files) - `server/pxf-service/src/main/java/.../service/rest/**` (4 files without a header) - `server/pxf-service/src/main/java/.../service/spring/**` (9 files) ## Why Wholesale directory excludes mean RAT cannot catch a *new* file added to those directories without an ASF license header. Listing the legacy Greenplum-derived files individually keeps the same behavior for the existing files (they stay untouched, per the documented rationale in `pom.xml`) while making RAT effective for everything added in the future. As a side effect, `PxfBackendControlResource.java` and `PxfStatActivityResource.java`, which already carry an ASF header, go back through the normal RAT scan (they are approved). ## Verification `mvn apache-rat:check` still passes: `Unapproved: 0, unknown: 0, approved: 318` (previously 312, the difference being the two rest files now scanned). `target/rat.txt` shows both as `AL`. -- 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]
