zstan commented on PR #1469: URL: https://github.com/apache/ignite-3/pull/1469#issuecomment-1371826199
I don`t like new approach, cause : 1. In case of index scan with concrete bounds (check ItSecondaryIndexTest#testSelectWithRanges) we call StorageScanNode#convertPublisher -> Subscriber#onNext more than 5 times, while in previous approach it raises only 5 (limit param) times and as a consequence we call StorageScanNode#convert more than 5 times. 2. we still have test hanging:  -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
