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 #5793 (closed)

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

BUG: Delete tagset failed

Reported by: atarkowska Owned by: jamoore
Priority: blocker Milestone: OMERO-Beta4.3
Component: Services Version: n.a.
Keywords: n.a. Cc: jburel
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-06-16 (14)

Description (last modified by atarkowska)

 Delete tagset 8902 Failed 1 error(s) (ConstraintViolation: fkannotationannotationlink_parent_annotation)

Change History (7)

comment:1 Changed 13 years ago by atarkowska

server log:

2011-06-06 15:31:35,662 INFO  [ ome.services.graphs.AnnotationGraphSpec] (3-thread-2) Graphd 0 annotation links2011-06-06 15:31:35,662 INFO  [                 org.perf4j.TimingLogger] (3-thread-2) start[1307370695654] time[8] tag[omero.delete.top.
8902]2011-06-06 15:31:35,667 INFO  [          ome.services.graphs.GraphState] (3-thread-2) Failed to delete TagAnnotation: 8902 due to Constr
aintViolation: fkannotationannotationlink_parent_annotation
2011-06-06 15:31:35,667 INFO  [                 org.perf4j.TimingLogger] (3-thread-2) start[1307370695653] time[14] tag[omero.delete.ste
p.0]2011-06-06 15:31:35,668 INFO  [                 org.perf4j.TimingLogger] (3-thread-2) start[1307370695643] time[25] tag[omero.delete.com
mand.0]2011-06-06 15:31:35,668 INFO  [                 org.perf4j.TimingLogger] (3-thread-2) start[1307370695634] time[34] tag[omero.call.excep
tion]2011-06-06 15:31:35,668 INFO  [        ome.services.util.ServiceHandler] (3-thread-2)  Excp:    ome.services.blitz.impl.DeleteHandleI$Ca
ncel: Cancelled by ConstraintViolation: fkannotationannotationlink_parent_annotation

comment:2 Changed 13 years ago by atarkowska

  • Component changed from General to Services
  • Description modified (diff)
  • Priority changed from minor to blocker

comment:3 Changed 13 years ago by jmoore

Ola, how was the data created and whom did it belong to?

comment:4 Changed 13 years ago by Aleksandra Tarkowska <aleksandrat@…>

comment:5 Changed 13 years ago by jmoore

  • Remaining Time set to 0.5
  • Status changed from new to accepted

Looks like the issue is the discriminator in the following query:

2011-06-07 16:30:05,306 INFO  [   ome.services.blitz.impl.DeleteHandleI] (3-thread-5) Deleting /Annotation:1888
2011-06-07 16:30:05,307 DEBUG [                       org.hibernate.SQL] (3-thread-5) select annotation0_.id as col_0_0_, annotation1_.id as col_1_0_ from annotation
 annotation0_ inner join annotationannotationlink annotation1_ on annotation0_.id=annotation1_.parent and (

  ( annotation1_.group_id = ? AND 
     ( 1 = ? OR 
       1 = ? OR 
       annotation1_.owner_id = ?
     )
  ) OR
  annotation1_.group_id = 1 OR 
 1 = ?
) where (

  ( annotation0_.group_id = ? AND 
     ( 1 = ? OR 
       1 = ? OR 
       annotation0_.owner_id = ?
     )
  ) OR
  annotation0_.group_id = 1 OR 
 1 = ?
) and annotation0_.id=? and annotation0_.discriminator='/'

We actually want it to delete ALL links.

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

  • Remaining Time changed from 0.5 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed

(In [ebd0ba990c984f7d4a8bc6f816ddabf539f151d6/ome.git] on branch develop) Skipping ROOT.class if 'Annotation' for link deletion (Fix #5793)

comment:7 Changed 13 years ago by jmoore

  • Cc jburel added

Note: the deletion now passes but does not delete the tags within the tagset. For that would we need another delete spec. Please open up a ticket for that if desired.

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

We're Hiring!