syk-coder commented on PR #20250: URL: https://github.com/apache/pulsar/pull/20250#issuecomment-1540531752
> > Could you also help change `orElse` to `orElseGet` at line 1713 and 1763 > > Even at those places and the remaining places where we are creating **new LocalPolicies()** we are not using the bundle size from the configuration. Should I change it to have the bundle size supplied by the user from the configuration? Should we do it in this PR or shall we create another bug and link it and fix them there? As this PR is already approved, created a new issue for this and will push the changes in a new PR -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
