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

Ftr/triple #302

Merged
merged 17 commits into from
Nov 24, 2021
Merged

Ftr/triple #302

merged 17 commits into from
Nov 24, 2021

Conversation

LaurenceLiZhixin
Copy link
Contributor

@LaurenceLiZhixin LaurenceLiZhixin commented Nov 21, 2021

  • Ftr: http -> triple using dynamic API generation support
  • Transfer dubbo-go server's ip:port from API to Client.
  • Import formatted
  • Nacos registry support
  • HTTP random loadbalancer
  • go version upgrate to 1.15

@codecov-commenter
Copy link

codecov-commenter commented Nov 21, 2021

Codecov Report

Merging #302 (7790be6) into develop (d1930df) will decrease coverage by 0.22%.
The diff coverage is 14.28%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #302      +/-   ##
===========================================
- Coverage    33.93%   33.70%   -0.23%     
===========================================
  Files           61       61              
  Lines         3165     3198      +33     
===========================================
+ Hits          1074     1078       +4     
- Misses        1979     2008      +29     
  Partials       112      112              
Impacted Files Coverage Δ
...egistry/registry/zookeeper/application_listener.go 0.00% <ø> (ø)
...registry/zookeeper/application_service_listener.go 0.00% <0.00%> (ø)
...oregistry/registry/zookeeper/interface_listener.go 10.00% <ø> (ø)
...apter/dubboregistry/registry/zookeeper/registry.go 47.50% <ø> (ø)
...bboregistry/registry/zookeeper/service_listener.go 0.00% <0.00%> (ø)
pkg/client/dubbo/dubbo.go 20.00% <0.00%> (-0.28%) ⬇️
pkg/client/http/http.go 42.16% <ø> (ø)
pkg/config/api_config.go 3.13% <ø> (ø)
pkg/config/config_load.go 68.81% <ø> (ø)
pkg/filter/http/apiconfig/api/discovery_service.go 41.40% <0.00%> (-1.00%) ⬇️
... and 5 more

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 d1930df...7790be6. Read the comment docs.

Comment on lines 28 to 36
import (
fc "github.com/dubbogo/dubbo-go-pixiu-filter/pkg/api/config"

etcdv3 "github.com/dubbogo/gost/database/kv/etcd/v3"

perrors "github.com/pkg/errors"

"go.etcd.io/etcd/api/v3/mvccpb"
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Those blank line kind of strange, isn't it ?

Copy link
Member

Choose a reason for hiding this comment

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

Those blank line kind of strange, isn't it ?

+1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cai boss said follow the code formatting tool

Comment on lines 24 to 34
import (
"github.com/dubbogo/dubbo-go-pixiu-filter/pkg/api/config"
"github.com/dubbogo/dubbo-go-pixiu-filter/pkg/router"

"github.com/pkg/errors"
)

import (
"github.com/apache/dubbo-go-pixiu/pkg/model"
"github.com/dubbogo/dubbo-go-pixiu-filter/pkg/api/config"
"github.com/dubbogo/dubbo-go-pixiu-filter/pkg/router"
)

Copy link
Contributor

Choose a reason for hiding this comment

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

Imports here is not correct as well

@MasterKenway
Copy link
Contributor

MasterKenway commented Nov 21, 2021

@PhilYue As we discussed tonight, after server reflection is done, the github.com/mercari/grpc-http-proxy could be replaced

@LaurenceLiZhixin
Copy link
Contributor Author

LaurenceLiZhixin commented Nov 22, 2021

我今天搞下nacos 支持,以及负载均衡,还有上述问题。import 块的问题都是按照 import-formatter 来规范的,这块是否还有问题?

@LaurenceLiZhixin
Copy link
Contributor Author

LaurenceLiZhixin commented Nov 23, 2021

@MasterKenway I guess this pr is able to be merged.

@AlexStocks AlexStocks merged commit 425b380 into apache:develop Nov 24, 2021
@LaurenceLiZhixin LaurenceLiZhixin deleted the ftr/triple branch November 24, 2021 06:37
bobtthp pushed a commit to bobtthp/dubbo-go-pixiu that referenced this pull request Dec 12, 2022
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.

None yet

6 participants