Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.6
    • Labels:
      None

      Description

      We are calling an SP to retrieve a timestamp from the DB but the mapping of the nano seconds is not being done correctly. Only milliseconds are being returned although the SP is returning the nano seconds.
      After debugging through the apache db utils, we found that in the class BeanProcessor (method callSetter), the setting of timestamp is not being done properly:

      if ("java.sql.Timestamp".equals(targetType)) {
           value = new java.sql.Timestamp(((java.util.Date) value).getTime());
       }
      

      We have modified the codes as follows so as to get the nano seconds:

      if ("java.sql.Timestamp".equals(targetType))
      {
       int nanos = ((Timestamp) value).getNanos(); 
      value = new java.sql.Timestamp(((Timestamp) value).getTime());
       ((Timestamp) value).setNanos(nanos); 
      } 
      

      Can you please provide a fix as soon as possible. (v1.6 would be great)

        Activity

        Hide
        Benedikt Ritter added a comment -

        Fixed in rev. 16110066. Thanks for contributing!

        Show
        Benedikt Ritter added a comment - Fixed in rev. 16110066. Thanks for contributing!
        Hide
        Benedikt Ritter added a comment -

        Feysal Rujbally I'm currently working on the open issues and expect to cut Dbutil 1.6 RC1 anytime this week.

        Show
        Benedikt Ritter added a comment - Feysal Rujbally I'm currently working on the open issues and expect to cut Dbutil 1.6 RC1 anytime this week.
        Hide
        Feysal Rujbally added a comment -

        Can you please advise when a new version will be available.

        Show
        Feysal Rujbally added a comment - Can you please advise when a new version will be available.
        Hide
        Daniele Cremonini added a comment -

        Plese find attached patch. Tests were modified to check the suggested implementation.

        Show
        Daniele Cremonini added a comment - Plese find attached patch. Tests were modified to check the suggested implementation.

          People

          • Assignee:
            Benedikt Ritter
            Reporter:
            Feysal Rujbally
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development