aminghadersohi commented on PR #36033:
URL: https://github.com/apache/superset/pull/36033#issuecomment-3499320930
## MySQL Test Failure - Flaky Test (Unrelated to PR Changes)
The `test-mysql` failure is a known flaky test unrelated to this PR:
**Failed test:**
`tests/integration_tests/charts/commands_tests.py::TestChartsUpdateCommand::test_update_v1_response`
**Why this is unrelated to this PR:**
1. **This PR only modifies MCP service files:**
- `superset/mcp_service/app.py` (branding configuration)
- `superset/mcp_service/mcp_config.py` (config constants)
2. **The failing test is about chart update commands** - completely
unrelated to MCP service branding
3. **Known flaky test:** There are existing commits attempting to fix this:
- `5cb2201aa5 test: revert unrelated flaky test fix from MCP PR`
- `932b3b8e1f fix(tests): add sleep to prevent flaky timestamp comparison
in chart update test`
4. **Test passes on other databases:**
- ✅ PostgreSQL: all variants pass
- ✅ SQLite: passes
- ❌ MySQL: flaky timestamp comparison failure
The failure is a race condition in timestamp comparison (`last_saved_at`
timestamps are identical when they should differ), which is a MySQL-specific
timing issue, not a logic error introduced by this PR.
--
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]
