Skip to content

Add Javadoc to specify that CSOT does not limit socket writes#1791

Merged
vbabanin merged 8 commits intomongodb:mainfrom
vbabanin:JAVA-5569
Mar 20, 2026
Merged

Add Javadoc to specify that CSOT does not limit socket writes#1791
vbabanin merged 8 commits intomongodb:mainfrom
vbabanin:JAVA-5569

Conversation

@vbabanin
Copy link
Copy Markdown
Member

@vbabanin vbabanin requested a review from a team as a code owner August 15, 2025 01:03
@vbabanin vbabanin requested review from stIncMale and removed request for a team August 15, 2025 01:03
@vbabanin vbabanin changed the title Add Javadoc to specify that CSOT does not limit socket writes in sync… Add Javadoc to specify that CSOT does not limit socket writes Aug 15, 2025
@vbabanin vbabanin marked this pull request as draft August 20, 2025 19:39
@vbabanin vbabanin self-assigned this Dec 11, 2025
# Conflicts:
#	driver-core/src/test/resources/specifications
JAVA-5569
@vbabanin vbabanin marked this pull request as ready for review January 26, 2026 16:13
Copy link
Copy Markdown
Member

@stIncMale stIncMale left a comment

Choose a reason for hiding this comment

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

The last reviewed commit is c331d15.

Comment thread driver-legacy/src/main/com/mongodb/MongoClientOptions.java Outdated
Comment thread driver-core/src/main/com/mongodb/MongoClientSettings.java Outdated
Comment thread driver-legacy/src/main/com/mongodb/MongoClientOptions.java
Comment thread driver-core/src/main/com/mongodb/MongoClientSettings.java
Comment thread driver-sync/src/main/com/mongodb/client/gridfs/GridFSBucket.java Outdated
Comment thread driver-kotlin-sync/src/main/kotlin/com/mongodb/kotlin/client/MongoCluster.kt Outdated
Comment thread driver-sync/src/main/com/mongodb/client/MongoCluster.java Outdated
Comment thread driver-sync/src/main/com/mongodb/client/MongoCluster.java
Comment thread driver-core/src/test/resources/specifications Outdated
# Conflicts:
#	driver-core/src/test/resources/specifications
- Fix note format from <strong>NOTE:</strong> to <p>Note: across driver-core, driver-sync, and driver-legacy to match codebase conventions
- Use plain Note: (no HTML) in driver-kotlin-sync KDoc to match Kotlin package conventions
- Keep Note that format in MongoClientOptions to match existing convention within that class
- Add socket-writes caveat note to all getTimeout/getDefaultTimeout getter methods
- Fix @SInCE 4.x to @SInCE 5.2 in GridFSBucket.getTimeout and GridFSBucket.withTimeout

JAVA-5569
* @param timeUnit the time unit
* @return the timeout in the given time unit
* @since 4.x
* @since 5.2
Copy link
Copy Markdown
Member Author

@vbabanin vbabanin Mar 15, 2026

Choose a reason for hiding this comment

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

I changed @since 4.x to @since 5.2 on GridFSBucket.getTimeout and GridFSBucket.withTimeout. These methods were introduced in 5.2 as part of CSOT (commit 01aff5a078), and the @since tag wasn’t updated accordingly.

@vbabanin vbabanin requested a review from stIncMale March 15, 2026 21:57
Copy link
Copy Markdown
Member

@stIncMale stIncMale left a comment

Choose a reason for hiding this comment

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

The last reviewed commit is 224f406.

Comment thread driver-core/src/main/com/mongodb/ClientEncryptionSettings.java Outdated
@stIncMale
Copy link
Copy Markdown
Member

Note also that the Evergreen build failed.

@vbabanin vbabanin requested a review from stIncMale March 19, 2026 04:13
Copy link
Copy Markdown
Member

@stIncMale stIncMale left a comment

Choose a reason for hiding this comment

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

The last reviewed commit is 1b206ac.

@vbabanin vbabanin merged commit 2e24eb6 into mongodb:main Mar 20, 2026
53 checks passed
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