dubbo icon indicating copy to clipboard operation
dubbo copied to clipboard

[Feature] Add AsyncRpcResult test sample in dubbo-samples

Open CAICAIIs opened this issue 5 months ago • 5 comments

Pre-check

  • [x] I am sure that all the content I provide is in English.

Search before asking

  • [x] I had searched in the issues and found no similar feature requirement.

Apache Dubbo Component

Java Samples (apache/dubbo-samples)

Descriptions

Background

AsyncRpcResult is the key class for asynchronous calls in Dubbo Java SDK, but the official dubbo-samples repository does not contain any test case or example to demonstrate its correct usage. This makes it hard for users to understand how to:

  1. Obtain the result via AsyncRpcResult after an async call.
  2. Handle timeout and exceptions properly.
  3. Ensure compatibility across Dubbo versions.

Proposal

Create a new sub-module dubbo-samples-async-result under 2-advanced/ with:

  • A provider that exposes an async service.
  • A consumer that uses AsyncRpcResult to get the response and handle timeout/exception.
  • Docker-based integration tests similar to existing samples (dubbo-samples-annotation, etc.).

Acceptance Criteria

  • [ ] New module builds and passes ./test/run-tests.sh 2-advanced/dubbo-samples-async-result.
  • [ ] README explains how to run the demo and the key points of AsyncRpcResult.
  • [ ] No breaking changes to existing samples.

Volunteer

I’m willing to submit a PR once the feature request is accepted.

Related issues

No response

Are you willing to submit a pull request to fix on your own?

  • [ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

CAICAIIs avatar Jul 20 '25 06:07 CAICAIIs

Hi! I'd like to contribute to this issue and can start working on it right away. If it's still unassigned, could you please assign it to me? Thank you!

yashvi1912 avatar Aug 05 '25 10:08 yashvi1912

Hi! I'd like to contribute to this issue and can start working on it right away. If it's still unassigned, could you please assign it to me? Thank you!

I don't seem to have the authority to assign it to you, but please go ahead!

CAICAIIs avatar Aug 05 '25 13:08 CAICAIIs

Hi! Thank you for the confirmation. I'll go ahead and start working on the issue.

yashvi1912 avatar Aug 06 '25 01:08 yashvi1912

Thank you, I am working on it.

yashvi1912 avatar Aug 08 '25 05:08 yashvi1912

Could you please assign it to me? Thank you!

redoom avatar Nov 07 '25 06:11 redoom