roryqi commented on PR #12850: URL: https://github.com/apache/gravitino/pull/12850#issuecomment-5523066285
Addressed the overall review in e7f6ccfa2. The contract is now explicit: job.namespace is the single event metalake (organization) when authorization is enabled; every dataset namespace must match it; missing datasetType defaults to TABLE; supported types and external-target behavior are documented. Inputs and outputs intentionally use the same CAN_ACCESS_METADATA check because this issue is scoped to metadata visibility, not read/write lineage attestation. OpenAPI now documents 201, 400, and 403. Tests cover every input/output position, exact identifiers for all supported types, malformed/denied/internal status mapping, authorization-disabled generic namespaces, authorization-enabled interception, and no dispatch for rejected events. -- 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]
