terrymanu commented on PR #39081: URL: https://github.com/apache/shardingsphere/pull/39081#issuecomment-5100735795
Thanks for the work on this PR. #39050 has now been merged, but it does not implement op_seek_blob; the current master still explicitly reports SEEK BLOB as unsupported, so #39051 should remain open. However, #39050 replaced the BLOB state-management and package structure on which this PR is based, and most of the broader BLOB fixes included here are now already covered. Updating this branch would effectively require a new implementation rather than a normal rebase. I suggest closing this PR as superseded by the BLOB architecture introduced in #39050—not as fixed or duplicate—and opening a focused follow-up PR against the latest master. The new PR should extend the current Proxy-side BLOB read state with cursor and BLOB-type information, implement the three seek modes, preserve deferred-handle resolution, and reject seek for non-stream BLOBs. If you would still like to work on this, we would be happy for you to continue with that focused follow-up PR. -- 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]
