Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Dulwich performance workarounds #258

Open
jonashaag opened this issue Oct 8, 2020 · 1 comment
Open

Dulwich performance workarounds #258

jonashaag opened this issue Oct 8, 2020 · 1 comment

Comments

@jonashaag
Copy link
Owner

From #257, @jelmer, see

output = subprocess.check_output(cmd, cwd=os.path.abspath(self.path))
,
cmd = ['git', 'blame', '-ls', '--root', decode_from_git(commit.id), '--', path]

I don't have up-to-date speed comparisons of these but when I initially wrote the code, Dulwich was an order of magnitude slower.

@jelmer
Copy link
Contributor

jelmer commented Oct 8, 2020

Thanks, I'll have a look. These are both operations that scale with the length of history per file.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants