SeqWriter.add is waiting 10 sec in a worse case before calling a flush. This was there to guarantee that the data has been written to HDFS before returning the ack.
It used to work well with 0.18 but in 0.20 the flush is a nohup so there's no point in waiting 10 sec for nothing.
The real flush will give us the ability to write redo logs directly to HDFS so the ack implementation will have to change anyway.
So I vote for removing the 10 sec ack from the add method.
|Transition||Time In Source Status||Execution Times||Last Executer||Last Execution Date|
|2h 19m||1||Jerome Boulon||18/Jun/09 23:55|
|2h 1m||1||Eric Yang||19/Jun/09 01:57|
|Status||Patch Available [ 10002 ]||Resolved [ 5 ]|
|Resolution||Fixed [ 1 ]|
|Status||Open [ 1 ]||Patch Available [ 10002 ]|
|Affects Version/s||0.2.0 [ 12313934 ]|
|Affects Version/s||0.3.0 [ 12314032 ]|
|Fix Version/s||0.2.0 [ 12313934 ]|
|Fix Version/s||0.3.0 [ 12314032 ]|