On 07/15 12:56, Jeremy Evans wrote: > On 07/15 04:45, Eric Wong wrote: > > Jeremy Evans <[email protected]> wrote: > > > All of Sequel's postgres adapter tests still pass with this, so I merged > > > this into the master branch. I'll do some more testing of my apps, but > > > unless I run into problems I plan to release this as sequel_pg 1.7.1 > > > early next week. > > > > Thanks for the update. Btw, did you get a chance to test with > > GC.stress? It's not 100% reliable (and it is slow), but > > probably could've caught problems like this one. > > I hadn't tested with GC.stress before. You weren't kidding about it being > slow. I'll let it run overnight with the previous code (without your > patch), to see if this is something it would have caught.
Running with GC.stress didn't catch the error for me. But I'm using a fairly old compiler (GCC 4.2.1, the OpenBSD default), so this may be something that only shows up on a newer compiler that does more optimizations. Thanks, Jeremy -- unsubscribe: [email protected] archive: https://bogomips.org/unicorn-public/
