Skip to content

Conversation

@mukund-thakur
Copy link
Contributor

Ran test in ap-south-1 bucket with command:
mvn clean verify -Ds3guard -Ddynamo -Dparallel-tests -DtestsThreadCount=8
All good.

Contributed by Mukund Thakur

Optimize S3AFileSystem.listLocatedStatus() to perform list
operations directly and then fallback to head checks for files
@mukund-thakur mukund-thakur changed the base branch from trunk to branch-3.3 April 15, 2020 11:50
@mukund-thakur mukund-thakur changed the title Forked branch 3.3 HADOOP-16465 listLocatedStatus() optimisation (#1943) Apr 15, 2020
@mukund-thakur
Copy link
Contributor Author

@steveloughran Backporting #1943 to branch 3.3

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 14s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 1 new or modified test files.
_ branch-3.3 Compile Tests _
+1 💚 mvninstall 22m 8s branch-3.3 passed
+1 💚 compile 0m 32s branch-3.3 passed
+1 💚 checkstyle 0m 23s branch-3.3 passed
+1 💚 mvnsite 0m 36s branch-3.3 passed
+1 💚 shadedclient 16m 26s branch has no errors when building and testing our client artifacts.
+1 💚 javadoc 0m 26s branch-3.3 passed
+0 🆗 spotbugs 1m 1s Used deprecated FindBugs config; considering switching to SpotBugs.
+1 💚 findbugs 0m 59s branch-3.3 passed
_ Patch Compile Tests _
+1 💚 mvninstall 0m 32s the patch passed
+1 💚 compile 0m 26s the patch passed
+1 💚 javac 0m 26s the patch passed
-0 ⚠️ checkstyle 0m 18s hadoop-tools/hadoop-aws: The patch generated 1 new + 10 unchanged - 0 fixed = 11 total (was 10)
+1 💚 mvnsite 0m 30s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 shadedclient 15m 20s patch has no errors when building and testing our client artifacts.
+1 💚 javadoc 0m 22s the patch passed
+1 💚 findbugs 1m 4s the patch passed
_ Other Tests _
+1 💚 unit 1m 9s hadoop-aws in the patch passed.
+1 💚 asflicense 0m 29s The patch does not generate ASF License warnings.
64m 19s
Subsystem Report/Notes
Docker Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1960/1/artifact/out/Dockerfile
GITHUB PR #1960
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle
uname Linux dd3572e35b17 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision branch-3.3 / 4bd37f2
Default Java 1.8.0_242
checkstyle https://builds.apache.org/job/hadoop-multibranch/job/PR-1960/1/artifact/out/diff-checkstyle-hadoop-tools_hadoop-aws.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-1960/1/testReport/
Max. process+thread count 426 (vs. ulimit of 5500)
modules C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-1960/1/console
versions git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@steveloughran
Copy link
Contributor

merged into 3.3 by way of a cherrypick of trunk and local test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants