-
Notifications
You must be signed in to change notification settings - Fork 1k
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
remove unused reference.conf
from DistributedData.LightningDb
#7294
remove unused reference.conf
from DistributedData.LightningDb
#7294
Conversation
These configuration values are read from and used by `DistributedData/reference.conf`, so this unused duplicate file is confusing and unnecessary.
reference.confg
from DistributedData.LightningDbreference.conf
from DistributedData.LightningDb
On this block of code: WithSystem(_configForNewMode, typeName, "NewMode", (system, region, rememberedEntitiesProbe) =>
{
AssertRegionRegistrationComplete(region);
var probe = CreateTestProbe(system);
region.Tell(new Message(1), probe.Ref);
probe.ExpectMsg("ack");
ImmutableHashSet.Create(
rememberedEntitiesProbe.ExpectMsg<string>(),
rememberedEntitiesProbe.ExpectMsg<string>(),
rememberedEntitiesProbe.ExpectMsg<string>()).Should().BeEquivalentTo("1", "2", "3"); // 1-2 from the snapshot, 3 from a replayed message
rememberedEntitiesProbe.ExpectNoMsg();
}); The issue occurred because the sharding system had to retry remembering entity
All 3 entities get started, but due to the retry to entity 1 the |
// due to retries in the remember-entities system, we have to tolerate | ||
// potentially receiving a duplicate message for the same entity | ||
// therefore, we need to wait for at least 3 distinct messages or until the timeout | ||
var maxTimeout = TimeSpan.FromSeconds(5); |
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.
Hardened the spec here - the comment above explains the change.
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.
LGTM
Changes
These configuration values are read from and used by
DistributedData/reference.conf
, so this unused duplicate file is confusing and unnecessary.