Project

General

Profile

Feature #1386

ckDestroy for Groups and NodeGroups

Added by Sam White over 2 years ago. Updated 11 months ago.

Status:
Rejected
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
01/28/2017
Due date:
% Done:

0%


Description

Jozsef Bakosi pointed out on the charm mailing list that we currently do not support deleting Groups. ckDestroy is only defined for Chare Arrays.

History

#1 Updated by Eric Bohm over 2 years ago

  • Target version changed from 6.8.0 to 6.8.1

Retargetting this to a later version until someone makes a case for it being urgently required in 6.8.0.

#2 Updated by Sam White over 2 years ago

  • Assignee set to Eric Mikida

#3 Updated by Sam White almost 2 years ago

  • Target version changed from 6.8.1 to 6.9.0

#4 Updated by Eric Mikida almost 2 years ago

  • Target version deleted (6.9.0)

#5 Updated by Sam White over 1 year ago

  • Subject changed from ckDestroy for Groups to ckDestroy for Groups and NodeGroups

#6 Updated by Sam White 11 months ago

  • Assignee changed from Eric Mikida to Evan Ramos

Re-assigning to Evan since Quinoa was originally the motivation for this.

#7 Updated by Evan Ramos 11 months ago

  • Status changed from New to Rejected

For CProxy_MyChareArray array;, calling array.ckDestroy(); only serves to broadcast a call to ArrayElement::ckDestroy. This removes all the elements from the array but does not delete the array itself, if such a concept even exists. CkArray::ckDestroy is never called. In testing I was able to insert new elements into the array after such a ckDestroy broadcast completed. Groups and nodegroups do not allow insertion or removal of elements so ckDestroy does not make sense for them in light of this fact. We would need to redefine ckDestroy which is out of scope of this issue.

#8 Updated by Sam White 11 months ago

Jozsef's original email to the Charm mailing list was IIRC more interested in how to best free the memory associated with a Group that is only transiently used. Even if it shouldn't be named ckDestroy() for semantic reasons, we should provide an API for freeing Groups, even if it's just a wrapper around 'delete this' for each element of the Group. We should also better document the semantics of ckDestroy(), the manual looks to be lacking here at the moment.

Also available in: Atom PDF