Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Kafka Connector]Fix BadRequest exception when using serverless database account #43125

Merged
merged 6 commits into from
Jan 3, 2025

Conversation

xinlian12
Copy link
Member

Issue

When customer using Event stream in Fabric with Serverless CosmosDB database account, the connector failed with:
Setting offer throughput or autopilot on container is not supported for serverless accounts

Root cause

When using event stream in Fabric with kafka connector, for azure.cosmos.source.metadata.storage.type will be default to Cosmos. Internally if the metadata container does not exists, kafka connector will try to create one with autoscale 4000 RU. However creating a container with throughput(neither manual nor autoscale) on serverless account is not supported, hence the error

Fix

When Kafka connector try to create the metadata container, if it fails with 400, then will retry to create the metadata container without throughput configured

Tests

Manually test with serverless database account

@xinlian12 xinlian12 requested review from kirankumarkolli and a team as code owners November 27, 2024 23:10
@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

Copy link
Member

@tvaron3 tvaron3 left a comment

Choose a reason for hiding this comment

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

LGTM

@tvaron3
Copy link
Member

tvaron3 commented Jan 2, 2025

/azp run java - cosmos - ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tvaron3
Copy link
Member

tvaron3 commented Jan 2, 2025

/azp run java - cosmos - kafka

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@FabianMeiswinkel FabianMeiswinkel left a comment

Choose a reason for hiding this comment

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

LGTM - Thanks

@tvaron3
Copy link
Member

tvaron3 commented Jan 3, 2025

/check-enforcer override

@tvaron3 tvaron3 enabled auto-merge (squash) January 3, 2025 19:08
@tvaron3 tvaron3 merged commit 765874a into Azure:main Jan 3, 2025
53 checks passed
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.

5 participants