RFR: 2400: Avoid overwriting mbox archive [v2]
Zhao Song
zsong at openjdk.org
Mon Nov 18 23:48:26 UTC 2024
> Sometimes, the ArchiveWorkItem fails to retrieve the current contents of an mbox file while processing a PR. When this happens, it will resend all emails. Erik pointed out that in GitLab, creating a new file uses a POST request, while updating an existing file uses PUT. Similarly, GitHub requires the current "SHA" to update a file.
>
> The ArchiveWorkItem can determine whether it found existing content in the mbox archive and choose the right method accordingly. If it incorrectly fails to retrieve the existing content from GitLab or GitHub, it will attempt to create a new file. This attempt should fail, triggering a retry of the WorkItem, which would ideally succeed in retrieving the existing content.
Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
review comment
-------------
Changes:
- all: https://git.openjdk.org/skara/pull/1696/files
- new: https://git.openjdk.org/skara/pull/1696/files/9bc5ad18..26998838
Webrevs:
- full: https://webrevs.openjdk.org/?repo=skara&pr=1696&range=01
- incr: https://webrevs.openjdk.org/?repo=skara&pr=1696&range=00-01
Stats: 25 lines in 3 files changed: 19 ins; 1 del; 5 mod
Patch: https://git.openjdk.org/skara/pull/1696.diff
Fetch: git fetch https://git.openjdk.org/skara.git pull/1696/head:pull/1696
PR: https://git.openjdk.org/skara/pull/1696
More information about the skara-dev
mailing list