-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
HADOOP-18959. Use builder for prefetch CachingBlockManager. (#6240) #6302
HADOOP-18959. Use builder for prefetch CachingBlockManager. (#6240) #6302
Conversation
Testing in progress |
FYI @ahmarsuhail |
Tested against
|
🎊 +1 overall
This message was automatically generated. |
@ahmarsuhail since the original PR is merged to trunk and this is backport with clean build and scale testing done, could you please help review/merge so that upcoming 3.3.x or 3.4.0 release doesn't miss it? (otherwise we might have to deal with backward compatibility issues) :) |
@virajjasani I recently cut the 3.4.0 branch..so I'm just checking where things should be backported to now. Will do this once I have some clarity there. |
Sounds good, thanks @ahmarsuhail. For branch-3.4, this will be clean cherry-pick since it was just cut recently. |
@virajjasani Thanks for your reminder! I modified the version to 3.4.0, the PR will be merged to branch3.3 soon. @ahmarsuhail Can you help review this PR? Thank you very much! |
@virajjasani Thanks for the contribution! merge into branch-3.3. |
branch-3.3 backport PR of #6240