You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "ankrgyl (via GitHub)" <gi...@apache.org> on 2025/07/08 03:08:18 UTC
[PR] Add conditional deletes [arrow-rs-object-store]
ankrgyl opened a new pull request, #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427
# Which issue does this PR close?
Closes #298
# Rationale for this change
It's useful to have conditional deletes while implementing concurrent primitives on object stores, like distributed locks.
# What changes are included in this PR?
Mirror the conditional put implementation to support conditional deletes
# Are there any user-facing changes?
Yes a new `delete_opts` method that accepts deletion opts.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "crepererum (via GitHub)" <gi...@apache.org>.
crepererum closed pull request #427: Add conditional deletes
URL: https://github.com/apache/arrow-rs-object-store/pull/427
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "ankrgyl (via GitHub)" <gi...@apache.org>.
ankrgyl commented on PR #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427#issuecomment-3050680885
Yep very happy to fill in more tests. Do you just generally run them against your own object storage accounts?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "tustvold (via GitHub)" <gi...@apache.org>.
tustvold commented on PR #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427#issuecomment-3047805161
Thank you for this, I think this needs some tests
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "rbehjati (via GitHub)" <gi...@apache.org>.
rbehjati commented on PR #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427#issuecomment-3365705493
It seems that as of September 16th, if-match is supported for S3 general purpose buckets ([source](https://aws.amazon.com/about-aws/whats-new/2025/09/amazon-s3-conditional-deletes-s3-general-purpose-buckets/)). Can the work on this PR be continued now?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "crepererum (via GitHub)" <gi...@apache.org>.
crepererum commented on PR #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427#issuecomment-3876773897
Discussion & impl. continues in #610. Thank you @ankrgyl for your initial work on this feature!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "ankrgyl (via GitHub)" <gi...@apache.org>.
ankrgyl commented on PR #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427#issuecomment-3369527933
> It seems that as of September 16th, if-match is supported for S3 general purpose buckets ([source](https://aws.amazon.com/about-aws/whats-new/2025/09/amazon-s3-conditional-deletes-s3-general-purpose-buckets/)). Can the work on this PR be continued now?
Feel free to take it over!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "ankrgyl (via GitHub)" <gi...@apache.org>.
ankrgyl commented on PR #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427#issuecomment-3053010739
> As far as I know, aws only support delete with if match on s3 express.
Ah yes that is correct!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo commented on PR #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427#issuecomment-3051267552
As far as I know, aws only support delete with if match on s3 express.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add conditional deletes [arrow-rs-object-store]
Posted by "pdeva (via GitHub)" <gi...@apache.org>.
pdeva commented on PR #427:
URL: https://github.com/apache/arrow-rs-object-store/pull/427#issuecomment-3761550799
i took inspiration from this and created a fresh PR here #610
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org