Re: [PR] Minor change to IndexOrDocValuesQuery#toString [lucene]

2023-11-16 Thread via GitHub
shubhamvishu commented on PR #12791: URL: https://github.com/apache/lucene/pull/12791#issuecomment-1814515539 > I assume this is because this is a DoubleDocValuesField which encodes the double using NumericUtils.doubleToSortableLong @mikemccand Is it possible to fix `NumericUtils.doub

Re: [PR] Minor change to IndexOrDocValuesQuery#toString [lucene]

2023-11-15 Thread via GitHub
mikemccand commented on PR #12791: URL: https://github.com/apache/lucene/pull/12791#issuecomment-1812469386 Nightly sparse (NYC taxis) benchy was a bit unhappy with this change because it (weirdly) relies on `Query.toString` (I tried to fix the benchy [here](https://github.com/mikemccand/lu

Re: [PR] Minor change to IndexOrDocValuesQuery#toString [lucene]

2023-11-13 Thread via GitHub
mikemccand merged PR #12791: URL: https://github.com/apache/lucene/pull/12791 -- 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: issues-unsubscr...@lucene.

[PR] Minor change to IndexOrDocValuesQuery#toString [lucene]

2023-11-10 Thread via GitHub
shubhamvishu opened a new pull request, #12791: URL: https://github.com/apache/lucene/pull/12791 ### Description Adds doc value query to the `IndexOrDocValuesQuery#toString` -- This is an automated message from the Apache Git Service. To respond to the message, please log on