Skip to content
This repository has been archived by the owner on May 10, 2022. It is now read-only.

fix: handle case where Future.cause() is null in FutureGroup #52

Merged
merged 6 commits into from
Aug 22, 2019

Conversation

neverchanje
Copy link

@neverchanje neverchanje commented Aug 21, 2019

waitAllCompleteOrOneFail will throw NullPointException when timeout expired in current implementation. Because Future.cause() could be null when the result is not available in time.

This SNAPSHOT version was deployed on one of our users' environment (c3srv-xiaoai/ai_profile_data).

Copy link
Member

@qinzuoyan qinzuoyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个是彻底修复上次那个hang住的问题了吗?

pom.xml Outdated Show resolved Hide resolved
neverchanje added 2 commits August 21, 2019 16:39
@@ -72,7 +72,7 @@ function java_client_get_next_version()
esac

if [ ${versions[2]} == "SNAPSHOT" ]; then
echo ${versions[0]}.${versions[1]}.SNAPSHOT-$staging_branch
echo ${versions[0]}.${versions[1]}-$staging_branch-SNAPSHOT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里SNAPSHOT为什么要放在后面?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nexus 只把末尾是 snapshot 的当做 snapshot 包。
1.12.SNAPSHOT 是 snapshot,但是 1.12.SNAPSHOT-thrift-0.11.0-inlined 会被认作是 release。1.12-thrift-0.11.0-inlined-SNAPSHOT 就是正确的 snapshot。亲测。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

那么pom.xml里面现在也要改为 1.12-thrift-0.11.0-inlined-SNAPSHOT 吧

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已改

@neverchanje neverchanje changed the title bugfix on FutureGroup fix: handle case where Future.cause() is null in FutureGroup Aug 22, 2019
@neverchanje neverchanje merged commit c924e2c into XiaoMi:thrift-0.11.0-inlined Aug 22, 2019
@neverchanje neverchanje deleted the release branch August 22, 2019 05:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants