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

Apache Pulsar: Add support for partitioned topics #3834

Merged

Conversation

michaeljmarshall
Copy link
Contributor

@michaeljmarshall michaeljmarshall commented Nov 10, 2022

Signed-off-by: Michael Marshall [email protected]

Apache Pulsar has topics and partitioned topics. In order to support scaling partitioned topics, the Scaler needs to call a different stats endpoint. The data returned by that endpoint is in the same format as the one returned by the non-partitioned stats endpoint.

Checklist

  • When introducing a new scaler, I agree with the scaling governance policy
  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Tests have been added
  • A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
  • A PR is opened to update the documentation on (repo) (if applicable)
  • Changelog has been updated and is aligned with our changelog requirements

Fixes #3833
Relates kedacore/keda-docs#975

@michaeljmarshall michaeljmarshall requested a review from a team as a code owner November 10, 2022 00:12
@zroubalik
Copy link
Member

@michaeljmarshall looking good, could you please rebase the branch?

Signed-off-by: Michael Marshall <[email protected]>
@zroubalik
Copy link
Member

zroubalik commented Nov 10, 2022

/run-e2e pulsar*
Update: You can check the progress here

@zroubalik
Copy link
Member

@michaeljmarshall e2e test failed.

Signed-off-by: Michael Marshall <[email protected]>
@michaeljmarshall
Copy link
Contributor Author

Thanks @zroubalik. I tested locally and think these changes should work now.

Copy link

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

LGTM
nice work!

@zroubalik
Copy link
Member

zroubalik commented Nov 11, 2022

/run-e2e pulsar*
Update: You can check the progress here

Copy link
Member

@zroubalik zroubalik 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 for the contribution!

@zroubalik zroubalik merged commit b175e17 into kedacore:main Nov 11, 2022
@michaeljmarshall michaeljmarshall deleted the support-pulsar-partitioned-topics branch November 11, 2022 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apache Pulsar: Support Partitioned Topics
3 participants