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"

User Story #67 (closed)

Opened 18 years ago

Closed 15 years ago

Unify Options (ome.util.builders) and QueryParameter (ome.services.query) APIs.

Reported by: jamoore Owned by: jamoore
Priority: critical Milestone: OMERO-Beta4
Component: Remoting Keywords: n.a.
Cc: jburel, atarkowska Story Points: n.a.
Sprint: n.a. Importance: n.a.
Total Remaining Time: n.a. Estimated Remaining Time: n.a.

Description

QueryParameter? instances are intended to permit C++-templates-esque parameter
passing for dynamically generated queries. QueryParameters? can be mapped in our
cross-language remoting platform (problem is touched on for ICE at
http://zeroc.com/vbulletin/showthread.php?t=565) and as such be accessible to
languages other than Java. But to do this, it will need to be in the /common
component.

As long as we are making that change, we could combine QueryParameters? with the
Builders (see PojoBuilder?) to make the builders more type safe and, in general,
more useful.

Originally : http://bugs.openmicroscopy.org.uk/show_bug.cgi?id=663

Change History (11)

comment:1 Changed 18 years ago by jmoore

  • Milestone changed from 3.0-M2 to 3.0-M3

comment:2 Changed 18 years ago by jmoore

  • Milestone changed from 3.0-M3 to Unscheduled

Was being over optimistic. Post-poning.

comment:3 Changed 18 years ago by jmoore

  • Milestone changed from Unscheduled to 3.0-M4
  • Priority changed from minor to critical

This is a fairly substantial API change. Upgrading.

comment:4 Changed 17 years ago by jmoore

  • Milestone changed from 3.0-Beta2 to 3.0-Beta3

r1516 took a step in this direction.

comment:5 Changed 16 years ago by jmoore

  • Cc jburel added
  • Milestone changed from 3.0-Beta3 to 3.0-Beta4

milestone:3.0-Beta3 would actually be a great place to do this, but unfortunately there won't be enough time. Are you alright with the IPojos.method(..., Map map) parameters changing in Beta4, Jean-Marie?

comment:6 Changed 16 years ago by jburel

Beta4 is probably the right time to do it.

comment:7 Changed 15 years ago by atarkowska

  • Cc atarkowska added

comment:8 Changed 15 years ago by jmoore

r3976 to r3979 makes changes to the method signatures. What's remaining is to move as much out of the Map<String, RString> and into real objects like omero.sys.Filter.

comment:9 Changed 15 years ago by jmoore

r3995 and r3997 remove PojoOptions and move fully to Parameters

comment:10 Changed 15 years ago by jmoore

r3999 adds missing Options.java

comment:11 Changed 15 years ago by jmoore

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

API changes are done. As long as the semantics of no methods has changed, this is complete. For any new options or methods in the ParametersI implementations, please open a new ticket.

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

We're Hiring!