snoopdave commented on PR #225:
URL: https://github.com/apache/roller/pull/225#issuecomment-6453919560

   **🤖 GLM: PR-Review — General Issues**
   
   The following issues were found but cannot be attached to a specific line in 
the diff:
   
   - **Important:** CHANGES.md has no entry for this change. Other PRs in the 
6.1.7 batch each carry one. The user-facing parts belong in the 6.1.7 notes: 
queries that name an index field outside the search fields are now a parse 
error (so e.g. `name:`-prefixed or `email:`-prefixed terms no longer work in 
the public search box), and a search index rebuilt after upgrading drops the 
email field entirely — existing indexes keep the field but the query check 
covers them.
   
   Strengths (not action items): overriding the full set of parser hooks 
(field/fuzzy/prefix/wildcard/regexp/range, both getFieldQuery signatures) 
closes the bypass shapes, not just the plain prefix form, and the tests 
enumerate exactly those shapes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to