4 ms·
Good point on 2 and 3. Should just involve checking the previous commands return code before echoing the uuid. I'll get that patched up and ship a new version i
by nodesocket 10y ago
Good point on 2 and 3. Should just involve checking the previous commands return code before echoing the uuid. I'll get that patched up and ship a new version in a bit.
Not sure what you mean exactly by 1 and 4 though.
- jakewins 10y agoMany operating systems do not handle tens of thousands of file handles open simultaneously very well. For instance, if you run this database on Linux, the RAM overhead for each document is 1K just for the file handle, see the comment at the bottom here http://lxr.free-electrons.com/source/fs/file_table.c http://lxr.free-electrons.com/source/fs/file_table.c
- CJefferson 10y agoWith 1, you are trusting the directory returned by uuid will be unique --it should be, but it might not be, in particular if it is based on time and I run in two threads. With 4, I find once a directory has about 100,000 files in it, things get bad, from the simple (ls * won't work) to the nastier (git starts using huge amounts of space, you'll hit github's size limit even though all your files are quite small).
- avip 10y agoIn all fairness, I find this (and only this) criticism unsubstantiated/nitpicky. Many systems are relying on the statistical improbability of 32-hexdigits uuid collision (name-dropping one - Disque).
- sz4kerto 10y agoI would not let anyone close to the code I'm working on if he wanted to verify whether a properly generated UUID clashes with some previously generated UUID. You could just as do stuff like "int x=1; assert(x == 1);" and so on. And it would be just as futile.
- CJefferson 10y agoIn this case, you are relying on what installed program or function is called when you invoke uuid in the shell. It seems reasonable to me to check if it is having sensibly, if the reliability of your program counts on it. I have no idea what dodgy uuid programs you might find installed.
- tyingq 10y agoOn 4, the point might be that thousands of files in a single directory can create performance issues with readdir(). The usual workaround to that is to create subdirectories based on the first few characters of the filename. Since you're using bash, you could do something like: $ echo 1df8be33-4392-471a-99af-3df967b87cb6 |sed -e 's/^\(.\)\(.\)/\1\/\2\/\1\2/' 1/d/1df8be33-4392-471a-99af-3df967b87cb6 Edit: You may want more than 2 levels of directories, or directories with 2 character names instead of 1, etc. Apache's mod_cache defaults to 2 levels of 2 character names, but the filenames are base 64, so more possibilities than hex.