[Internal] Update SDK to latest OpenAPI spec #1209
Conversation
tanmay-db
temporarily deployed
to
test-trigger-is
GitHub Actions
Inactive
tanmay-db
temporarily deployed
to
test-trigger-is
GitHub Actions
Inactive
tanmay-db
force-pushed
the
update-sdk-24april
branch
from
06de5cb to
7447a2d
Compare
tanmay-db
temporarily deployed
to
test-trigger-is
GitHub Actions
Inactive
tanmay-db
temporarily deployed
to
test-trigger-is
GitHub Actions
Inactive
NEXT_CHANGELOG.md
Outdated
| * [Breaking] Changed `CreateIndex` method for [w.VectorSearchIndexes](https://pkg.go.dev/github.com/databricks/databricks-sdk-go/service/vectorsearch#VectorSearchIndexesAPI) workspace-level service to return [vectorsearch.VectorIndex](https://pkg.go.dev/github.com/databricks/databricks-sdk-go/service/vectorsearch#VectorIndex). | ||
| * [Breaking] Changed `DeleteDataVectorIndex` method for [w.VectorSearchIndexes](https://pkg.go.dev/github.com/databricks/databricks-sdk-go/service/vectorsearch#VectorSearchIndexesAPI) workspace-level service . HTTP method/verb has changed. | ||
| * [Breaking] Changed `DeleteDataVectorIndex` method for [w.VectorSearchIndexes](https://pkg.go.dev/github.com/databricks/databricks-sdk-go/service/vectorsearch#VectorSearchIndexesAPI) workspace-level service with new required argument order. | ||
| * [Breaking] Changed [vectorsearch.List](https://pkg.go.dev/github.com/databricks/databricks-sdk-go/service/vectorsearch#List) to. |
There was a problem hiding this comment.
@hectorcast-db this message seems a bit off; is this known?
There was a problem hiding this comment.
I went through all the vector search changes, the list related changes are covered in below point regarding DataArray (ListValueList), not sure where this is coming from. Going to remove this line and follow up with release.
There was a problem hiding this comment.
No, but I can guess what it is.
I approved the following change:
A list of list of strings seems something that we did not considered (and cannot be expressed in protobuf). We should double check, but I would not fix this, since generated specs and new APIs won't have this.
tanmay-db
temporarily deployed
to
test-trigger-is
GitHub Actions
Inactive
|
If integration tests don't run automatically, an authorized user can run them manually by following the instructions below: Trigger: Inputs:
Checks will be approved automatically on success. |
tanmay-db
temporarily deployed
to
test-trigger-is
GitHub Actions
Inactive
What changes are proposed in this pull request?
Update SDK to latest OpenAPI spec
How is this tested?
N/A