search

Alluxio / Community

New Contributor Tasks for Alluxio
20 stars 38 forks source link

Change Alluxio shell L100 to STDERR #597

Closed LuQQiu closed 3 years ago

LuQQiu commented 3 years ago

Prerequisite: Alluxio code convention: https://docs.alluxio.io/os/user/stable/en/contributor/Code-Conventions.html Google shell guide: https://google.github.io/styleguide/shellguide.html

Change https://github.com/Alluxio/alluxio/blob/837a41c68b956ded56ae0fd41f0bf26b52ffbf49/bin/alluxio#L100

echo "Process did not complete after "${maxWait}" seconds, killing."

to 

echo "Process did not complete after "${maxWait}" seconds, killing." >&2