Re: [elephant-devel] bug in map-inverted-index

2008-05-14 Thread Robert L. Read
On Tue, 2008-05-13 at 10:17 -0400, Ian Eslick wrote: > Robert or Alex, would you mind back patching on the main branch? I have now applied Richards patch to the main branch -- thanks! (I didn't run the tests, which is a bit naughty, but time is short.) > > We should deprecate the main branch an

Re: [elephant-devel] bug in map-inverted-index

2008-05-13 Thread Ian Eslick
Robert or Alex, would you mind back patching on the main branch? We should deprecate the main branch and move everyone to the new branch as soon as possible. Alex, any update on when we'll get the postmodern backend to work on the new branch with dup-btrees, etc? Thank you! Ian On May 13,

Re: [elephant-devel] bug in map-inverted-index

2008-05-13 Thread Ryszard Szopa
On Tue, May 13, 2008 at 2:05 AM, Ian Eslick <[EMAIL PROTECTED]> wrote: > The problem is on both branches then. I've fixed it on unstable. I don't > intend to back patch, although the patch is trivial if someone wants to make > it. I wanted to. The patch is attached. -- http://szopa.tasak.gda.p

Re: [elephant-devel] bug in map-inverted-index

2008-05-12 Thread Ian Eslick
The problem is on both branches then. I've fixed it on unstable. I don't intend to back patch, although the patch is trivial if someone wants to make it. Ian On May 12, 2008, at 7:55 PM, Ryszard Szopa wrote: Huh... This is one of those things that are kind of hard to admit publicly... Ap

Re: [elephant-devel] bug in map-inverted-index

2008-05-12 Thread Ryszard Szopa
Huh... This is one of those things that are kind of hard to admit publicly... Apparently, I tricked myself into thinking that I was using elephant-unstable, while in fact I was using Elephant stable (please don't ask how this was possible). So, the bug I've mentioned in this thread and for which I'

Re: [elephant-devel] bug in map-inverted-index

2008-05-12 Thread Ryszard Szopa
On Mon, May 12, 2008 at 8:34 PM, Ian Eslick <[EMAIL PROTECTED]> wrote: > map-index was missing a check for a valid key after the first value query. > I've patched, verified and pushed a fix along with the test case to > elephant-unstable. > > Thank you for finding this! My pleasure. Thank you fo

Re: [elephant-devel] bug in map-inverted-index

2008-05-12 Thread Ian Eslick
map-index was missing a check for a valid key after the first value query. I've patched, verified and pushed a fix along with the test case to elephant-unstable. Thank you for finding this! Ian On May 12, 2008, at 2:24 PM, Ian Eslick wrote: That test was great. I reproduced this on my Ma

Re: [elephant-devel] bug in map-inverted-index

2008-05-12 Thread Ian Eslick
That test was great. I reproduced this on my Mac under Allegro with BDB 4.6. Will let you know what I find... I also added the test to testindexing.lisp. On May 12, 2008, at 2:08 PM, Ryszard Szopa wrote: Hey, map-inverted-index shows some strange behavior. When you pass it `start' and `

Re: [elephant-devel] bug in map-inverted-index

2008-05-12 Thread Ian Eslick
I thought I had fixed all of these - I'll look into it... Ian On May 12, 2008, at 2:08 PM, Ryszard Szopa wrote: Hey, map-inverted-index shows some strange behavior. When you pass it `start' and `end' arguments and there are no objects that satisfy the range, it maps over the first object gre