pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Alexey Chernyshov <a.chernyshov@postgrespro.ru>
To: pgsql-hackers@postgresql.org
Cc: Alexander Kuzmenkov <a.kuzmenkov@postgrespro.ru>
Subject: Re: index-only count(*) for indexes supporting bitmap scans
Date: Thu, 07 Sep 2017 13:18:37 +0000
Message-ID: <20170907131837.18925.19752.pgcf@coridan.postgresql.org> (raw)
In-Reply-To: <a6434d5c-ed8d-b09c-a7c3-b2d1677e35b3@postgrespro.ru>
References: <239a8955-c0fc-f506-026d-c837e86c827b@postgrespro.ru>
	<a6434d5c-ed8d-b09c-a7c3-b2d1677e35b3@postgrespro.ru>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>

The following review has been posted through the commitfest application:
make installcheck-world:  tested, passed
Implements feature:       tested, passed
Spec compliant:           tested, passed
Documentation:            tested, passed

One thing I have noticed is a trailing whitespace after "bogus one":

123 +            * If we don't have to fetch the tuple, just return a
124 +            * bogus one 
125 +            */

The new status of this patch is: Ready for Committer

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


view thread (15+ messages)  latest in thread

Message-ID: <20170907131837.18925.19752.pgcf@coridan.postgresql.org>
Permalink:  ../20170907131837.18925.19752.pgcf@coridan.postgresql.org/
Also on:    postgresql.org/message-id/20170907131837.18925.19752.pgcf@coridan.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: a.chernyshov@postgrespro.ru, a.kuzmenkov@postgrespro.ru
  Subject: Re: index-only count(*) for indexes supporting bitmap scans
  In-Reply-To: <20170907131837.18925.19752.pgcf@coridan.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox