commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (Jira)" <j...@apache.org>
Subject [jira] [Work logged] (DBCP-509) Not all methods are consistent in PerUserPooldataSource and InstanceKeyDataSource
Date Tue, 08 Sep 2020 21:26:00 GMT

     [ https://issues.apache.org/jira/browse/DBCP-509?focusedWorklogId=480449&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-480449
]

ASF GitHub Bot logged work on DBCP-509:
---------------------------------------

                Author: ASF GitHub Bot
            Created on: 08/Sep/20 21:25
            Start Date: 08/Sep/20 21:25
    Worklog Time Spent: 10m 
      Work Description: kinow commented on pull request #14:
URL: https://github.com/apache/commons-dbcp/pull/14#issuecomment-689144262


   No worries @garydgregory . Had completely forgotten about it, but no bandwidth to work
on this issue now. We can revisit it later on :+1: Thanks!


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 480449)
    Time Spent: 40m  (was: 0.5h)

> Not all methods are consistent in PerUserPooldataSource and InstanceKeyDataSource
> ---------------------------------------------------------------------------------
>
>                 Key: DBCP-509
>                 URL: https://issues.apache.org/jira/browse/DBCP-509
>             Project: Commons DBCP
>          Issue Type: Improvement
>    Affects Versions: 2.4.0
>            Reporter: Bruno P. Kinoshita
>            Priority: Major
>             Fix For: 2.8.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> While working on DBCP-504 tests, I wrote a few files to help me automating some of the
tests (e.g. https://gist.github.com/kinow/053b6d1f293fdc208a2a14571f246786).
> In PerUserPooldataSource, I realized I had to change some tests that were failing to
handle null values. But not all methods. So I had a look at the methods, and the majority
was following a pattern
> * using primitives
> * default'ing to the class/parent method getDefaultPropertyZ() whenever Z property was
null
> But three values were using objects instead of primitives, and allowing null's:
> * defaultAutoCommit
> * defaultReadOnly
> * perUserDefaultAutoCommit
> I prepared a pull request that falls back to the default method's values.
> It's more of a discussion issue, just to document what I found during DBCP-504.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Mime
View raw message