feat: Add if_none_match for write

[ForestLH]

Which issue does this PR close?

Closes #5097

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Yes, add an optionif_none_match for write_with.

[Xuanwo]

It mostly looks good to me. To make this feature work, we should also add a flag in Capability called write_with_if_none_match. Additionally, we can add a behavior test based on this.

[Xuanwo]

Perhaps we can also provide an example to demonstrate how to handle this error.

[ForestLH]

It mostly looks good to me. To make this feature work, we should also add a flag in Capability called write_with_if_none_match. Additionally, we can add a behavior test based on this.

Thank you very much for your careful help. I have revised all your comments. 😄

[Xuanwo]

Hi, the tests failed because the version we used for testing doesn't support this feature yet. The upstream added this feature in this commit.

Would you like to upgrade MinIO to the latest version, such as RELEASE.2024-09-13T20-26-02Z, in a new PR?

[ForestLH]

Hi, the tests failed because the version we used for testing doesn't support this feature yet. The upstream added this feature in this commit.

Would you like to upgrade MinIO to the latest version, such as RELEASE.2024-09-13T20-26-02Z, in a new PR?

I'm sorry I didn't think of that. Thank you for your careful help.
And you mean upgrade MinIO version at here? Like this PR?

[tisonkun]

And you mean upgrade MinIO version at here? Like this #5142?

@ForestLH Yes. Please try it out.

FYI @Xuanwo reaction on comments won't send a notification so that the author may assume he/she isn't replied.

[ForestLH]

FYI @Xuanwo reaction on comments won't send a notification so that the author may assume he/she isn't replied.

Thanks for the tip!

[ForestLH]

Can you help me review the code, I have rebase the code to the latest.
Because I don't have a relevant environment, and I'm still learning OpenDAL, the code I write may not be comprehensive.
@Xuanwo

[Xuanwo]

Nice PR! Thanks you @ForestLH for the work, and also thanks @tisonkun for the review and notes.

[Xuanwo]

if_none_match also supported by gcs and azblob, would you like to implement them too? cc @ForestLH

[ForestLH]

if_none_match also supported by gcs and azblob, would you like to implement them too? cc @ForestLH

Sure, I appreciate to be able to do this work! 😄 But I don't have gcs and azblob acounts. What better way than to sign up for gcs and azblob account?
ps, Shall I open another issue about this?