Return-Path: Delivered-To: apmail-hadoop-hbase-dev-archive@minotaur.apache.org Received: (qmail 89336 invoked from network); 29 Apr 2009 19:27:54 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 29 Apr 2009 19:27:54 -0000 Received: (qmail 97541 invoked by uid 500); 29 Apr 2009 19:27:53 -0000 Delivered-To: apmail-hadoop-hbase-dev-archive@hadoop.apache.org Received: (qmail 97501 invoked by uid 500); 29 Apr 2009 19:27:53 -0000 Mailing-List: contact hbase-dev-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: hbase-dev@hadoop.apache.org Delivered-To: mailing list hbase-dev@hadoop.apache.org Received: (qmail 97491 invoked by uid 99); 29 Apr 2009 19:27:53 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 29 Apr 2009 19:27:53 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 29 Apr 2009 19:27:51 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 66946234C04C for ; Wed, 29 Apr 2009 12:27:30 -0700 (PDT) Message-ID: <71165324.1241033250419.JavaMail.jira@brutus> Date: Wed, 29 Apr 2009 12:27:30 -0700 (PDT) From: "Erik Holstad (JIRA)" To: hbase-dev@hadoop.apache.org Subject: [jira] Commented: (HBASE-1249) Rearchitecting of server, client, API, key format, etc for 0.20 In-Reply-To: <728871099.1236620030451.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/HBASE-1249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12704272#action_12704272 ] Erik Holstad commented on HBASE-1249: ------------------------------------- @Stack {quote}1. My sense is that the delete check is not costly, not in the scheme of things.{quote} I would say that it is costly in cases where you have many deletes. I also see no downside or big areas where the new design would perform any worse. We also need to think about memory allocation. {quote}2. There is caching of lengths being done in many of those cases, right?{quote} I'm not sure exactly what you mean, but yes there may be. But without it being clear, it seems we would want to ensure in this very important (for performance) code path is optimally reusing them. One of the problems with java is that you can't return multiple things at the same time in an easy way. For me trying to make the server implementation faster I'd rather have duplication of code than make it slower. I totally agree that we should try to keep duplication to a minimum, but I rather focus on speed first and then work in minimizing the code base, might be the wrong approach. When it comes to the part of duplicating complex code I totally agree, this makes it more prone to errors and should definitely be avoided when possible. The code that I duplicate is mostly the compare code for 2 KeyValues, so it is pretty straightforward, not like the more complex merge codes which > Rearchitecting of server, client, API, key format, etc for 0.20 > --------------------------------------------------------------- > > Key: HBASE-1249 > URL: https://issues.apache.org/jira/browse/HBASE-1249 > Project: Hadoop HBase > Issue Type: Improvement > Reporter: Jonathan Gray > Priority: Blocker > Fix For: 0.20.0 > > Attachments: HBASE-1249-Example-v1.pdf, HBASE-1249-Example-v2.pdf, HBASE-1249-GetQuery-v1.pdf, HBASE-1249-GetQuery-v2.pdf, HBASE-1249-GetQuery-v3.pdf, HBASE-1249-GetQuery-v4.pdf, HBASE-1249-StoreFile-v1.pdf, HBASE-1249-StoreFile-v4.pdf > > > To discuss all the new and potential issues coming out of the change in key format (HBASE-1234): zero-copy reads, client binary protocol, update of API (HBASE-880), server optimizations, etc... -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.