[cmake] bugfix: rerun build.sh will cause pyarrow files missing

[chuxi]

What do these changes do?

1. move make clean before cmake command, because cmake will build some modules by checking some directories exist or not
2. avoid always running mvn install plasma java lib

question: do we have better solution of skipping build of some modules? including pyarrow, plasma_java.

Related issue number

[chuxi]

cc @robertnishihara @pcmoritz

[AmplabJenkins]

Merged build finished. Test FAILed.

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/8596/
Test FAILed.

[guoyuhong]

@AmplabJenkins retest this, please.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/8597/
Test PASSed.

[pcmoritz]

Which kinds of failures can happen here? Previously there was no || true necessary here, right?

[chuxi]

the script will exit if error happen as set -e at the begin.

So the first time, make clean in an empty directory which leads to makefile not found error.

|| true will redirect the error and keep the script to continue.

I use some directories existence to justify whether the lib is built already. once make clean is called, the directories will be removed.

However, if I call cmake command first, then cmake will see the directories exist. And it will not build the libs any more. Then I can make clean, which leads to the directories removed. So the error happens.

[pcmoritz]

Thanks for the explanation!