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

Grpc backend prototype #583

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Grpc backend prototype #583

wants to merge 20 commits into from

Conversation

flier
Copy link

@flier flier commented Jun 30, 2016

prototype resolve #582


This change is Reviewable

@@ -39,6 +39,9 @@ var (
clientParamTemplate *template.Template
clientResponseTemplate *template.Template
clientFacadeTemplate *template.Template
gRpcDefTemplate *template.Template
gRpcServerImplTemplate *template.Template
gRpcClientImplTemplate *template.Template

Choose a reason for hiding this comment

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

var gRpcClientImplTemplate should be gRPCClientImplTemplate

@casualjim
Copy link
Member

this looks great :)

I updated the build on master to only use 1.6 I think that might help with the way you've written the templates.

TLSCACertificate flags.Filename `long:"tls-ca-certificate" description:"the CA certificate to use for secure connections" env:"TLS_CA_CERTIFICATE"`
}

func NewClient() *Client {

Choose a reason for hiding this comment

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

exported function NewClient should have comment or be unexported

Choose a reason for hiding this comment

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

exported function NewClient should have comment or be unexported

@casualjim
Copy link
Member

are you done with this?

@flier
Copy link
Author

flier commented Jul 22, 2016

not yet, just moved focus to other emergency on work :S

Ivan Porto Carrero notifications@github.com于2016年7月22日周五 上午1:11写道:

are you done with this?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#583 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADwsgsGNkL5jS6PD-sb5u0jjQ1sDhJJks5qX6hdgaJpZM4JCApZ
.

@GlenDC
Copy link
Member

GlenDC commented Jan 11, 2017

Hi @flier how is your situation at the moment. Thank you very much for your work so far in this PR, would it be a lot of work to finish this of? What would you like to see happening with this PR? :) Thanks again, your time and effort is very well appreciated!

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.

Support gRPC backend
4 participants