[GitHub] [ozone] kaijchen commented on pull request #3391: HDDS-6710. EC: Add EC pipeline minimum to MiniOzoneCluster

[GitBox <gi...@apache.org>]

kaijchen commented on PR #3391:
URL: https://github.com/apache/ozone/pull/3391#issuecomment-1124731223

   > Thanks @kaijchen for working on this.
   > 
   > I'm not sure we should add new methods to `MiniOzoneCluster.Builder` to set config properties, unless it is a widely used setting with different values in various tests.
   > 
   > When building a `MiniOzoneCluster`, tests provide an `OzoneConfiguration`, which they can already customize.
   > 
   > If the production default value is not suitable for test environment, we can override it for all integration tests in `hadoop-ozone/integration-test/src/test/resources/ozone-site.xml`.
   > 
   > Can you please share more details on the expected usage of this property?
   
   I was working on EC offline recovery in my local branch, and I need to verify interfaces like `ListBlock` and `ReadChunk`.
   
   I limit EC pipeline number to 1 in tests to ensure all the data is written into the same container group.


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org