adityamparikh opened a new pull request, #59:
URL: https://github.com/apache/solr-mcp/pull/59
## Summary
Verifies that the JSON wire format (introduced in #48) works correctly with
Solr 10, and extends the CI matrix to cover all supported versions.
- **Solr 10 compatibility verified**: all integration tests pass with
`solr:10-slim` using the JSON wire format (no JavaBin)
- **`/admin/mbeans` removed in Solr 10**: `getCacheMetrics()` and
`getHandlerMetrics()` in `CollectionService` now catch `RuntimeException`
(which covers `RemoteSolrException`) so collection stats degrade gracefully to
`null` instead of crashing
- **CI matrix extended**: `build-and-publish.yml` now tests against Solr
8.11, 9.4, 9.9, 9.10, and 10 on every PR and push to main
- **Docs updated**: `AGENTS.md` updated to reflect Solr 10 as a tested,
supported version
## Changes
### `CollectionService.java`
Added `| RuntimeException` to catch clauses in `getCacheMetrics()` and
`getHandlerMetrics()`:
```java
} catch (SolrServerException | IOException | RuntimeException e) {
// RuntimeException covers SolrException subclasses (e.g.
RemoteSolrException)
// thrown when the /admin/mbeans endpoint is unavailable (removed in
Solr 10).
return null;
}
```
### `.github/workflows/build-and-publish.yml`
New `solr-compatibility` job with matrix strategy:
```yaml
matrix:
solr-version: ["8.11-slim", "9.4-slim", "9.9-slim", "9.10-slim", "10-slim"]
```
### `AGENTS.md`
Updated Solr 10 section from "upcoming breaking changes" to current
compatibility status.
## Test plan
- [x] `./gradlew build` passes (default Solr 9.9)
- [x] `./gradlew test -Dsolr.test.image=solr:10-slim` passes (0 new failures)
- [x] `./gradlew test -Dsolr.test.image=solr:9.10-slim` passes
- [x] All existing Solr 8.11, 9.4, 9.9 tests continue to pass
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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]
