Skip to content

Fix the classic query package-loading cutoff optimization with external workspaces.#12595

Closed
benjaminp wants to merge 1 commit intobazelbuild:masterfrom
benjaminp:query-loading
Closed

Fix the classic query package-loading cutoff optimization with external workspaces.#12595
benjaminp wants to merge 1 commit intobazelbuild:masterfrom
benjaminp:query-loading

Conversation

@benjaminp
Copy link
Copy Markdown
Collaborator

A package's path fragment is not unambiguous when external workspaces are involved.

Fixes #12497.

@google-cla google-cla Bot added the cla: yes label Dec 1, 2020
@jin jin added team-Core Skyframe, bazel query, BEP, options parsing, bazelrc team-Configurability platforms, toolchains, cquery, select(), config transitions and removed team-Core Skyframe, bazel query, BEP, options parsing, bazelrc labels Dec 4, 2020
@jin jin requested review from gregestren and gyias December 4, 2020 05:27
@gregestren gregestren assigned meisterT and unassigned gregestren Dec 4, 2020
@gregestren
Copy link
Copy Markdown
Contributor

Reassigned to @meisterT for further triage since this is query (vs. cquery) logic.

@gregestren gregestren removed their request for review December 4, 2020 15:44
Copy link
Copy Markdown
Member

@meisterT meisterT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a simple test?

@benjaminp
Copy link
Copy Markdown
Collaborator Author

Are there a lot of internal tests for query? src/test/java/com/google/devtools/build/lib/query2/ seems pretty thin in this repository.

@meisterT
Copy link
Copy Markdown
Member

meisterT commented Dec 4, 2020

@benjaminp
Copy link
Copy Markdown
Collaborator Author

I can do that. Normally, I make some effort to not write shell integration tests, though, since bash is not a very good programming language.

…al workspaces.

A package's path fragment is not unambiguous when external workspaces are involved.

Fixes bazelbuild#12497.
@bazelbuild bazelbuild locked and limited conversation to collaborators Dec 4, 2020
@bazelbuild bazelbuild unlocked this conversation Dec 7, 2020
@bazel-io bazel-io closed this in 8555789 Dec 7, 2020
@benjaminp benjaminp deleted the query-loading branch December 7, 2020 14:45
philwo pushed a commit that referenced this pull request Dec 7, 2020
…al workspaces.

A package's path fragment is not unambiguous when external workspaces are involved.

Fixes #12497.

Closes #12595.

PiperOrigin-RevId: 346048937
philwo pushed a commit that referenced this pull request Dec 7, 2020
…al workspaces.

A package's path fragment is not unambiguous when external workspaces are involved.

Fixes #12497.

Closes #12595.

PiperOrigin-RevId: 346048937
philwo pushed a commit that referenced this pull request Dec 9, 2020
…al workspaces.

A package's path fragment is not unambiguous when external workspaces are involved.

Fixes #12497.

Closes #12595.

PiperOrigin-RevId: 346048937
philwo pushed a commit that referenced this pull request Dec 10, 2020
…al workspaces.

A package's path fragment is not unambiguous when external workspaces are involved.

Fixes #12497.

Closes #12595.

PiperOrigin-RevId: 346048937
ulfjack pushed a commit to EngFlow/bazel that referenced this pull request Mar 5, 2021
…al workspaces.

A package's path fragment is not unambiguous when external workspaces are involved.

Fixes bazelbuild#12497.

Closes bazelbuild#12595.

PiperOrigin-RevId: 346048937
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes team-Configurability platforms, toolchains, cquery, select(), config transitions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Querying "//..." fails if a top-level target exists and any external dependency can't be fetched

4 participants