Age | Commit message (Collapse) | Author |
|
Updates several targets within grpc-grpc-java to allow for Android and Host targets to allow the building of a gRPC client library that uess OkHttp and Protobuf Lite. This provides a lightweight client library for gRPC clients that works on both Android and Host since it uses lite protos rather than full protos.
Test: m grpc-java-okhttp-client-lite
Test: mm
Test: Manual testing using client library on Android.
Change-Id: Ib9dc14a44856914b38662c01905337f98bd8d108
|
|
Test: make grpc-java
Bug: None
Change-Id: I55dc3d537c87c7ffc80b23cdec48fa830e9c87db
|
|
Add basic build files for the grpc-java modules.
Still need to be added: netty, protobuf
Test: make
Bug: None
Change-Id: I9248dd23c0ec24ec0f9a15aa6f6d826b7b90d617
|
|
|
|
Resolves #3605
|
|
The docs refer to the wrong places where this feature is ignored.
|
|
* Reflowed some method parameters to be on the same line, else one
parameter per line
* Used `@link` where appropriate
* Made some parameters non-final where it had no effect
* Renamed some parameters to be consistent
|
|
|
|
Additionally, make Status*Exception.getTrailers() non experimental
|
|
The trick wasn't necessary, as any java_library can mark itself as
neverlink which applies to its dependencies.
|
|
|
|
DoNotMock was removed from error_prone_annotations in 2.1.3, because
there was no enforcement mechanism (which is in google/error-prone#572).
Guava and Trust also depend on error_prone_annotations and are beginning
to use newer versions, so our usage of DoNotMock is causing diamond
dependency problems. This allows us to update to 2.2.0.
The annotations were useful internally; we're solving that in cl/205294089.
|
|
|
|
|
|
This annotation will enable Java APT to generate code.
Addresses part of #3173.
|
|
|
|
This PR adds an automatic gradle format checker and reformats all the *.gradle files. After this, new changes to *.gradle files will fail to build if not in good format, just like checkStyle failure.
|
|
|
|
|
|
|
|
|
|
Fixes: #3633.
Test Plan:
On most recent Bazel version run:
$ bazel --host_javabase=/usr/lib64/jvm/java-9-openjdk \
build --javacopt='--release 9' \
--java_toolchain=@bazel_tools//tools/jdk:toolchain_jdk9 \
examples:helloworld_java_grpc
|
|
The fixes could have subtle side-effects, but I did take care when making them.
|
|
|
|
This mainly copies documentation from other places, like StreamObserver
and ClientCall, but does fix some missing important threading notes.
Fixes #3413
|
|
Additionally:
* Prefer RuntimeException from status
* remove redundant "static" on interfaces
* use checkState to shorten code
|
|
|
|
|
|
@com_google_errorprone_error_prone_annotations//jar (#3584)
|
|
|
|
This commit aligns the naming of the Bazel Maven jars with the names
used by Bazel's migration-tooling project:
https://github.com/bazelbuild/migration-tooling
Unfortunately, we can't fix @com_google_protobuf_java because it's
required by Bazel itself.
Fixes #3328
|
|
As discussed in #1914, we need CallCredentials and MoreCallCredentials
to be stable, but there's less of a strong need for the contents of
CallCredentials to be stable. We're willing to commit to the name,
without needing to commit to the plumbing.
|
|
|
|
|
|
|
|
This has a number of small benefits. First, it makes stack traces
easier to read. For example:
Old class names:
```
ServerCalls$1$1.class
ServerCalls$1.class
ServerCalls$2$1.class
ServerCalls$2.class
```
New class names:
```
ServerCalls$StreamingServerCallHandler.class
ServerCalls$StreamingServerCallHandler$StreamingServerCallListener.class
ServerCalls$UnaryServerCallHandler.class
ServerCalls$UnaryServerCallHandler$UnaryServerCallListener.class
```
This is much easier to read quickly, espcially if line numbers don't
match between HEAD and the code that prints the stack trace.
Another benefit of this is that it drops a class file from the jar
(`EmptyServerCallListener`).
Lastly, it makes it easier in the future to test, since the specific
class can be referenced from a test. Traditionally this class
hasn't been easy to test.
|
|
|
|
|
|
|
|
This is important for stable builds, as if the signature changes the old source
may no longer validate.
|
|
one step for #3105
|
|
Bazel third party dependencies are specified in repositories.bzl which
gives the consumer the ability to opt-out of any dependencies they use
directly in their own project.
Fixes #2756
|
|
The current check in ServerCallImpl is theoretically unsafe (#3059). Move that check into the stub, and expand the unit tests to cover other interesting edge cases on the server side:
client sends one, but zero requests received at onHalfClose
client sends one, but > 1 requests received at onHalfClose
server sends one, but zero responses sent at onComplete
server sends one, but > 1 responses sent via onNext
fixes #2243
fixes #3059
|
|
|
|
Also, update the authors.
|
|
|
|
Initated package javadoc for
* `io.grpc.stub`,
* `io.grpc.auth`,
* `io.grpc.util`,
* `io.grpc.protobuf`,
* `io.grpc.protobuf.lite`,
* `io.grpc.protobuf.nano`
|
|
All that the `call` field was used for was to call `request` and nothing else. The `request` can be done by the `adapter` field, so the `call` field is redundant.
|
|
|
|
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.
|