Free cookie consent management tool by TermsFeed Policy Generator

Opened 9 years ago

Closed 8 years ago

#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 9 years 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 9 years ago by pfleck

  • Status changed from new to accepted

comment:3 Changed 9 years ago by pfleck

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

comment:4 Changed 9 years ago by pfleck

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

comment:5 Changed 9 years ago by pfleck

  • Status changed from assigned to accepted

comment:6 Changed 9 years 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 9 years ago by pfleck

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

comment:8 Changed 9 years ago by pfleck

  • Owner changed from abeham to mkommend

comment:9 Changed 9 years 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 9 years ago by pfleck

  • Status changed from assigned to accepted

comment:11 Changed 9 years 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 9 years ago by mkommend

  • Status changed from reviewing to readytorelease

Reviewed r13676.

Last edited 8 years ago by mkommend (previous) (diff)

comment:13 Changed 8 years 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.