Skip to content

Conversation

@Vito-Yang
Copy link
Contributor

Description

Migrate remaining std::unordered_map to absl::flat_hash_map

Related issues

Link related issues: "Related to #59886 ".

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request migrates several instances of std::unordered_map to absl::flat_hash_map in the core worker implementation, which is a good practice for performance. The changes are correct and align with the goal of the PR. I have one minor suggestion to remove a now-unused header include.

@Vito-Yang Vito-Yang marked this pull request as ready for review January 7, 2026 07:15
@Vito-Yang Vito-Yang requested a review from a team as a code owner January 7, 2026 07:15
@ray-gardener ray-gardener bot added core Issues that should be addressed in Ray Core community-contribution Contributed by the community labels Jan 7, 2026
@edoakes edoakes added the go add ONLY when ready to merge, run all tests label Jan 7, 2026
@edoakes edoakes enabled auto-merge (squash) January 7, 2026 15:22
@edoakes edoakes merged commit caa936c into ray-project:master Jan 7, 2026
8 checks passed
AYou0207 pushed a commit to AYou0207/ray that referenced this pull request Jan 13, 2026
…ay-project#59921)

Migrate remaining std::unordered_map to absl::flat_hash_map

---------

Signed-off-by: yang <[email protected]>
Signed-off-by: jasonwrwang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Contributed by the community core Issues that should be addressed in Ray Core go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants