Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
MB-52355: Add more logging when maxCompositeFilters > Num SecKeys
* Adding the print in fillScans and not setExplodePositions to capture protoscans received from the query * Index out of range panic was seen in setExplodedPositions function when accessing explodePositions array of size len(IndexInst.Defn.SecExprs) * -- panic: runtime error: index out of range [4] with length 4 * Print the scans details and return error than panic * Example print 2022-06-17T02:37:02.185+05:30 [Error] SCAN##2 ReqID: ee9939aa-d17c-4705-9d60-91d773053bbd Defn: 9151033855720878966 SecExprs: [`age`] Scans: <ud>([{ [150] 3 filterRange [{[{ 100 2}] [100] 3 } {[{50 150 3}] [50] [150] 3 }] <nil>} {[200] [200] 3 range [{[{200 200 3}] [200] [200] 3 }] <nil>}])</ud> MaxCompositeFilters: 1 ProtoScans: protobuf scan 0 filters:<low:"null" high:"100" inclusion:2 > protobuf scan 1 filters:<low:"50" high:"150" inclusion:3 > protobuf scan 2 filters:<low:"200" high:"200" inclusion:3 > Change-Id: Ibb017edb6e84af3d963e80f2549e28b69dc82d57
- Loading branch information