On 6 September 2017 at 10:27, Tom Lane wrote:
> Simon Riggs writes:
>> Other than that, I'm good to commit.
>> Any last minute objections?
>
> The docs and comments could stand a bit closer copy-editing by a
> native English speaker. Other than that, it seemed sane in a
> quick read-through.
Wi
Simon Riggs writes:
> Other than that, I'm good to commit.
> Any last minute objections?
The docs and comments could stand a bit closer copy-editing by a
native English speaker. Other than that, it seemed sane in a
quick read-through.
regards, tom lane
--
Sent via pgs
On 4 September 2017 at 10:30, Adrien Nayrat wrote:
> On 09/04/2017 06:16 PM, Alexander Korotkov wrote:
>> Looks good for me. I've integrated those changes in the patch.
>> New revision is attached.
>
> Thanks, I changed status to "ready for commiter".
This looks useful and good to me.
I've chan
On 09/04/2017 06:16 PM, Alexander Korotkov wrote:
> Looks good for me. I've integrated those changes in the patch.
> New revision is attached.
Thanks, I changed status to "ready for commiter".
--
Adrien NAYRAT
http://dalibo.com - http://dalibo.org
signature.asc
Description: OpenPGP digital
Hi, Adrien!
On Mon, Sep 4, 2017 at 3:57 PM, Adrien Nayrat
wrote:
> On 08/30/2017 02:26 PM, Alexander Korotkov wrote:
> I reviewed this patch. It works as expected but I have few remarks :
>
Thank you for reviewing my patch!.
> * There is a warning during compilation :
>
> gram.y: In functio
On 08/30/2017 02:26 PM, Alexander Korotkov wrote:
> Patch rebased to current master is attached.
Hello,
I reviewed this patch. It works as expected but I have few remarks :
* There is a warning during compilation :
gram.y: In function ‘base_yyparse’:
gram.y:2090:6: warning: ISO C90 forbids mi
On Thu, Jun 1, 2017 at 4:10 PM, Alexander Korotkov <
a.korot...@postgrespro.ru> wrote:
> On Wed, May 31, 2017 at 7:18 PM, Alexander Korotkov <
> a.korot...@postgrespro.ru> wrote:
>
>> On Wed, May 31, 2017 at 6:53 PM, Tom Lane wrote:
>>
>>> Alexander Korotkov writes:
>>> > I've discovered that Po
On Sun, Jun 4, 2017 at 10:11 AM, Tom Lane wrote:
> Amit Kapila writes:
>> In order to avoid losing track of this patch, I think it is better to
>> add it in open items list for 10.
>
> This is an entirely new feature, not a bug fix, and thus certainly not an
> open item for v10. Please stick it
Amit Kapila writes:
> In order to avoid losing track of this patch, I think it is better to
> add it in open items list for 10.
This is an entirely new feature, not a bug fix, and thus certainly not an
open item for v10. Please stick it in the next commitfest, instead.
r
On Thu, Jun 1, 2017 at 6:40 PM, Alexander Korotkov
wrote:
> On Wed, May 31, 2017 at 7:18 PM, Alexander Korotkov
> wrote:
>>
>>>
>>> Don't like either of those particularly, but what about just targeting
>>> a column by column number, independently of whether it's an expression?
>>>
>>> AL
On Wed, May 31, 2017 at 7:18 PM, Alexander Korotkov <
a.korot...@postgrespro.ru> wrote:
> On Wed, May 31, 2017 at 6:53 PM, Tom Lane wrote:
>
>> Alexander Korotkov writes:
>> > I've discovered that PostgreSQL is able to run following kind of
>> queries in
>> > order to change statistic-gathering
On Wed, May 31, 2017 at 6:53 PM, Tom Lane wrote:
> Alexander Korotkov writes:
> > I've discovered that PostgreSQL is able to run following kind of queries
> in
> > order to change statistic-gathering target for an indexed expression.
>
> > ALTER INDEX index_name ALTER COLUMN expr SET STATISTICS
Alexander Korotkov writes:
> I've discovered that PostgreSQL is able to run following kind of queries in
> order to change statistic-gathering target for an indexed expression.
> ALTER INDEX index_name ALTER COLUMN expr SET STATISTICS stat_target;
> It's been previously discussed in [1].
> I th
Hackers,
I've discovered that PostgreSQL is able to run following kind of queries in
order to change statistic-gathering target for an indexed expression.
ALTER INDEX index_name ALTER COLUMN expr SET STATISTICS stat_target;
It's been previously discussed in [1].
I think this should be fixed not
14 matches
Mail list logo
