Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use InetAddressResolverProvider and add tests #1354

Merged
merged 2 commits into from
Mar 28, 2024

Conversation

stIncMale
Copy link
Member

@stIncMale stIncMale commented Mar 28, 2024

This is a backport of #1353.

The changes made to the :graalvm-native-image-app Gradle project were not included, neither were the API documentation changes made to MongoClientSettings, the rest of the changes was taken as is.

JAVA-5390

This is a backport of 9a02f81
from `master` to `5.0.x`

JAVA-5390
@stIncMale stIncMale requested a review from jyemin March 28, 2024 14:06
@stIncMale stIncMale self-assigned this Mar 28, 2024
*
* <p>If set, it will be used to look up the {@link java.net.InetAddress} for each host, via
* {@link InetAddressResolver#lookupByName(String)}. Otherwise, {@link java.net.InetAddress#getAllByName(String)} will be used.
* Gets the explicitly set {@link InetAddressResolver} to use for looking up the {@link java.net.InetAddress} instances for each host.
Copy link
Contributor

Choose a reason for hiding this comment

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

Just FYI, we generally don't update Javadoc in patch releases, because we don't republish the Javadoc for patch releases. But it doesn't do any harm, so it's ok to leave this.

Copy link
Member Author

@stIncMale stIncMale Mar 28, 2024

Choose a reason for hiding this comment

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

Right, thanks! It's still better to have the source code consistent with the published API documentation, so I rolled the change back.

@stIncMale stIncMale merged commit 1b8bce4 into mongodb:5.0.x Mar 28, 2024
2 checks passed
@stIncMale stIncMale deleted the JAVA-5390_backport branch March 28, 2024 15:13
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