Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Bug #684 (closed)

Opened 17 years ago

Closed 17 years ago

blitz implementation of Pojos.findAnnotations throws a ClassCastException

Reported by: jamoore Owned by: jamoore
Priority: major Cc:
Sprint: n.a.
Total Remaining Time: n.a.

Description

The mapping logic doesn't recurse down into the Map<Long,Set> to convert it to a List. Easiest solution is probably to enforce "No sets in Ice" since the other language bindings don't have them anyway.

Change History (1)

comment:1 Changed 17 years ago by jmoore

  • Resolution set to fixed
  • Status changed from new to closed

r1497 closes. In the Java/Ice? mapping, everything is now a List for simplicity. Some cleaning up needs to be done in the code-generation to reflect this.

Another bug was uncovered with the switch: apparently, the hash function for empty collections (lists only?) was inexact enough that model objects were sharing collections. E.g. after mapping, a ProjectI and a DatasetI would both have the same collection instance. This caused duplicates in the collections. Solved by changing HashMap to IdentityHashMap in common/.../ModelMapper.java.

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.64041 sec.)

We're Hiring!