Project

General

Profile

Story #7324

Synchronization is setting default values for origin and authoritativeMN if left null by the source MN.

Added by Rob Nahf about 9 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Architecture Design
Target version:
Start date:
2015-09-01
Due date:
% Done:

100%

Story Points:
Sprint:

Description

In issue #3062, it was agreed that the CN should not be setting the authoritativeMN for an object, yet it still does for cases where those fields are null from the MN being synchronized, setting it using the nodeID in the associated syncTask (the systemMetadata's source MN).

The CN does need an authoritativeMN for further processing, but can we assume that the source node is the right one?
How does this jive with pre-replicated content (where there's already a replica on a member node) and the possibility of getting the replica before the authoritative? (what instructions have we given MNs who do this)

For example, if both MemberNodes have the same value for the originNode, but leave the authoritativeMN field blank, the authoritativeMN will be set to whichever node sync's first. It seems that if the originMN is passed in by the source node, and the authMN field is null, it should be populated with the same node as originMN, not the source node.

Associated revisions

Revision 16243
Added by Rob Nahf about 9 years ago

refs: #7314, #7324: Removed restriction on dateSystemMetadtataModified field that was preventing update of an object. Added logic for rejecting updates if the CN is authoritative for the systemMetadata (v1 read-write nodes). Added missing business-rule validation on systemMetadata.

Revision 16243
Added by Rob Nahf about 9 years ago

refs: #7314, #7324: Removed restriction on dateSystemMetadtataModified field that was preventing update of an object. Added logic for rejecting updates if the CN is authoritative for the systemMetadata (v1 read-write nodes). Added missing business-rule validation on systemMetadata.

History

#1 Updated by Rob Nahf about 9 years ago

  • Description updated (diff)

#2 Updated by Rob Nahf about 9 years ago

  • Description updated (diff)

#3 Updated by Rob Nahf about 9 years ago

Chris suggests that we instead return InvalidSystemMetadata because the fields are blank. Agreed.

#4 Updated by Rob Nahf about 9 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

created unit test for the situation and got it to pass.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 14.8 MB)