Source:
<code>public LockInfo(ZNRecord znRecord) { this(); if (znRecord != null) { String ownerId = znRecord.getSimpleField(LockInfoAttribute.OWNER.name()); String message = znRecord.getSimpleField(LockInfoAttribute.MESSAGE.name()); <START> long timeout = znRecord <END> .getLongField(LockInfoAttribute.TIMEOUT.name(), LockConstants.DEFAULT_TIMEOUT_LONG); int priority = znRecord .getIntField(LockInfoAttribute.PRIORITY.name(), LockConstants.DEFAULT_PRIORITY_INT); long waitingTimeout = znRecord.getLongField(LockInfoAttribute.WAITING_TIMEOUT.name(), LockConstants.DEFAULT_WAITING_TIMEOUT_LONG); long cleanupTimeout = znRecord.getLongField(LockInfoAttribute.CLEANUP_TIMEOUT.name(), LockConstants.DEFAULT_CLEANUP_TIMEOUT_LONG); String requestorId = znRecord.getSimpleField(LockInfoAttribute.REQUESTOR_ID.name()); int requestorPriority = znRecord.getIntField(LockInfoAttribute.REQUESTOR_PRIORITY.name(), LockConstants.DEFAULT_REQUESTOR_PRIORITY_INT); long requestorWaitingTimeout = znRecord .getLongField(LockInfoAttribute.REQUESTOR_WAITING_TIMEOUT.name(), LockConstants.DEFAULT_REQUESTOR_WAITING_TIMEOUT_LONG); long requestorRequestingTimestamp = znRecord .getLongField(LockInfoAttribute.REQUESTOR_REQUESTING_TIMESTAMP.name(), LockConstants.DEFAULT_REQUESTOR_REQUESTING_TIMESTAMP_LONG); setLockInfoFields(ownerId, message, timeout, priority, waitingTimeout, cleanupTimeout, requestorId, requestorPriority, requestorWaitingTimeout, requestorRequestingTimestamp); } }</code><technical_language>nit, list here, "this()" call is necessary. current code set fields twice, is necessary all</technical_language>

Target:
public LockInfo(ZNRecord znRecord) { if (znRecord == null) { znRecord = new ZNRecord(ZNODE_ID); } String ownerId = znRecord.getSimpleField(LockInfoAttribute.OWNER.name()) == null ? LockConstants.DEFAULT_USER_ID : znRecord.getSimpleField(LockInfoAttribute.OWNER.name()); String message = znRecord.getSimpleField(LockInfoAttribute.MESSAGE.name()) == null ? LockConstants.DEFAULT_MESSAGE_TEXT : znRecord.getSimpleField(LockInfoAttribute.MESSAGE.name()); long timeout = znRecord.getLongField(LockInfoAttribute.TIMEOUT.name(), LockConstants.DEFAULT_TIMEOUT_LONG); int priority = znRecord.getIntField(LockInfoAttribute.PRIORITY.name(), LockConstants.DEFAULT_PRIORITY_INT); long waitingTimeout = znRecord.getLongField(LockInfoAttribute.WAITING_TIMEOUT.name(), LockConstants.DEFAULT_WAITING_TIMEOUT_LONG); long cleanupTimeout = znRecord.getLongField(LockInfoAttribute.CLEANUP_TIMEOUT.name(), LockConstants.DEFAULT_CLEANUP_TIMEOUT_LONG); String requestorId = znRecord.getSimpleField(LockInfoAttribute.REQUESTOR_ID.name()); int requestorPriority = znRecord.getIntField(LockInfoAttribute.REQUESTOR_PRIORITY.name(), LockConstants.DEFAULT_PRIORITY_INT); long requestorWaitingTimeout = znRecord .getLongField(LockInfoAttribute.REQUESTOR_WAITING_TIMEOUT.name(), LockConstants.DEFAULT_WAITING_TIMEOUT_LONG); long requestingTimestamp = znRecord .getLongField(LockInfoAttribute.REQUESTING_TIMESTAMP.name(), LockConstants.DEFAULT_REQUESTING_TIMESTAMP_LONG); setLockInfoFields(ownerId, message, timeout, priority, waitingTimeout, cleanupTimeout, requestorId, requestorPriority, requestorWaitingTimeout, requestingTimestamp); }