You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@eventmesh.apache.org by GitBox <gi...@apache.org> on 2022/06/06 10:13:55 UTC
[GitHub] [incubator-eventmesh] walleliu1016 opened a new pull request, #911: Go SDK for EventMesh gRPC
walleliu1016 opened a new pull request, #911:
URL: https://github.com/apache/incubator-eventmesh/pull/911
merge the golang sdk to master branch
https://github.com/apache/incubator-eventmesh/issues/86 add golang sdk GRPC
- support webhook subscription and stream subscription
- support async and RR message
- only support eventmeshmessage now
- add multiple loadbalancer strategy, support multiple connection later
- add logger with zap.Logger
- support uuid and sonyflake to generate unique ID
- suport atomic.Seqence ID
TODO
- [ ] support cloudevent message
- [ ] support multiple eventmesh server connection
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org
[GitHub] [incubator-eventmesh] qqeasonchen commented on pull request #911: Go SDK for EventMesh gRPC
Posted by GitBox <gi...@apache.org>.
qqeasonchen commented on PR #911:
URL: https://github.com/apache/incubator-eventmesh/pull/911#issuecomment-1147339244
WARNING Failed to determine the comment style of file: eventmesh-sdk-go/go.sum
WARNING Failed to create review comment, fallback to a plain comment: POST https://api.github.com/repos/apache/incubator-eventmesh/pulls/911/reviews: 403 Resource not accessible by integration []
WARNING Failed to create review comments POST https://api.github.com/repos/apache/incubator-eventmesh/pulls/911/reviews: 403 Resource not accessible by integration []
ERROR the following files don't have a valid license header:
eventmesh-sdk-go/common/protocol/http/body/client/heartbeat_request_body.go
eventmesh-sdk-go/common/protocol/http/body/client/subscribe_request_body.go
eventmesh-sdk-go/common/seq/num.go
eventmesh-sdk-go/go.mod
eventmesh-sdk-go/go.sum
eventmesh-sdk-go/grpc/client_test.go
eventmesh-sdk-go/grpc/conf/config_test.go
eventmesh-sdk-go/grpc/heartbeat_test.go
eventmesh-sdk-go/grpc/loadbalancer/loadbalancer_test.go
eventmesh-sdk-go/grpc/loadbalancer/rule_iphash_test.go
eventmesh-sdk-go/grpc/loadbalancer/rule_random_test.go
eventmesh-sdk-go/grpc/loadbalancer/rule_roundrobin_test.go
eventmesh-sdk-go/grpc/loadbalancer/status_server_test.go
eventmesh-sdk-go/grpc/proto/eventmesh-client_grpc.pb.go
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org
[GitHub] [incubator-eventmesh] codecov[bot] commented on pull request #911: Go SDK for EventMesh gRPC
Posted by GitBox <gi...@apache.org>.
codecov[bot] commented on PR #911:
URL: https://github.com/apache/incubator-eventmesh/pull/911#issuecomment-1147302754
# [Codecov](https://codecov.io/gh/apache/incubator-eventmesh/pull/911?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#911](https://codecov.io/gh/apache/incubator-eventmesh/pull/911?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (5b85819) into [master](https://codecov.io/gh/apache/incubator-eventmesh/commit/755df48f541d47020f47df3bbd3f928c2bf5a447?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (755df48) will **decrease** coverage by `0.03%`.
> The diff coverage is `n/a`.
```diff
@@ Coverage Diff @@
## master #911 +/- ##
===========================================
- Coverage 8.91% 8.88% -0.04%
+ Complexity 529 528 -1
===========================================
Files 347 347
Lines 21773 21769 -4
Branches 2426 2422 -4
===========================================
- Hits 1941 1934 -7
- Misses 19682 19684 +2
- Partials 150 151 +1
```
| [Impacted Files](https://codecov.io/gh/apache/incubator-eventmesh/pull/911?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
|---|---|---|
| [...ava/org/apache/eventmesh/common/utils/IPUtils.java](https://codecov.io/gh/apache/incubator-eventmesh/pull/911/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZXZlbnRtZXNoLWNvbW1vbi9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZXZlbnRtZXNoL2NvbW1vbi91dGlscy9JUFV0aWxzLmphdmE=) | `33.33% <0.00%> (-6.49%)` | :arrow_down: |
| [...rg/apache/eventmesh/runtime/trace/LogExporter.java](https://codecov.io/gh/apache/incubator-eventmesh/pull/911/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZXZlbnRtZXNoLXJ1bnRpbWUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2V2ZW50bWVzaC9ydW50aW1lL3RyYWNlL0xvZ0V4cG9ydGVyLmphdmE=) | `0.00% <0.00%> (ø)` | |
| [...l/tcp/client/recommend/EventMeshRecommendImpl.java](https://codecov.io/gh/apache/incubator-eventmesh/pull/911/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZXZlbnRtZXNoLXJ1bnRpbWUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2V2ZW50bWVzaC9ydW50aW1lL2NvcmUvcHJvdG9jb2wvdGNwL2NsaWVudC9yZWNvbW1lbmQvRXZlbnRNZXNoUmVjb21tZW5kSW1wbC5qYXZh) | `0.00% <0.00%> (ø)` | |
| [.../tcp/client/session/push/DownStreamMsgContext.java](https://codecov.io/gh/apache/incubator-eventmesh/pull/911/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZXZlbnRtZXNoLXJ1bnRpbWUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2V2ZW50bWVzaC9ydW50aW1lL2NvcmUvcHJvdG9jb2wvdGNwL2NsaWVudC9zZXNzaW9uL3B1c2gvRG93blN0cmVhbU1zZ0NvbnRleHQuamF2YQ==) | `0.00% <0.00%> (ø)` | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-eventmesh/pull/911?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/incubator-eventmesh/pull/911?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [755df48...5b85819](https://codecov.io/gh/apache/incubator-eventmesh/pull/911?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org
[GitHub] [incubator-eventmesh] qqeasonchen commented on pull request #911: Go SDK for EventMesh gRPC
Posted by GitBox <gi...@apache.org>.
qqeasonchen commented on PR #911:
URL: https://github.com/apache/incubator-eventmesh/pull/911#issuecomment-1147292206
@walleliu1016 please take a look about license check fail
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org
[GitHub] [incubator-eventmesh] qqeasonchen merged pull request #911: Go SDK for EventMesh gRPC
Posted by GitBox <gi...@apache.org>.
qqeasonchen merged PR #911:
URL: https://github.com/apache/incubator-eventmesh/pull/911
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org