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 obsolete methods and properties from Akka.Cluster #6445
Remove obsolete methods and properties from Akka.Cluster #6445
Conversation
/// Obsolete. Use <see cref="P:Cluster.DowningProvider.DownRemovalMargin"/>. | ||
/// </summary> | ||
[Obsolete("Use Cluster.DowningProvider.DownRemovalMargin [1.1.2]")] | ||
public TimeSpan DownRemovalMargin { get; } |
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.
Obsolete since v1.1.2
/// <param name="allowLocalRoutees">N/A</param> | ||
/// <param name="routeesPaths">N/A</param> | ||
[Obsolete("This method is deprecated [1.1.0]")] | ||
public ClusterRouterGroupSettings(int totalInstances, bool allowLocalRoutees, IEnumerable<string> routeesPaths) |
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.
Obsolete since v1.1.0
/// <param name="useRole">N/A</param> | ||
/// <param name="routeesPaths">N/A</param> | ||
[Obsolete("This method is deprecated [1.1.0]")] | ||
public ClusterRouterGroupSettings(int totalInstances, bool allowLocalRoutees, string useRole, ImmutableHashSet<string> routeesPaths) |
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.
Obsolete since v1.1.0
/// <param name="allowLocalRoutees">N/A</param> | ||
/// <param name="maxInstancesPerNode">N/A</param> | ||
[Obsolete("This method is deprecated [1.1.0]")] | ||
public ClusterRouterPoolSettings(int totalInstances, bool allowLocalRoutees, int maxInstancesPerNode) |
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.
Obsolete since 1.1.0
/// <param name="useRole">N/A</param> | ||
/// <param name="maxInstancesPerNode">N/A</param> | ||
[Obsolete("This method is deprecated [1.1.0]")] | ||
public ClusterRouterPoolSettings(int totalInstances, bool allowLocalRoutees, string useRole, int maxInstancesPerNode) |
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.
Obsolete since v1.1.0
@@ -166,18 +166,6 @@ public KeepOldestSettings(bool downIfAlone, string role) | |||
|
|||
public sealed class LeaseMajoritySettings | |||
{ | |||
[Obsolete] | |||
public LeaseMajoritySettings(string leaseImplementation, TimeSpan acquireLeaseDelayForMinority, string role) |
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.
Obsolete since v1.4.22
} | ||
|
||
[Obsolete] | ||
public LeaseMajoritySettings(string leaseImplementation, TimeSpan acquireLeaseDelayForMinority, string role, string leaseName) |
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.
Obsolete since v1.4.22
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
No description provided.