Aias00 commented on PR #4388:
URL:
https://github.com/apache/rocketmq-dashboard/pull/4388#issuecomment-5756242740
Addressed the requested contract direction in `327c199f0`:
1. Removed `page/pageSize`; both query tools now accept one `limit` with
default 20 and a
server-side cap of 100.
2. Responses retain `resultMayBeTruncated` and add `skippedCount`, the exact
number omitted by
the local limit from the provider-bounded result.
3. Both YAML descriptions explicitly identify `includeBody=false` as a
breaking output change and
tell existing `items[].body` callers to set `includeBody=true`.
4. `rmq.message.query_by_offset` now explicitly documents that it always
includes body fields and
has no `includeBody` flag.
5. Regenerated `rmqctl/internal/catalog/catalog_gen.go` with the updated
digest.
Verification: focused Java tests pass 23/23; all Go packages pass; `make
verify` passes race,
format, catalog drift, vet, and CLI build; Maven package passes with 0
Checkstyle violations. The full
server run passed 2420/2421 tests, with only the unrelated existing
59-vs-60-second session timing
boundary, which reproduced in isolation.
--
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]
