DO NOT REPLY [Bug 35046] New: - [transaction] GenericLockManager constructor ignores LoggerFacade argument

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35046>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35046

           Summary: [transaction] GenericLockManager constructor ignores
                    LoggerFacade argument
           Product: Commons
           Version: 1.1.0
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P3
         Component: Transaction
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: ky.vong@gmail.com


In the following constructor for GenericLockManager, the LoggerFacade
argument is ignored and a new LoggerFacade is always created.  Thus, the
LoggerFacade that I'm passing in is never used.  This affects the
subclasses of GenericLockManager as well since they all call this
constructor.  Someone probably just did this temporarily for some debugging and
forgot to set this.logger back to the LoggerFacade argument.

Can this be fixed for 1.1?

Thanks,
Ky Vong


    public GenericLockManager(int maxLockLevel, LoggerFacade logger, long
timeoutMSecs,
            long checkThreshholdMSecs) throws IllegalArgumentException {
        if (maxLockLevel < 1)
            throw new IllegalArgumentException("The maximum lock level must be
at least 1 ("
                    + maxLockLevel + " was specified)");
        this.maxLockLevel = maxLockLevel;
        this.logger = logger.createLogger("Locking");
        this.globalTimeoutMSecs = timeoutMSecs;
        this.checkThreshhold = checkThreshholdMSecs;
    }

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org