mobile: More JNI cleanup #34181
Merged
mobile: More JNI cleanup #34181
CI (Envoy) / Mobile/Release validation
succeeded
May 15, 2024 in 11m 29s
Mobile/Release validation (success)
Check has finished
Details
Check run finished (success ✔️)
The check run can be viewed here:
Mobile/Release validation (pr/34181/main@6ff01c7)
Check started by
Request (pr/34181/main@6ff01c7)
@fredyw 6ff01c7
#34181 merge
main@ece9170
mobile: More JNI cleanup
This PR cleans up the JNI code by removing the
Envoy::JNI::Exception
andEnvoy::JNI::String
and addsexceptionClear
in theJniHelper
so that everything behaves the same way without more going through many layers.Risk Level: low
Testing: unit tests
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: mobile
Environment
Request variables
Key | Value |
---|---|
ref | a09817c |
sha | 6ff01c7 |
pr | 34181 |
base-sha | ece9170 |
actor | @fredyw |
message | mobile: More JNI cleanup... |
started | 1715802238.161436 |
target-branch | main |
trusted | false |
Build image
Container image/s (as used in this CI run)
Key | Value |
---|---|
default | envoyproxy/envoy-build-ubuntu:f94a38f62220a2b017878b790b6ea98a0f6c5f9c |
mobile | envoyproxy/envoy-build-ubuntu:mobile-f94a38f62220a2b017878b790b6ea98a0f6c5f9c |
Version
Envoy version (as used in this CI run)
Key | Value |
---|---|
major | 1 |
minor | 31 |
patch | 0 |
dev | true |
Loading