accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mike Drob" <md...@mdrob.com>
Subject Re: Review Request 16857: ACCUMULO-2182 - first set of backports to 1.4.5-SNAPSHOT
Date Tue, 14 Jan 2014 17:51:56 GMT

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

Ship it!



src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/State.java
<https://reviews.apache.org/r/16857/#comment60465>

    It looks like there is a potential concurrency issue between this method and isMultiTableBatchWriterInitialized,
but I haven't traced deep enough to discover if that's a real problem for us or not. Also,
the issue exists upstream, so this is a faithful backport, if nothing else. Maybe it makes
sense to open a new JIRA for the bug.


- Mike Drob


On Jan. 14, 2014, 2:09 p.m., Bill Havanki wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16857/
> -----------------------------------------------------------
> 
> (Updated Jan. 14, 2014, 2:09 p.m.)
> 
> 
> Review request for accumulo and Josh Elser.
> 
> 
> Bugs: ACCUMULO-2182
>     https://issues.apache.org/jira/browse/ACCUMULO-2182
> 
> 
> Repository: accumulo
> 
> 
> Description
> -------
> 
> Backports of the following to 1.4.5-SNAPSHOT: ACCUMULO-1123, ACCUMULO-2104 / ACCUMULO-2106.
This review request is a combined diff; I have a patch file with separate commits for -1123
and -2104/-2106.
> 
> These backports were done "by hand" rather than cherry-picking, so double-checking against
the original tickets is most welcome.
> 
> 
> Diffs
> -----
> 
>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/State.java 1c73414

>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/image/ImageFixture.java
fe406e7 
>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/multitable/MultiTableFixture.java
d7f65ae 
>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/ChangePass.java
429c8cd 
>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/CreateUser.java
afefee8 
>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityHelper.java
52a4672 
>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/TableOp.java
e0b8deb 
>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/sequential/SequentialFixture.java
5714893 
>   src/server/src/main/java/org/apache/accumulo/server/test/randomwalk/shard/ShardFixture.java
b91c81c 
> 
> Diff: https://reviews.apache.org/r/16857/diff/
> 
> 
> Testing
> -------
> 
> Successfully ran following randomwalk tests under 1.4.x: Image, Sequential, Shard, Multitable.
Security test running 17 hours so far without failure due to BAD_CREDENTIALS, which is what
-1123 fixes.
> 
> 
> Thanks,
> 
> Bill Havanki
> 
>


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