databricks-sdk-py icon indicating copy to clipboard operation
databricks-sdk-py copied to clipboard
verified publisher icon databricks

Remove Unused Request Messages

Open parthban-db opened this issue 8 months ago • 3 comments

What changes are proposed in this pull request?

This PR removes Unused Request Messages. It is the result of resolving a bug in the code generator. Currently, many Request messages are generated but never used, as in the Python SDK, we are inlining all the elements of request messages in the method.

How is this tested?

Existing CI.

parthban-db avatar Jun 14 '25 21:06 parthban-db

Please ensure that the NEXT_CHANGELOG.md file is updated with any relevant changes. If this is not necessary for your PR, please include the following in your PR description: NO_CHANGELOG=true and rerun the job.

github-actions[bot] avatar Jun 14 '25 21:06 github-actions[bot]

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger: go/deco-tests-run/sdk-py

Inputs:

  • PR number: 991
  • Commit SHA: 837a48e727c9834080d4be927b953fdd468acb2f

Checks will be approved automatically on success.

github-actions[bot] avatar Jun 15 '25 16:06 github-actions[bot]

@renaudhartert-db @hectorcast-db, Do you think it is safe to merge this PR? Although these messages are never used in the SDKs, but still this removes a lot of unused messages.

parthban-db avatar Jun 15 '25 16:06 parthban-db