mikebridge commented on PR #42797: URL: https://github.com/apache/superset/pull/42797#issuecomment-5194870900
Launch-review follow-up at `4fd4a80d92`: - Required UUID identity in `identity_filter`, `current_version_number`, and `current_live_transaction_id`; production helpers can no longer fall back to ambiguous id-only reads. - Kept id-only queries only as explicit regression-test controls. - Made both committed integration-test cleanup paths failure-safe with `finally`. - Reflowed the constitution-violating docstring summaries. - Reduced the identity helper documentation to its stable contract. - Updated the PR description and seven-test inventory to match HEAD. Verification: 13 focused tests pass; MyPy and affected-file pre-commit pass; full pre-commit passes apart from the documented unrelated frontend formatter skip. -- 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]
