-
Notifications
You must be signed in to change notification settings - Fork 857
SOLR-18352: Improve DX by more accurately communicating deprecated code status #4750
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
5c605f9
5d4ff3a
70e7233
b368750
6e2e5c0
2e6a5d6
68a96b7
7859627
2dc896a
510811d
fb37320
7c0c8b7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These deprecated methods were deprecated on 10.0; look at branch_10_0 to see for yourself. Thus we may remove them anytime in 10.x. 9.x had no alternative so devs that worked on this didn't deprecate there.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i thought if we deprecated on 10, then we remove on 11? I will go back and check the deprecation tags for the 10 versus 10.1... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Arguably this entire package can be seen as having a consistent status: The code is associated with "Trie" numerics. Anticipated to be fully resolved (replacements) for Solr 11. @HoustonPutman is leading the way here with progress.
Maybe the BBox stuff could separately be decoupled from that and be ported to Points + NumericDocValues but someone has not undertaken that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that if you created a nice JIRA for that, then it COULD be.... I don't know enough about BBox to say.