[DERBY-5285] [patch] avoid npe if tempCongloms is null

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Duplicate
• Affects Version/s: 10.8.2.2
• Fix Version/s: 10.10.1.1
• Component/s: Store
• Urgency: Low
• Assignee: Mike Matrigali
• Reporter: Dave Brosius
• Created: 19/Jun/11 07:45
• Updated: 17/Jun/13 09:19
• Resolved: 25/May/12 10:49

Description

if (tempCongloms != null)
tempCongloms.remove(new Long(conglomId));
tempCongloms.put(new Long(conglomId), conglom);

code guards against tempCongloms being null for the remove, but not for the put. so put the put in the if statement.
Then realized that the remove is superfluous, as the put will overright so just removed the remove.

Issue Links

• duplicates: DERBY-3371 Strange (and untested) code fragment in RAMTransaction.addColumnToConglomerate()