search

aegif / CmisSync

Synchronize content between a CMIS repository and your desktop. Like Dropbox for Enterprise Content Management!
http://CmisSync.com
164 stars 123 forks source link

Add repository name in log #482

Open ymolinet opened 9 years ago

ymolinet commented 9 years ago 
2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - Created CMIS session: DotCMIS.Client.Impl.Session
2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - ChangeLog capability: True
2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - Get folder tree support: True
2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - Get descendants support: True
2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - Chunked Up/Download disabled
2014-11-15 09:30:53,060 [4] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - First sync, invoke a full crawl sync
2014-11-15 09:30:53,066 [10] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - First sync, invoke a full crawl sync
2014-11-15 09:30:53,086 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - First sync, invoke a full crawl sync
nicolas-raoul commented 9 years ago

What exactly do you mean? Adding a new line like below?

2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - Repository name: Main repository

?

ymolinet commented 9 years ago

I think it's better to add the repo name on each line because more than one repo could be in sync at the same time.

2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - [RepoName] - Created CMIS session: DotCMIS.Client.Impl.Session 2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - [RepoName] - ChangeLog capability: True 2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - Get folder tree support: True 2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - Get descendants support: True 2014-11-15 09:30:53,045 [9] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - Chunked Up/Download disabled 2014-11-15 09:30:53,060 [4] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - First sync, invoke a full crawl sync 2014-11-15 09:30:53,066 [10] DEBUG CmisSync.Lib.Sync.CmisRepo+SynchronizedFolder [(null)] - [RepoName] - First sync, invoke a full crawl sync

nicolas-raoul commented 9 years ago

Lines are already quite long, but that can be useful indeed... how about a number like 1, 2 ,3, meaning it is the first or second repo, for instance?

ymolinet commented 9 years ago

I think that name is more efficient. I don't remember if Context Menu sort repo by name, but if the order if not the same, it could be more difficult to understand the repo used in log file.

ymolinet commented 9 years ago

We are in a log file, in debug mode, we need a lot of information for debugging. Reduce log verbose, reduce log entries.