[GitHub] [hudi] xushiyan commented on pull request #1746: [HUDI-996] Add SharedResources for functional tests

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

xushiyan commented on pull request #1746:
URL: https://github.com/apache/hudi/pull/1746#issuecomment-646396974


   > FYI: The total travis test time has increased by 5 min which seem to be the time for newly added functional test suite. I dont know enough about surefire so cannot verify but it seems the functional tests are also being executed as part of the unit tests?
   
   Thank you for checking @prashantwason ! The overall test time fluctuates with a few min difference. I would pay attention to the module test time printed at the end of the tests.
   
   The last 2 success runs ended up with ~9 min for `hudi-utilities`
   
   ```
   [INFO] hudi-utilities_2.11 ................................ SUCCESS [09:13 min]
   [INFO] hudi-utilities_2.11 ................................ SUCCESS [09:07 min]
   ```
   
    The [last master build](https://travis-ci.org/github/apache/hudi/jobs/699696974) used almost 11 min for `hudi-utilities`
   
   ```
   [INFO] hudi-utilities_2.11 ................................ SUCCESS [10:50 min]
   ```
   
   Also did a text search from the travis logs that the tests included in the functional suite are not found, so they are excluded. By separating a few tests to the functional suite, we started to save time from unit tests execution. Incrementally we'll save more. :) 
   
   cc @n3nash @vinothchandar


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