diff options
author | George Goldberg <george@gberg.me> | 2016-11-22 15:14:25 +0000 |
---|---|---|
committer | Christopher Speller <crspeller@gmail.com> | 2016-11-22 10:14:25 -0500 |
commit | e1fca412fc9c1f98f1682052fa8ce52166181a44 (patch) | |
tree | 19bb89069088c829cb87f9eafb806f92ef4849dd /model/preference_test.go | |
parent | 3c65a20f4587da6315d47222b5d9852d7e5ad66f (diff) | |
download | chat-e1fca412fc9c1f98f1682052fa8ce52166181a44.tar.gz chat-e1fca412fc9c1f98f1682052fa8ce52166181a44.tar.bz2 chat-e1fca412fc9c1f98f1682052fa8ce52166181a44.zip |
PLT-4442 Generate preview images sequentially in Slack importer (#4617)
* Break out image preparation to its own function.
This is preparatory work to make it easier to handle the thumbnail image
generation in a non-racy way in the Slack command line importer.
* Build preview images sequentially in Slack import.
This removes the use of goroutines from the image preview generation
code run when importing Slack attachments. This slows things down, but
it has important benefits:
1) Avoids a race condition with the goroutines and the command line
exiting.
2) Avoids the problem of massive memory consumption when importing a
Slack channel with a lot of large images attached in quick succession.
Fixes PLT-4442.
* Use mutliple return values instead of struct.
Diffstat (limited to 'model/preference_test.go')
0 files changed, 0 insertions, 0 deletions