shubhamvishu opened a new issue, #15840:
URL: https://github.com/apache/lucene/issues/15840

   ### Description
   
   Currently, NativeVectorUtilSupport is only exercised in tests when the user 
explicitly enables it and provides a native binary with overridden 
implementations. Even then, not all code paths and VectorUtil 
   operations are covered but only the ones overridden in the provided binary 
(e.g., the issue @ExE-Boss found 
[here](https://github.com/apache/lucene/pull/15508#discussion_r2960053681)). 
The native path is not tested automatically in our CI workflow which 
disconnects it from other implementations.
   
   One option is to generate a simple native binary purely for test coverage - 
not architecture-specific optimized code, just trivial C implementations that 
exercise the FFM calling convention and code paths. This 
   would keep maintenance overhead low since the binary wouldn't need per-arch 
tuning and only supposed to be used for testing(and also provide as base for 
users to build on if they want to write more optimized c code). Or we could 
just have fake java methods calling DefaultVectorUtil and use those to test the 
native path? That being said, there may be better approaches and I'm looking 
for suggestions and ideas to improve coverage here.


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to