Skip to content

File writing with temporary suffix#48

Merged
MarkRivers merged 7 commits intomasterfrom
TempSuffix
Jan 8, 2015
Merged

File writing with temporary suffix#48
MarkRivers merged 7 commits intomasterfrom
TempSuffix

Conversation

@ulrikpedersen
Copy link
Copy Markdown
Member

This is a feature which was implemented April 2014 at DLS by @NicholasRees

When the temporary suffix string (NDFileTempSuffix octet parameter) is not empty it will be used as a temporary suffix on the output filename while the file is being written. When writing is complete, the file is closed and then renamed to have the suffix removed.

The rename operation is atomic from a filesystem view and can be used by monitoring applications like rsync or inotify to kick off processing applications.

Nick Rees and others added 6 commits April 24, 2014 12:01
   - Added parameter TEMP_SUFFIX to NDPluginFile parameters
   - Added corresponding protected class variable NDTempSuffix
   - Made necessary mods in NDPluginFile.cpp/.h
   - Added records in NDFile.template
   - Updated NDFileHDF5Details.edl. This included adding LazyOpen parameter
     since it wasn't there yet.
 * Had to overwrite the TempSufix changes to ADApp/op/edl/NDFileHDF5Details.edl as EDM files just dont merge nicely.
 * Resolved conflict in ADApp/pluginSrc/NDPluginFile.h as the LazyOpen parameter has since been moved elsewhere.
While doing this move, renamed the parameter from TempSuffix to FileTempSuffix
as per the naming convention of file related parameters.
Now with both TempSuffix and CreateDir PVs
Now with description of the temporary suffix feature NDFileTempSuffix.
MarkRivers added a commit that referenced this pull request Jan 8, 2015
File writing with temporary suffix
@MarkRivers MarkRivers merged commit 928101c into master Jan 8, 2015
@ulrikpedersen ulrikpedersen deleted the TempSuffix branch May 1, 2015 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants