mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Shorten prefix to 2 characters

Browse source 
Using 2 hex characters [0-9a-f] will give us 16 * 16 = 256 root folders which
seems more manageable than 4096 (16^3). Assuming a user has 10,000 attachments,
they should roughly distribute at ~40 per folder with prefix length 2 rather
than ~2.5 per folder with a prefix of 3.
This commit is contained in:
Daniel Gasienica 2018-03-15 16:03:56 -04:00
parent 2cd3d5ac16
commit 1262d1d696
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/types/attachment/write_attachment_data.js
View file

@ -35,6 +35,6 @@ exports._getAttachmentName = () => {
// _getAttachmentPath :: Unit -> IO Path // _getAttachmentPath :: Unit -> IO Path
exports._getAttachmentPath = () => { exports._getAttachmentPath = () => {
const name = exports._getAttachmentName(); const name = exports._getAttachmentName();
const prefix = name.slice(0, 3); const prefix = name.slice(0, 2);
return Path.join(prefix, name); return Path.join(prefix, name);
}; };

2
test/app/types/attachment/write_attachment_data_test.js
View file

@ -12,7 +12,7 @@ const {
} = require('../../../../app/types/attachment/write_attachment_data'); } = require('../../../../app/types/attachment/write_attachment_data');
const PREFIX_LENGTH = 3; const PREFIX_LENGTH = 2;
const NUM_SEPARATORS = 1; const NUM_SEPARATORS = 1;
const NAME_LENGTH = 64; const NAME_LENGTH = 64;
const PATH_LENGTH = PREFIX_LENGTH + NUM_SEPARATORS + NAME_LENGTH; const PATH_LENGTH = PREFIX_LENGTH + NUM_SEPARATORS + NAME_LENGTH;