Paul Masurel comments

Results 327 comments of


                                            Paul Masurel

change debug_assert to assert in term length

I agree with the change. Can you check that it does not provoke a regression however?

Store DateTime as nanos in docstore

We need to be careful with backward compatiblity with that one. Is it motivated by a problem observed on quickwit side? Is there a ticket open somewhere?

Be able to use range query with values containing space

The query parser expects you date in the format rfc3339. In this format, the space is replaced by a T `timestamp:[2024-03-04T16:45:00 TO 2024-03-04T16:46:03]`

Panic occured while committing writer with sort by text field.

I suggest we just remove the ability to sort the index by something. It has brought more bugs, confusion than any other feature.

Panic occured while committing writer with sort by text field.

I don't understand your sentence.

to what extent could a Rust DB working group benefit our shared goals?

I'm mostly interested to hear about testing tips. `loom` and `fail-rs` is something that is in my radar for instance. I would also love to see a convergence on best...

Added is_closed method

Is there a release planned @faern ?

feat(query): Make `BooleanQuery` supports `minimum_number_should_match`

@LebranceBW don't forget to reapply for review once you have added the requested changes

feat(query): Make `BooleanQuery` supports `minimum_number_should_match`

@PSeitz can you help @LebranceBW and take over the code review?

keys should be increasing panic

The two document indeed contains the following: ``` {"o_params": {"\u0000\">alert(309)": "1"}} {"o_params": {" ADw-script AD4-alert(312) ADw-/script AD4-": "1"}} ```