adoroszlai opened a new pull request, #6161:
URL: https://github.com/apache/ozone/pull/6161
## What changes were proposed in this pull request?
1. `setStreamBufferSize` is never used: it can be removed
2. `enableContainerDatastream`, `numOfOmHandlers`, `numOfScmHandlers` are
never changed:
- values can be defined in `ozone-site.xml`
- member variables and their usage can be removed
3. `setClusterId`, `setOmId` and `setScmId` are only called with
`UUID.randomUUID().toString()`:
- `setOmId`, `setScmId` can be completely removed
- `setClusterId` is kept for internal use (`MiniOzoneCluster` and
`MiniOzoneClusterProvider`)
- few tests that need the actual `clusterId` value are changed to use
`cluster.getClusterId()`
Additional minor cleanup in `TestOzoneFsHAURLs` and
`TestOzoneManagerConfiguration`, mostly required by the removal of `omId`:
- remove unnecessary creation of `OMStorage`
- use `@TempDir`
https://issues.apache.org/jira/browse/HDDS-10289
## How was this patch tested?
CI:
https://github.com/adoroszlai/ozone/actions/runs/7773770796
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
