[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:
Related
is related to SERVER-19168 Finer default indexed level for index... Closed
is related to SERVER-10363 User should be able to specify 2dsphe... Closed
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 SERVER-769 in 3.4, it is not allowed, even though the other two 2dsphere index parameters are allowed. We should add "maxCellsInCovering" to the allowed filed list and consider documenting it.



 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 SERVER-10363 and introduced in SERVER-19168.

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.

Generated at Thu Feb 08 05:21:58 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.