Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 36F72200CFA for ; Tue, 5 Sep 2017 22:52:09 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 355031609ED; Tue, 5 Sep 2017 20:52:09 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 7AE601609EA for ; Tue, 5 Sep 2017 22:52:08 +0200 (CEST) Received: (qmail 312 invoked by uid 500); 5 Sep 2017 20:52:07 -0000 Mailing-List: contact hdfs-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list hdfs-issues@hadoop.apache.org Received: (qmail 301 invoked by uid 99); 5 Sep 2017 20:52:07 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Sep 2017 20:52:07 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id 9AF9AD4DD2 for ; Tue, 5 Sep 2017 20:52:06 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -99.502 X-Spam-Level: X-Spam-Status: No, score=-99.502 tagged_above=-999 required=6.31 tests=[KAM_NUMSUBJECT=0.5, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id Sbmu4GkuWXFr for ; Tue, 5 Sep 2017 20:52:02 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id 577385F569 for ; Tue, 5 Sep 2017 20:52:01 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id 988A8E00C9 for ; Tue, 5 Sep 2017 20:52:00 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id 53C172414B for ; Tue, 5 Sep 2017 20:52:00 +0000 (UTC) Date: Tue, 5 Sep 2017 20:52:00 +0000 (UTC) From: "Chen Liang (JIRA)" To: hdfs-issues@hadoop.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (HDFS-12000) Ozone: Container : Add key versioning support-1 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Tue, 05 Sep 2017 20:52:09 -0000 [ https://issues.apache.org/jira/browse/HDFS-12000?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chen Liang updated HDFS-12000: ------------------------------ Attachment: HDFS-12000-HDFS-7240.004.patch Thanks [~xyao] for the comments! Addressed all your comments in v004 patch except the following: bq. Line 281: Please file followup JIRAs for TODO bq. Line 81: NIT: getLatestVersionList -> getLatestLocations, can we modify this function to take version as parameter so that this can be reused later when versions other than latest are fully supported. Will definitely do. Since it will actually involve some more code of retrieving the version, new APIs of requesting/returning specific version etc, I was planning for having a separate JIRA about getting specific version of block. there will be a couple of more JIRAs for versioning to work fully, reading a specific version will be one of the next JIRAs for sure. bq. Since we only update the latestVersion at commit time, there could be multiple clients that allocate blocks before the first commit is done. Once the first commit is done, all the allocate will be invalid to commit due to the logic between line 109-115. This is exactly what I was trying to do in this part. When multiple clients try to write the data, only one will succeed, the others will get exception, and maybe retry later. This works just same purpose as a write lock, such that when multiple writes are racing, we will never end up having a version that is an uncertain mix of multiple different writes. In addition, since delete key work is in-progress, I will need to look closer to see if there is anything conflicting between delete key and this versioning code. Submitting v004 patch anyway for now mainly for early review purpose. > Ozone: Container : Add key versioning support-1 > ----------------------------------------------- > > Key: HDFS-12000 > URL: https://issues.apache.org/jira/browse/HDFS-12000 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: ozone > Affects Versions: HDFS-7240 > Reporter: Anu Engineer > Assignee: Chen Liang > Attachments: HDFS-12000-HDFS-7240.001.patch, HDFS-12000-HDFS-7240.002.patch, HDFS-12000-HDFS-7240.003.patch, HDFS-12000-HDFS-7240.004.patch, OzoneVersion.001.pdf > > > The rest interface of ozone supports versioning of keys. This support comes from the containers and how chunks are managed to support this feature. This JIRA tracks that feature. Will post a detailed design doc so that we can talk about this feature. -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscribe@hadoop.apache.org For additional commands, e-mail: hdfs-issues-help@hadoop.apache.org