[GitHub] [lucene] alessandrobenedetti opened a new issue, #12099: introduce support in KnnVectorQuery for getters
alessandrobenedetti opened a new issue, #12099: URL: https://github.com/apache/lucene/issues/12099 ### Description Knn Queries are locked currently, it would be beneficial for applications using them to have access to getters and setters. An example is how filter queries are managed in Apache Solr: the processing of pre-filters and post-filters could benefit from opening up the access to such variables. Especially the pre-filter support introduced in Solr 9.1 could get great benefits from being able to set the filter, after the query has been parsed. See: https://github.com/apache/solr/pull/1245 -- 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.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
[GitHub] [lucene] janhoy commented on pull request #12096: remove username from MANIFEST.MF in build artifacts
janhoy commented on PR #12096: URL: https://github.com/apache/lucene/pull/12096#issuecomment-1399301456 As long as we have not decided to aim for reproducible builds (it will take lots and lots of effort) there’s little use in such a change.. You should present your proposal on the dev list as a first step. -- 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
[GitHub] [lucene] magibney commented on pull request #12096: remove username from MANIFEST.MF in build artifacts
magibney commented on PR #12096: URL: https://github.com/apache/lucene/pull/12096#issuecomment-1399322839 I think reproducible builds would be an great goal! (and second Jan's suggestion of proposing on dev list). Paradoxically, further consideration of the issues that "reproducible builds" seek to address has reinforced some nagging doubts I had about removing the userid from the manifest. It's about trust. We don't have `@author` tags etc. in code, but the project does have usernames all over it -- in the form author/committer fields of commits. The process of preparing a release is mechanical, but it definitely requires trust in the RM (`>=` trust required for a commit). And I realize that the gpg signing of release artifacts fills this role (and in a more substantial way), so username in MANIFEST.MF is arguably redundant. But I think broadcasting the identity of the user who produced the artifact makes sense. When the same user _signs_ the artifacts, there's an implicit endorsement of the build user/agent as recorded in the MANIFEST.MF. I'm not going to make an issue of it one way or another; and I'm still happy to merge this PR. But tbh my personal inclination would be to leave the userid in the manifest. It's trust and provenance for the "last mile" that bridges the gap between source code and the artifacts that most users will actually interact with; and even if in a sense it's redundant, declaring it explicitly as part of the build artifact strikes me as appropriate. -- 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
[GitHub] [lucene] gus-asf commented on pull request #12096: remove username from MANIFEST.MF in build artifacts
gus-asf commented on PR #12096: URL: https://github.com/apache/lucene/pull/12096#issuecomment-139933 Discussion email sent. -- 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
