[git] trac_user_rlookup is ineffective with cached repository

[Ryan J Ollos]

The [git] trac_user_rlookup has not effect when [git] cached_repository is true.

[Jun Omae]

Works for me….

$ git --git-dir /dev/shm/tmp/t11864-git/.git show e370d900fa0a0b9c67b23e1ea0b53b7c28bdb376 | head -3
commit e370d900fa0a0b9c67b23e1ea0b53b7c28bdb376
Author: jomae <jun66j5@gmail.com>
Date:   Thu Dec 4 03:59:19 2014 +0900

Trac [/dev/shm/tmp/t11864]> session list

SID                 Auth  Last Visit  Name      Email
-----------------------------------------------------------------
rlookuped-username  1     2014-12-05  Jun Omae  jun66j5@gmail.com

Trac [/dev/shm/tmp/t11864]> config get git cached_repository
enabled
Trac [/dev/shm/tmp/t11864]> config get git trac_user_rlookup
enabled
Trac [/dev/shm/tmp/t11864]> repository list

Name       Type  Alias  Directory
-----------------------------------------------
(default)  git          /dev/shm/tmp/t11864-git

Trac [/dev/shm/tmp/t11864]> repository resync ''
Resyncing repository history for (default)...
10494 revisions cached.29c0c9bc1cd9c451b13]
Done.

$ sqlite3 /dev/shm/tmp/t11864/db/trac.db
sqlite> .mode line
sqlite> select * from revision where rev='e370d900fa0a0b9c67b23e1ea0b53b7c28bdb376';
  repos = 1
    rev = e370d900fa0a0b9c67b23e1ea0b53b7c28bdb376
   time = 1417633159000000
 author = rlookuped-username
message = [#11503] 1.0.3dev: fixed raising `UnicodeDecodeError` when url has no handler and unicode characters with trailing slash

[Jun Omae]

After disabling [git] trac_user_rlookup and syncing the commit, its git author is stored in revision.author with no changes.

Trac [/dev/shm/tmp/t11864]> config get git trac_user_rlookup
disabled
Trac [/dev/shm/tmp/t11864]> repository sync '' e370d900fa0a0b9c67b23e1ea0b53b7c28bdb376
e370d900fa0a0b9c67b23e1ea0b53b7c28bdb376 resynced on (default).
Trac [/dev/shm/tmp/t11864]>
$ sqlite3 /dev/shm/tmp/t11864/db/trac.db
SQLite version 3.7.9 2011-11-01 00:52:41
Enter ".help" for instructions
Enter SQL statements terminated with a ";"
sqlite> .mode line
sqlite> select * from revision where rev='e370d900fa0a0b9c67b23e1ea0b53b7c28bdb376';
  repos = 1
    rev = e370d900fa0a0b9c67b23e1ea0b53b7c28bdb376
   time = 1417633159000000
 author = jomae <jun66j5@gmail.com>
message = [#11503] 1.0.3dev: fixed raising `UnicodeDecodeError` when url has no handler and unicode characters with trailing slash

[Ryan J Ollos]

Replying to jomae:

After disabling [git] trac_user_rlookup and syncing the commit, its git author is stored in revision.author with no changes.

You are right, it works. I was expecting the lookup to occur dynamically regardless of whether the changeset is cached or not. Just changing trac_user_rlookup from false → true is not enough, the repository must be resynced.

Since the replaced username is saved in the Revision table, the option documentation that states (costly if you have many users) seem slightly misleading because while it might slow down the sync, it shouldn't slow down any of the views, right?

[Jun Omae]

You are right, it works. I was expecting the lookup to occur dynamically regardless of whether the changeset is cached or not. Just changing trac_user_rlookup from false → true is not enough, the repository must be resynced.

Okay. I understand this issue.

Since the replaced username is saved in the Revision table, the option documentation that states costly if you have many users seem slightly misleading because while it might slow down the sync, it shouldn't slow down any of the views, right? I'm assuming costly implies that requests might be slower.

Enabling the option on cached repository would lead slowness of sync. Because usernames and emails are retrieved from sessions using Environment.get_known_users().

If the option is disabled and cached_repository is enabled, I suppose that requests wouldn't be slow because GitCachedChangeset.author property just returns revision.author column with no changes. But if cached_repository is disabled, requests would be very slow because GitChangeset.author property calls git cat-file and Environment.get_known_user() every time.

[Ryan J Ollos]

Thanks, I'll propose a documentation patch before closing this ticket.

[Ryan J Ollos]

Proposed patch:

tracopt/versioncontrol/git/git_fs.py

@@ -282 +282 @@
         """)
 
     trac_user_rlookup = BoolOption('git', 'trac_user_rlookup', 'false',
-        """Enable reverse mapping of git email addresses to trac user ids
-        (costly if you have many users).""")
+        """Enable reverse mapping of git email addresses to trac user ids.
+        Performance will be reduced if there are many users and the
+        `cached_repository` option is `disabled`.
+
+        A repository resync is required after changing the value of this
+        option.
+        """)
 
     use_committer_id = BoolOption('git', 'use_committer_id', 'true',
         """Use git-committer id instead of git-author id for the

[Ryan J Ollos]

Committed to 1.0-stable in [13467], merged to trunk in [13468].