[SERVER-50172] Allow maxCellsInCovering index parameter Created: 07/Aug/20 Updated: 06/Dec/22 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | Index Maintenance, Querying |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Siyuan Zhou | Assignee: | Backlog - Query Execution |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | neweng, qexec-team | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Query Execution
|
||||||||||||
| Operating System: | ALL | ||||||||||||
| Sprint: | Query 2020-12-14, Query 2020-12-28, Query 2021-01-11, Query 2021-01-25, Query Execution 2021-02-22, Query Execution 2021-03-08, Query Execution 2021-03-22, Query Execution 2021-04-05, Query Execution 2021-04-19, Query Execution 2021-05-03, Query Execution 2021-05-17, Query Execution 2021-05-31, Query Execution 2021-06-14, Query Execution 2021-06-28, Query Execution 2021-07-12 | ||||||||||||
| Participants: | |||||||||||||
| Description |
|
maxCellsInCovering is probably the most important index parameter of 2dsphere index, added in 3.2. It is an advisory upper bound of the number of cells (index keys) of a geometry, but it's not documented. Since |
| Comments |
| Comment by Katya Kamenieva [ 18/Nov/20 ] |
|
Flagging for re-triage based on the additional info from siyuan.zhou |
| Comment by Siyuan Zhou [ 17/Nov/20 ] |
|
This parameter was originally requested by users in We actually didn't remove the support for configurable maxCellsInCovering. It's parsed here and used here. It's just not allowed when validating the index parameters. Unlike other parameters, we don't have testing for this parameter, so maybe when we added strict validation, it was omitted. The proposal is to add "maxCellsInCovering" into this list, do some sanity check (e.g. greater than 0) after parsing here and add test cases in geo_s2index.js like this. |
| Comment by Ana Meza [ 17/Nov/20 ] |
|
Katia to review |
| Comment by David Storch [ 20/Aug/20 ] |
|
justin.seyster I'm going to shift this into a future sprint to allow you to focus on SBE work. |