Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HDFS-17303 Make WINDOW_SIZE and NUM_WINDOWS configurable. #6801

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

huangzhaobo99
Copy link
Contributor

Description of PR

  1. The delay reported by DN to NN is an average delay of 3 hours.
    MutableRollingAverages: WINDOW_SIZE_MS_DEFAULT = 300_000 NUM_WINDOWS_DEFAULT = 36
  2. Changeing this window size and num to configurable param.

How was this patch tested?

Add UT

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
_ Prechecks _
+1 💚 dupname 0m 01s No case conflicting files found.
+0 🆗 spotbugs 0m 00s spotbugs executables are not available.
+0 🆗 codespell 0m 01s codespell was not available.
+0 🆗 detsecrets 0m 01s detect-secrets was not available.
+1 💚 @author 0m 00s The patch does not contain any @author tags.
+1 💚 test4tests 0m 00s The patch appears to include 1 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 6m 18s Maven dependency ordering for branch
+1 💚 mvninstall 130m 01s trunk passed
+1 💚 compile 62m 06s trunk passed
+1 💚 checkstyle 9m 12s trunk passed
-1 ❌ mvnsite 6m 50s /branch-mvnsite-hadoop-common-project_hadoop-common.txt hadoop-common in trunk failed.
+1 💚 javadoc 15m 56s trunk passed
+1 💚 shadedclient 232m 52s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 3m 39s Maven dependency ordering for patch
+1 💚 mvninstall 14m 32s the patch passed
+1 💚 compile 58m 01s the patch passed
+1 💚 javac 58m 00s the patch passed
+1 💚 blanks 0m 00s The patch has no blanks issues.
+1 💚 checkstyle 11m 40s the patch passed
-1 ❌ mvnsite 7m 17s /patch-mvnsite-hadoop-common-project_hadoop-common.txt hadoop-common in the patch failed.
+1 💚 javadoc 16m 29s the patch passed
+1 💚 shadedclient 255m 38s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 9m 50s The patch does not generate ASF License warnings.
783m 09s
Subsystem Report/Notes
GITHUB PR #6801
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets
uname MINGW64_NT-10.0-17763 a41ee85e8d5b 3.4.10-87d57229.x86_64 2024-02-14 20:17 UTC x86_64 Msys
Build tool maven
Personality /c/hadoop/dev-support/bin/hadoop.sh
git revision trunk / 3572446
Default Java Azul Systems, Inc.-1.8.0_332-b09
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch-windows-10/job/PR-6801/1/testReport/
modules C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch-windows-10/job/PR-6801/1/console
versions git=2.44.0.windows.1
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants