mikebridge commented on PR #44009: URL: https://github.com/apache/superset/pull/44009#issuecomment-5708834444
Prepared by Codex at Mike's request. Author-side accounting at `d6e3d4f11007e3cb0cdbc6a8d8fb7d7660ef2dba`: the identity-lookup guard, removal of the test-only wrapper and attempted-read wording are linked with controls in https://github.com/apache/superset/pull/44009#discussion_r4031458610 , https://github.com/apache/superset/pull/44009#discussion_r4031458772 and https://github.com/apache/superset/pull/44009#discussion_r4031458886 . The earlier two-stage timing, per-entity isolation and warning-only fixes retain their individual replies; persistence-spy coverage is not being relabeled as end-to-end coverage. This supersedes my earlier migration-blocked CI update: repaired-master PostgreSQL run https://github.com/apache/superset/actions/runs/35166197245/job/105027822900 completes with `2790 passed, 165 skipped`; the current rollup has no failing or pending checks. The metric namespace concern is explicitly deferred in https://github.com/apache/superset/pull/44009#discussion_r4033213896 . Reviewer confirmation remains separate; no thread has been automatically resolved. -- 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]
