Skip to content

Conversation

@sakshamVerma08
Copy link

Linked Issue) #2373

📄 Summary
This PR introduces a new feature to the automq-perf-test.sh script, allowing users to reuse existing topics for catch-up read (cold read) performance testing. By specifying a topic prefix, the script can identify and utilize pre-existing topics, thereby eliminating the need to create new ones and wait for message accumulation. This enhancement significantly reduces the time required to set up and execute catch-up read tests.

🛠️ Implementation Details
New Parameter: Added --catchup-topic-prefix to specify the prefix of existing topics to be reused.

Topic Discovery: The script lists existing topics matching the provided prefix and reuses them for testing.

Skip Accumulation Phase: When existing topics are reused, the script bypasses the message accumulation phase, expediting the testing process.

Code Modifications: Updated the PerfCommand class to handle the new parameter and implement the logic for reusing topics.

@SCNieh
Copy link
Contributor

SCNieh commented May 6, 2025

@sakshamVerma08 the code you submitted does not even compile and doesn't make any sense, please note that it's a violation of GitHub Community Code of Conduct since you have submitted same invalid code more than once, if you keep being irresponsible to your submission we'll report your action to Github and block you from the community

@SCNieh SCNieh closed this May 6, 2025
@sakshamVerma08 sakshamVerma08 deleted the sakshamVerma08/catchup-read-solution branch May 6, 2025 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants