goutamadwant commented on PR #12619: URL: https://github.com/apache/seatunnel/pull/12619#issuecomment-5977039969
Thanks @SEZ9. Updated in 4b9e8ee: 1. Byte check now runs after the name and after each kept path; test `rejectsOneLargeVertexBeforeRetainingAllPaths` fails if a path is read past the limit. 2. Any non-null `long` job ID is accepted; `acceptsAnyNonNullJobId` covers 0 and negatives. 3. Not reachable: `TablePath` rejects a null/empty table name ([TablePath.java#L38-L45](https://github.com/apache/seatunnel/blob/309b15effe99423f73b6007cbe61b97bb6af7b24/seatunnel-api/src/main/java/org/apache/seatunnel/api/table/catalog/TablePath.java#L38-L45)). 4. Transform chains are typed `TRANSFORM` by [ActionUtils.java#L24-L33](https://github.com/apache/seatunnel/blob/309b15effe99423f73b6007cbe61b97bb6af7b24/seatunnel-engine/seatunnel-engine-core/src/main/java/org/apache/seatunnel/engine/core/dag/actions/ActionUtils.java#L24-L33); a real transform job is covered by the Phase 3 E2E. 5. Added `PARTIAL` with `DATASET_METADATA_PARTIAL`; STIP body updated. 6. Javadoc: the Phase 3 servlet maps reasons to 409/413. 7. Javadoc on `DEFAULT` citing STIP section 4; fixed in V1. 8. `requireNonNull(limits)` with a test. -- 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]
