Adding log sort flag to disable stable sort and get better memory performance in some situations

[YuanHao97]

fix #17111
add log sorting flag to disable stable sort and get better memory performance in some situations.
command line flag: --execution_log_sort
possible values:
true : use stable sort as before when writing logs to a file. See com.google.devtools.build.lib.bazel.execlog.StableSort#stableSort
false : do not sort logs. Write directly from input stream to output stream.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[YuanHao97]

@tjgq Hi, I've updated some files and got 5 failed tests. How can I resolve those failed checks? Thanks!

[tjgq]

@tjgq Hi, I've updated some files and got 5 failed tests. How can I resolve those failed checks? Thanks!

Please ignore the failing tests on MacOS for now; they're unrelated to your PR. Someone else is working on fixing them.

[tjgq]

@YuanHao97 at least one of the failures seems to be caused by your PR:

src/main/java/com/google/devtools/build/lib/bazel/SpawnLogModule.java:190: error: could not resolve MessageOutputStream
--
  | private void ignoreSort(InputStream in, MessageOutputStream out) throws IOException {
  | ^

In addition, please do the following before this PR can be merged:

• Sign the CLA
• Accept my suggested edit to the flag description in ExecutionOptions.java (if you agree with it)

I think the other broken tests have now been fixed, so you should get a cleaner result when you push again.

[YuanHao97]

@tjgq Sorry, I got some compiling problems before push, so it's difficult to do verification.
BTW, I'm using Intellij IDEA and seems some packages are not found:
import com.google.common.collect.ImmutableList;
import com.google.devtools.build.lib.exec.Protos.SpawnExec;
I'm not sure why com.google.common.collect.ImmutableList is not found. Is there any more dependencies I should add?
I guess com.google.devtools.build.lib.exec.Protos.SpawnExec should be generated by some protobuf plugins, at least for maven project. But how to do that in Bazel project?

[tjgq]

@tjgq Sorry, I got some compiling problems before push, so it's difficult to do verification. BTW, I'm using Intellij IDEA and seems some packages are not found: import com.google.common.collect.ImmutableList; import com.google.devtools.build.lib.exec.Protos.SpawnExec; I'm not sure why com.google.common.collect.ImmutableList is not found. Is there any more dependencies I should add? I guess com.google.devtools.build.lib.exec.Protos.SpawnExec should be generated by some protobuf plugins, at least for maven project. But how to do that in Bazel project?

To get Intellij support for Bazel builds, see https://ij.bazel.build/.

I think this looks good now. Could you please sign the CLA (#17354 (comment)) so I can start the import process?

[YuanHao97]

@tjgq I've already siged cla. Please take a look at the check list.

[tjgq]

@tjgq I've already siged cla. Please take a look at the check list.

My bad, sorry. I'll start the import process.

[brentleyjones]

@bazel-io flag

[ShreeM01]

@bazel-io fork 6.1.0