Age | Commit message (Collapse) | Author |
|
am: 7c9e109202
Change-Id: I6b8df3261cf63dbc04b5cd7db7ee2be20c56ad56
|
|
Bug: 126125118
Change-Id: Idca5084114ed8679246d5fb83c9f1433be1a5eb2
|
|
Bug: 120848293
Change-Id: I68b32ed3587b817251f466d9306ed1119e453f7f
|
|
am: 12519662e1
Change-Id: If1e1e3cab4035f8d2eabdcc2fb201a363c2d8bda
|
|
am: d7556eb784
Change-Id: I6e0d55619a526e95d2e6cdd0792a8737b7ff2f59
|
|
am: 44482cd5fe
Change-Id: Ic8bc6f4f714b53b796134ce32de884eb30445b6e
|
|
Bug: 33166666
Test: gerrit uploader
Change-Id: Ic14812359b38f4a6c70b1841ca8229fc41db77b5
|
|
am: 701a5d1190
Change-Id: I94022b55658fb30ee6b64a2c66fce4cbf5ff9c6c
|
|
am: 54952e8341
Change-Id: I400d0cd48a8b702551cf2f125f2062e07b7d4334
|
|
am: 4d84fe10a4
Change-Id: Ib153a300dd96da04a2a4069d52de7a10d5f1ab02
|
|
Test: make grpc-java
Bug: None
Change-Id: I55dc3d537c87c7ffc80b23cdec48fa830e9c87db
|
|
am: 9b8fa11637
Change-Id: I0e66dc03939af25f17225de32cd6bdaa2fc9fe2c
|
|
am: d3eaec606d
Change-Id: Idcb8e53160ed9721118423dd2fad1d9555c6bf8c
|
|
am: 76deef7817
Change-Id: I8c88a66c442b802afafb2694257dbfa705f991d8
|
|
Test: make grpc-java
Bug: None
Change-Id: Ibf9d369637f8cd780c42936c11b4bd67541f1a93
|
|
am: 12d4d8bcf8
Change-Id: Ib6146c2fa44b091ce1c6c5ea168a156c4780a449
|
|
am: 78b77df0ba
Change-Id: I7f7ddbfe7739cb1196fa88ba0efead9b5dcb48bb
|
|
am: af14f34438
Change-Id: Ic06f167ff2add3cd4629a2cb4bc4c7d7c39fda15
|
|
Add basic build files for the grpc-java modules.
Still need to be added: netty, protobuf
Test: make
Bug: None
Change-Id: I9248dd23c0ec24ec0f9a15aa6f6d826b7b90d617
|
|
am: c19c7d7025
am: e52cd2ac97
Change-Id: Ifd70d53ca36e401d096ea7c7f72cb636410c18b3
|
|
am: c19c7d7025
Change-Id: Ia3a4bec9428850f4d10da5a1e1fd727f91fa01b4
|
|
am: ed6077b99e
Change-Id: Ibe69e20bf92a86f44798df6741885b19fcfa9c50
|
|
Change-Id: I3bfaf72376866c93610dc6f8807edaeadc44ad5f
|
|
This simplifies the construction process, as we have fewer "interesting"
phases to weed through.
|
|
|
|
|
|
* Remove redundant SubchannelPicker refreshes in RoundRobinLoadBalancer
- Ensure active subchannel list and round-robin index is only
regenerated/refreshed when it changes
- Make it so that Subchannels exist in subchannels map iff their state
!= SHUTDOWN
- Add EmptyPicker class since logic for this case is disjoint from the
non-empty case
* remove explicit initialization of boolean ready field
per @carl-mastrangelo's review comment
* minor restructuring to make logic clearer; more explanatory comments
* move some checks inside updateBalancingState method for clarity
* store current state and picker in RRLB, only update when new one is diff
* some more simplification/refactoring; improve test coverage
- remove now redundant check in handleSubchannelState
- collapse getAggregatedState() and getAggregatedError() into
handleBalancingState()
- have both pickers extend new RoundRobinPicker, move
areEquivalentPickers() logic into RoundRobinPicker.isEquivalentTo()
- extend unit tests to cover some additional cases
* Address latest review comments from @zhangkun83
- Use explicit check for non-empty list instead of assert
- Change EmptyPicker.status to be non-nullable
- Further test coverage improvement including explicit picker comparison
tests
* use EMPTY_OK instead of Status.OK for initial empty picker
|
|
|
|
This simplifies the construction paradigm and leads to the eventual
removal of TransportCreationParamsFilterFactory. The eventual end goal
is to be able to shut down ProtocolNegotiators as is necessary for ALTS.
The only reason the initialization was delayed was for 'authority', so
we now plumb the authority through GrpcHttp2ConnectionHandler.
|
|
|
|
|
|
The CSRF protection has been moved to the proxy implementation. This
is cleaner since the gRPC service should not need to know about
grpc-web.
|
|
It appears to be stable now. Ran for 1000s of times. I do see some
general flakiness in TransportTest, but it applies to the tests in
general and isn't specific to this one test. It is:
```
org.mockito.exceptions.verification.WantedButNotInvoked:
Wanted but not invoked:
listener.transportReady();
-> at io.grpc.internal.testing.AbstractTransportTest.startTransport(AbstractTransportTest.java:1815)
Actually, there were zero interactions with this mock.
```
This flake is not seen often because it occurs less frequently when
running all the tests (~.1% vs 1%). One of the early tests must warm
something up to make it less likely.
|
|
This attempts to fix a flake seen exactly once with the
currently-disabled OkHttpTransportTest.flowControlPushBack:
```
java.lang.AssertionError
at org.junit.Assert.fail(Assert.java:86)
at org.junit.Assert.assertTrue(Assert.java:41)
at org.junit.Assert.assertTrue(Assert.java:52)
at io.grpc.internal.testing.AbstractTransportTest.flowControlPushBack(AbstractTransportTest.java:1300)
```
That was a failure for assertTrue(serverStream.isReady()), because the
awaitOnReady was finding the previous invocation of onReady. We now
track how many times it has been called. This was a bug introduced in
a8db154702 but wouldn't have been noticed since the in-process transport
is deterministic.
|
|
The exact census span behavior wrt gRPC is not yet defined, so let's
punt on tight integration.
It may be fine to log grpc-trace-bin on server side because it is a
key visible to the application.
|
|
The addresses from the string dump of the LoadBalanceResponse proto is
in binary format and not human-readable. We will log the
BackendAddressGroups when using a new list from the balancer. The
original logging of LoadBalanceResponse is downgraded to FINER level.
|
|
DnsNameResolver caches refresh using java security property networkaddress.cache.ttl.
Resolves #4745
|
|
interop-testing: add very_large_request case to test client
|
|
okhttp: setting acks back after apply settings before sending any data as a result of the change.
Resolves #4809 also, make #4816 the not flaky.
|
|
This is a safer way to hide the classes, because they will not appear
in public targets for some build configurations.
|
|
deflake kokoro." (#4821)
Revert "examples: Add google mirrored maven central to examples pom.xml to deflake kokoro. (#4813)"
This reverts commit 54f76dfeafe8ac9cf908ceb49ab87bd8348d2dd9.
|
|
This is an API used to coordinate across packages and must live in
`io.grpc`.
Prepending `Internal` makes it easier to detect and hide this class
from public visibility when using certain build tools.
fixes #4796
|
|
|
|
|
|
|
|
kokoro. (#4813)
Add google mirrored maven central to examples pom.xml to deflake kokoro.
|
|
|
|
Resolves #3605
|
|
alts: convert handshaker service channel to SharedResourceHolder
|
|
alts: if ALTS is not running on GCP, fails call rather than RuntimeException
|