XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: 0.9.2
    • Fix Version/s: 0.10.0
    • Component/s: None
    • Labels:
      None
    • Operating System:
      Multiple

      Description

      As part of the v4/Java 8 changes, we need to review the use of date/time APIs.

      There is one use of Joda-Time in ComponentInfo, which uses Joda-Time's DateTime as start/end instants. These are based on The ISO chronology and the UTC time zone, presumably so that if you just .toString them, you'll get something sane and consistent. It seems to me likely, however, that these should in fact be {{Instant}}s both in Joda-Time and under Java 8 and JSR-310, with time zone assignment and formatting left to the caller. There are no real uses of these objects within the MDA codebase at present, so this looks low impact.

      Joda-Time conversion stuff:

      http://blog.joda.org/2014/11/converting-from-joda-time-to-javatime.html

      http://blog.joda.org/2009/11/why-jsr-310-isn-joda-time_4941.html

      Most other time-related functionality seems to be based around millisecond representations. I guess that's fine with the presence of the @Duration annotation and the DurationToLongConverter in XML configurations. We should review anyway, though.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              ian@iay.org.uk Ian Young
              Reporter:
              ian@iay.org.uk Ian Young
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 6 hours
                  6h