Skip to content

Conversation

@ueshin
Copy link
Member

@ueshin ueshin commented Jul 22, 2020

What changes were proposed in this pull request?

This is a follow-up of #28852.

This PR to use only config name; otherwise the doc for the config entry shows the entire details of the referring configs.

Why are the changes needed?

The doc for the newly introduced config entry shows the entire details of the referring configs.

Does this PR introduce any user-facing change?

The doc for the config entry will show only the referring config keys.

How was this patch tested?

Existing tests.

@ueshin
Copy link
Member Author

ueshin commented Jul 22, 2020

Also cc @sap1ens

@sap1ens
Copy link
Contributor

sap1ens commented Jul 22, 2020

Ah, thanks so much for fixing this 👍

@SparkQA
Copy link

SparkQA commented Jul 23, 2020

Test build #126361 has finished for PR 29194 at commit f9b8691.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan
Copy link
Contributor

thanks, merging to master!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants