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

Refactor tests #1

Open
wants to merge 24 commits into
base: parallelize_tests
Choose a base branch
from
Open

Refactor tests #1

wants to merge 24 commits into from

Conversation

Ponewor
Copy link
Owner

@Ponewor Ponewor commented Apr 16, 2022

No description provided.

piodul and others added 5 commits April 20, 2022 08:44
Check if replication went correctly after each query
Replicator consumer wrongly assumed that the values for original
table's clustering key coming from CDC Log table are not null.
However for some queries like range deletes clustering key may
be unspecified. An example of such a query is:

SELECT FROM table WHERE pk = 0;

This would result in two rows in CDC Log table both having null
as a clustering key value.

As a remedy, types for clustering key values and in consequence
all the other columns' values have been changed from &CqlValue
to Option<&CqlValue>.
@Ponewor Ponewor force-pushed the refactor_tests branch 6 times, most recently from a5efa25 to cd3bf66 Compare April 20, 2022 16:52
@Ponewor Ponewor force-pushed the refactor_tests branch 2 times, most recently from 1390f9c to d7fd267 Compare April 21, 2022 01:14
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.

2 participants