Return-Path: Delivered-To: apmail-incubator-cassandra-dev-archive@minotaur.apache.org Received: (qmail 99948 invoked from network); 4 Apr 2009 07:48:56 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 4 Apr 2009 07:48:56 -0000 Received: (qmail 30385 invoked by uid 500); 4 Apr 2009 07:48:56 -0000 Delivered-To: apmail-incubator-cassandra-dev-archive@incubator.apache.org Received: (qmail 30340 invoked by uid 500); 4 Apr 2009 07:48:56 -0000 Mailing-List: contact cassandra-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cassandra-dev@incubator.apache.org Delivered-To: mailing list cassandra-dev@incubator.apache.org Received: (qmail 30330 invoked by uid 99); 4 Apr 2009 07:48:56 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 04 Apr 2009 07:48:56 +0000 X-ASF-Spam-Status: No, hits=2.2 required=10.0 tests=HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of schumi.han@gmail.com designates 209.85.146.178 as permitted sender) Received: from [209.85.146.178] (HELO wa-out-1112.google.com) (209.85.146.178) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 04 Apr 2009 07:48:49 +0000 Received: by wa-out-1112.google.com with SMTP id n7so750806wag.12 for ; Sat, 04 Apr 2009 00:48:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:content-type; bh=TYG902F/KTGBBb9mSK5JnmI5wwDqtR9Fh1dqORFSsPo=; b=T9PyTA866hOi62kGP8AvGU2iwD7hbh7cb/hO1bc+AWkJi0qIXSwAWD8qooHArGq036 xZ8D3pgtAEJGqK+i01fqsHTj9Ohs5OXQUlc/ASgf5ZMoQVW/Hcuav7msn/V+MePuCnWR V738yz1OhzJOFeDw2tdQQUkp4RO4lOZyuqudI= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; b=JGEmwNBVp6uTNLKMX6t56QxKY01GDXwJByGqrbXtG9aMyTL0ecYAYwObWj4dggGVEu IJMt1eEVsNsWYTImF3WfqKkcbL92dKWhY3iaIRb4SXMAqVnMklP86Ocz9ykNG2sZhyef Q8TKwimLN9IZv330jfQ2idA2fShgdCQNrowxc= MIME-Version: 1.0 Received: by 10.115.22.1 with SMTP id z1mr1035233wai.202.1238831309411; Sat, 04 Apr 2009 00:48:29 -0700 (PDT) In-Reply-To: <23048D59-145B-4BB5-B58C-038C42F81474@Holsman.net> References: <23048D59-145B-4BB5-B58C-038C42F81474@Holsman.net> Date: Sat, 4 Apr 2009 15:48:29 +0800 Message-ID: <4e777ed10904040048w1330fb30p590c260f13ee65bb@mail.gmail.com> Subject: Re: Random Checkins From: Zhu Han To: cassandra-dev@incubator.apache.org Content-Type: multipart/alternative; boundary=0016e649c0a62255ce0466b5e44a X-Virus-Checked: Checked by ClamAV on apache.org --0016e649c0a62255ce0466b5e44a Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit +1 for this proposal. best regards, hanzhu On Fri, Apr 3, 2009 at 10:52 AM, Ian Holsman wrote: > > > ideally the test cases that your working with could be made public, but > until then is it possible for you to run a hudson like service inside of FB > and mail the test result output to the list so people know that tests are > failing. > > > On 03/04/2009, at 1:21 PM, Jonathan Ellis wrote: > > On Thu, Apr 2, 2009 at 6:37 PM, Avinash Lakshman >> wrote: >> >>> Another issue with ConcurrentHashMap is that the dude is a memory hog. We >>> got rid of it over a year ago because of the very same reason. >>> >> >> Re CHM: what do you suggest instead? I assume you are talking about >> for EfficientBidiMap. Do you want the Getter to return a copy of the >> ColumnFamily and stick with non-concurrent structures? Or use >> NonBlockingHashMap? >> >> Also for >>> everything that is done there is a reason. >>> >> >> I don't see any harm in, for instance, me making a commit, you saying >> "ConcurrentHashMap is a memory hog", and me saying, "okay, what do you >> suggest instead?" Nothing wrong with post-commit review in trunk. >> >> I think we are not asking for >>> much to apart from run it by someone before what are basically changes >>> that >>> suit your style. >>> >> >> We've tried waiting for you or Prashant to review things. That didn't >> work over on the code.google project and it hasn't worked here. My >> remove patches sat unreviewed for weeks, literally. More recently, a >> request for a list of known issues and my attempt to start a >> conversation about a roadmap have gone without comment so far. >> >> I'm not trying to dump on you; we know you're busy. But we have >> deadlines too and we can't hold every patch waiting days or weeks for >> a review. >> >> Tell you what. I will try to get changes reviewed by someone in the >> community before committing. We will see how that works. That should >> at least give you some confidence that it's not just me off being a >> cowboy. >> >> -Jonathan >> > > -- > Ian Holsman > Ian@Holsman.net > > > > --0016e649c0a62255ce0466b5e44a--