revlog: open files in 'r+' instead of 'a+'
Needs ReviewPublic
Actions

Authored by marmoute on Mon, May 3, 7:52 AM.

Details

Reviewers

Group Reviewers

Summary

The code doing actual writing is already doing the necessary seeking, so we
could safely use 'r+'. This make the file objecs usable in more situation, like
updating the sidedata information during pull.

revlog: forcibly move the file cursor at the right location before writing

This is a paranoid change in case the changelog computation moved the cursors
under our feets.

This is not known to happens right now.

Diff Detail

Repository

rHG Mercurial

Branch

default

Lint

No Linters Available

Unit

No Unit Test Coverage

Event Timeline

marmoute created this revision.Mon, May 3, 7:52 AM

Herald added a reviewer: indygreg. · View Herald TranscriptMon, May 3, 7:52 AM

Herald added a reviewer: hg-reviewers. · View Herald Transcript

Herald added a subscriber: mercurial-patches. · View Herald Transcript

marmoute added a child revision: D10609: revlog: use `_writing` in `rewrite_sidedata`.Mon, May 3, 7:52 AM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

Revision Contents
Changeset List

			Path	Packages
M			mercurial/revlog.py (20 lines)
M			mercurial/store.py (2 lines)

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	27385		Mon, May 3, 7:52 AM	★	★
Diff 2	27523		Mon, May 3, 9:44 PM	★	★
Diff 3	27854		Tue, May 11, 1:42 PM	★	★

