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

Add bindings rocketmq #815

Merged
merged 12 commits into from
May 13, 2021
Merged

Add bindings rocketmq #815

merged 12 commits into from
May 13, 2021

Conversation

cinience
Copy link
Contributor

Description

A binding that allows reading / writing from Alibaba Cloud RocketMQ.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #812

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

@cinience cinience changed the title Add bindings RocketMQ Add bindings rocketmq Apr 13, 2021
@pkedy pkedy self-assigned this Apr 21, 2021
@pkedy pkedy added this to the v1.2 milestone Apr 28, 2021
@pkedy
Copy link
Member

pkedy commented May 5, 2021

@cinience Like with the nacos binding, I'm making tweaks to use "the settings pattern".

Copy link
Member

@pkedy pkedy left a comment

Choose a reason for hiding this comment

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

@cinience In general I just made some comments that would make this component cleaner if you want to act on them. Also asked a question if using /internal/retry makes sense at all.

@cinience cinience requested a review from pkedy May 11, 2021 07:00
pkedy
pkedy previously approved these changes May 11, 2021
@pkedy
Copy link
Member

pkedy commented May 13, 2021

/cc: @artursouza

@codecov
Copy link

codecov bot commented May 13, 2021

Codecov Report

Merging #815 (935b26b) into master (5e547d7) will decrease coverage by 0.51%.
The diff coverage is 2.53%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #815      +/-   ##
==========================================
- Coverage   36.59%   36.07%   -0.52%     
==========================================
  Files         127      129       +2     
  Lines       10283    10441     +158     
==========================================
+ Hits         3763     3767       +4     
- Misses       6127     6280     +153     
- Partials      393      394       +1     
Impacted Files Coverage Δ
bindings/alicloud/rocketmq/rocketmq.go 0.00% <0.00%> (ø)
bindings/alicloud/rocketmq/settings.go 22.22% <22.22%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e547d7...935b26b. Read the comment docs.

@dapr-bot dapr-bot merged commit de417e4 into dapr:master May 13, 2021
Taction added a commit to Taction/components-contrib that referenced this pull request May 21, 2021
* master: (23 commits)
  Add bindings dingtalk webhook (dapr#817)
  add BulkGetResponse comments (dapr#845)
  Fix race in pulsar pubsub component (dapr#868)
  Add pubsub rocketmq (dapr#816)
  Add bindings rocketmq (dapr#815)
  Add bindings nacos (dapr#814)
  Sentinel middleware support (dapr#829)
  Added vaultToken support (dapr#796)
  Update Aerospike to v4.5.0
  Add function to generate CloudEvent for raw subscribed events. (dapr#864)
  Configurable retry (dapr#854)
  Update redis metadata comments to reflect the new maxRetry default from v8 (dapr#856)
  fix calc ipv6num use ipv4 slice (dapr#838)
  Do not block indefinitely on XGroupRead for redis subscriber (dapr#855)
  fix go mod
  fix lint
  fix add add a ctx context variable in the Redis struct. And implement `Close() error` function.
  fix add add a ctx context variable in the Redis struct. And implemet `Close() error` function
  fix use r.ctx instead of context.Background() in pubsub redis component
  Fixing go.mod
  ...

# Conflicts:
#	secretstores/hashicorp/vault/vault.go
#	secretstores/hashicorp/vault/vault_test.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Alibaba Cloud RocketMQ Binding
4 participants