accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dave Marion <dlmar...@hotmail.com>
Subject Re: Review Request 43957: ACCUMULO-1755: removed synchronized modifier from TabletServerBatchWriterstartProcessing()
Date Wed, 24 Feb 2016 19:29:52 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43957/
-----------------------------------------------------------

(Updated Feb. 24, 2016, 7:29 p.m.)


Review request for accumulo and Josh Elser.


Changes
-------

modified startProcessing() so that duplicates would not be added to the writer. Inside of
startProcessing(), the MutationSet could have been swapped out by another thread between each
call to mutations.get()


Repository: accumulo


Description
-------

ACCUMULO-1755: removed synchronized modifier from TabletServerBatchWriterstartProcessing()


Diffs (updated)
-----

  core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java bc90d00


Diff: https://reviews.apache.org/r/43957/diff/


Testing
-------

unit tests in core pass


Thanks,

Dave Marion


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message