Status	Author	Revision
Needs Review	marmoute	D10667 changelogv2: use a dedicated on disk format for changelogv2
Needs Review	marmoute	D10666 revlog: do not call Rust code if the index is not compatible with it
Needs Review	marmoute	D10665 revlog: signal which revlog index are compatible with Rust
Needs Review	marmoute	D10664 revlog: pass around the `rev` we deal with when packing/unpacking entry
Needs Review	marmoute	D10663 changelogv2: use a dedicated version number
Needs Review	marmoute	D10662 changelogv2: `copies-side-data` now implies `changelogv2`
Needs Review	marmoute	D10661 changelogv2: allow upgrade from and to this format
Needs Review	marmoute	D10660 changelogv2: introduce a "changelogv2" feature
Needs Review	marmoute	D10659 revlogv2: drop the code related to inlined revlogv2
Needs Review	marmoute	D10658 revlogv2: no longer attempt to use inline for new revlog
Needs Review	marmoute	D10657 sidedata: use the "feature" to detect that sidedata are enabled
Needs Review	marmoute	D10656 revlog: compress sidedata when doing "post-pull" sidedata update
Needs Review	marmoute	D10655 revlog: compress sidedata in `_writeentry`
Needs Review	marmoute	D10654 revlog: apply compression mode while reading sidedata
Needs Review	marmoute	D10653 revlog: introduce a compression mode for sidedata in the revlog index
Needs Review	marmoute	D10652 revlog: implement a "default compression" mode
Needs Review	marmoute	D10651 revlog: add a `_get_decompressor` method
Needs Review	marmoute	D10650 revlog: introduce a plain compression mode
Needs Review	marmoute	D10649 revlog: use an intermediate variable for `data` in `_chunk`
Needs Review	marmoute	D10648 revlogv2: preserve the compression mode on disk
Needs Review	marmoute	D10647 revlog: use `format_version` to dispatch different version in index_get
Needs Review	marmoute	D10646 revlog: add a "data compression mode" entry in the index tuple
Needs Review	marmoute	D10645 revlog: fix the `null_item` attribute for V0
Needs Review	marmoute	D10644 revlog: create a create `null_item` attribute for V0
Needs Review	marmoute	D10643 revlog: improve documentation of the entry tuple
Needs Review	marmoute	D10642 revlog: simplify entry update logic in `rewrite_sidedata`
Needs Review	marmoute	D10641 revlog: simplify the replace_sidedata_info code
Needs Review	marmoute	D10640 revlog: use `rev` instead of `i` in replace_sidedata_info
Needs Review	marmoute	D10632 revlogv2: also keep track for the size of the "data" file
Needs Review	marmoute	D10631 revlogv2: track pending write in the docket and expose it to hooks
Needs Review	marmoute	D10630 revlog: move the `trypending` logic from the `changelog` to the `revlog`
Needs Review	marmoute	D10629 revlogv2: delay the update of the changelog docket to transaction end
Needs Review	marmoute	D10628 revlogv2: track current index size in the docket
Needs Review	marmoute	D10627 revlogv2: also test transactionality of revlog v2
Needs Review	marmoute	D10626 revlog: add a new test file focussed on testing transactionally issue
Needs Review	marmoute	D10625 revlogv2: store version information in the docket only
Needs Review	marmoute	D10624 revlogv2: introduce a very basic docket file
Needs Review	marmoute	D10623 revlogv2: mark revlogv2 as requires a full upgrade
Needs Review	marmoute	D10622 revlog: unify flag processing when loading index
Needs Review	marmoute	D10621 revlog: unify checks for supported flag
Needs Review	marmoute	D10620 side-data: drop the associated config and requirements
Needs Review	marmoute	D10619 revlog: fix capitalisation of an error
Needs Review	marmoute	D10618 test: no longer directly use the sidedata config and requirements
Needs Review	marmoute	D10617 sidedata: add a 'side-data' repository feature and use it
Needs Review	marmoute	D10616 statichttp: add the missing `features` attribute
Needs Review	marmoute	D10615 sidedata: use revlogv2 requirement in the test helper
Needs Review	marmoute	D10614 upgrade: display sidedata before upgrade
Needs Review	marmoute	D10613 revlog: determine sidedata support based on the revlog version
Needs Review	marmoute	D10612 requirements: no longer drop `generaldelta` requirement with revlogv2
Needs Review	marmoute	D10611 config: drop the `format.exp-revlogv2.2` option
Needs Review	marmoute	D10610 revlog: only use the `_indexfp` method for read operation
Needs Review	marmoute	D10609 revlog: use `_writing` in `rewrite_sidedata`
Needs Review	marmoute	D10608 revlog: open files in 'r+' instead of 'a+'
Needs Review	marmoute	D10607 revlog: pass a transaction object to `rewrite_sidedata`
Needs Review	marmoute	D10606 revlog: rename variable in `rewrite_sidedata` to match other code
Needs Review	marmoute	D10605 revlog: introduce a mandatory `_writing` context to update revlog content
Needs Review	marmoute	D10604 revlog: preindent some code in _enforceinlinesize
Needs Review	marmoute	D10603 revlog: preindent some code in addgroup
Closed	marmoute	D10602 revlog: rename `indexdata` to entry_data
Closed	marmoute	D10601 revlog: use "entry_point" phrasing for loading the revlog
Closed	marmoute	D10600 revlog: directly use self._format_flags when loading index
Closed	marmoute	D10599 revlog: directly use self._format_version when loading index
Closed	marmoute	D10598 revlog: use `_format_flags` to access flags instead of `header`
Closed	marmoute	D10597 revlog: rename `newversionflags` to `new_header`
Closed	marmoute	D10596 revlog: rename `versionflags` to header
Closed	marmoute	D10595 revlog: move index reading logic in a dedicated method
Closed	marmoute	D10594 revlog: define the actual index and datafile at loading time
Closed	marmoute	D10593 revlog: simplify a conditionnal in _enforceinlinesize
Closed	marmoute	D10592 revlog: drop `flush` parameter from `_peek_iscensored`
Closed	marmoute	D10591 revlog: fix error message when data are missing
Closed	marmoute	D10590 revlog: rename `nodemap_file` to `_nodemap_file`
Closed	marmoute	D10589 revlog: use revlog.display_id in error related to bad revisions
Closed	marmoute	D10588 revlog: use revlog.display_id in "revision too big" errors
Closed	marmoute	D10587 revlog: use revlog.display_id in censor related errors
Closed	marmoute	D10586 revlog: use revlog.display_id in integrity error
Closed	marmoute	D10585 revlog: use revlog.display_id in ambiguity errors
Closed	marmoute	D10584 revlog: use revlog.display_id for corruption error
Closed	marmoute	D10583 revlog: use revlog.display_id in format related errors
Closed	marmoute	D10582 revlog: use revlog.display_id in narrow error message
Closed	marmoute	D10581 revlog: use revlog.display_id in LookupError
Closed	marmoute	D10580 revlog: use revlog.display_id for FilteredLookupError
Closed	marmoute	D10579 revlog: introduce a `display_id` property
Closed	marmoute	D10578 revlog: also use radix when computing nodemap data file
Closed	marmoute	D10577 revlog: stop usage of `_indexfile` to computing nodemap path
Closed	marmoute	D10576 revlog: use a "radix" to address revlog
Closed	marmoute	D10575 revlog: rename `datafile` to `datafile`
Closed	marmoute	D10574 revlog: rename `indexfile` to `_indexfile`
Closed	marmoute	D10573 filelog: drop `indexfile` from `filelog`
Closed	marmoute	D10572 manifest: drop the `indexfile` from `manifestrevlog`
Closed	marmoute	D10571 revlog: deal with special "postfix" explicitely
Closed	marmoute	D10570 revlog: split the option initialisation in its own method
Closed	marmoute	D10569 revlog: always "append" full size tuple
Closed	marmoute	D10568 revlog: make the index always return the same tuple
Closed	marmoute	D10567 revlog: introduce an explicit `format_version` member in the index struct
Closed	marmoute	D10566 revlog: rename `hdrsize` to `entry_size` in the C code
Closed	marmoute	D10565 revlog: split the `version` attribute into its two components
Closed	marmoute	D10564 verify: pass a revlog to `_checkrevlog` in `_verifymanifest`
Closed	marmoute	D10563 revlog: replace flag check related to generaldelta with attribute check
Closed	marmoute	D10562 revlog: replace REVLOGV2 check related to sidedata with `hassidedata` checks
Closed	marmoute	D10561 revlog: explicitely pass the "indexfile" parameter
Closed	marmoute	D10544 revlog: highlight current incompatibility in `rewrite_sidedata`
Closed	marmoute	D10543 revlog: adjust rewrite_sidedata code to not delete existing revlog content
Closed	Alphare	D10350 test-copies: test that copies' sidedata can get computed during push
Closed	Alphare	D10349 test-copies: test that copies' sidedata can get computed during pull
Closed	Alphare	D10348 test-copies: test that copies' sidedata does not get corrupted during push
Closed	Alphare	D10347 test-copies: test that copies' sidedata does not get corrupted during pull
Closed	Alphare	D10346 test-copies: simplify some conditional output
Closed	Alphare	D10361 sidedata: move documentation about sidedata helpers to sidedata module
Closed	Alphare	D10360 sidedata: move sidedata-related utils to the dedicated module
Closed	Alphare	D10359 sidedata: replace sidedata upgrade mechanism with the new one
Closed	Alphare	D10358 sidedata: add a way of replacing an existing sidedata computer
Closed	Alphare	D10345 bundle2: remove restriction around sidedata
Closed	Alphare	D10344 sidedata: enable sidedata computers to optionally rewrite flags
Closed	Alphare	D10343 cg4: introduce protocol flag to signify the presence of sidedata
Closed	Alphare	D10342 changegroup: don't limit cgv4 to revlogv2 repos
Closed	Alphare	D10341 sidedata: gate sidedata functionality to revlogv2 in more places
Closed	Alphare	D10340 sidedata: register copies sidedata computer regardless of the revlog version
Closed	Alphare	D10353 revlog: replace the old `revlog_kind` approach with the new `target` one
Closed	Alphare	D10352 revlog: introduce an explicit tracking of what the revlog is about

revlog: open files in 'r+' instead of 'a+'Needs ReviewPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 27854

mercurial/revlog.py

mercurial/store.py

revlog: open files in 'r+' instead of 'a+'
Needs ReviewPublic
Actions

Revision Contents
Changeset List