Skip to content
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

Metadata timestamps returned for objects and collections #581

Open
paulborgermans opened this issue Jul 9, 2024 · 9 comments
Open

Metadata timestamps returned for objects and collections #581

paulborgermans opened this issue Jul 9, 2024 · 9 comments

Comments

@paulborgermans
Copy link
Contributor

When querying <data_object|collection>metadata(timestamps=True).items() it returns the timestamps from the avu creation, not when it was added to an item (the timestamps in the r_objt_metamap table for example).

Could this be specified in the query / future API calls or is there another way to obtain those timestamps with GenQuery / GenQuery2 ?

Thanks

@paulborgermans
Copy link
Contributor Author

Bacground: we want to use those timestamps in workflow/validation scenarios

@d-w-moore
Copy link
Collaborator

d-w-moore commented Jul 10, 2024

Hmm, there don't seem to be any GenQuery1 columns that reflect the timestamp of AVU attachment (ie set/add). The META_*_MODIFY_TIME and META_MET2_MODIFY_TIME, in my experimentation , seem to reflect the the creation (or modification) of the AVU itself. Maybe GenQuery2 expands available columns to include the timestamps in the r_objt_metamap table, which do seem to reflect the attachment-ts values we're looking for? @korydraughn ?

@d-w-moore
Copy link
Collaborator

d-w-moore commented Jul 10, 2024

If not , I guess specific query will be the workaround / answer. In that case though, baking it into the PRC as a fixture, ie object attribute, will only be appropriate if these columns are available by means of a built-in (non-deletable) specific query.

@korydraughn
Copy link
Contributor

@d-w-moore is correct. That information is not exposed through GenQuery1 or GenQuery2 today.

The only way to fetch that information is by using a SpecificQuery.

I'll open an issue in irods/irods for this.

@alanking
Copy link
Contributor

So, we need to...

Am I missing anything?

@d-w-moore
Copy link
Collaborator

d-w-moore commented Jul 10, 2024

@alanking sounds right. The PRC can expose the new timestamp attributes conditionally , based on an optional- or lazy-load policy , informed by whether our new specific query for those timestamps is loaded or not. (as opposed to making the query undeletable, which I'm guessing is not feasible.)

@alanking
Copy link
Contributor

Okay. I'm going to mark this as an enhancement, but I'm willing to be convinced that it's something else. :)

@korydraughn
Copy link
Contributor

korydraughn commented Jul 10, 2024

@alanking sounds right. The PRC can expose the new timestamp attributes conditionally , based on an optional- or lazy-load policy , informed by whether our new specific query for those timestamps is loaded or not. (as opposed to making the query undeletable, which I'm guessing is not feasible.)

The decision to provide a SpecificQuery is up to the admin.

The only things for us to do are:

  • Expose the columns via GenQuery
  • Decide whether the columns get an entry in library_features.h
  • Update the PRC to conditionally include the columns if the iRODS zone supports them

@alanking Agreed. Marking as enhancement is good.

@paulborgermans
Copy link
Contributor Author

Thanks for this, until it lands in a release, I will use a specific query

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants