Re: GSoC 2017 weekly progress reports (week 2)

From: Shubham Barai <shubhambaraiss(at)gmail(dot)com>
To: amborodin(at)acm(dot)org
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Kevin Grittner <kgrittn(at)gmail(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, David Steele <david(at)pgmasters(dot)net>
Subject: Re: GSoC 2017 weekly progress reports (week 2)
Date: 2017-06-14 15:50:12
Message-ID: CALxAEPtLbHbrH3rVncmXzquaGmi9966BY+6G16mHfKYzYQrgkg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

I have made some changes in tests and pushed them to my branch.

Thanks for helping me out with testing.

Now, current head produces false positives but, with my patch, it doesn't.

Here is the link for updated tests:
https://github.com/shubhambaraiss/postgres/commit/2c02685a50a2b30654beb5c52542a57a46219c39

Regards,
Shubham

<https://mailtrack.io/> Sent with Mailtrack
<https://mailtrack.io/install?source=signature&lang=en&referral=shubhambaraiss(at)gmail(dot)com&idSignature=22>

On 13 June 2017 at 23:32, Andrew Borodin <borodin(at)octonica(dot)com> wrote:

> 2017-06-13 18:00 GMT+05:00 Shubham Barai <shubhambaraiss(at)gmail(dot)com>:
> >
> > Project: Explicitly support predicate locks in index AMs besides b-tree
> >
> Hi, Shubham
> Good job!
>
> So, in current HEAD test predicate_gist_2.spec generate false
> positives, but with your patch, it does not?
> I'd suggest keeping spec tests with your code in the same branch, it's
> easier. Also it worth to clean up specs style and add some words to
> documentation.
>
> Kevin, all, how do you think, is it necessary to expand these tests
> not only on Index Only Scan, but also on Bitmap Index Scan? And may be
> KNN version of scan too?
> I couldn't find such tests for B-tree, do we have them?
>
> Best regards, Andrey Borodin, Octonica.
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2017-06-14 15:57:59 Re: WIP: Data at rest encryption
Previous Message Robert Haas 2017-06-14 15:49:24 Re: outfuncs.c utility statement support