Skip to content

Support a custom DnsAddressResolverGroup builder function#2745

Merged
pderop merged 1 commit intoreactor:mainfrom
samueldlightfoot:address-resolver-provider
Mar 27, 2023
Merged

Support a custom DnsAddressResolverGroup builder function#2745
pderop merged 1 commit intoreactor:mainfrom
samueldlightfoot:address-resolver-provider

Conversation

@samueldlightfoot
Copy link
Copy Markdown
Contributor

Support a user provided function to build a DnsAddressResolverGroup

Copy link
Copy Markdown
Contributor

@pderop pderop left a comment

Choose a reason for hiding this comment

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

Hi @samueldlightfoot ,

thanks for this PR.

One minor thing: can you update the equals and hashCode methods with the new introduced dnsAddressResolverGroupProvider attribute ?

@samueldlightfoot samueldlightfoot force-pushed the address-resolver-provider branch from ba4d8ba to 378a0e0 Compare March 27, 2023 15:14
@samueldlightfoot
Copy link
Copy Markdown
Contributor Author

Hi @samueldlightfoot ,

thanks for this PR.

One minor thing: can you update the equals and hashCode methods with the new introduced dnsAddressResolverGroupProvider attribute ?

Nice spot. Done.

@samueldlightfoot samueldlightfoot requested a review from pderop March 27, 2023 15:54
@violetagg violetagg changed the title Support a custom DnsAddressResolverGroup builder function Support a custom DnsAddressResolverGroup builder function Mar 27, 2023
@pderop pderop merged commit 4effedc into reactor:main Mar 27, 2023
@samueldlightfoot samueldlightfoot deleted the address-resolver-provider branch March 27, 2023 18:13
pderop added a commit that referenced this pull request Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants