I fixed the failure in TestNRTThreads, but in the process tripped an assert because SegmentReader.doCommit isn't sync'd.
So I sync'd it, but I don't think the norms APIs need to be sync'd – we populate norms up front and then never change them. Un-sync'ing them is important so that in the NRT case calling IW.commit doesn't block searches trying to pull norms.
Also some small code refactoring.
|Fix Version/s||3.2 [ 12316070 ]|
|Fix Version/s||3.1 [ 12314822 ]|
|Status||Open [ 1 ]||Resolved [ 5 ]|
|Resolution||Fixed [ 1 ]|
|Status||Resolved [ 5 ]||Closed [ 6 ]|
|Transition||Time In Source Status||Execution Times||Last Executer||Last Execution Date|
|54d 22m||1||Michael McCandless||06/May/11 00:28|
|28d 17h 8m||1||Robert Muir||03/Jun/11 17:37|