Skip to content

Fix component name for aws dynamodb #5095

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

Merged
merged 2 commits into from
Aug 29, 2022

Conversation

sunzhaochang
Copy link
Contributor

@sunzhaochang sunzhaochang commented Aug 29, 2022

Signed-off-by: sunzhaochang zhchsun1992@gmail.com

Description

Issue reference

Please reference the issue this PR will close: #5089

Checklist

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

Signed-off-by: sunzhaochang <zhchsun1992@gmail.com>
@sunzhaochang sunzhaochang requested review from a team as code owners August 29, 2022 11:44
@codecov
Copy link

codecov bot commented Aug 29, 2022

Codecov Report

Merging #5095 (fdefefb) into master (5c42e8f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5095   +/-   ##
=======================================
  Coverage   65.45%   65.45%           
=======================================
  Files         113      113           
  Lines       12723    12723           
=======================================
  Hits         8328     8328           
  Misses       3811     3811           
  Partials      584      584           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@ItalyPaleAle ItalyPaleAle left a comment

Choose a reason for hiding this comment

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

Thanks for catching this

@artursouza artursouza merged commit 996cc62 into dapr:master Aug 29, 2022
@sunzhaochang sunzhaochang deleted the fix-state-dynamodb branch August 30, 2022 01:56
Taction added a commit to Taction/dapr that referenced this pull request Sep 5, 2022
* master: (134 commits)
  Record metric for service updates (dapr#5114)
  chore: remove duplicate word in comments (dapr#5099)
  Run e2e tests in previous version of K8s on AKS to see if it's breaking Windows tests (dapr#5104)
  add {namespace} option to set consumerID
  Add tracing passthrough for bindings (dapr#5049)
  Improve component init error logs and metrics (dapr#4975)
  Improvements to E2E test apps (dapr#5042)
  Fix component name for aws dynamodb (dapr#5095)
  Fix perf tests not building (dapr#5096)
  Fix missing components in Metadata API result. (dapr#5052)
  Fixed log message (dapr#5088)
  Disable IPFS binding for now (dapr#5092)
  optimize/concurrency: pubsub concurrency map writes in runtime unit test. (dapr#5086)
  Configuration API response to use dictionaries - based on PR dapr#4695 (dapr#5083)
  correct compile error
  correct checking for actors with same id
  fix type error in print
  fix typo
  add log when actor start time and end time is not expected
  Components auto-registration (dapr#5062)
  ...

Signed-off-by: zhangchao <zchao9100@gmail.com>

# Conflicts:
#	go.mod
#	pkg/grpc/api.go
#	pkg/grpc/api_test.go
#	pkg/http/api.go
#	pkg/runtime/runtime.go
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.

couldn't find state store state.aws.dynamodb/v1
3 participants