On Thu, Sep 24, 2015 at 10:22 PM, Tom Lane <t...@sss.pgh.pa.us> wrote:
> Jeevan Chalke <jeevan.cha...@enterprisedb.com> writes: > > On Wed, Sep 23, 2015 at 10:15 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: > >> After a bit more thinking and experimentation, I propose the attached > >> patch. > > > I had a look over the patch and reviewed it. It is in excellent state to > > check-in. > > After further thought I decided that the base case for > Const/Param/non-foreign-Vars wasn't quite right either. If we don't like > the collation we should just set the state to UNSAFE not fail immediately, > because it might appear in a context where collation doesn't matter. > An example is "var IS NOT NULL". > Make sense. > > So I've committed the attached modification of that patch. > > Thanks > regards, tom lane > > -- Jeevan B Chalke Principal Software Engineer, Product Development EnterpriseDB Corporation The Enterprise PostgreSQL Company