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

Autosharding tests: Data manipulation #161

Closed
2 of 4 tasks
donhardman opened this issue Sep 26, 2023 · 6 comments
Closed
2 of 4 tasks

Autosharding tests: Data manipulation #161

donhardman opened this issue Sep 26, 2023 · 6 comments
Assignees

Comments

@donhardman
Copy link
Contributor

donhardman commented Sep 26, 2023

  • Ensure we can insert multiple rows into distributed data, and it's distributed across shards.
  • Check that the data we inserted are returned in the select statement.
  • Verify that we cannot alter or change the sharded table from the user side (only Buddy).
  • Confirm that we cannot alter or change the replicated cluster from the user side (only Buddy).
@PavelShilin89
Copy link

@donhardman At the moment only insertion can be performed for this task. But maybe I found a bug in its work, I have given examples in #pavel-onboarding, the rest of the functionality is not working yet, it is not possible to test it. Please let me know when I can proceed with the issue and reassign it to me.

@donhardman
Copy link
Contributor Author

While fixing a race condition issue in a multi-node environment, I introduced a bug yesterday. It should be fixed now and work as expected. Please check and let me know if it doesn't.

@PavelShilin89
Copy link

@donhardman I've done the first two items. Please explain in more detail what you mean in point 3 and 4.

If it is not possible to test everything specified in the task at the moment, I can merge what is already working and tested.
The rest can be placed in a separate task or you can specify what blocks its execution.

@donhardman
Copy link
Contributor Author

@donhardman
Copy link
Contributor Author

I've resolved the issue in Buddy that led to this bug after we addressed the double thread spawn. Please confirm that it's working now. Additionally, I discovered that the configuration for data validation is incorrect.

To handle data manipulation properly, we need to use sharding/inserts from Buddy.

Please double-check and add the required line to the config ONLY for data manipulation tests.

Image

@PavelShilin89
Copy link

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

No branches or pull requests

3 participants