#2557 closed defect (done)

Cached objects are troublesome for object graph traversal

Reported by: abeham Owned by: mkommend
Priority: medium Milestone: HeuristicLab 3.3.14
Component: Problems.ExternalEvaluation Version: 3.3.13
Keywords: Cc:

Description

If there are objects in the cache they may prevent object graph traversal from completing successfully.

Change History (13)

comment:1 Changed 21 months ago by pfleck

r13494 Excluded all Types from System.Reflection.Emit from object graph traversal.

The problem occured during object-graph traversal, where GetObjectGraphObjects accesses the GoogleProtoBuff-Message in the CacheEntry. Accessing some of their members causes Reflection.Emit to create code that throws an exception.

When calling GetObjectGraphObjects, some types are already excluded from being returned, like the SignatureHelper from Reflection.Emit (due to simmilar problems). To fix problems with Reflection.Emit types, all types from "System.Reflection.Emit" are excluded now.

comment:2 Changed 21 months ago by pfleck

  • Status changed from new to accepted

comment:3 Changed 21 months ago by pfleck

  • Owner changed from pfleck to mkommend
  • Status changed from accepted to reviewing

comment:4 Changed 21 months ago by pfleck

  • Owner changed from mkommend to pfleck
  • Status changed from reviewing to assigned

comment:5 Changed 21 months ago by pfleck

  • Status changed from assigned to accepted

comment:6 Changed 21 months ago by pfleck

r13510 Fixed wrong excluding of types in object graph traversal (no members were traversed any more). Added a unit test that verifies that GetObjectGraphObjects yields a reasonable number of objects.

comment:7 Changed 19 months ago by pfleck

  • Owner changed from pfleck to abeham
  • Status changed from accepted to reviewing

comment:8 Changed 19 months ago by pfleck

  • Owner changed from abeham to mkommend

comment:9 Changed 19 months ago by mkommend

  • Owner changed from mkommend to pfleck
  • Status changed from reviewing to assigned

Reviewed r13494 and r13510.

Please change the unit test name to a more descriptive one.

comment:10 Changed 19 months ago by pfleck

  • Status changed from assigned to accepted

comment:11 Changed 19 months ago by pfleck

  • Owner changed from pfleck to mkommend
  • Status changed from accepted to reviewing

r13676: Renamed test method and added an additional description.

comment:12 Changed 19 months ago by mkommend

  • Status changed from reviewing to readytorelease

Reviewed r13676.

Last edited 16 months ago by mkommend (previous) (diff)

comment:13 Changed 16 months ago by mkommend

  • Resolution set to done
  • Status changed from readytorelease to closed

r13879: Merged r13494, r13510 and r13676 into stable.

Note: See TracTickets for help on using tickets.