Skip to content

Methods in KafkaConnectionDetails are named inconsistently#35733

Closed
izeye wants to merge 1 commit intospring-projects:mainfrom
izeye:polish-gh-34770
Closed

Methods in KafkaConnectionDetails are named inconsistently#35733
izeye wants to merge 1 commit intospring-projects:mainfrom
izeye:polish-gh-34770

Conversation

@izeye
Copy link
Copy Markdown
Contributor

@izeye izeye commented Jun 5, 2023

This PR polishes the changes made in gh-34770 a bit by completing renaming nodes to servers.

It contains a breaking change, but I didn't do anything for it as I don't know what the preference for it is here.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 5, 2023
@philwebb philwebb added the for: team-attention An issue we'd like other members of the team to review label Jun 5, 2023
@philwebb
Copy link
Copy Markdown
Member

philwebb commented Jun 5, 2023

Thanks @izeye! I wonder what we should do about that method rename since 3.1 is already out. We could deprecate the old one or we could risk it since 3.1 hasn't been out that long and we're probably the only implementers right now.

@philwebb philwebb changed the title Polish gh-34770 Polish KafkaConnectionDetails Jun 5, 2023
@wilkinsona
Copy link
Copy Markdown
Member

wilkinsona commented Jun 6, 2023

@izeye If it's not too much trouble, would you mind rebasing this PR on top of the latest main? I'm trying to figure out why the PR build ran for 6 hours(!) and hoping that rebasing on top of 3922f1b will help with that.

@izeye izeye force-pushed the polish-gh-34770 branch from 595d7b7 to 60ffe79 Compare June 6, 2023 23:32
@izeye
Copy link
Copy Markdown
Contributor Author

izeye commented Jun 6, 2023

@wilkinsona Sure, done!

@wilkinsona
Copy link
Copy Markdown
Member

wilkinsona commented Jun 7, 2023

Thanks very much, @izeye. The PR build was cancelled but we got a thread dump this time which revealed #35771. If you can be bothered, another rebase would be much appreciated.

@izeye izeye force-pushed the polish-gh-34770 branch from 60ffe79 to 62ac0fc Compare June 7, 2023 11:19
@izeye
Copy link
Copy Markdown
Contributor Author

izeye commented Jun 7, 2023

@wilkinsona Great, done again.

@wilkinsona
Copy link
Copy Markdown
Member

wilkinsona commented Jun 23, 2023

We discussed this and decided that we'd risk the breaking API change. It's a shame we didn't get this into 3.1.1 but I don't think that should change the decision.

@wilkinsona wilkinsona removed for: team-attention An issue we'd like other members of the team to review status: waiting-for-triage An issue we've not yet triaged labels Jun 23, 2023
@wilkinsona wilkinsona changed the title Polish KafkaConnectionDetails Methods in KafkaConnectionDetails are named inconsistently Jun 23, 2023
@wilkinsona wilkinsona added the type: bug A general bug label Jun 23, 2023
@wilkinsona wilkinsona added this to the 3.1.x milestone Jun 23, 2023
@wilkinsona wilkinsona self-assigned this Jun 30, 2023
@wilkinsona wilkinsona mentioned this pull request Jun 30, 2023
Closed
@wilkinsona wilkinsona modified the milestones: 3.1.x, 3.1.2 Jun 30, 2023
wilkinsona pushed a commit that referenced this pull request Jun 30, 2023
@izeye @wilkinsona
Name methods consistently in KafkaConnectionDetails
58a9961 
See gh-35733
@wilkinsona
Copy link
Copy Markdown
Member

wilkinsona commented Jun 30, 2023

Thanks very much, @izeye.

@izeye izeye deleted the polish-gh-34770 branch June 30, 2023 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wilkinsona wilkinsona wilkinsona approved these changes

Assignees

@wilkinsona wilkinsona

Labels

type: bug A general bug

Projects

None yet

Milestone

3.1.2

Development

Successfully merging this pull request may close these issues.

4 participants

@izeye @philwebb @wilkinsona @spring-projects-issues