Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2.2
    • Fix Version/s: 2.0
    • Labels:
      None
    • Environment:

      All.

      Description

      Request to add support for testOnCreate, to validate connections only when they are created, much less frequent than on borrow.

      This will solve issues where for example driver logs you into master instead of db on the connection string because db is not available (e.g. Sybase) and then rest of the queries fail. This only happens on first login. Hence validating the correct db using sql can test such a case so that borrowers are guaranteed correct db without having to testOnBorrow every time.

      May also be worthwhile to support different validation queries for testing at different times.

      Thanks.
      Parag

        Activity

        Parag Mehta created issue -
        Phil Steitz made changes -
        Field Original Value New Value
        Affects Version/s 1.2.2 [ 12311976 ]
        Fix Version/s 1.4 [ 12312615 ]
        Affects Version/s 1.4 [ 12312615 ]
        Affects Version/s 1.3 [ 12311977 ]
        Phil Steitz made changes -
        Fix Version/s 2.0 [ 12313721 ]
        Fix Version/s 1.4 [ 12312615 ]
        Hide
        Mark Thomas added a comment -

        I can see the use case for testOnCreate. Support for this will need to be added to Commons Pool. It will only have an effect if testOnBorrow is false, since testOnBorrow triggers a validation test on create already.

        Show
        Mark Thomas added a comment - I can see the use case for testOnCreate. Support for this will need to be added to Commons Pool. It will only have an effect if testOnBorrow is false, since testOnBorrow triggers a validation test on create already.
        Mark Thomas made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Phil Steitz made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Parag Mehta
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development