Re: [PR] Fix AbstractRangeQueryNode#toQueryString [lucene]

2025-05-24 Thread via GitHub
stefanvodita commented on code in PR #14697: URL: https://github.com/apache/lucene/pull/14697#discussion_r2105767826 ## lucene/CHANGES.txt: ## @@ -123,6 +127,10 @@ Bug Fixes * GITHUB#14161: PointInSetQuery's constructor now throws IllegalArgumentException instead of Unsuppo

Re: [PR] Fix AbstractRangeQueryNode#toQueryString [lucene]

2025-05-22 Thread via GitHub
phb-ig commented on code in PR #14697: URL: https://github.com/apache/lucene/pull/14697#discussion_r2102606331 ## lucene/queryparser/src/test/org/apache/lucene/queryparser/flexible/standard/nodes/TestAbstractRangeQueryNode.java: ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache

Re: [PR] Fix AbstractRangeQueryNode#toQueryString [lucene]

2025-05-22 Thread via GitHub
phb-ig commented on code in PR #14697: URL: https://github.com/apache/lucene/pull/14697#discussion_r2102594352 ## lucene/CHANGES.txt: ## @@ -854,7 +862,7 @@ Improvements * GITHUB#13285: Early terminate graph searches of AbstractVectorSimilarityQuery to follow timeout set fro

Re: [PR] Fix AbstractRangeQueryNode#toQueryString [lucene]

2025-05-22 Thread via GitHub
phb-ig commented on PR #14697: URL: https://github.com/apache/lucene/pull/14697#issuecomment-2901267587 @stefanvodita I am the one who originally submitted the patches all that time ago because the work I was doing at the time required inspecting the query string and altering, then re-parsi

Re: [PR] Fix AbstractRangeQueryNode#toQueryString [lucene]

2025-05-21 Thread via GitHub
stefanvodita commented on code in PR #14697: URL: https://github.com/apache/lucene/pull/14697#discussion_r2101276705 ## lucene/queryparser/src/test/org/apache/lucene/queryparser/flexible/standard/nodes/TestAbstractRangeQueryNode.java: ## @@ -0,0 +1,69 @@ +/* + * Licensed to the