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

chore: Activate dataset query based on new search graph #1591

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

eikek
Copy link
Member

@eikek eikek commented Jul 5, 2023

This activates the new dataset query that improves performance, but also changes the resulting data slightly.

/deploy #persist extra-values=core.sentry.dsn=https://[email protected]/2,core.sentry.environment=renku-dev,core.sentry.enabled=true

@eikek eikek force-pushed the 1478-remove-old-dataset-query branch from 56f0a31 to 831a7b7 Compare July 5, 2023 12:44
@eikek eikek temporarily deployed to renku-ci-gr-1591 July 5, 2023 13:13 — with GitHub Actions Inactive
@RenkuBot
Copy link
Collaborator

RenkuBot commented Jul 5, 2023

You can access the deployment of this PR at https://renku-ci-gr-1591.dev.renku.ch

@eikek eikek marked this pull request as ready for review July 5, 2023 14:10
@eikek eikek requested a review from a team as a code owner July 5, 2023 14:11
@eikek eikek temporarily deployed to renku-ci-gr-1591 July 6, 2023 07:25 — with GitHub Actions Inactive
@eikek eikek temporarily deployed to renku-ci-gr-1591 July 6, 2023 08:40 — with GitHub Actions Inactive
@jachro jachro linked an issue Jul 11, 2023 that may be closed by this pull request
Copy link
Contributor

@jachro jachro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-16k lines :)

@eikek eikek temporarily deployed to renku-ci-gr-1591 July 12, 2023 12:29 — with GitHub Actions Inactive
@eikek eikek force-pushed the 1478-remove-old-dataset-query branch from 831a7b7 to 3079bc9 Compare July 12, 2023 13:22
@eikek eikek temporarily deployed to renku-ci-gr-1591 July 12, 2023 13:22 — with GitHub Actions Inactive
@eikek eikek force-pushed the 1478-remove-old-dataset-query branch from 3079bc9 to 0c97505 Compare July 14, 2023 04:59
@eikek eikek temporarily deployed to renku-ci-gr-1591 July 14, 2023 04:59 — with GitHub Actions Inactive
@eikek eikek force-pushed the 1478-remove-old-dataset-query branch from 0c97505 to a960d15 Compare July 14, 2023 13:29
@eikek eikek temporarily deployed to renku-ci-gr-1591 July 14, 2023 13:29 — with GitHub Actions Inactive
@eikek eikek deployed to renku-ci-gr-1591 July 14, 2023 14:48 — with GitHub Actions Active
@eikek eikek merged commit ba51063 into development Jul 18, 2023
@eikek eikek deleted the 1478-remove-old-dataset-query branch July 18, 2023 09:24
@jachro jachro mentioned this pull request Jul 25, 2023
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.

Remove old datasets query code
3 participants