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
Sharding: Add some tests for sharding codepaths #615
base: master
Are you sure you want to change the base?
Conversation
License: MIT Signed-off-by: Hector Sanjuan <code@hector.link>
Actually one of the tests should fail because it tests the correct unpinning of shared shards. But let's leave this here so I don't forget I need to fix it. |
// the files resulting from GetShardingDir* | ||
// They have been obtained by adding the "shardTesting" folder | ||
// the files resulting from shardingTestDir/shadringTestTree folder. | ||
// They have been obtained by adding the that folder |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the that folder
???
pinDelay() | ||
}) | ||
|
||
t.Run("shard", func(t *testing.T) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great if each of these tests could have a bit more explanation as to what they are testing as the testcase is hard to grok from the code.
@hsanjuan mind cleaning this up? Was reading it and then noticed that date :D |
I had these changes laying around on a different branch (along some other code not included here). They are the ones that surfaced the problems with the batching DAGService. Since that was fixed, we can merge these so they are not lost (I think).