Re: Adding arm64 support to travis-ci in storm

2020-06-22 Thread Rahul Aggarwal
Hi sir, Thank you for the suggestion. Yes, pip2 install mock has solved the mock import problem. But I am again coming up with some test errors. I am linking my travis-ci job logs below. https://travis-ci.com/github/rahulgit-ps/storm/jobs/352331819 I am guessing that the code is making some

Re: Adding arm64 support to travis-ci in storm

2020-06-22 Thread Govind Menon
>From the logs it looks like the JAVA_HOME isn't set. And when I looked at the job config it seems you've commented out that section. Could you also file a JIRA? - the CLI should fail more gracefully when the JAVA_HOME isn't set. On Mon, 22 Jun 2020 at 13:36, Rahul Aggarwal <

Re: Adding arm64 support to travis-ci in storm

2020-06-22 Thread Sheng Liu
Hi Rahul, Thanks for bringing this up. My colleague and I have done some attempts about adding ARM CI support for Storm by using Travis CI. here is an example of Travis results[1], we have also met some issues: 1. The "No module named mock" error you have mentioned, that is because in the Travis

Re: Adding arm64 support to travis-ci in storm

2020-06-22 Thread Yikun Jiang
Hi, We have some invistatgation on ARM CI support of STORM. https://github.com/Yikun/storm/pull/2/files Some introduction of our invesitagation, FYI: *1. We should set the JAVA_HOME, because the travis issue see more in [2]* if [ "$(uname -m)" == aarch64 ]; then sudo apt-get -y install

[GitHub] [storm] kishorvpatil opened a new pull request #3290: MINOR: Adding Supervisor StormTimer thread names

2020-06-22 Thread GitBox
kishorvpatil opened a new pull request #3290: URL: https://github.com/apache/storm/pull/3290 ## What is the purpose of the change * _StormTimer_ threads are not names properly on Supervisor. ## How was the change tested Running unit test and jstack shows Thread names

Adding arm64 support to travis-ci in storm

2020-06-22 Thread Rahul Aggarwal
Hi Team, I am adding arm64 support to storm's travis-ci with jdk 8 and 11. During build, I am getting an error related to mock as below : ImportError: No module named mock I have manually installed mock via pip install mock and pip3 install mock. But the error is still the same. Can you please

[GitHub] [storm] RuiLi8080 opened a new pull request #3289: [STORM-3654] remove executor id from JCQueue receive-queue name

2020-06-22 Thread GitBox
RuiLi8080 opened a new pull request #3289: URL: https://github.com/apache/storm/pull/3289 ## What is the purpose of the change This name is mainly used in two places: 1. JCQueue metric name: Add unique executor id to metric name could be bad if we want to aggregate metrics

Re: Adding arm64 support to travis-ci in storm

2020-06-22 Thread Govind Menon
What is the Python version which is showing the error? It's possible your default Python is also 3 - try pip2 install mock as well. On Mon, 22 Jun 2020 at 04:26, Rahul Aggarwal < rahul.aggar...@puresoftware.com> wrote: > Hi Team, > > I am adding arm64 support to storm's travis-ci with jdk 8 and