Kontinuation opened a new pull request, #36: URL: https://github.com/apache/sedona-db/pull/36
`limit` in `RecordBatchReaderExec` is supposed to be a number of rows. However, it is currently being applied as a number of batches. This patch fixes this by implementing a structure that keeps track of the number of iterated rows and stops early. This patch also fixes another problem: the usage of `RwLock` in `RecordBatchReaderExec` is inappropriate, since we only access the underlying reader protected by the lock in write mode. We use a `Mutex` here instead, and removed `unsafe impl Sync for RecordBatchReaderExec`. -- 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]
