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"

Task #2142 (new)

Opened 14 years ago

Last modified 12 years ago

Remove all compile warnings from SPR 3.0 and HIB 3.5

Reported by: jamoore Owned by: jamoore
Priority: critical Milestone: Unscheduled
Component: Deployment Version: n.a.
Keywords: n.a. Cc: cxallan, jburel, wmoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.5d
Sprint: n.a.

Description (last modified by jmoore)

Several APIs were deprecated in the two and various checks for annotation misuse were added:

  • Cascade type deprecations are particularly numerous. (delete goes to OneToMany(orphanRemoval=true)
  • @DocumentId specified on an entity which is not indexed by itself. Annotation gets ignored. Use @Field instead.

Change History (13)

comment:1 Changed 14 years ago by jmoore

  • Description modified (diff)

comment:2 Changed 14 years ago by jmoore

  • Description modified (diff)

comment:3 Changed 14 years ago by jmoore

  • Sprint set to 2010-06-10 (11)

Should possibly consider doing this for release to prevent user confusion.

comment:4 Changed 14 years ago by jburel

  • Sprint changed from 2010-06-10 (11) to 2010-06-24 (12)

comment:6 Changed 14 years ago by jmoore

(In [7364]) Removing significant numbers of deprecation warnings (See #2142)

comment:7 Changed 14 years ago by jmoore

  • Cc cxallan jburel wmoore added

r7364's use of orphanRemoval has led to odd errors:

  • unexpected end of subtree [select rdef from ome.model.display.RenderingDef
  • Deleting ome.model.meta.EventLog:Id_5446 not allowed.

Upgrading to 3.5.3 may be necessary, but a quicker solution for now will be to disable orphanRemoval. That may leave orphaned objects hanging around, but we can take care of those with later upgrade scripts.

Other issues concerning the new delete cascade:

comment:8 Changed 14 years ago by jmoore

(In [7377]) Temporary fix for strange orphanRemoval (See #2142)

comment:9 Changed 14 years ago by jmoore

  • Milestone changed from OMERO-Beta4.2 to Unscheduled
  • Priority changed from major to critical
  • Sprint 2010-06-24 (12) deleted

Not much we can do to fix the orphan issue at this stage. Pushing, but also increasing priority for a 4.2 re-evaluation.

comment:10 Changed 14 years ago by jmoore

(In [7413]) Readding the old DELETE_ORPHAN cascade (See #2547, #2142)

comment:11 Changed 14 years ago by jmoore

(In [7418]) Adding DELETE_ORPHAN to all ordered collections (Fix #1679, See #2142)

comment:12 Changed 13 years ago by jmoore <josh@…>

(In [fda3c786c5d0e9ae56d8aa95a2ede73cdd49ec36/ome.git] on branch dev_4_1_custom) Adding DELETE_ORPHAN to all ordered collections (Fix #1679, See #2142)

git-svn-id: ​file:///home/svn/omero/trunk@7418 05709c45-44f0-0310-885b-81a1db45b4a6

Conflicts:

components/server/test/ome/server/itests/update/UpdateTest.java
components/tools/OmeroJava/test/integration/UpdateServiceTest.java

comment:13 Changed 12 years ago by jmoore

Referencing ticket #1260 has changed sprint.

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.68117 sec.)

© 2000-2014 University of Dundee & Open Microscopy Environment. Creative Commons Attribution 3.0 Unported License
OME source code is available under the GNU General public license or through commercial license from Glencoe Software

We're Hiring!