Skip to content

Replaced Utils.join() with JDK API.#791

Open
Suchi Amalapurapu (samalapurapu) wants to merge 1 commit into
masterfrom
CC-28213_remove_utils.join
Open

Replaced Utils.join() with JDK API.#791
Suchi Amalapurapu (samalapurapu) wants to merge 1 commit into
masterfrom
CC-28213_remove_utils.join

Conversation

@samalapurapu

Copy link
Copy Markdown
Member

Problem

Solution

Does this solution apply anywhere else?
  • yes
  • no
If yes, where?

Test Strategy

Testing done:
  • Unit tests
  • Integration tests
  • System tests
  • Manual tests

Release Plan

@confluent-cla-assistant

Copy link
Copy Markdown

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

@cla-assistant

cla-assistant Bot commented Oct 11, 2024

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@cla-assistant

cla-assistant Bot commented Oct 11, 2024

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@vbalani002 Vikas Balani (vbalani002) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks Suchi Amalapurapu (@samalapurapu) ! Can we backport this change to the oldest supported feature branch please?

@sappusaketh

Copy link
Copy Markdown

Suchi Amalapurapu (@samalapurapu) or Vikas Balani (@vbalani002) can this be merged? This is blocking us from upgrading to kafka 3.8+

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.

3 participants