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