aminghadersohi commented on PR #36038: URL: https://github.com/apache/superset/pull/36038#issuecomment-3505492958
## MySQL Test Failure - Flaky Test (Unrelated to Changes) The `test-mysql` failure is a **flaky test** unrelated to the `stateless_http` parameter changes: ### Failed Test: ``` FAILED tests/integration_tests/charts/commands_tests.py::TestChartsUpdateCommand::test_update_v1_response assert datetime.datetime(2025, 11, 8, 0, 21, 1) != datetime.datetime(2025, 11, 8, 0, 21, 1) ``` ### Root Cause: This is a timing issue where MySQL's timestamp precision causes `last_saved_at` to not change when updates happen within the same second. This is a known flaky test. ### Why It's Unrelated: This PR only modifies: - `superset/mcp_service/__main__.py` - Added env var configuration - `superset/mcp_service/app.py` - Removed deprecated parameter placement - `superset/mcp_service/server.py` - Added env var configuration None of these changes affect chart update timestamps or database operations. ### All Other Tests: ✅ PASSING - ✅ test-postgres (current, next, previous) - ✅ test-sqlite - ✅ unit-tests (current, next, previous) - ✅ All Cypress/Playwright tests - ✅ Pre-commit hooks (current, next, previous) This is a known flaky test and should be retriggered or can be safely ignored as it's unrelated to the MCP service changes. -- 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]
