You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by "terrymanu (via GitHub)" <gi...@apache.org> on 2026/07/29 11:41:37 UTC
[PR] Simplify MCP LLM E2E test coverage [shardingsphere]
terrymanu opened a new pull request, #39271:
URL: https://github.com/apache/shardingsphere/pull/39271
- consolidate LLM validation into four autonomous HTTP scenarios
- remove guided and duplicated conversation harness code
- parallelize LLM scenarios and streamline model readiness checks
- add MCP transport timeouts and logging conformance coverage
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Simplify MCP LLM E2E test coverage [shardingsphere]
Posted by "terrymanu (via GitHub)" <gi...@apache.org>.
terrymanu merged PR #39271:
URL: https://github.com/apache/shardingsphere/pull/39271
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Simplify MCP LLM E2E test coverage [shardingsphere]
Posted by "terrymanu (via GitHub)" <gi...@apache.org>.
terrymanu commented on PR #39271:
URL: https://github.com/apache/shardingsphere/pull/39271#issuecomment-5118419860
### Result
**Review Result: Mergeable**
No code-correctness issues requiring changes were found at the latest public head.
### Evidence
- Reviewed head: `84360ec776a1d0d13f8e261c8c13f6cf89308900`
- `.github/workflows/e2e-mcp.yml:270-282`: The four matrix scenarios now have an explicit concurrency limit.
- `docs/document/content/test-manual/mcp-e2e-test/_index.en.md:16,96,122` and the corresponding Chinese content now match the four HTTP scenarios and the `answer.txt` artifact.
- Remaining references to deleted types and `expected-answer.txt`: 0.
- Focused tests: all 39 tests passed with exit code 0.
- Spotless, Checkstyle, and YAML parsing all passed.
### Coverage
- Authoritative GitHub changed files: 57; the local triple-dot file list matched exactly.
- Seven behavior clusters were reviewed: workflow, HTTP scenarios, conversation orchestration, artifacts, model readiness, transport lifecycle, and bilingual manuals.
- Root Cause, Blast Radius, Tests/Runtime, and Convergence passes found no new issues.
- No unresolved evidence or coverage gap could change the blocker set.
- This is a Code Correctness result only. CI was not reviewed, and the Docker- and real-model-dependent LLM E2E suite was not run locally.
--
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: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org