Details
-
Bug
-
Resolution: Fixed
-
Critical
-
7.1.0
-
Untriaged
-
1
-
Unknown
Description
The goal is to get rid of the lock introduced with - http://review.couchbase.org/c/n1fty/+/158589
Recently discovered the query parameter - for concurrent access during verify/eval. This defaults to 1. Contention for this new lock will kick in only if the query parallelization is greater than 1.
I can think of 3 options at the moment ..
- Leave the locking in place to serialize evaluation (undesirable).
- We can update the NewVerify API for query to share the parallelization factor - so we'll need to update the sear index to be able to hold that many documents at a time (against what sear brings to the table).
- Thirdly, based on the parallelization factor - set up that many sear indexes for evaluation.
Attachments
Issue Links
- relates to
-
MB-47260 [FTS] Drop in throughput for N1fty QF2 and QF3 queries
- Closed
For Gerrit Dashboard: MB-47699 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
158674,2 | MB-47699. Pass max_parallelism to verify | master | query | Status: ABANDONED | 0 | 0 |
158679,4 | MB-47699: Pass max_parallelism to n1fty's NewVerify(..) | master | query | Status: MERGED | +2 | +1 |
158680,4 | MB-47699: Obtain max_parallelism for NewVerify from query | master | n1fty | Status: MERGED | +2 | +1 |
158748,7 | MB-47699: Support as many sear indexes as parallelism demands | master | n1fty | Status: MERGED | +2 | +1 |
159642,3 | MB-47699: FIFO queue to push-pop bleve.Indexes for Verify/Eval | master | n1fty | Status: MERGED | +2 | +1 |