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

Return the data version #188

Merged
merged 4 commits into from
Aug 4, 2023
Merged

Return the data version #188

merged 4 commits into from
Aug 4, 2023

Conversation

JonasKellerer
Copy link
Contributor

Resolves #130

The data version is returned in the header as Data-Version and not Lapis-Data-Version as requested in the ticked, since it is the data version from silo.

This should be reworked, with the verbose preprocessing.

@JonasKellerer JonasKellerer linked an issue Aug 3, 2023 that may be closed by this pull request
endToEndTests/test/info.test.js Outdated Show resolved Hide resolved
src/silo/database.cpp Outdated Show resolved Hide resolved
src/silo/database.cpp Outdated Show resolved Hide resolved
src/silo_api/query_handler.cpp Outdated Show resolved Hide resolved
src/silo_api/query_handler.cpp Outdated Show resolved Hide resolved
src/silo_api/info_handler.cpp Show resolved Hide resolved
@JonasKellerer JonasKellerer force-pushed the 130-return-the-data-version branch 2 times, most recently from a9161a3 to 71d453f Compare August 3, 2023 12:20
include/silo/database.h Outdated Show resolved Hide resolved
@Taepper
Copy link
Collaborator

Taepper commented Aug 3, 2023

Are we sure, that we do not want to set the data version during preprocessing? And then the build only reads the data version directly from the file.

This would enable us to use the data version during change detection. And set a new data version if changes have been detected. This would also make it more flexible to already get the data version from the preprocessed input, in case we decide that SILO should not be the authority to mint new versions

@JonasKellerer
Copy link
Contributor Author

So far I have set the point where data version is mined at the end of the build process, which is also the end of the preprocessing step; just to have a first point and that it can be used in LAPIS #272. When looking at the preprocessing in #185 again we could also look at this again and find out a better spot?

@JonasKellerer JonasKellerer merged commit 80e79a2 into main Aug 4, 2023
5 checks passed
@JonasKellerer JonasKellerer deleted the 130-return-the-data-version branch August 4, 2023 07:24
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.

Return the data version
3 participants