This is an archive of the discontinued Mercurial Phabricator instance.

export: use unfiltered repo if we're exporting the working copy parent
AbandonedPublic

Authored by martinvonz on Oct 22 2019, 1:55 PM.

Reviewers

durin42
marmoute

Group Reviewers

hg-reviewers

Summary

Same reasoning as in the previous patch. hg export drops from 0.180s
to 0.140s.

Repository

rHG Mercurial

Lint

Lint Skipped

Unit

Unit Tests Skipped

Do you have performance number for this?

martinvonz updated this revision to Diff 18047.

This revision is now accepted and ready to land.Nov 13 2019, 6:08 PM

This was superseeded by another mechanism and can be safely abandonned.

This revision now requires changes to proceed.Apr 22 2020, 11:59 AM

			Path	Packages
M			mercurial/commands.py (5 lines)

Commit	Parents	Author	Summary	Date
617f252663db	fb4c66fede45	Martin von Zweigbergk		Oct 22 2019, 1:40 PM

Status	Author	Revision
Abandoned	martinvonz	D7149 export: use unfiltered repo if we're exporting the working copy parent
Abandoned	martinvonz	D7148 annotate: use unfiltered repo if we're annotating the working copy
Abandoned	martinvonz	D7147 diff: use unfiltered repo if we're diffing the working copy
Abandoned	martinvonz	D7144 status: use unfiltered repo if we're getting status of working copy
Closed	martinvonz	D7145 status: move initialization closer together
Closed	martinvonz	D7143 largefiles: avoid dynamically subclassing context instances
Closed	martinvonz	D7142 largefiles: delete unused repo.status_nolfiles()
Closed	martinvonz	D7141 largefiles: use context manager for wlock in repo.status() override
Closed	martinvonz	D7140 largefiles: allow "lfstatus" context manager to set value to False
Closed	martinvonz	D7139 largefiles: move lfstatus context manager to lfutil

	"""			"""
	opts = pycompat.byteskwargs(opts)			opts = pycompat.byteskwargs(opts)
	bookmark = opts.get(b'bookmark')			bookmark = opts.get(b'bookmark')
	changesets += tuple(opts.get(b'rev', []))			changesets += tuple(opts.get(b'rev', []))

	if bookmark and changesets:			if bookmark and changesets:
	raise error.Abort(_(b"-r and -B are mutually exclusive"))			raise error.Abort(_(b"-r and -B are mutually exclusive"))

				if not bookmark and not changesets:
				# Avoid loading obsmarkers if we're accessing only the working copy
				# parent (which will never be hidden).
				repo = repo.unfiltered()

	if bookmark:			if bookmark:
	if bookmark not in repo._bookmarks:			if bookmark not in repo._bookmarks:
	raise error.Abort(_(b"bookmark '%s' not found") % bookmark)			raise error.Abort(_(b"bookmark '%s' not found") % bookmark)

	revs = scmutil.bookmarkrevs(repo, bookmark)			revs = scmutil.bookmarkrevs(repo, bookmark)
	else:			else:
	if not changesets:			if not changesets:
	changesets = [b'.']			changesets = [b'.']