##// END OF EJS Templates
default-reviewers: diff data should load more things lazy for better performance.
marcink -
r4508:2365f5e7 stable
parent child
Show More
@@ -154,9 +154,10 def get_diff_info(
154
154
155 commits = []
155 commits = []
156 if get_commit_authors:
156 if get_commit_authors:
157 log.debug('Obtaining commit authors from set of commits')
157 commits = target_scm.compare(
158 commits = target_scm.compare(
158 target_ref, source_ref, source_scm, merge=True,
159 target_ref, source_ref, source_scm, merge=True,
159 pre_load=["author"])
160 pre_load=["author", "date", "message", "branch", "parents"])
160
161
161 for commit in commits:
162 for commit in commits:
162 user = User.get_from_cs_author(commit.author)
163 user = User.get_from_cs_author(commit.author)
@@ -165,6 +166,7 def get_diff_info(
165
166
166 # lines
167 # lines
167 if get_authors:
168 if get_authors:
169 log.debug('Calculating authors of changed files')
168 target_commit = source_repo.get_commit(ancestor_id)
170 target_commit = source_repo.get_commit(ancestor_id)
169
171
170 for fname, lines in changed_lines.items():
172 for fname, lines in changed_lines.items():
@@ -190,6 +192,8 def get_diff_info(
190 author_counts[author] = author_counts.get(author, 0) + 1
192 author_counts[author] = author_counts.get(author, 0) + 1
191 email_counts[email] = email_counts.get(email, 0) + 1
193 email_counts[email] = email_counts.get(email, 0) + 1
192
194
195 log.debug('Default reviewers processing finished')
196
193 return {
197 return {
194 'commits': commits,
198 'commits': commits,
195 'files': all_files_changes,
199 'files': all_files_changes,
General Comments 0
You need to be logged in to leave comments. Login now