Skip to content

chore(sources/dataplex)!: Renaming changes across all the files related to Dataplex Toolbox#2958

Open
anikasharma03 wants to merge 5 commits intogoogleapis:mainfrom
anikasharma03:rename-toolbox
Open

chore(sources/dataplex)!: Renaming changes across all the files related to Dataplex Toolbox#2958
anikasharma03 wants to merge 5 commits intogoogleapis:mainfrom
anikasharma03:rename-toolbox

Conversation

@anikasharma03
Copy link
Copy Markdown
Contributor

@anikasharma03 anikasharma03 commented Apr 6, 2026

Description

Perform a full rename across files and directories related to the Dataplex Toolbox . This wide-scale refactoring is needed to ensure consistency across the entire codebase and documentation.

Key Changes

File and Directory Renaming:

Moved and renamed all directories and files associated with the dataplex implementation to follow the new naming convention (e.g., in internal/tools/, internal/sources/, and tests/).

Documentation Updates:

Updated references in README.md, integration guides, and other documentation files to reflect the new component naming.

Code Refactoring:

Updated internal references, variable names, and configuration keys to match the new naming standard while maintaining functional compatibility.

Test Updates:

Renamed test files and updated test cases to ensure they correctly validate the renamed components.

PR Checklist

Thank you for opening a Pull Request! Before submitting your PR, there are a
few things you can do to make sure it goes smoothly:

  • Make sure you reviewed
    CONTRIBUTING.md
  • Make sure to open an issue as a
    bug/issue
    before writing your code! That way we can discuss the change, evaluate
    designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)
  • Make sure to add ! if this involve a breaking change

🛠️ Fixes #2956

@anikasharma03 anikasharma03 requested review from a team as code owners April 6, 2026 10:13
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request renames "Dataplex" to "Knowledge Catalog" across the entire repository, including CI configurations, documentation, internal Go packages, and integration tests. The feedback provided focuses on maintaining consistent naming conventions in the documentation and fixing a minor typo in the description of the search aspect types tool.

@Yuan325 Yuan325 changed the title chore(renaming): Renaming changes across all the files related to Dataplex Toolbox chore(sources/dataplex)!: Renaming changes across all the files related to Dataplex Toolbox Apr 6, 2026
Copy link
Copy Markdown
Contributor

@Yuan325 Yuan325 left a comment

Choose a reason for hiding this comment

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

Might be good to update the project env var as well so users are not confused (having to use type as "knowledge-catalog" but env var with the "dataplex" name). Please update it across the docs~

we're missing updates on these files:

  • .github/CODEOWNERS - folder name (we can keep the mdb group as is for now)
  • .github/blunderbuss.yml - update the label name (can keep the group as is)
  • .github/labels.yaml

Will dataplex be updating the official cloud docs page link? e.g. https://cloud.google.com/dataplex? If yes, we might want to update that as well. seems like some of bigquery doc that points towards that too.

Thank you!

anikasharma03 added a commit to anikasharma03/genai-toolbox that referenced this pull request Apr 7, 2026
anikasharma03 added a commit to anikasharma03/genai-toolbox that referenced this pull request Apr 7, 2026
@Yuan325 Yuan325 added release candidate Use label to signal PR should be included in the next release. do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Apr 7, 2026
@Yuan325
Copy link
Copy Markdown
Contributor

Yuan325 commented Apr 9, 2026

/gcbrun

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

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing. release candidate Use label to signal PR should be included in the next release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Renaming across files and directories related to Dataplex Toolbox

2 participants