1705 Commits

Author SHA1 Message Date
e8bd16ecfa Merge pull request #766 from iamqizhao/master
Fixed some bugs
2016-07-18 11:01:24 -07:00
1ea428008b add the missing close(s.done) 2016-07-15 16:55:51 -07:00
55b7bd0424 add the missing close(s.done) 2016-07-15 16:39:31 -07:00
07bf108290 fix Invoke to accommodate the change 2016-07-15 16:02:21 -07:00
0bfa80150a add a TODO 2016-07-15 15:43:42 -07:00
df0a2ae779 finish trace with the right status 2016-07-15 15:06:00 -07:00
c9f0b89a96 clean up underlying footprint when Stream.Done() is read 2016-07-15 14:53:49 -07:00
0223eb7067 remove remaining debugging logs 2016-07-15 11:21:50 -07:00
1391d99481 clear dead code 2016-07-15 11:18:18 -07:00
ec7eacfcc4 some fix 2016-07-15 11:02:03 -07:00
69e799fc76 fix a comment 2016-07-14 18:41:53 -07:00
0f1aeede97 do not cancel stream in http2_client.go, use done chanel instead 2016-07-14 18:38:43 -07:00
7ce43067bd more comments 2016-07-14 17:17:46 -07:00
e558cf17ff fix comments 2016-07-14 17:13:20 -07:00
7b7cb1ae8c fix bugs and flaky tests 2016-07-14 17:12:11 -07:00
e5011a450e Merge branch 'master' of https://github.com/grpc/grpc-go 2016-07-14 15:10:08 -07:00
02fca896ff Merge pull request #756 from heyitsanthony/fix-cred-clobber
credentials: don't overwrite ServerName in given config
2016-07-14 15:09:51 -07:00
9a97937ed3 credentials: don't overwrite ServerName in given config
The first endpoint will set the ServerName which will then be used by
the second endpoint, causing the transport to reject the second endpoint
since the server cert won't match the server name.
2016-07-14 11:17:40 -07:00
7428593149 Merge pull request #759 from menghanl/streams_in_serviceinfo_comments
Fix comments
2016-07-14 10:55:28 -07:00
108308b1d3 Merge pull request #762 from LK4D4/prevent_double_close
server: close only non-closed listeners
2016-07-14 10:55:03 -07:00
311bcfb8e8 fix some comments 2016-07-14 10:49:31 -07:00
99c9776beb add more comments 2016-07-13 18:18:35 -07:00
48068bbe62 remove debugging info 2016-07-13 18:13:28 -07:00
6205cb25ab fix some bugs 2016-07-13 18:05:02 -07:00
47de9c3564 server: close only non-closed listeners
The listener can be closed twice: in Close and in Serve. It might lead to
pretty bad things, for example, https://golang.org/src/net/unixsock_posix.go#L340
can delete a file which created by another listener.

Signed-off-by: Alexander Morozov <lk4d4math@gmail.com>
2016-07-12 21:16:45 -07:00
f53faa647d minor fix 2016-07-12 19:26:12 -07:00
2e7aa9a2b1 Error out the send call for a client streaming rpc if the server has returned an error. 2016-07-12 19:24:33 -07:00
42a2879989 Modify comments 2016-07-11 16:25:52 -07:00
13edeeffde Merge pull request #749 from menghanl/streams_in_serviceinfo
Split methods and streams in service info
2016-07-11 16:17:12 -07:00
bc8885608f Merge unary RPC info and streaming RPC info 2016-07-11 13:09:50 -07:00
0ea9f970c0 Add StreamInfo for streaming types 2016-07-11 12:09:05 -07:00
a518fa911d Split methods and streams in service info 2016-07-11 12:09:05 -07:00
565b602168 Merge pull request #755 from menghanl/errorf_pointer
Make grpc.Errorf return struct pointer
2016-07-11 10:11:51 -07:00
1b5902d709 Fix review comments 2016-07-08 14:15:16 -07:00
ffdfb592e8 Fix type assertion error after rebase 2016-07-08 10:35:38 -07:00
562a45da66 add test to verify that errors defined with same code and description are not considered equal via the error interface 2016-07-08 10:28:36 -07:00
d2125eeb91 fix error comparison 2016-07-08 10:28:36 -07:00
55fbdb44ea compare error strings, not interfaces 2016-07-08 10:28:36 -07:00
3802318f46 Make Errorf return pointer to rpcError 2016-07-08 10:28:36 -07:00
461dac9997 Merge pull request #738 from iamqizhao/master
[Pre-1.0 Semantics Change] Support Fail-fast and make it the default setup
2016-07-08 10:28:17 -07:00
9a76919800 Merge branch 'master' of https://github.com/grpc/grpc-go 2016-07-06 17:18:01 -07:00
4f14d195bf Merge pull request #753 from iamqizhao/fix
Add the missing address metadata
2016-07-06 17:17:39 -07:00
f492622da4 rebase 2016-07-06 14:16:27 -07:00
daeb9cc0f2 Merge pull request #752 from tamird/dupe-balancer
remove duplicate balancer field
2016-07-06 14:09:59 -07:00
5688542ad1 Add the missing address metadata 2016-07-06 11:40:57 -07:00
ebd9633b55 remove duplicate balancer field 2016-07-05 14:52:25 -04:00
605ce23104 Merge branch 'master' of https://github.com/grpc/grpc-go 2016-06-30 15:11:56 -07:00
a0ff1e78a9 Merge pull request #743 from menghanl/go_vet
Fix go vet
2016-06-30 15:10:19 -07:00
c477cb3b14 fix the flaky test 2016-06-30 14:26:31 -07:00
be59d023f2 refactor error handling a bit 2016-06-29 15:21:44 -07:00