virajjasani commented on PR #5893: URL: https://github.com/apache/hadoop/pull/5893#issuecomment-1704478561
`us-west-2`: `mvn clean verify -Dparallel-tests -DtestsThreadCount=8 -Dscale -Dprefetch` Checkstyle complains about 8th param added to CachingBlockManager c'tor. Shall we mark it ignored for now and use builder class (for both CachingBlockManager and S3ACachingBlockManager) later for this patch to not focus on refactor work? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org