Skip to content

Split up the TargetParsingCompleteEvent (BEP PatternExpanded) message #31282

Description

@BarrettStephen

Description of the feature request:

We recently started failing BES upload to BuildBuddy as we passed their 50 MB default GRPC message limit: https://github.com/buildbuddy-io/buildbuddy/blob/master/server/util/grpc_server/grpc_server.go#L42
On this particular message. I worked around it by increasing that configuration to 200 MB, but overall it would be better if this message didnt scale so huge.

The issue is the TargetConfigured id for every target. We have a ton of targets in a large monorepo. Can we stop stuffing all this into one event and instead announce them in batched Progress events (same idea as --build_event_max_named_set_of_file_entries)?

Which category does this issue belong to?

No response

What underlying problem are you trying to solve with this feature?

Large GRPC messages hitting server side reasonable limits

Which operating system are you running Bazel on?

CentOS 9 Stream

What is the output of bazel info release?

release 9.2.0

If bazel info release returns development version or (@non-git), tell us how you built Bazel.

No response

What's the output of git remote get-url origin; git rev-parse HEAD ?

N/A

Have you found anything relevant by searching the web?

Similar #12050

Any other information, logs, or outputs that you want to share?

No response

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    team-CoreSkyframe, bazel query, BEP, options parsing, bazelrctype: feature requestuntriagedHas not yet been seen by appropriate subteam

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions