[GitHub] [lucene] ChrisHegarty commented on issue #12396: Make ForUtil Vectorized

2023-07-04 Thread via GitHub
ChrisHegarty commented on issue #12396: URL: https://github.com/apache/lucene/issues/12396#issuecomment-1620696514 I would like to pause a little, double check where we're going, and reset if needed. It's become clear to me that we're not quite aligned. The main issue I see is with th

[GitHub] [lucene] ChrisHegarty commented on issue #12396: Make ForUtil Vectorized

2023-07-03 Thread via GitHub
ChrisHegarty commented on issue #12396: URL: https://github.com/apache/lucene/issues/12396#issuecomment-1617801212 ++ Let's use the format that vectorizes the best. And yes, we will need a scalar implementation of that too, but to get started let's assume that the format changeable. -- T

[GitHub] [lucene] ChrisHegarty commented on issue #12396: Make ForUtil Vectorized

2023-07-01 Thread via GitHub
ChrisHegarty commented on issue #12396: URL: https://github.com/apache/lucene/issues/12396#issuecomment-1615898202 @tang-hi thanks for the update. I have some time later this weekend, I’ll see if I can get this started, and take a look at your code. -- This is an automated message from th

[GitHub] [lucene] ChrisHegarty commented on issue #12396: Make ForUtil Vectorized

2023-07-01 Thread via GitHub
ChrisHegarty commented on issue #12396: URL: https://github.com/apache/lucene/issues/12396#issuecomment-1615897183 Thanks @uschindler, no great urgency on this part. As you have said earlier, work on the actual vector implementation can proceed independently of the general infrastructure -

[GitHub] [lucene] ChrisHegarty commented on issue #12396: Make ForUtil Vectorized

2023-07-01 Thread via GitHub
ChrisHegarty commented on issue #12396: URL: https://github.com/apache/lucene/issues/12396#issuecomment-1615888789 @uschindler Ok, thanks. Sounds like you have this under control. And what you say makes sense. Did the small sketch of the ForUtil interface I provided above make sense?

[GitHub] [lucene] ChrisHegarty commented on issue #12396: Make ForUtil Vectorized

2023-07-01 Thread via GitHub
ChrisHegarty commented on issue #12396: URL: https://github.com/apache/lucene/issues/12396#issuecomment-1615848352 If I'm not mistaken, the specific interface required here to abstract out `ForUtil` would look something like this? ``` interface ForUtilXXX { /** Number of bytes req