JDK-6759617 : Rename DEFAULT_LEASE_TIMEOUT to DEFAULT_REQUESTED_LEASE_TIMEOUT
  • Type: Enhancement
  • Component: core-svc
  • Sub-Component: javax.management
  • Affected Version: 7
  • Priority: P4
  • Status: Closed
  • Resolution: Duplicate
  • OS: generic
  • CPU: generic
  • Submitted: 2008-10-15
  • Updated: 2010-07-29
  • Resolved: 2008-10-28
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 7
7Resolved
Related Reports
Duplicate :  
Description
As of JDK 7 b37 the EventClient API makes use of a requestedlease, first as a parameter of one of the constructors, second by defining a method getRequestedLease().
Considering the most common use case is perhaps not the constructor that takes a lease, I suggest to rename the parameter 'lease' and the method 'getLease()'.
By the way the API defines the field DEFAULT_LEASE_TIMEOUT, not DEFAULT_REQUESTED_LEASE_TIMEOUT or the like.

Comments
EVALUATION This will be fixed as part of the fix for 5072267.
28-10-2008

EVALUATION A lease, in the real world, is a contract that allows you to own something for a limited period of time. It is not the period of time itself. So I think the names requestedLeaseTime and getRequestedLeaseTime are OK as they stand and the proposed change would be for the worse. On the other hand, we could reasonably rename DEFAULT_LEASE_TIMEOUT to DEFAULT_REQUESTED_LEASE_TIMEOUT.
15-10-2008