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

Data manipulation #1828

Closed
wants to merge 23 commits into from
Closed

Data manipulation #1828

wants to merge 23 commits into from

Conversation

PavelShilin89
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Feb 12, 2024

Linux debug test results

 10 files   10 suites   7m 47s ⏱️
978 tests 929 ✅ 49 💤 0 ❌
992 runs  943 ✅ 49 💤 0 ❌

Results for commit b67fc55.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 12, 2024

Linux release test results

 10 files   10 suites   6m 8s ⏱️
978 tests 929 ✅ 49 💤 0 ❌
992 runs  943 ✅ 49 💤 0 ❌

Results for commit b67fc55.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Feb 12, 2024

Windows test results

  3 files    3 suites   16m 54s ⏱️
954 tests 905 ✅ 49 💤 0 ❌
958 runs  909 ✅ 49 💤 0 ❌

Results for commit b67fc55.

♻️ This comment has been updated with latest results.

@PavelShilin89 PavelShilin89 removed the request for review from djklim87 February 20, 2024 09:37
Copy link
Contributor

@donhardman donhardman left a comment

Choose a reason for hiding this comment

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

Looks good just couple issues and some questions

@PavelShilin89 PavelShilin89 requested review from donhardman and removed request for donhardman February 26, 2024 07:46
Copy link
Contributor

@donhardman donhardman left a comment

Choose a reason for hiding this comment

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

Looks good for me. I suggest to merge it to tests/sharding branch first, and after collecting all tests for sharding we can merge to the master

@donhardman
Copy link
Contributor

I've rebased and merged it into the tests/sharding branch first. Once we gather all the working sharding tests there, we can proceed to merge them into the master branch.

@donhardman donhardman closed this Feb 27, 2024
@donhardman
Copy link
Contributor

Link to the pull request: #1478

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

Successfully merging this pull request may close these issues.

2 participants