Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

S3DataSegmentMover: No need to move files onto themselves

This commit is contained in:
Gian Merlino 2013-12-18 18:30:01 -08:00
parent 8d8ec51c9c
commit 6478c9a162
1 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
s3-extensions/src/main/java/io/druid/storage/s3/S3DataSegmentMover.java
View File

@ -114,6 +114,9 @@ public class S3DataSegmentMover implements DataSegmentMover
public Void call() throws Exception
{
if (s3Client.isObjectInBucket(s3Bucket, s3Path)) {
if (s3Bucket.equals(targetS3Bucket) && s3Path.equals(targetS3Path)) {
log.info("No need to move file[s3://%s/%s] onto itself", s3Bucket, s3Path);
} else {
log.info(
"Moving file[s3://%s/%s] to [s3://%s/%s]",
s3Bucket,
@ -122,6 +125,7 @@ public class S3DataSegmentMover implements DataSegmentMover
targetS3Path
);
s3Client.moveObject(s3Bucket, s3Path, targetS3Bucket, new S3Object(targetS3Path), false);
}
} else {
// ensure object exists in target location
if (s3Client.isObjectInBucket(targetS3Bucket, targetS3Path)) {