Log for tique

• Revert "Temporarily ignore `Dismax::explanation()` test" 💬 by Caio 4 years ago

  This reverts commit 8e231cbf5b1cc537d2d18180c11b493a5563d700.
  

• Formalize that unsafe code is not allowed by Caio 4 years ago
• Move QueryParser docs into the module by Caio 4 years ago
• Drop cargo-readme template by Caio 4 years ago
• Upgrade nom dependency to 6.+ by Caio 5 years ago
• Temporarily ignore `Dismax::explanation()` test 💬 by Caio 5 years ago

  Since a DisMaxQuery is a combination of TermQuery items,
  trying to `explain()` it triggers a bug in tantivy that
  leads to a crash in debug builds.
  
  Upstream issue: tantivy-search/tantivy#915
  

• Ensure we don't seek back when reading postings 💬 by Caio 5 years ago

  This works around the debug-only crash when trying to seek
  to a doc_id that would have appeared before the current
  SegmentPostings cursor.
  
  Since DocSets now come already initialized, `.seek()`ing
  without checking if we're already at the desired position
  is likely a bug (hence the `debug_assert!` biting me, I
  suspect).
  
  On tantivy.git @ 730cceff one can see the subtle bug that
  the assertion can catch. Code that once looked like:
  
      let mut scorer = create_scorer();
      if scorer.seek(doc) != doc {
          ...
      }
  
  Should now look like:
  
      let mut scorer = create_scorer();
      if scorer.doc() > doc || scorer.seek(doc) != doc {
          ...
      }
  

• Use new DocSet/Scorer API 💬 by Caio 5 years ago

  Introduced on tantivy.git @ e25284ba
  
  This changeset is sufficient, however upstream's f71b04acb introduced
  a `debug_assert!(self.doc() <= target)` for `SegmentPostings::seek`
  that looks overzealous to me.
  
  In release mode all tests pass, but given that lot has changed since
  last I looked I'll be double checking the affected functionality
  prior to letting this go wild.
  

• Remove useless closure by Caio 5 years ago
• No need to consume the keyword acceptor by Caio 5 years ago
• Use a more concise iteration style by Caio 5 years ago
• Replace `map -> unwrap_or` with `map_or` by Caio 5 years ago
• Document known error conditions by Caio 5 years ago
• Replace `map -> flatten` with `and_then` by Caio 5 years ago
• Add missing docs to exported things 💬 by Caio 5 years ago

  And enable `missing_docs` and `missing_doc_code_examples`
  

• Do not execute assertion-less doc code examples by Caio 5 years ago
• Prepare for v0.4.0 release by Caio 5 years ago
• Add QueryParser docs by Caio 5 years ago
• Rename tique's `unstable` feature to `queryparser` 💬 by Caio 5 years ago

  ... And somehow `tique` can't be built because `cantine` wants a
  feature that stopped existing? That kinda diminishes the usefulness
  of workspaces.
  

• Move definitions around 💬 by Caio 5 years ago

  No changes
  

• Add support for parsing queries using DisMax by Caio 5 years ago
• Fix goofy pure negative query detection 💬 by Caio 5 years ago

  And the weird leftofver copy-pasta in the test got removed
  

• Add initial DisMaxQuery implementation 💬 by Caio 5 years ago

  I was (unintentionally?) made aware that tantivy doesn't have a dismax
  query when @jackdoe pointed me at his cool new project. So I wrote one.
  
  Since I'm hacking on a dumb query parser that allows multiple fields
  and boosts, this will come in handy very soon.
  
  Ref: https://github.com/jackdoe/octopus_query/
  

• Add support for changing field name 💬 by Caio 5 years ago

  And rename `queryparser::interpreter` mod to `parser`
  

• Decide the Occur at the raw parser level by Caio 5 years ago
• First working field-aware QueryParser by Caio 5 years ago
• Rename queryparser::parser to queryparser::raw by Caio 5 years ago
• Ensure it's hard to cause an Err() with this parser by Caio 5 years ago
• Add support for parsing +mandatory queries 💬 by Caio 5 years ago

  And rename `negated` to `prohibited`
  

• Add support for strict field names by Caio 5 years ago