Return-Path: 
 <dev-return-17574-apmail-phoenix-dev-archive=phoenix.apache.org@phoenix.apache.org>
X-Original-To: apmail-phoenix-dev-archive@minotaur.apache.org
Delivered-To: apmail-phoenix-dev-archive@minotaur.apache.org
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by minotaur.apache.org (Postfix) with SMTP id 5A0CE172AB
	for <apmail-phoenix-dev-archive@minotaur.apache.org>;
 Mon,  1 Jun 2015 06:55:39 +0000 (UTC)
Received: (qmail 47951 invoked by uid 500); 1 Jun 2015 06:55:34 -0000
Delivered-To: apmail-phoenix-dev-archive@phoenix.apache.org
Received: (qmail 47896 invoked by uid 500); 1 Jun 2015 06:55:34 -0000
Mailing-List: contact dev-help@phoenix.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:dev-help@phoenix.apache.org>
List-Unsubscribe: <mailto:dev-unsubscribe@phoenix.apache.org>
List-Post: <mailto:dev@phoenix.apache.org>
List-Id: <dev.phoenix.apache.org>
Reply-To: dev@phoenix.apache.org
Delivered-To: mailing list dev@phoenix.apache.org
Received: (qmail 47883 invoked by uid 99); 1 Jun 2015 06:55:33 -0000
Received: from Unknown (HELO spamd1-us-west.apache.org) (209.188.14.142)
    by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 01 Jun 2015 06:55:33 +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 66EC1CA437
	for <dev@phoenix.apache.org>; Mon,  1 Jun 2015 06:55:33 +0000 (UTC)
X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org
X-Spam-Flag: NO
X-Spam-Score: -0.121
X-Spam-Level: 
X-Spam-Status: No, score=-0.121 tagged_above=-999 required=6.31
	tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,
	RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001]
	autolearn=disabled
Authentication-Results: spamd1-us-west.apache.org (amavisd-new);
	dkim=pass (2048-bit key) header.d=gmail.com
Received: from mx1-us-east.apache.org ([10.40.0.8])
	by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new,
 port 10024)
	with ESMTP id 24RyEcFSg1mg for <dev@phoenix.apache.org>;
	Mon,  1 Jun 2015 06:55:32 +0000 (UTC)
Received: from mail-la0-f53.google.com (mail-la0-f53.google.com
 [209.85.215.53])
	by mx1-us-east.apache.org (ASF Mail Server at mx1-us-east.apache.org) with
 ESMTPS id 67D5E47BEB
	for <dev@phoenix.apache.org>; Mon,  1 Jun 2015 06:55:32 +0000 (UTC)
Received: by labko7 with SMTP id ko7so90471695lab.2
        for <dev@phoenix.apache.org>; Sun, 31 May 2015 23:54:46 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=20120113;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :content-type;
        bh=cUaELih/7P9Bujfwc26vH+vBN8CFo9neETomVv2wNGE=;
        b=iw0JY0AnG5VdI1aAGyHOLRy9KVT8IIp2l72fFdgR3eqtpwqr+PbfNvYHpsqUiYtQ+Q
         zSSA/kE/mRIJ40bN7d3z8hdMIi1MGGpST++1FMNBH4SKqipfiFJ9VMNgbD62FwrcpbQp
         KWe2Jf7ZkV77pccimGNJu2W7duasWPSVhcvmYRP+0TYsS5wiZ+tWmx7mVtkUuAXADlgn
         9vQ7nqhHAzoRui3BhwgH7zjd9Z0vj7LX3a/LKSq2rHKTBo3dipQ40vZuLxOrhEOStknc
         Y6EsP94uT0MmECrerl19aQQcFJv2bieLLdyAlk2wTKkMdjbPVYgEUtKEX8K5G6eXQk+V
         F8XQ==
MIME-Version: 1.0
X-Received: by 10.152.44.225 with SMTP id h1mr19472845lam.5.1433141686116;
 Sun, 31 May 2015 23:54:46 -0700 (PDT)
Received: by 10.25.206.71 with HTTP; Sun, 31 May 2015 23:54:46 -0700 (PDT)
In-Reply-To: 
 <CAH7WEUDErFHe-fo_YPxMWXVwamEtuQN1ehT_7VXiir6VZrhtfg@mail.gmail.com>
References: 
 <CAH7WEUDErFHe-fo_YPxMWXVwamEtuQN1ehT_7VXiir6VZrhtfg@mail.gmail.com>
Date: Mon, 1 Jun 2015 08:54:46 +0200
Message-ID: 
 <CAA5C_pvoUzDh4ewZK47skf8qnYsYNBRXBWNzmVP5N1TNobASpA@mail.gmail.com>
Subject: Re: Infinite recursive loop in next(List<Cell> results) method of
 BaseRegionScanner
From: Gabriel Reid <gabriel.reid@gmail.com>
To: "dev@phoenix.apache.org" <dev@phoenix.apache.org>
Content-Type: text/plain; charset=UTF-8

Good catch Yuhao! I've created PHOENIX-2022 to track this.

- Gabriel

On Sat, May 30, 2015 at 10:20 AM, Yuhao Bi <byh0831@gmail.com> wrote:
> Hi there,
>
> I'm a little confused about the implement of the next(List<Cell> results)
> method in BaseRegionScanner.
> It seems to be a infinite recursive loop there.
>
>     @Override
>     public boolean next(List<Cell> results) throws IOException {
>         return next(results);
>     }
>
> I noticed that, each place of using "return new BaseRegionScanner()..." has
> overrided this method.
>
> So, is it necessary to slightly modify the implement of this method?
>
>
> Thanks,
> Yuhao Bi