Skip to content

Update ColocationGroupProcDir.java#59685

Open
shawn6616 wants to merge 1 commit into
apache:masterfrom
shawn6616:patch-1
Open

Update ColocationGroupProcDir.java#59685
shawn6616 wants to merge 1 commit into
apache:masterfrom
shawn6616:patch-1

Conversation

@shawn6616

Copy link
Copy Markdown

In the lookup function, add a judgment for the cloud environment, call CloudReplica.getColocatedBeId to obtain the BE ID, and construct beSeqs that are compatible with the original format.

What problem does this PR solve?

In the lookup function, add a judgment for the cloud environment, call CloudReplica.getColocatedBeId to obtain the BE ID, and construct beSeqs that are compatible with the original format.

Issue Number: close #xxx

solve issue #55756

In the lookup function, add a judgment for the cloud environment, call CloudReplica.getColocatedBeId to obtain the BE ID, and construct beSeqs that are compatible with the original format.
@hello-stephen

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@deardeng

deardeng commented Jan 8, 2026

Copy link
Copy Markdown
Collaborator

plz add a test case , you can use feut or regression case

@deardeng

deardeng commented Jan 8, 2026

Copy link
Copy Markdown
Collaborator

PR title, suggestion: [fix] Cloud support should show proc colocationgroup in the cloud.

@shawn6616

Copy link
Copy Markdown
Author

plz add a test case , you can use feut or regression case

#59687 (comment)
add a new feut case check plz

@shawn6616

Copy link
Copy Markdown
Author

PR title, suggestion: [fix] Cloud support should show proc colocationgroup in the cloud.

i'll do it next time

@deardeng

deardeng commented Jan 9, 2026

Copy link
Copy Markdown
Collaborator

plz add a test case , you can use feut or regression case

#59687 (comment) add a new feut case check plz

测试case 放到同一个pr里面呀

  1. git checkout 到pr 的分支
  2. 在pr 分支添加case
  3. git commit
  4. git push

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

We're closing this PR because it hasn't been updated in a while.
This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and feel free a maintainer to remove the Stale tag!

@github-actions github-actions Bot added the Stale label Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants