Bump upstream SNAPSHOTS Adopt: - odlparent-10.0.0 - yangtools-8.0.0-SNAPSHOT - mdsal-9.0.0-SNAPSHOT Change-Id: I60991718a6d1ef6f108c2b1f7b2c09b0c5f973c4 Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Do not leak DataTree from backend actor YANG tools is now exposing the read-only part of DataTree as ReadOnlyDataTree. Use this interface to restrict the set of methods accessible to frontend. Change-Id: I9426d4cee1e68e3cc21f7a39322a0df09739e770 Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Migrate nullness annotations This mass-migrates sal-distributed-datastore to use JDT nullness annotations instead of JSR305. Change-Id: I2cd739e3db8c074a55e335e17297a23065775d6a Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Fix sonar warnings in sal-distributed-datastore These come from squid: - String literals should not be duplicated - Modifiers should be declared in the correct order - Lambdas and anonymous classes should not have too many lines - Nested blocks of code should not be left empty - Local variables should not shadow class fields - Exception handlers should preserve the original exception - Utility classes should not have public constructors - Overriding methods should do more than simply call the same method in the super class - Unused private fields should be removed I fixed quite a few of them. Others we'd have to suppress or modify the sonar config to be more lenient. Change-Id: I7ce7b2a05feac9844fd9c37927de82b7b8b68ee5 Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
String performance and maintenability Use StringBuilder for strings built in loops. Use plain concatenation for string construction not involving loops. Change-Id: I8299eb54a347d132f0d19dfb39c8fb52366049f2 Signed-off-by: Stephen Kitt <skitt@redhat.com>
Return local shard tree from FindPrimaryShard Added a new ShardLeaderStateChanged message that includes the Shard's DataTree as an Optional. If the shard is the leader, it returns it's local DataTree, oherwise returns absent. The ShardManager now returns a LocalPrimaryShardFound response to FindPrimary if the shard's DataTree is present. Otherwise it returns RemotePrimaryShardFound (renamed from PrimaryFound). Change-Id: I4413aacfff3d3d2ee89df7c4a3a1d7f7c3d2c486 Signed-off-by: Tom Pantelis <tpanteli@brocade.com> (cherry picked from commit fdddb482b07c3ee2f3ca853d09ee9a6ecdd7eb2a)