chatman commented on PR #14131: URL: https://github.com/apache/lucene/pull/14131#issuecomment-2598991862
> I haven't touched on the validity of having an NVIDIA only GPU backed index in Lucene sandbox directly. The new dependencies are huge. IDK if whomever downloads and builds lucene then now have to download and build with these? I am unsure how the sandbox stuff works. This is something we need to work out as we want this to be. Here are my thoughts at the moment, and things we need consensus on: * Right now, the `cuvs-java` dependency comes from local Maven and it should come from Maven central once the artifacts are available. * TODO: If a system doesn't have cuda or GPUs, these codepaths should gracefully fail and indicate that support not available. * Continuous testing can be enabled on GPU enabled Jenkins instances (we can have a discussion around that). * To validate the integration at API level, some mock tests (that simulate the same functionality using the CPU) can be added to the cuvs-java API. * If we can choose, we can ignore shipping this by default. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
