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

feature(materialized-views): add cli flags to disable materialized views #424

Merged
merged 1 commit into from
Sep 12, 2024
Merged

feature(materialized-views): add cli flags to disable materialized views #424

merged 1 commit into from
Sep 12, 2024

Conversation

CodeLieutenant
Copy link
Contributor

There is a crash in gemini, which is deeped then we expected first, and it comes down to the materialized views generation and validation. Something there is wrong, until further investigation (and possibly gemini redisign, this issue cannot be fixed). As I've discussed with @fruch about this issue, we concluded that the best way forward for now is to disable materialized views until the redisign of the project.

Simplest solution is just to add the CLI flag and propagate it to the creation of the materialized views.

Jenkins Issue: https://jenkins.scylladb.com/job/enterprise-2024.1/job/gemini-/job/gemini-3h-with-nemesis-test/27/

There is a crash in gemini, which is deeped then we expected first, and
it comes down to the materialized views generation and validation.
Something there is wrong, until further investigation (and possibly
gemini redisign, this issue cannot be fixed). As I've discussed with
@fruch about this issue, we concluded that the best way forward for now
is to disable materialized views until the redisign of the project.

Simplest solution is just to add the CLI flag and propagate it to the
creation of the materialized views.

Signed-off-by: Dusan Malusev <[email protected]>
@CodeLieutenant CodeLieutenant self-assigned this Sep 9, 2024
@CodeLieutenant CodeLieutenant added bug Something isn't working crash labels Sep 9, 2024
@CodeLieutenant CodeLieutenant marked this pull request as ready for review September 12, 2024 10:53
@dkropachev dkropachev merged commit accfcf2 into scylladb:master Sep 12, 2024
1 check passed
@CodeLieutenant CodeLieutenant deleted the feat/disable-materialized-views branch September 12, 2024 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working crash
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants