MetadataConstraints used to use the presence of a directory column to detect a split mutation. In version 4.0 split mutations will now set an operation id so this change updates the metadata constraints code to check that id vs the directory.
This closes #4857
Also, I added in a TODO marker in the code. It's also a placeholder because #4913 will add in a new validation check when merged and also because I was wondering if it would make sense to still validate that the directory exists in addition to the opid or not. The marker is obliviously temporary and can be removed when those questions are figured out.
MetadataConstraints used to use the presence of a directory column to detect a split mutation. In version 4.0 split mutations will now set an operation id so this change updates the metadata constraints code to check that id vs the directory.
This closes #4857
Also, I added in a TODO marker in the code. It's also a placeholder because #4913 will add in a new validation check when merged and also because I was wondering if it would make sense to still validate that the directory exists in addition to the opid or not. The marker is obliviously temporary and can be removed when those questions are figured out.