[GitHub] [lucene] msokolov commented on pull request #238: LUCENE-10016: add support for Knnvector index and query to the demo

[GitBox <gi...@apache.org>]

msokolov commented on pull request #238:
URL: https://github.com/apache/lucene/pull/238#issuecomment-894706593


   Hmm I had a spotless problem that didn't show up when I ran `check` locally or `precommit`? Also locally I am getting a rat failure:
   
   ```
   Execution failed for task ':lucene:demo:rat'.
   > Found 1 file(s) with errors:
       - Unknown license: /home/ANT.AMAZON.COM/sokolovm/workspace/lucene/lucene/demo/src/test/org/apache/lucene/demo/test-files/knn-dict/knn-token-vectors
   ```
   which is totally reasonable (how does it know!). I can see I need to add the `pddl-1.0` license to the lucene/licenses folder to cover that content (which is from GloVe) but how can I assert that the contents there are covered by that license?
   


-- 
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-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org