Skip to content

Add known issue for Java SDK #2604

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

Merged
merged 3 commits into from
Dec 4, 2024

Conversation

pauljewellmsft
Copy link
Contributor

No description provided.

@pauljewellmsft pauljewellmsft marked this pull request as draft December 4, 2024 20:41

| Client library | Versions impacted | Minimum safe version | Recommended action |
| --- | --- | --- | --- |
| Azure File Share | 12.15.0 to 12.25.0 | 12.26.0-beta.1 | [Update to latest version or minimum 12.26.0-beta.1](https://mvnrepository.com/artifact/com.azure/azure-storage-file-share) |
Copy link
Contributor

Choose a reason for hiding this comment

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

we should call out 12.16.0 as the minimum safe version. We can't realistically expect customers to upgrade to a beta release

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Updated to 12.26.0

Choose a reason for hiding this comment

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

Let's change this to 12.25.1, the release today is a patch release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated to 12.25.1 - thx for clarifying.

Copy link
Contributor

Learn Build status updates of commit 454c400:

⚠️ Validation status: warnings

File Status Preview URL Details
docs-ref-services/latest/storage.md ⚠️Warning View (azure-java-stable) Details

docs-ref-services/latest/storage.md

  • Line 7, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'storage'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 45557b4:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-services/latest/storage.md ✅Succeeded View (azure-java-stable)

For more details, please refer to the build report.

For any questions, please:


#### Issue description

In the **azure-storage-file-share** package, the `ShareFileClient.downloadToFile()` method can write incorrect data to a file when more than 5 retries occur due to partial network responses while streaming data. The maximum number of retries is unintentionally higher (up to a maximum of 15), which can result in writing to incorrect positions in the file when the operation is retried more than 5 times. The data in Storage is still correct.

Choose a reason for hiding this comment

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

The data in Storage is still correct.

Maybe we should say "This issue does not affect data in the Storage service, only the data being written to the local filesystem."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated. thx

Copy link
Contributor

Learn Build status updates of commit 1f9cc51:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-services/latest/storage.md ✅Succeeded View (azure-java-stable)

For more details, please refer to the build report.

For any questions, please:

@pauljewellmsft pauljewellmsft marked this pull request as ready for review December 4, 2024 22:17
@pauljewellmsft pauljewellmsft merged commit c8fd9ee into MicrosoftDocs:main Dec 4, 2024
2 checks passed
@pauljewellmsft pauljewellmsft deleted the java-known-issues branch December 4, 2024 22:19
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.

3 participants