[Tm4j-developers] [ tm4j-Bugs-1704268 ] Occurence may be member of zero to many different Topics

[SourceForge.net <no...@so...>]

Bugs item #1704268, was opened at 2007-04-20 13:58
Message generated for change (Comment added) made by mediumnet
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=391879&aid=1704268&group_id=27895

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: In-Memory Impl
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Xuan Baldauf (mediumnet)
Assigned to: Nobody/Anonymous (nobody)
Summary: Occurence may be member of zero to many different Topics

Initial Comment:
By analyzing the current code, I found out following:

(1) Occurences may be removed from their parent topic by calling setOccurences(new Occurence[0]) on the parent topic. Then, these occurences are orphaned.
(2) Occurences may be added to a different topic by calling setOccurences(new Occurence[foreignOccurence]) or by calling addOccurrence(foreignOccurence);
(3) Occurences may be added to a different topic by calling newTopic.setOccurences(new Occurence[foreignOccurence]) while still not being removed from the old parent.

Orphaned occurences are not supported by TMDM or by TMAPI. Moving occurences between different topics is also not supported by TMAPI. Support for orphaning occurences (for other reasons than to finally destroying them) and adding orphaned occurences should thus be removed. Thus, as a first step, the corresponding methods should be deprecated.

I'm not sure wether any TM4J application exists which moves occurences between topics.

I'm also not sure wether any TM4J applications exists which implicitly removes occurences from their parentTopic by calling parentTopic.setOccurences(smallerOccurenceArray) as opposed to just explicitly removing occurences by calling occurence.destroy().

If no such application exists, the methods Topic.addOccurrence() and Topic.setOccurrences() could be simply removed (the remaining functionality of Topic.setOccurences() would be just removing occurences, but this functionality is also available by calling occurence.destroy().)


----------------------------------------------------------------------

>Comment By: Xuan Baldauf (mediumnet)
Date: 2007-04-20 15:43

Message:
Logged In: YES 
user_id=506885
Originator: YES

Lars Heuer voiced concerns regarding the depreciation (and later removal)
of Topic.addOccurence() (using private e-mail, because the SF bug tracker
did not work temporarily). He also pointed out that this issue does not
only apply to Occurences.

It would be nice if you, Lars, could elaborate your point again here, if
you like.

I want to clarify that I do not intend to remove the implementation of
addOccurence() in TopicImpl, but I want to remove addOccurence() from the
interface Topic. Reasons are that else Occurences could be (at least
temporarily) without a parenting Topic, which would complicate indexing of
TopicMaps, be in contradiction to TMDM, be in contradiction to TMAPI.
Additionally, it would create more memory requirements, as then (new)
backends would be forced to support Occurences which belong to a particular
TopicMap, while not belonging to a particular topic, thus requiring the
Occurence to know its TopicMap explicitly (and not implictly by knowing its
Topic) which would require an extra pointer per Occurence (at least if
implemented straightforwardly).


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=391879&aid=1704268&group_id=27895