eolivelli opened a new pull request, #3608:
URL: https://github.com/apache/celeborn/pull/3608

   ### What changes were proposed in this pull request?
   This commit adds only tests and some useful debug information about using 
MEMORY and S3 storage.
   
   ### Why are the changes needed?
   Because there is not enough code coverage on some configurations that may 
happen in production,  in particular about:
   - using MEMORY storage
   - using only S3 storage
   - using MEMORY with eviction to S3
   
   There is an interesting case to test: when you configure MEMORY to S3 
eviction and the dataset is small.
   It is important to ensure that no file is created in S3
   
   ### Does this PR resolve a correctness bug?
   
   No
   
   ### Does this PR introduce _any_ user-facing change?
   
   No
   
   ### How was this patch tested?
   
   It adds new integration tests


-- 
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]

Reply via email to