Age | Commit message (Collapse) | Author |
|
|
|
* MethodDescriptor is lazy loaded, so protobuf loading only happens on demand. This also means tracing registration happens on demand.
* The names of the getters all being with `method`. This makes it harder for autocomplete to pick them up.
* A new field is used, which matches the getter name. Rather than make the new-getters reference the old-fields, make the old-fields reference the new getters. This makes removal of the old-fields a simple operation.
* The getters may not be inlineable, but thats an easy fix if it ends up being a problem. Not worth premature optimization (but is worth future work).
The expected timeline for this is adding this to the 1.8 cut, and deprecating the old-fields. They will be removed in 1.9.
|
|
|
|
|
|
This is a more favorable approach than #3467. Doing the registration
in MethodDescriptor should allow us to deregister in case the
generated stub and its MethodDescriptors are garbage-collected
routinely, e.g., if they are loaded by a separate ClassLoader.
|
|
|
|
|
|
If the option becomes popular, we can just remove the version.
|
|
This bump changelist is applied a bit late with respect to the
1.6.0 branch cut. Look at the 1.6.0 to see the source of truth of
where it was cut. Do not assume it is the commit that precedes
this one.
|
|
|
|
This is important for stable builds, as if the signature changes the old source
may no longer validate.
|
|
|
|
Also, update the authors.
|
|
|
|
|
|
`HealthStatusManager` should not expose server stub methods, because it is wrong to call server stub methods directly.
|
|
FutureStubs only support unary calls
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
This fix addresses https://github.com/grpc/grpc-java/issues/2689
|
|
|
|
The new plugin uses a newer version of animalsniffer, allows overriding
the animalsniffer version used, and has up-to-date handling. The
up-to-date handling cuts fully incremental parallel build times in half,
from 5.5s to 2.7s.
The previous plugin was supposed to be verifying tests. However, either
it wasn't verifying them or its verification was broken.
|
|
|
|
|
|
|
|
|
|
|
|
Now that the fix for #2444 is in, there's no need for the initial noop call.
|
|
Updates the proto reflection service to process immutable services once and
mutable services as-needed.
|
|
not necessary to synchronze every time calling
getServiceDescriptor(), if the descriptor has been created already;
go with the double-checked locking idom
|
|
So that it won't be overridden by Mockito when it creates a mock for
the server interface.
|
|
|
|
|
|
|
|
core: adds @Nullable Object getAttachedObject() to ServiceDescriptor
compiler: Plumbing necessary to access proto file descriptors via
the reflection service
|
|
|
|
Fixes #2086
|
|
|
|
partially resolving #1469
The added option for java_plugin `enable_deprecated` is `true` by default in `java_plugin.cpp`, so the generated code for `TestService.java` (`compiler/build.gradle` not setting this option) has all deprecated interfaces and static bindService method.
`./build.gradle` and `examples/build.gradle` set this option explicitly to `false`, so all the other generated classes do not have deprecated code.
Will set `enable_deprecated` to `false` by default in future PR when we are ready.
|
|
|
|
|
|
first step to address issue #1469:
- leave and deprecate interfaces in codegen
- introduce `ServiceImplBase`,
- `AbstractService` is deprecated and extends `ServiceImplBase`
- static `bindService()` is deprecated
|
|
This allows us to play with zero-copy and proto3 support for lite.
Unfortunately, it introduced some warnings, so deprecated warnings are
now ignored for benchmarks and interop-testing.
|
|
servers"
This reverts commit ef178304cb93ee660219e60b1a6553c1ace835de, which
itself was a revert.
|
|
This reverts commit 3df1446debe7e0da964b57306c2e42a74a8005b5.
The commit was adding to the difficulty of integration for testing. By
itself it isn't bad, so this is a temporary revert until the many other
commits are absorbed and then it will be reapplied.
This does have a manual edit for ClientCallsTest.
|