Conversation
623c27d to
7f14f3e
Compare
philiplantz
left a comment
There was a problem hiding this comment.
Some changes suggested but none required.
config-vdms.json
Outdated
| // Network | ||
| "port": 55555, // Default is 55555 | ||
| "max_simultaneous_clients": 20, // Default is 500 | ||
| "max_simultaneous_clients": 100, // Default is 500 |
There was a problem hiding this comment.
Why does the comment not match the value?
And why would the comment be needed at all, if it did match the value?
config-vdms.json
Outdated
| "tdb_path": "db/images/tiledb/tdb/", | ||
| "blob_path":"db/blobs/", | ||
| // For more path configuration options, check documentation. | ||
| "db_root_path": "db", // Default is "db" |
There was a problem hiding this comment.
Is this comment useful?
(This also applies in other places the comment simply states the value in the initializer.)
src/VDMSConfig.cc
Outdated
| // IMAGES - TDB | ||
| path_tdb = path_images + "/" + DEFAULT_PATH_TDB; | ||
| path_tdb = get_string_value(PARAM_DB_TDB, path_tdb); | ||
| check_or_create(path_tdb); |
There was a problem hiding this comment.
no need, it just looked better. but you are right, no good reason.
tests/python/config-tests.json
Outdated
| { | ||
| // Network | ||
| "port": 55557 // Default is 55555 | ||
| "port": 55557, // Default is 55555 |
There was a problem hiding this comment.
The comment doesn't match the value.
src/VDMSConfig.cc
Outdated
| int success = 0; | ||
| if (dir_exist(path)) { | ||
| return; | ||
| } |
There was a problem hiding this comment.
There's no need to call dir_exist. Create_dir checks and returns success if the directory already exists.
Dir_exist returns failure if the directory exists and is searchable but not readable, even though that may not actually be a problem.
Also I think it returns success if the directory is readable but not searchable, which would actually be a problem.
src/VDMSConfig.h
Outdated
| #include <sys/stat.h> | ||
| #include <unistd.h> | ||
| #include <dirent.h> | ||
|
|
There was a problem hiding this comment.
Is this module not meant to be portable?
7f14f3e to
0be89f7
Compare
Fix #12 and #38