Re: [PR] Move some DocValuesConsumer merge logic to helper methods. [lucene]

2025-04-15 Thread via GitHub
martijnvg merged PR #14459: URL: https://github.com/apache/lucene/pull/14459 -- 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.a

Re: [PR] Move some DocValuesConsumer merge logic to helper methods. [lucene]

2025-04-15 Thread via GitHub
martijnvg commented on PR #14459: URL: https://github.com/apache/lucene/pull/14459#issuecomment-2807092322 Thanks for the reviews @dnhatn and @stefanvodita. -- 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

Re: [PR] Move some DocValuesConsumer merge logic to helper methods. [lucene]

2025-04-15 Thread via GitHub
stefanvodita commented on PR #14459: URL: https://github.com/apache/lucene/pull/14459#issuecomment-2804349293 Thanks for the explanation! The new revision looks good as well. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Move some DocValuesConsumer merge logic to helper methods. [lucene]

2025-04-15 Thread via GitHub
martijnvg commented on PR #14459: URL: https://github.com/apache/lucene/pull/14459#issuecomment-2804123675 Thanks for taking a look @stefanvodita! I've added the CHANGES entry. > Curious if you can share more about what you're trying to achieve! We have [a specialized doc values