rmcyang commented on PR #39725: URL: https://github.com/apache/spark/pull/39725#issuecomment-1404317258
Thanks @otterc and @mridulm ! Have updated the PR trying to cover below proposal: - For `onComplete` and `onFailure`, add the data not being used from the push to ignored bytes - For `onData`, add the received data not being used to ignored bytes (include the bytes during which the server encounters IOException) - updated the documents for `ignoredBytes` to be clearer on what it tries to collect Please let me know if you if any thoughts. -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org