Return-Path: X-Original-To: apmail-hbase-dev-archive@www.apache.org Delivered-To: apmail-hbase-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 720FE10FE4 for ; Sat, 3 Aug 2013 18:13:59 +0000 (UTC) Received: (qmail 5758 invoked by uid 500); 3 Aug 2013 18:13:58 -0000 Delivered-To: apmail-hbase-dev-archive@hbase.apache.org Received: (qmail 5715 invoked by uid 500); 3 Aug 2013 18:13:58 -0000 Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hbase.apache.org Delivered-To: mailing list dev@hbase.apache.org Received: (qmail 5707 invoked by uid 99); 3 Aug 2013 18:13:56 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 03 Aug 2013 18:13:56 +0000 X-ASF-Spam-Status: No, hits=2.5 required=5.0 tests=FREEMAIL_REPLY,HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of ndimiduk@gmail.com designates 209.85.128.180 as permitted sender) Received: from [209.85.128.180] (HELO mail-ve0-f180.google.com) (209.85.128.180) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 03 Aug 2013 18:13:52 +0000 Received: by mail-ve0-f180.google.com with SMTP id pb11so1829352veb.25 for ; Sat, 03 Aug 2013 11:13:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type; bh=igsT1iKtZdFZnAxLeqATIQhOF1O/xMzXuyZ8jUXMb0s=; b=E45sd+7A5tO2DaLpBTg0VowguC44vW4gPNaOj3/h/Q7IxPjoHlWKveEjtxU5y4JSkn IuEnIo9TkzguKiHFG0Yl6Amy9KuCCJ7zIfA81Xh/HOvjajgoI+pZf+Ui8UfjNrA88mbW ExBjOXXrJG4Wk2iWdQ7z4AwNozHI+/VqxhS+NBhp7dj4G5sqD9JXwZ6azOUE1JcoQUl2 mM6L7ZvK8/zSqMSQo6gAmTWxJ7J7+HDpnYKFL9JjqZUFCwqXP9yEUDnnQWd2Hluj5pa+ UJIk2zm3xjax8b1wcdkk2a9W0i3YeQOL2q6w6GvBEdpNbYFTBUqzNvi4Xzwxp92RPCiD gjIQ== X-Received: by 10.220.44.195 with SMTP id b3mr3743738vcf.62.1375553612102; Sat, 03 Aug 2013 11:13:32 -0700 (PDT) MIME-Version: 1.0 Received: by 10.58.97.204 with HTTP; Sat, 3 Aug 2013 11:13:11 -0700 (PDT) In-Reply-To: References: From: Nick Dimiduk Date: Sat, 3 Aug 2013 11:13:11 -0700 Message-ID: Subject: Re: Data types stage 1 is ready for reivew To: dev@hbase.apache.org Content-Type: multipart/alternative; boundary=001a11c2c9ac6b440304e30f0877 X-Virus-Checked: Checked by ClamAV on apache.org --001a11c2c9ac6b440304e30f0877 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Sat, Aug 3, 2013 at 10:33 AM, Michael Segel w= rote: > How do you plan on enforcing data types within the engine? > Data types, at this point, are a client-side feature, not enforced by the engine in anyway. On Aug 1, 2013, at 10:57 PM, Matt Corgan wrote: > > > Looks great to me. Without the strict dependencies on hadoop or hbase > > it'll be easy to pull into its own standalone module or new project if > > there's demand. > > > > > > On Thu, Aug 1, 2013 at 6:30 PM, Nick Dimiduk wrote= : > > > >> Finally-for-real-this-time patches posted. I'll take your +1's any tim= e > now > >> ;) > >> > >> Thanks, > >> Nick > >> > >> On Wed, Jul 24, 2013 at 11:27 AM, Nick Dimiduk > wrote: > >> > >>> Hi everyone, > >>> > >>> As of yesterday, I've posted "final" patched on both HBASE-8201< > >> https://issues.apache.org/jira/browse/HBASE-8201>and > >>> HBASE-8693 . The > >> former > >>> specifies on-disk format and the latter is the user-facing API. If > you've > >>> already left me a review, thank you; please have another look at thes= e > >>> patches. If you have an opinion here and haven't voiced it, we're > >>> approaching the "forever hold your peace" part of the ceremony. > >>> > >>> Thanks, > >>> Nick > >>> > >>> > >>> On Wed, Jul 17, 2013 at 9:33 AM, Nick Dimiduk > >> wrote: > >>> > >>>> Thanks for having a look. If you don't mind terribly, I responded to > >> your > >>>> comments on JIRA [0]. > >>>> > >>>> Thanks, > >>>> Nick > >>>> > >>>> [0]: > https://issues.apache.org/jira/browse/HBASE-8693#comment-13711250 > >>>> > >>>> > >>>> On Wed, Jul 17, 2013 at 1:42 AM, Matteo Bertozzi < > >> theo.bertozzi@gmail.com > >>>>> wrote: > >>>> > >>>>> I was looking at the HBASE-8693 patch, and looks good to me for the > >>>>> primitive types. > >>>>> but I can't see how do you plan to evolve stuff like the struct. > >>>>> By "evolve" I mean add/remove fields, or just query it with a subse= t > of > >>>>> fields. > >>>>> the fields don't have an id, and on read you must specify all of th= em > >> in > >>>>> the same order as you've used for write. > >>>>> (but maybe is just an immutable/fixed list of fields, and I'm ok wi= th > >>>>> just > >>>>> adding that info to the comment on top of the class) > >>>>> > >>>>> > >>>>> Matteo > >>>>> > >>>>> > >>>>> > >>>>> On Wed, Jul 17, 2013 at 12:39 AM, Nick Dimiduk > >>>>> wrote: > >>>>> > >>>>>> New patch posted. What do you think about the new isSkippable() an= d > >> the > >>>>>> associated limitation in Struct? > >>>>>> > >>>>>> I also posted some "dogfeed" per Enis's suggestion. > >>>>>> > >>>>>> -n > >>>>>> > >>>>>> On Fri, Jul 12, 2013 at 1:38 PM, Stack wrote: > >>>>>> > >>>>>>> On Fri, Jul 12, 2013 at 1:10 PM, Enis S=C3=B6ztutar > >>>>> wrote: > >>>>>>> > >>>>>>>> Did some chatting with Nick today. > >>>>>>>> > >>>>>>>> I think it is really important to get this right, and for that w= e > >>>>> would > >>>>>>>> definitely need more eyes towards it. The current patch set is > >> in a > >>>>>> good > >>>>>>>> state to bolster the discussion. > >>>>>>>> > >>>>>>>> > >>>>>>> > >>>>>>> I'll do another pass (Kicking others to give it a looksee too). > >>>>>>> St.Ack > >>>>>>> > >>>>>> > >>>>> > >>>> > >>>> > >>> > >> > > --001a11c2c9ac6b440304e30f0877--