virajjasani commented on PR #1736: URL: https://github.com/apache/phoenix/pull/1736#issuecomment-1897974489
Offset is broken, without paging. Offset maintains row count at server side in memory and increments it until offset is reached, so even if we restrict dummy processing until offset is reached, it can still cause issues with valid row. e.g. rows are scanned until offset is reached and then we return valid row, and then region moves. Now when scan is resumed on moved region, row count state is lost and it is reset to 0 to begin with. This can easily lead to data integrity issues for offset because the expected behavior in this case is to resume scanning rows considering row count is already at offset. I am working on fixing this. -- 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]
