You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudberry.apache.org by "tuhaihe (via GitHub)" <gi...@apache.org> on 2026/09/23 02:00:10 UTC
[PR] Replace directory-wide RAT excludes with per-file excludes [cloudberry-pxf]
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: commits-unsubscribe@cloudberry.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cloudberry.apache.org
For additional commands, e-mail: commits-help@cloudberry.apache.org
Re: [PR] Replace directory-wide RAT excludes with per-file excludes [cloudberry-pxf]
Posted by "tuhaihe (via GitHub)" <gi...@apache.org>.
tuhaihe merged PR #160:
URL: https://github.com/apache/cloudberry-pxf/pull/160
--
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: commits-unsubscribe@cloudberry.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cloudberry.apache.org
For additional commands, e-mail: commits-help@cloudberry.apache.org