Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
Received-SPF: pass (nike.apache.org: domain of jdcryans@gmail.com designates
 74.125.82.41 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:sender:in-reply-to:references:date
         :x-google-sender-auth:message-id:subject:from:to:content-type
         :content-transfer-encoding;
        b=sg4H7vmTi7SLhjT8k3r8ACMG2jY+YLbN1Aic0SxBq+24Db+kgUpY16LNT1UXsDm87E
         uAeJ11ZNElqcds8/SCncoQtRFnV926xWJC+qEVPCGdF9cmJ6v6TKEA1U5voQdzmxkD5e
         JHBZdodn7NW40oN07J4SHRs5gjYXC/0nwDm8c=
MIME-Version: 1.0
Sender: jdcryans@gmail.com
In-Reply-To: <AANLkTimUBxiftO1DPt5oRyKeKy+p+AiLyrOjhTk6FVvG@mail.gmail.com>
References: <20100825185756.27362.57859@ip-10-202-7-187.ec2.internal>
	<AANLkTimUBxiftO1DPt5oRyKeKy+p+AiLyrOjhTk6FVvG@mail.gmail.com>
Date: Wed, 25 Aug 2010 13:19:33 -0700
Message-ID: <AANLkTik9eszKbvmJ5bR0pzuBQL-1F8KL-WPfn4S-Zwei@mail.gmail.com>
Subject: Re: Review Request: RegionServer should refuse to be assigned a
 region that use LZO when LZO isn't available
From: Jean-Daniel Cryans <jdcryans@apache.org>
To: dev@hbase.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Sure, but currently there's no infrastructure for that. It could be
part of the master rewrite, but it'd be low priority as we are still
in the process of having a functional master. Also we need that kind
of reporting outside of the web UIs as some production environment may
not have those.

J-D

On Wed, Aug 25, 2010 at 1:11 PM, Ted Yu <yuzhihong@gmail.com> wrote:
> Can we display region server(s) which has this issue in a different color=
 in
> Region Servers table on master.jsp ?
> Or better, can we add a column showing brief error message in Region Serv=
ers
> table ?
>
> On Wed, Aug 25, 2010 at 11:57 AM, Jean-Daniel Cryans <jdcryans@apache.org=
>wrote:
>
>>
>> -----------------------------------------------------------
>> This is an automatically generated e-mail. To reply, visit:
>> http://review.cloudera.org/r/719/
>> -----------------------------------------------------------
>>
>> Review request for hbase.
>>
>>
>> Summary
>> -------
>>
>> Trying to make some progress on this issue, here's an untested patch tha=
t
>> adds testing of codecs when the region server starts, then uses that to
>> check the schema of regions it tries to open. That's basically what this
>> jira's scope is about.
>>
>> Some things I don't like the way I did it:
>> =A0- For all the users that don't do LZO, they will have a WARN every ti=
me a
>> RS starts (in its log)
>> =A0- Does not cover cases where the files are LZOed, but the schema is N=
ONE
>> or GZ
>>
>> Finally, there's no way to let user know about the errors unless he chec=
ks
>> the logs, but a least we won't throw ugly exceptions.
>>
>>
>> This addresses bug HBASE-2514.
>> =A0 =A0http://issues.apache.org/jira/browse/HBASE-2514
>>
>>
>> Diffs
>> -----
>>
>> =A0/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServ=
er.java
>> 989204
>> =A0/trunk/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.jav=
a
>> 989204
>>
>> Diff: http://review.cloudera.org/r/719/diff
>>
>>
>> Testing
>> -------
>>
>>
>> Thanks,
>>
>> Jean-Daniel
>>
>>
>