mikebridge commented on PR #43493: URL: https://github.com/apache/superset/pull/43493#issuecomment-5413829396
Thanks, Bito. The operational kill-switch guidance was removed from `init_versioning()` intentionally to avoid duplicating operator-facing rationale in implementation docstrings. The full guidance remains in `UPDATING.md` under “Version history is on by default”: it explains that `ENABLE_VERSIONING_CAPTURE` is a permanent operational kill-switch, stops capture within a restart without revert-and-redeploy, preserves existing rows for read-only access, and disables Restore. The initializer docstring retains the non-derivable implementation details: which listener groups are gated/detached and why its fallback is `False`. No additional duplication is needed here. -- 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]
