Skip to content

Address review comments#221

Merged
fvaleri merged 10 commits into
showuon:cl3from
fvaleri:jun-fixes
Jul 14, 2026
Merged

Address review comments#221
fvaleri merged 10 commits into
showuon:cl3from
fvaleri:jun-fixes

Conversation

@fvaleri

@fvaleri fvaleri commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Mainly javadoc and description updates from review comments.

fvaleri added 5 commits July 13, 2026 13:00
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
@fvaleri
fvaleri requested a review from showuon July 13, 2026 16:03
@fvaleri
fvaleri marked this pull request as draft July 13, 2026 16:03
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
@fvaleri
fvaleri marked this pull request as ready for review July 13, 2026 19:10

@showuon showuon left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@showuon

showuon commented Jul 14, 2026

Copy link
Copy Markdown
Owner

fvaleri#9 to address JR39 and JR42. cc @fvaleri

@fvaleri

fvaleri commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator Author

Mmm, topicName to topicId change broke the code.

fvaleri added 3 commits July 14, 2026 16:45
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
@fvaleri

fvaleri commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator Author
================================================================================
SESSION REPORT (ALL TESTS)
ducktape version: 0.14.0
session_id:       2026-07-14--002
run time:         15 minutes 49.485 seconds
tests run:        24
passed:           24
flaky:            0
failed:           0
ignored:          0
================================================================================

@fvaleri
fvaleri merged commit 8fa30ea into showuon:cl3 Jul 14, 2026
1 check passed
@fvaleri
fvaleri deleted the jun-fixes branch July 14, 2026 15:15
@showuon

showuon commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Ah, sorry for breaking the build. Thanks for fixing it.

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