----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30059/#review68685 -----------------------------------------------------------
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/SendingAccountor.java <https://reviews.apache.org/r/30059/#comment113035> Let's not change this everywhere. exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/SendingAccountor.java <https://reviews.apache.org/r/30059/#comment113036> Synchronized is fine. You can remove comments. exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/SendingAccountor.java <https://reviews.apache.org/r/30059/#comment113037> Yes, it should. - Jacques Nadeau On Jan. 20, 2015, 12:45 a.m., Daniel Barclay wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30059/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2015, 12:45 a.m.) > > > Review request for drill, Jacques Nadeau and Jason Altekruse. > > > Repository: drill-git > > > Description > ------- > > Technical change: Added "synchronized" to increment(). > > Other changes: Documentation and renaming for clarity. > > > Diffs > ----- > > > exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/SendingAccountor.java > 7af7b65 > > Diff: https://reviews.apache.org/r/30059/diff/ > > > Testing > ------- > > Ran developer and QA tests; got no errors other than errors that appear for > master branch. > > > Thanks, > > Daniel Barclay > >
