You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hbase.apache.org by Ted Yu <yu...@gmail.com> on 2010/08/01 00:13:11 UTC

minor bug in ServerManager.ServerMonitor.chore()

Hi,
I see a minor issue in ServerManager.ServerMonitor.chore() where first would
always be true.
The following line should be moved outside if block:
            first = false;

Cheers

Re: minor bug in ServerManager.ServerMonitor.chore()

Posted by Ryan Rawson <ry...@gmail.com>.

If its a bonafide bug please file a JIRA outlining the reproduction steps,
the expected outcome and impact.

On Jul 31, 2010 3:13 PM, "Ted Yu" <yu...@gmail.com> wrote:
> Hi,
> I see a minor issue in ServerManager.ServerMonitor.chore() where first
would
> always be true.
> The following line should be moved outside if block:
> first = false;
>
> Cheers

RE: minor bug in ServerManager.ServerMonitor.chore()

Posted by Jonathan Gray <jg...@facebook.com>.

Can you file a jira and put a patch up?

> -----Original Message-----
> From: Ted Yu [mailto:yuzhihong@gmail.com]
> Sent: Saturday, July 31, 2010 3:13 PM
> To: dev@hbase.apache.org
> Subject: minor bug in ServerManager.ServerMonitor.chore()
> 
> Hi,
> I see a minor issue in ServerManager.ServerMonitor.chore() where first
> would
> always be true.
> The following line should be moved outside if block:
>             first = false;
> 
> Cheers