Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

grpc 1.2.0 having issues with netty #579

Closed
travisci-hrijp opened this issue Mar 24, 2017 · 8 comments
Closed

grpc 1.2.0 having issues with netty #579

travisci-hrijp opened this issue Mar 24, 2017 · 8 comments
Assignees
Labels
type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@travisci-hrijp
Copy link

travisci-hrijp commented Mar 24, 2017

userpc:grpc username$ bin/speech-sample-async.sh --host=speech.googleapis.com --port=443 \
> --uri=resources/audio.raw --sampling=16000
Exception in thread "main" java.lang.NoSuchMethodError: io.grpc.netty.NettyChannelBuilder.maxInboundMessageSize()I
	at io.grpc.netty.NettyChannelBuilder.buildTransportFactory(NettyChannelBuilder.java:274)
	at io.grpc.internal.AbstractManagedChannelImplBuilder.build(AbstractManagedChannelImplBuilder.java:239)
	at io.grpc.internal.AbstractManagedChannelImplBuilder.build(AbstractManagedChannelImplBuilder.java:69)
	at com.examples.cloud.speech.AsyncRecognizeClient.createChannel(AsyncRecognizeClient.java:90)
	at com.examples.cloud.speech.AsyncRecognizeClient.main(AsyncRecognizeClient.java:229)

In which file did you encounter the issue?

git/java-docs-samples/speech/grpc/pom.xml

Did you change the file? If so, how?

userpc:grpc username$ git diff
diff --git a/speech/grpc/pom.xml b/speech/grpc/pom.xml
index 21b508a..70c1507 100644
--- a/speech/grpc/pom.xml
+++ b/speech/grpc/pom.xml
@@ -49,7 +49,7 @@ limitations under the License.
   <properties>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <codehaus-versions-maven-plugin-version>2.3</codehaus-versions-maven-plugin-version>
-    <grpc-version>1.2.0</grpc-version> <!-- 1.1.2 doesn't work - DO NOT UPGRADE -->
+    <grpc-version>1.0.3</grpc-version> <!-- 1.1.2 doesn't work - DO NOT UPGRADE -->
     <maven-compiler-plugin-version>3.6.0</maven-compiler-plugin-version>
     <xolstice-protobuf-maven-plugin-version>0.5.0</xolstice-protobuf-maven-plugin-version>

Describe the issue

There are issues finding Netty in the latest grpc version bump.

@travisci-hrijp travisci-hrijp changed the title Roll back GRPC version in pom.xml grpc 1.2.0 having issues with netty Mar 24, 2017
@lesv
Copy link
Contributor

lesv commented Mar 24, 2017

I've rolled back to 1.0.3 for now.

@lesv lesv added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. SPEECH labels Mar 24, 2017
@lesv
Copy link
Contributor

lesv commented Apr 22, 2017

@chrisgarry Is this still a problem for you?

@travisci-hrijp
Copy link
Author

@lesv nope my local version change resolved it. I haven't pulled in any recent changes to verify if you guys resolved.

@atejandro
Copy link

atejandro commented May 31, 2017

I have the same problem in a project I'm working on. This time however, it happens when I change from version 1.2.0 to version 1.3.0.

org.jini.rio.core.JSBInstantiationException: java.lang.NoSuchMethodError: io.grpc.netty.NettyChannelBuilder.maxInboundMessageSize()I at io.grpc.netty.NettyChannelBuilder.buildTransportFactory(NettyChannelBuilder.java:333) at io.grpc.internal.AbstractManagedChannelImplBuilder.build(AbstractManagedChannelImplBuilder.java:239) at io.grpc.internal.AbstractManagedChannelImplBuilder.build(AbstractManagedChannelImplBuilder.java:69) at com.pagosonline.maf.channel.GrpcManagedChannel.<init>(GrpcManagedChannel.java:32) at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) at

Is rolling back to 1.2.0 the only solution? Are you still having the same problem with this new release?

@travisci-hrijp
Copy link
Author

@atejandro This is the second version bump of GRPC that has had this issue. I wonder if the same change that broke this last time was reintroduced, or it wasn't patched in a sustainable way. I am no longer using this so I'm not sure. Good luck!

@lesv
Copy link
Contributor

lesv commented May 31, 2017

@gguuss PTAL

@gguuss
Copy link
Contributor

gguuss commented May 31, 2017

I just was looking at the java/speech samples for forever trying to figure this out. I believe we removed the GRPC samples in favor of those using Cloud Client libraries. Can you point me to the code you're running?

Do the samples in this folder resolve your issue? I cannot reproduce the issue.

@lesv
Copy link
Contributor

lesv commented Mar 6, 2018

If you are still having issues look at:
https://github.com/GoogleCloudPlatform/google-cloud-java/blob/master/TROUBLESHOOTING.md
Should be obsolete.

@lesv lesv closed this as completed Mar 6, 2018
minherz pushed a commit that referenced this issue Nov 9, 2022
minherz pushed a commit that referenced this issue Nov 10, 2022
minherz pushed a commit that referenced this issue Nov 10, 2022
charlieyu1996 pushed a commit that referenced this issue Nov 10, 2022
…579)

[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:libraries-bom](https://github.com/GoogleCloudPlatform/cloud-opensource-java) | `20.9.0` -> `21.0.0` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/21.0.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/21.0.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/21.0.0/compatibility-slim/20.9.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/21.0.0/confidence-slim/20.9.0)](https://docs.renovatebot.com/merge-confidence/) |

---

### Configuration

📅 **Schedule**: At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-talent).
Sita04 pushed a commit that referenced this issue Feb 7, 2023
🤖 I have created a release \*beep\* \*boop\*
---
### [2.4.5](https://www.github.com/googleapis/java-dlp/compare/v2.4.4...v2.4.5) (2021-07-06)


### Dependencies

* update dependency com.google.cloud:google-cloud-pubsub to v1.113.5 ([#578](https://www.github.com/googleapis/java-dlp/issues/578)) ([1a98232](https://www.github.com/googleapis/java-dlp/commit/1a982325885269d3d8ac24298dababb2e04d2f13))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

No branches or pull requests

5 participants