hschauhan / gosync

An Opensource Google Drive client written in Python
https://hschauhan.github.io/gosync/
GNU General Public License v2.0
281 stars 57 forks source link

Freezes: Refresh button, self.sync_lock.acquire() #27

Closed SGmuwa closed 4 years ago

SGmuwa commented 4 years ago

https://github.com/hschauhan/gosync/issues/25#issuecomment-624894302 The window freezes when the Refresh button is pressed. Sync folder ~/Google Drive/motherlode.muwa@gmail.com/ is empty, statistics is empty. Below I have depicted a call stack.

def SettingsPage.RefreshTree(self, event):
    ...
    driveTree = self.sync_model.GetDriveDirectoryTree()

def GetDriveDirectoryTree(self):
    ...
    self.sync_lock.acquire() ← The program freezes here.

The problem disappears after 10 minutes, but the program does not tell the user anything during this time.

SGmuwa commented 4 years ago

Duplicated: https://github.com/hschauhan/gosync/issues/9

SGmuwa commented 4 years ago

Between 2020-05-10 17:33:16,211 and 2020-05-10 17:42:04,356 = 0:08:48.145.

2020-05-10 17:33:16,211 - GoSync - DEBUG - Initialize - Started Initialize
2020-05-10 17:33:16,211 - GoSync - INFO - Initialize - Completed Credentials Verification
2020-05-10 17:33:39,899 - GoSync - DEBUG - Initialize - Completed Drive Quota Execution
2020-05-10 17:33:39,900 - GoSync - DEBUG - Initialize - Completed Account Information Load
2020-05-10 17:33:39,900 - GoSync - DEBUG - Initialize - Completed mirror_directory validation
2020-05-10 17:33:39,901 - GoSync - DEBUG - Initialize - Completed Default Config File Creation
2020-05-10 17:33:39,901 - GoSync - DEBUG - Initialize - Completed Config File Load
2020-05-10 17:33:39,901 - GoSync - DEBUG - Initialize - Completed GoogleDriveTree File
2020-05-10 17:33:39,901 - GoSync - DEBUG - Initialize - Completed Initialize
2020-05-10 17:37:56,290 - GoSync - INFO - Total files to check 4426

2020-05-10 17:42:04,355 - GoSync - INFO - ###############################################
2020-05-10 17:42:04,355 - GoSync - INFO - Start - Syncing remote directory
2020-05-10 17:42:04,356 - GoSync - INFO - ###############################################
2020-05-10 17:42:04,356 - GoSync - INFO - ### SyncRemoteDirectory: - Sync Started - Remote Directory () ... Recursive = True
arentoine commented 4 years ago

Hello !

I will create an environment to test this case.

I may need your help to reproduce the problem.

Thanks !

arentoine commented 4 years ago

Hello again,

we made several fixes to the synchronization code.

Can you update the source to the latest version and tell us if this works ?

Regards

SGmuwa commented 4 years ago

@arentoine https://github.com/hschauhan/gosync/issues/9#issuecomment-634565404