Return-Path: X-Original-To: apmail-hbase-issues-archive@www.apache.org Delivered-To: apmail-hbase-issues-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 4089F7AC2 for ; Sat, 13 Aug 2011 13:59:54 +0000 (UTC) Received: (qmail 92332 invoked by uid 500); 13 Aug 2011 13:59:54 -0000 Delivered-To: apmail-hbase-issues-archive@hbase.apache.org Received: (qmail 92313 invoked by uid 500); 13 Aug 2011 13:59:53 -0000 Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list issues@hbase.apache.org Received: (qmail 92304 invoked by uid 99); 13 Aug 2011 13:59:53 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 13 Aug 2011 13:59:53 +0000 X-ASF-Spam-Status: No, hits=-2000.8 required=5.0 tests=ALL_TRUSTED,RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 13 Aug 2011 13:59:50 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id A7938BBA25 for ; Sat, 13 Aug 2011 13:59:28 +0000 (UTC) Date: Sat, 13 Aug 2011 13:59:28 +0000 (UTC) From: "jiraposter@reviews.apache.org (JIRA)" To: issues@hbase.apache.org Message-ID: <154994800.35956.1313243968683.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <1766608612.27046.1313045727156.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (HBASE-4192) Optimize HLog for Throughput Using Delayed RPCs 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-4192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13084618#comment-13084618 ] jiraposter@reviews.apache.org commented on HBASE-4192: ------------------------------------------------------ ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1463/#review1440 ----------------------------------------------------------- src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java Some kind of counter for this case would be useful, I think. - Ted On 2011-08-12 23:44:23, Vlad Dogaru wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/1463/ bq. ----------------------------------------------------------- bq. bq. (Updated 2011-08-12 23:44:23) bq. bq. bq. Review request for hbase. bq. bq. bq. Summary bq. ------- bq. bq. Changes: bq. bq. 1. Add hbase.region.wal.batchentries configuration parameter. If this is bq. enabled, batch entries to the HLog in a queue. bq. 2. Use delayed RPCs for sync requests when aggresive batching is enabled. bq. This frees up RPC handler threads for the duration of the sync. bq. 3. Pass the RPC server instance all the way to down to HLog. This is needed bq. to find out the current remote call, mark it as delayed, and finally complete bq. it when the sync is done. bq. 4. Use the region read-write consistency control to avoid exposing to bq. RegionScanners edits which have not yet been synced. bq. 5. Change a few tests which directly create HRegions or HLogs. The bq. rpcServers passed in are null, HLog falls back to classic RPCs when it has no bq. knowledge of the RPC server. bq. 6. Add TestBatchEntriesLogRolling, which is identical to TestLogRolling, bq. except that it uses aggressive batching. I'm not sure how to add tests that bq. verify the same functionality but don't duplicate code, suggestion are bq. welcome. bq. bq. The new parameter is disabled by default. bq. bq. bq. This addresses bug HBASE-4192. bq. https://issues.apache.org/jira/browse/HBASE-4192 bq. bq. bq. Diffs bq. ----- bq. bq. src/main/java/org/apache/hadoop/hbase/ipc/HBaseServer.java 7117bce bq. src/main/java/org/apache/hadoop/hbase/master/HMaster.java a00b93d bq. src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 83ff7b2 bq. src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 7a917da bq. src/main/java/org/apache/hadoop/hbase/regionserver/ReadWriteConsistencyControl.java 8ec53d3 bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java 887f736 bq. src/main/resources/hbase-default.xml 66548ca bq. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestWALActionsListener.java dc43eb2 bq. src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java 381ac90 bq. bq. Diff: https://reviews.apache.org/r/1463/diff bq. bq. bq. Testing bq. ------- bq. bq. All unit tests run with aggressive batching turned on and off. bq. bq. bq. Thanks, bq. bq. Vlad bq. bq. > Optimize HLog for Throughput Using Delayed RPCs > ----------------------------------------------- > > Key: HBASE-4192 > URL: https://issues.apache.org/jira/browse/HBASE-4192 > Project: HBase > Issue Type: New Feature > Components: wal > Affects Versions: 0.92.0 > Reporter: Vlad Dogaru > Priority: Minor > > Introduce a new HLog configuration parameter (batchEntries) for more aggressive batching of appends. If this is enabled, HLog appends are not written to the HLog writer immediately, but batched and written either periodically or when a sync is requested. Because sync times become larger, they use delayed RPCs to free up RPC handler threads. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira