Return-Path: X-Original-To: apmail-accumulo-dev-archive@www.apache.org Delivered-To: apmail-accumulo-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 76D3A10D2E for ; Tue, 3 Dec 2013 19:11:30 +0000 (UTC) Received: (qmail 2223 invoked by uid 500); 3 Dec 2013 19:11:30 -0000 Delivered-To: apmail-accumulo-dev-archive@accumulo.apache.org Received: (qmail 2195 invoked by uid 500); 3 Dec 2013 19:11:30 -0000 Mailing-List: contact dev-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@accumulo.apache.org Delivered-To: mailing list dev@accumulo.apache.org Received: (qmail 2185 invoked by uid 99); 3 Dec 2013 19:11:30 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 03 Dec 2013 19:11:30 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 5CA9D1D3C7E; Tue, 3 Dec 2013 19:11:25 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============6927682760417581357==" MIME-Version: 1.0 Subject: Re: Review Request 15857: ACCUMULO-1931 - Javadoc for core.data From: "Bill Havanki" To: "Bill Havanki" , "accumulo" , keith@deenlo.com Date: Tue, 03 Dec 2013 19:11:25 -0000 Message-ID: <20131203191125.25728.11163@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Bill Havanki" X-ReviewGroup: accumulo X-ReviewRequest-URL: https://reviews.apache.org/r/15857/ X-Sender: "Bill Havanki" References: <20131203182445.25728.68547@reviews.apache.org> In-Reply-To: <20131203182445.25728.68547@reviews.apache.org> Reply-To: "Bill Havanki" X-ReviewRequest-Repository: accumulo --===============6927682760417581357== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit > On Dec. 3, 2013, 1:24 p.m., kturner wrote: > > core/src/main/java/org/apache/accumulo/core/data/Mutation.java, line 251 > > > > > > should comment about defensive copy be here? or maybe in class javadoc for all put methods. Added to each put() method. > On Dec. 3, 2013, 1:24 p.m., kturner wrote: > > core/src/main/java/org/apache/accumulo/core/data/Mutation.java, line 609 > > > > > > Should mention that after this method is called a mutation can no longer be modfied. Also it does a defensive copy, changing the arrays pointed to by column update will not affect mutation I'll add the needed verbiage, but the behavior you're describing isn't enforced by the class, i.e., you can still put things into a Mutation after calling getUpdates(), they just are (silently?) ignored. Perhaps that should be fixed? - Bill ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15857/#review29664 ----------------------------------------------------------- On Dec. 3, 2013, 12:31 p.m., Bill Havanki wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15857/ > ----------------------------------------------------------- > > (Updated Dec. 3, 2013, 12:31 p.m.) > > > Review request for accumulo. > > > Bugs: ACCUMULO-1931 > https://issues.apache.org/jira/browse/ACCUMULO-1931 > > > Repository: accumulo > > > Description > ------- > > Javadoc additions and updates to classes in org.apache.accumulo.core.data. > > > Diffs > ----- > > core/src/main/java/org/apache/accumulo/core/data/ArrayByteSequence.java eaa61b9950b691ed5955af3178a3aa939291e8bf > core/src/main/java/org/apache/accumulo/core/data/ByteSequence.java 4dc921c5efc44d374032c623f50e4218aa6c7299 > core/src/main/java/org/apache/accumulo/core/data/Column.java a56c01d6604a30153739ed1329d034e3932f2e2f > core/src/main/java/org/apache/accumulo/core/data/ColumnUpdate.java 641ca3a740ed384ab93f4b2d3c08c928daadfb2f > core/src/main/java/org/apache/accumulo/core/data/ComparableBytes.java ce61844375617c36ede5e0e411b439205329b023 > core/src/main/java/org/apache/accumulo/core/data/Condition.java fc8f2bf0394ae658bacfa94804b4b81f19a0c0c2 > core/src/main/java/org/apache/accumulo/core/data/ConstraintViolationSummary.java 2929bc610df9352374621d29caea7732bd50847d > core/src/main/java/org/apache/accumulo/core/data/Key.java de9e22d9ebff5442633dc578613fd7d033afc9c5 > core/src/main/java/org/apache/accumulo/core/data/KeyValue.java cc48322f9785ad508cc7925279f5e559629f3721 > core/src/main/java/org/apache/accumulo/core/data/Mutation.java 5e281f2bfef7e164c9f0e0c9a7132e94012d7186 > core/src/main/java/org/apache/accumulo/core/data/PartialKey.java 2049881fe26b5f237db39a412a88e2e2017f5cc9 > core/src/main/java/org/apache/accumulo/core/data/Range.java 70857348cefd0941e5ffcf6e5d569f09870101e4 > core/src/main/java/org/apache/accumulo/core/data/Value.java 7d3cf8f20aa12205d25027d6ad06e4a3f6062b2e > core/src/main/java/org/apache/accumulo/core/util/UnsynchronizedBuffer.java b640581bebbb87f1ac9a36f71275fa5dc8ef1223 > > Diff: https://reviews.apache.org/r/15857/diff/ > > > Testing > ------- > > Compiled successfully; mvn javadoc:javadoc ran with no errors from the javadoc in this patch. > > > Thanks, > > Bill Havanki > > --===============6927682760417581357==--