[ISSUE #299]polish(example) remove selection count

[ni-ze]

No description provided.

[ni-ze]

close #299

[RockteMQ-AI]

Review: Approved ✅

PR: #310 — Fix example
Type: Bug fix (1 file, +3/-3)

Assessment

Fixes example code. Minimal change.

Verdict

✅ Simple fix.

---

🤖 Automated review by oss-sentinel-ai

[RockteMQ-AI]

LGTM — Clean removal of unused count(SelectAction) overload, resetOffsetHandler field, and stale example.

Note: removing GroupedStream.count(SelectAction) is a breaking API change for downstream users. Consider mentioning this in release notes.

---

Automated review by github-manager-bot

[RockteMQ-AI]

Summary

Code cleanup — removes unused API surface and dead code:

1. GroupedStream.count(SelectAction) overload — the selectAction parameter was accepted but never used in the aggregation logic (always just agg + 1)
2. MessageQueueListenerWrapper.resetOffsetHandler — unused field and setter
3. Stale Demo.java example

Findings

• [Info] Removing the count(SelectAction) overload is a minor breaking API change for any external users of GroupedStream. Since the parameter was silently ignored, this is the right call — but worth noting in release notes.

LGTM.

[RockteMQ-AI]

Issue Evaluation

Category: enhancement (code cleanup) | Status: Stale — Needs Champion

This issue references #299 and proposes polishing examples by removing selection count. Opened August 2023, no activity since.

Assessment:

• This appears to be a minor example cleanup task
• No description or implementation details were provided
• The referenced issue There's no difference between the implementation for the two count functions of GroupedStreamImpl #299 may contain additional context

Recommendation: If still relevant, please provide:

1. Which example(s) need to be updated
2. What "selection count" refers to specifically
3. Or submit a PR with the proposed changes

If no activity occurs, this issue may be closed as stale.

---

Automated evaluation by @RockteMQ-AI