You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2019/12/05 18:31:58 UTC

[GitHub] [incubator-hudi] n3nash commented on issue #1057: Hudi Test Suite

n3nash commented on issue #1057:     Hudi Test Suite
URL: https://github.com/apache/incubator-hudi/pull/1057#issuecomment-562256132
 
 
   @yanghua Can we continue to use the same branch `hudi_test_suite_refactor` please ? You can continue to PR against that branch. 
   Secondly, I think we should name the package - `hudi-test-suite` instead of `hudi-end-to-end-tests`, it's more concise, WDYT ?
   
   Also, can we execute on the following plan :
   
   1) Please go over all the changes in the existing PR - I've addressed most of the comments. 
   2) Let's rename the package - like you've already done
   3) Both of us can start to use the test suite and run them locally for 2-3 days. This will give us confidence whether the test suite is ready to test large features.
   4) Let's merge this first version of the test suite to master so folks can start using it
   5) Both of us can continue to enhance the test suite - a) You can add some of the things that you mentioned earlier like DIstributedTestDataSource etc b) I want to add more use-cases
   
   Let me know if this sounds fair to you.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services