liusheng commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r539881557



##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
   include:
     - arch: s390x
       jdk: openjdk11
+    - dist: focal
+      group: edge

Review comment:
       sorry, this is unnecessary, my mistake.

##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
   include:
     - arch: s390x
       jdk: openjdk11
+    - dist: focal
+      group: edge
+      arch: arm64-graviton2

Review comment:
       Hmm,  we have tried using the common `arm64` resources, but many 
modules(especially Integration-Test) are unstable and easy to fail. the new 
added `arm64-graviton` is anounced a better performance ARM resources, so 
switch to use it. 
   Maybe we can try back to use the `arm64` resources as we have migrated from 
`travis-ci.org` to `travis-ci.com` to see if the problems still occur.

##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
   include:
     - arch: s390x
       jdk: openjdk11
+    - dist: focal

Review comment:
       Currently, the "arm64-graviton2" resources are only available in 
`Xenial`, `Bonic`, `Focal` dists of Ubuntu, please see the [Travis 
doc](https://docs.travis-ci.com/user/reference/overview#which-one-do-i-use), I 
just picked a lastest version.




----------------------------------------------------------------
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:
[email protected]


Reply via email to