Rename CollectionStatistics to FieldStats and TermStatistics to TermStats.#15929
Rename CollectionStatistics to FieldStats and TermStatistics to TermStats.#15929vsop-479 wants to merge 9 commits intoapache:mainfrom
Conversation
jainankitk
left a comment
There was a problem hiding this comment.
Without any context around why it was called CollectionStatistics, this change makes sense to me. Although, I am wondering if we should rename the corresponding methods and parameters as well to be inline with this change?
|
Thanks @jainankitk ! I will do it. |
|
I don't remember the reasoning behind the name, but I think FieldStatistics is better. One thing to consider, we also have Would just not be great to have to rename it yet again. |
Makes sense, I will rename |
|
I also added an entry in MEGRATE.md. |
No description provided.