Project

General

Profile

feature request #7199

Implement equals() for QuantitativeData

Added by Patrick Plitzner almost 4 years ago. Updated almost 4 years ago.

Status:
Rejected
Priority:
New
Assignee:
Patrick Plitzner
Category:
cdmlib
Target version:
-
Start date:
01/17/2018
Due date:
% Done:

0%

Severity:
normal
Tags:

Description

Compare all StatisticalMeasures


Related issues

Related to Edit - feature request #7198: Implement equals() forCategoricalData Rejected 01/17/2018
Related to Edit - bug #7155: VersionableEntity.equals() should take updated timestamp into account Closed 01/08/2018
Related to Edit - task #7201: [DISCUSS] Should we remove created comparison from CdmBase.equals? Closed 01/18/2018

Associated revisions

Revision 51a26cec (diff)
Added by Andreas Müller almost 4 years ago

ref #7155, ref #7198, ref #7199 make Cdm.equals final

  • remove sematically equal method from VersionableEntity
  • add javadoc to explain why it should be final
  • open issue: do we really need to compare created in equals ?

History

#1 Updated by Andreas Müller almost 4 years ago

  • Status changed from New to Rejected
  • Target version deleted (Release 4.13)

Equals for ALL CdmBase is defined in CdmBase itself and should not be overriden.
For discussion see also #7155.

If one wants to compare 2 CdmBase of the same class the class must implement the IMatchable interface and the attributes must be annotated correctly. Using MatchStrategy implementing IMatchStrategy (such as DefaultMatchStrategy) may then be used to compare instances of the class.
So we may open a ticket to implement IMatchable for QuantitativeData

#2 Updated by Andreas Müller almost 4 years ago

#3 Updated by Andreas Müller almost 4 years ago

  • Related to bug #7155: VersionableEntity.equals() should take updated timestamp into account added

#4 Updated by Andreas Müller almost 4 years ago

  • Related to task #7201: [DISCUSS] Should we remove created comparison from CdmBase.equals? added

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)