HDFS-16618. sync_file_range error should include more volume/file info (#4402)

Signed-off-by: Tao Li <tomscut@apache.org>
This commit is contained in:
Viraj Jasani 2022-06-07 01:53:45 -07:00 committed by GitHub
parent a234d00c1c
commit 0a65883c50
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -216,16 +216,20 @@ synchronized void shutdown() {
} }
} }
public void submitSyncFileRangeRequest(FsVolumeImpl volume, public void submitSyncFileRangeRequest(FsVolumeImpl volume, final ReplicaOutputStreams streams,
final ReplicaOutputStreams streams, final long offset, final long nbytes, final long offset, final long nbytes, final int flags) {
final int flags) { execute(volume, () -> {
execute(volume, new Runnable() { try {
@Override streams.syncFileRangeIfPossible(offset, nbytes, flags);
public void run() { } catch (NativeIOException e) {
try { try {
streams.syncFileRangeIfPossible(offset, nbytes, flags); LOG.warn("sync_file_range error. Volume: {}, Capacity: {}, Available space: {}, "
} catch (NativeIOException e) { + "File range offset: {}, length: {}, flags: {}", volume, volume.getCapacity(),
LOG.warn("sync_file_range error", e); volume.getAvailable(), offset, nbytes, flags, e);
} catch (IOException ioe) {
LOG.warn("sync_file_range error. Volume: {}, Capacity: {}, "
+ "File range offset: {}, length: {}, flags: {}", volume, volume.getCapacity(),
offset, nbytes, flags, e);
} }
} }
}); });