libgit2 / libgit2/libgit2sharp
repo.Commits.QueryBy(filename) slow on large repos
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- C#
- Star
- 3.5k
- Fork
- 925
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
Reproduction steps
1): Clone a large repo
2): run this function on that repo with some random file:
public IEnumerable<string> TestSlow(string filename)
{
using (var repo = new Repository(repoRoot))
{
string path = filename.Substring(repoRoot.Length + 1).Replace("\\", "/");
foreach (LogEntry entry in repo.Commits.QueryBy(path))
{
yield return entry.Commit.Author.ToString();
}
}
}
- run this command on the same file: "git log --follow --oneline -- "
Expected behavior
I expect similar time to be taken by TestSlow and the git log command above
Actual behavior
The git log command finishes in about 1.6 ms on my repo
The TestSlow command takes about 70 seconds.
Here is what I see in my profiler:

Version of LibGit2Sharp (release number or SHA1)
0.27.0-preview-0017
0.26.1
0.24.1
Operating system(s) tested; .NET runtime tested
.NET Framework 4.7.2 on Windows 10
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng cách tái hiện việc so sánh giữa TestSlow sử dụng repo.Commits.QueryBy(path) và git log --follow --oneline -- <file> trên một repository lớn. Sử dụng chế độ xem profiler và các mốc thời gian được báo cáo để xác định QueryBy dành thời gian ở đâu. Công việc được xem là hoàn tất khi thời gian chạy trên repository lớn được giảm xuống gần với lệnh git tương đương, đồng thời vẫn giữ nguyên các mục commit được trả về.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- csharp, git
- Lĩnh vực
- performance
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100