I3eka commented on PR #44561: URL: https://github.com/apache/superset/pull/44561#issuecomment-6435204046
Reviewed the current discussion and the focused implementation, and merged Apache master `037abb7d0b433c1274ab55f95e5613310f132462` without rewriting the branch. Published head: `04646bcb1b6b2ed6557c3e1e030abec7fa4d4964`. There are no inline review threads or outstanding substantive reviewer findings on this PR in this audit. I also checked the repeated automated migration notices (latest: https://github.com/apache/superset/pull/44561#issuecomment-6307964753); the actual revision-head test passes. The notice alone is not evidence of a broken migration graph. - 717 tests passed: the full AI unit suite and the single-migration-head check. - 105 AI frontend tests passed on the identical full frontend Git tree `91c05bee9972b32d81248e1a3575e8f4e31edd75` in #44497; reused that result. Frontend type/lint hooks were rerun on this branch. - All applicable pre-commit hooks passed for the 117 files in this PR delta. SQLite route/DAO/runtime tests exercise accepted metadata, duplicate claims and terminal replay. This does not provide whole-host crash recovery or live broker delivery proof. Updated the description to match this head and its validation. This stays **Draft / ON HOLD** behind #42805 and SIP #42806. No deployment, live database migration or external provider call was performed; GitHub CI is separate from these local checks. -- 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]
