> On 2 Sep 2022, at 21:22, Ranier Vilela <ranier...@gmail.com> wrote: > > Em sex., 2 de set. de 2022 às 16:15, Daniel Gustafsson <dan...@yesql.se > <mailto:dan...@yesql.se>> escreveu: > > On 2 Sep 2022, at 21:08, Ranier Vilela <ranier...@gmail.com > > <mailto:ranier...@gmail.com>> wrote: > > > At function circle_same the second isnan test is wrong. > > Yeah, that seems pretty wrong. Did you attempt to procure a test for when > this > yields the wrong result? > Hi Daniel, > Unfortunately not.
On HEAD, the below query yields what seems to be the wrong result for the "same as" operator: postgres=# select '<(0,0),NaN>'::circle ~= '<(0,0),1>'::circle; ?column? ---------- t (1 row) With the patch applied, it returns the expected: postgres=# select '<(0,0),NaN>'::circle ~= '<(0,0),1>'::circle; ?column? ---------- f (1 row) There seems to be surprisingly few tests around these geo operators? This was introduced in c4c340088 so any fix needs to be backpatched to v12. I will do some more testing and digging to verify and will take care of it. -- Daniel Gustafsson https://vmware.com/
