[GitHub] [incubator-heron] nicknezis commented on pull request #3779: Adding Bazel Platforms support

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

nicknezis commented on pull request #3779:
URL: https://github.com/apache/incubator-heron/pull/3779#issuecomment-1053659202


   I decided to remove the Travis CI special settings because some of them were outdated settings which were needed to use a modern C++ compiler. We are now on a more modern version of Ubuntu, so it's not needed anymore. Also With the other `.bazelrc` cleanup, I thought it would be a good opportunity to clean it up. 
   
   One of the benefits I see is that the build completed in 1 hr 45 min. This is down from the nearly 3 hour long builds. I'm not sure what the difference was. One theory is maybe it was due to us re-running the style check on each stage of the process (build, non-flaky unit test, flaky unit test). Another theory is that by limiting the memory used and also the jobs to 25, maybe that slowed things down. But I'm taking this opportunity to test TravisCI and it seems like it's working.


-- 
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: commits-unsubscribe@heron.apache.org

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