-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for phantom files #3956
Comments
This was referenced Mar 18, 2022
melainalegaspi
added
enh
[type] New feature or request
indexer
[subject] The indexer part of Azul
labels
Mar 21, 2022
58 tasks
nadove-ucsc
added a commit
that referenced
this issue
Apr 5, 2022
nadove-ucsc
added a commit
that referenced
this issue
Apr 5, 2022
achave11-ucsc
pushed a commit
that referenced
this issue
May 2, 2022
achave11-ucsc
pushed a commit
that referenced
this issue
Jun 3, 2022
achave11-ucsc
added a commit
that referenced
this issue
Jun 3, 2022
nadove-ucsc
added a commit
that referenced
this issue
Jun 10, 2022
35 tasks
achave11-ucsc
added a commit
that referenced
this issue
Jun 10, 2022
achave11-ucsc
added a commit
that referenced
this issue
Jun 21, 2022
hannes-ucsc
pushed a commit
that referenced
this issue
Nov 17, 2022
hannes-ucsc
pushed a commit
that referenced
this issue
Nov 17, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently we get
Phantom files are defined as files with no data file object in the staging area and
file_descriptor.drs_uri
set tonull
.https://github.com/HumanCellAtlas/dcp2/pull/55/files
https://github.com/HumanCellAtlas/metadata-schema/pull/1437/files
They will cause the TDR
file_id
column to benull
. We should allow that if, and only if thefile_descriptor
column contains JSON whosedrs_uri
property is alsonull
. If the property is not set, or if it is set to a value other thannull
and RequirementError should be raised. An unset property is invalid, support for non-null values will be added in #3631.Downstream from the indexer, the resulting
None
values indrs_path
in our index need to be handled gracefully.The text was updated successfully, but these errors were encountered: