Add test to coverage msg with the same packetId

Open
#635 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
35/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
java
Domain
backend, testing

Research direction

No file or existing test is named. Start by locating the MQTT packetId handling and related message-consumption tests; add coverage for broker-A consuming msg-1 with packetId=1 before broker-B sends msg-2 with the same packetId, and finish when the test clearly verifies whether msg-2 is discarded.

Written by the indexing model from the issue text.

Description

Motivation

Suppose there are 2 messages in the topic. One is consumed by consumer with packetId=1. but then broker-A crashes and broker-B owns the topic, it will send msg-2 with packetId=1.
Test to coverage if the msg-2 will be discarded.

Dominant language
Java
Stars
190
Forks
56
Avg merge
27m
Merged PRs (30d)
1

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from streamnative/mop

All issues in streamnative/mop

Similar issues

More Java issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.