Results 130 comments of Daniel Mewes

It actually turns out that this "eviction" proposal as I described it is pretty meaningless (noticed by @VeXocide). The imagined future variant that works for table-size based eviction doesn't play...

Some other random comments: @mlucy > (Also, what happens if the index is deleted or renamed or something? Should the expiration function silently disappear, or should we open an issue...

Thanks for reporting this @sagivf . It doesn't look like the Link header is valid in that response. According to https://tools.ietf.org/html/rfc5988 page 6, the `link-value` must consist of an URI-Reference...

Besides the escaped `>`, it's also missing a semicolon between `` part and the `rel="latest-version"` part I think.

Thank you for the suggestion @marshall007 . This looks useful. I don't think we'll get to it before our 2.0 release, but we'll keep it on our list.

@marshall007 That's a very neat work-around!

Thanks for the PR @danneu ! @segphault or @deontologician is one of you available for reviewing this change?

Our current focus is on developing some other features in the core server, but we'll look into this eventually. Thanks for bringing it up @babakness .

Yeah I think just taking parts of it wouldn't quite work for the reasons @chrisvariety mentions. (side note: @neumino is now at Google, but he's still maintaining and extending rethinkdbdash)

@marshall007 as far as I understand implementing the `Stream` interface on the cursor directly would be risky because of the special meaning of `null` in Streams (as discussed in #3600)....