[DOCS-13686] Investigate changes in SERVER-48529: Delete unused PlanCache::feedback() mechanism Created: 02/Jun/20 Updated: 13/Nov/23 Resolved: 14/Jun/21 |
|
| Status: | Closed |
| Project: | Documentation |
| Component/s: | manual, Server |
| Affects Version/s: | None |
| Fix Version/s: | 4.7.0, 4.4.3, Server_Docs_20231030, Server_Docs_20231106, Server_Docs_20231105, Server_Docs_20231113 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Backlog - Core Eng Program Management Team | Assignee: | Andrew Feierabend (Inactive) |
| Resolution: | Works as Designed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Participants: | |||||||||
| Days since reply: | 3 years, 36 weeks, 1 day ago | ||||||||
| Epic Link: | DOCSP-9747 | ||||||||
| Description |
DescriptionDownstream Change Summary This patch removes the setParameter internalQueryCacheFeedbacksStored. I think this parameter was pretty obscure and unlikely to be ever explicitly configured, but I'm notifying downstream teams just in case. The parameter does nothing in 4.4 and has been removed in 4.6. Any uses of this in command line parameters or config files should simply be removed. Description of Linked TicketIn previous versions, the PlanCache tracked the score associated with a cache entry for several historical executions of the cached plan. These scores were then surfaced via the planCacheListPlans comment, which was deprecated in 4.2 and removed in 4.4. The number of historical executions tracked was controlled by the internalQueryCacheFeedbacksStored setParameter. Since this information is no longer surfaced by the replacement for planCacheListPlans (the $planCacheStats agg stage), the code for tracking this "feedback" information can be deleted. The internalQueryCacheFeedbacksStored setParameter can also be deleted. Scope of changesImpact to Other DocsMVP (Work and Date)Resources (Scope or Design Docs, Invision, etc.) |