shaie commented on PR #841: URL: https://github.com/apache/lucene/pull/841#issuecomment-1142433913
> Is there a benefit to having it in `facets` vs `sandbox` I personally am not sure that we should worry about the big changes that the module will go under. API-wise we tag the classes as `@lucene.experimental` which should free us from worrying about the API for a while (but obviously at some point we'll want to declare the API stable!). As for the on-disk structure I also feel like we have some room with it. It's the `facet` module, not `core`. There are much less users of it and it's a completely new feature, so I think it's reasonable to declare the whole feature experimental. I worry that if we put it in sandbox, users might not even attempt to try it, even if technically they don't mind re-indexing on version upgrades. Because "sandbox" feels (to me) like half-baked stuff, while it's not true here - we do deliver value, it's just that the representation of things may change. -- 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
