mdmarshmallow commented on PR #841: URL: https://github.com/apache/lucene/pull/841#issuecomment-1142403496
There was an email thread where some other commiters suggested also developing this in sandbox. It does seem like this API could go through some heavy changes (I think we all agree on that here), so it seems like the `sandbox` module would make more sense for this? Is there a benefit to having it in `facets` vs `sandbox`? I think putting this in a field that extends `BDV` makes the most sense right now as well. I think the `VERSION` identifier might be a bit overkill though, especially if we decide to put this in sandbox. I don't think we should worry about making this backwards compatible. As for creating a new multidimensional version of the `Facets` API, I think what you're saying makes sense, but could we extend the existing `Facets` API as well, I think those methods would still be relevant right? -- 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]
