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

Clarify which Mongo properties are ignored when URI property is set #44384

Closed
wants to merge 1 commit into from

Conversation

nosan
Copy link
Contributor

@nosan nosan commented Feb 20, 2025

See #44382

@nosan nosan changed the title Update MongoProperties properties documentation Update MongoProperties documentation Feb 20, 2025
@nosan nosan changed the title Update MongoProperties documentation Clarify which properties are ignored when MongoProperties URI is specified Feb 20, 2025
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 20, 2025
…ified

Signed-off-by: Dmytro Nosan <dimanosan@gmail.com>
@wilkinsona wilkinsona added type: documentation A documentation update and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 20, 2025
@wilkinsona wilkinsona added this to the 3.3.x milestone Feb 20, 2025
@quaff
Copy link
Contributor

quaff commented Feb 21, 2025

It apply to RedisProperties and others.

@wilkinsona
Copy link
Member

I’m not sure that it does as RedisProperties does not use them same “cannot be set” language that caused confusion here. Let’s focus on Mongo for now. We can consider a broader sweep separately.

@snicoll snicoll changed the title Clarify which properties are ignored when MongoProperties URI is specified Clarify which Mongo properties are ignored when URI property is set Feb 22, 2025
@snicoll snicoll self-assigned this Feb 22, 2025
@snicoll snicoll modified the milestones: 3.3.x, 3.3.10 Feb 22, 2025
snicoll pushed a commit that referenced this pull request Feb 22, 2025
See gh-44384

Signed-off-by: Dmytro Nosan <dimanosan@gmail.com>
snicoll added a commit that referenced this pull request Feb 22, 2025
@snicoll snicoll closed this in acfdd11 Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants