Skip to content

[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy on InterruptedExcep… - #37304

Merged
BiteTheDDDDt merged 9 commits into
apache:masterfrom
BiteTheDDDDt:test_0704_2
Jul 18, 2024
Merged

[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy on InterruptedExcep…#37304
BiteTheDDDDt merged 9 commits into
apache:masterfrom
BiteTheDDDDt:test_0704_2

Conversation

@BiteTheDDDDt

@BiteTheDDDDtBiteTheDDDDt commented Jul 4, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

Proposed changes

  1. set grpc channel's keepAliveTime
  2. remove proxy on InterruptedException/TimeoutException to avoid channel unavailable

@BiteTheDDDDt

BiteTheDDDDt commented Jul 4, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

@BiteTheDDDDt

BiteTheDDDDt commented Jul 4, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

@github-actions

github-actionsBot commented Jul 5, 2024

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@BiteTheDDDDt

BiteTheDDDDt commented Jul 5, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

1 similar comment
@BiteTheDDDDt

BiteTheDDDDt commented Jul 5, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

@BiteTheDDDDt

BiteTheDDDDt commented Jul 5, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

@BiteTheDDDDt

BiteTheDDDDt commented Jul 9, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

@BiteTheDDDDtBiteTheDDDDt changed the title set grpc channel's keepAliveTime and remove proxy on InterruptedExcep…[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy on InterruptedExcep…Jul 9, 2024
morningman
morningman reviewed Jul 9, 2024
View reviewed changes
Comment threadfe/fe-common/src/main/java/org/apache/doris/common/Config.java Outdated
@BiteTheDDDDt

BiteTheDDDDt commented Jul 9, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

@BiteTheDDDDt

BiteTheDDDDt commented Jul 11, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

w41ter
w41ter reviewed Jul 11, 2024
View reviewed changes

@w41terw41ter left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reset LGTM

@BiteTheDDDDt

BiteTheDDDDt commented Jul 16, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

@BiteTheDDDDt

BiteTheDDDDt commented Jul 16, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

HappenLee
HappenLee approved these changes Jul 18, 2024
View reviewed changes

@HappenLeeHappenLee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@github-actions

github-actionsBot commented Jul 18, 2024

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actionsgithub-actionsBot added approved Indicates a PR has been approved by one committer. reviewed labels Jul 18, 2024
@github-actions

github-actionsBot commented Jul 18, 2024

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@BiteTheDDDDt
BiteTheDDDDt merged commit 881abbb into apache:masterJul 18, 2024
dataroaring pushed a commit that referenced this pull request Jul 19, 2024
@BiteTheDDDDt@dataroaring
[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy …
d54dbae
…on InterruptedExcep… (#37304)
## Proposed changes
1. set grpc channel's keepAliveTime
2. remove proxy on InterruptedException/TimeoutException to avoid
channel unavailable
BiteTheDDDDt added a commit to BiteTheDDDDt/incubator-doris that referenced this pull request Jul 25, 2024
@BiteTheDDDDt
[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy …
e59f8d0
…on InterruptedExcep… (apache#37304)
## Proposed changes
1. set grpc channel's keepAliveTime
2. remove proxy on InterruptedException/TimeoutException to avoid
channel unavailable
@BiteTheDDDDtBiteTheDDDDt mentioned this pull request Jul 25, 2024
Merged
BiteTheDDDDt added a commit to BiteTheDDDDt/incubator-doris that referenced this pull request Jul 25, 2024
@BiteTheDDDDt
[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy …
2ced17f
…on InterruptedExcep… (apache#37304)
## Proposed changes
1. set grpc channel's keepAliveTime
2. remove proxy on InterruptedException/TimeoutException to avoid
channel unavailable
@BiteTheDDDDtBiteTheDDDDt mentioned this pull request Jul 25, 2024
Merged
yiguolei pushed a commit that referenced this pull request Jul 25, 2024
@BiteTheDDDDt
[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy … (
b4e82d2
#38381)
…on InterruptedExcep… (#37304)
## Proposed changes
1. set grpc channel's keepAliveTime
2. remove proxy on InterruptedException/TimeoutException to avoid
channel unavailable
pick from #37304
## Proposed changes
Issue Number: close #xxx
<!--Describe your changes.-->
BiteTheDDDDt added a commit that referenced this pull request Jul 26, 2024
@BiteTheDDDDt
[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy … (
71c7c77
#38380)
…on InterruptedExcep… (#37304)
## Proposed changes
1. set grpc channel's keepAliveTime
2. remove proxy on InterruptedException/TimeoutException to avoid
channel unavailable
pick from #37304
## Proposed changes
Issue Number: close #xxx
<!--Describe your changes.-->
BiteTheDDDDt added a commit to BiteTheDDDDt/incubator-doris that referenced this pull request Aug 7, 2024
@BiteTheDDDDt
[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy … (
09aa556
apache#38380)
…on InterruptedExcep… (apache#37304)
## Proposed changes
1. set grpc channel's keepAliveTime
2. remove proxy on InterruptedException/TimeoutException to avoid
channel unavailable
pick from apache#37304
## Proposed changes
Issue Number: close #xxx
<!--Describe your changes.-->
mongo360 pushed a commit to mongo360/doris that referenced this pull request Aug 16, 2024
@BiteTheDDDDtweixingyu12
[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy … (
7b31e87
apache#38380)
…on InterruptedExcep… (apache#37304)
## Proposed changes
1. set grpc channel's keepAliveTime
2. remove proxy on InterruptedException/TimeoutException to avoid
channel unavailable
pick from apache#37304
## Proposed changes
Issue Number: close #xxx
<!--Describe your changes.-->
@gavinchougavinchou mentioned this pull request Aug 19, 2024
Closed
@yiguoleiyiguolei mentioned this pull request Sep 5, 2024
Closed
3 tasks
HappenLee pushed a commit to HappenLee/incubator-doris that referenced this pull request Apr 24, 2026
@BiteTheDDDDt@HappenLee
[Improvement](rpc) set grpc channel's keepAliveTime and remove proxy …
a64df85
…on InterruptedExcep… (apache#37304)
## Proposed changes
1. set grpc channel's keepAliveTime
2. remove proxy on InterruptedException/TimeoutException to avoid
channel unavailable
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morningmanmorningmanmorningman left review comments
@w41terw41terw41ter approved these changes
@HappenLeeHappenLeeHappenLee approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.dev/2.1.6-mergeddev/3.0.1-mergedreviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@BiteTheDDDDt@morningman@w41ter@HappenLee@yiguolei@dataroaring