wwj6591812 commented on PR #8219: URL: https://github.com/apache/paimon/pull/8219#issuecomment-4828849816
> Please carefully consider my question about forcefully judging in Writer. Choosing a static method may seem like a very perfunctory answer and solution. Please switch from AI to human state. Thanks for the feedback. @JingsongLi I removed the Writer-side 404 detection, reverted all batch/core changes, and limited this PR to the existing Flink path (FlinkRowWrapper + UriReaderFactory.exists()), with HTTP support added at the URI layer only. Sorry for the earlier over-scoped diff. On master, blob-write-null-on-missing-file already applies only to Flink writes: FlinkSinkBuilder → FlinkRowWrapper.isNullAt() → UriReaderFactory.exists() for descriptor BLOB fields (local files only; HTTP was always treated as existing). Batch/core writes do not use this option. This PR keeps the same scope — Flink only — and just extends that existing exists() check to HTTP URIs. -- 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]
