On Tue, Dec 6, 2016 at 9:19 AM, Andres Freund <and...@anarazel.de> wrote: > 0009 WIP: Add minimal keytest implementation. > > More or less experimental patch that tries to implement simple > expression of the OpExpr(ScalarVar, Const) into a single expression > evaluation step. The benefits probably aren't big enough iff we do end > up doing JITing of expressions.
Seems like we are try to achieve same thing with 'heap scan key push down patch'[1] as well. But I think with this patch you are covering OpExpr(ScalarVar, Const) for all the cases, wherein with [1] we are currently only doing it for seqscan and we are trying to make that generic for other node as well. So do you see any advantage of continuing [1] ? Is there something extra we can achieve with [1] what we can not get with this patch ? https://www.postgresql.org/message-id/CAFiTN-takT6Z4s3tGDwyC9bhYf%2B1gumpvW5bo_fpeNUy%2BrL-kg%40mail.gmail.com -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers