Skip to content

libswoc: Update IntrusiveHashMap in HTTP session manager. #9872

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
SolidWallOfCode merged 1 commit into from
Jun 26, 2023
Merged

libswoc: Update IntrusiveHashMap in HTTP session manager. #9872

SolidWallOfCode merged 1 commit into from
Jun 26, 2023

Conversation

@SolidWallOfCode
Copy link
Member

@SolidWallOfCode SolidWallOfCode commented Jun 17, 2023

Update old IntrusiveHashMap to libswoc version.

@SolidWallOfCode SolidWallOfCode self-assigned this Jun 17, 2023
@bryancall bryancall added this to the 10.0.0 milestone Jun 26, 2023
@SolidWallOfCode SolidWallOfCode merged commit ca4f8cd into apache:master Jun 26, 2023
@bneradt bneradt mentioned this pull request Sep 20, 2023
Closed
bneradt added a commit to bneradt/trafficserver that referenced this pull request Jul 12, 2024
@bneradt
YTSATS-4260: Revert libswoc IntrusiveHashMap (apache#853)
8e1053c 
Revert "libswoc: Update IntrusiveHashMap in HTTP session manager. (apache#9872)"

apache#9872

This reverts commit ca4f8cd.

I'm reverting this IntrusiveHashMap switch to libswoc because it
introduces a spike in our origin side connections.

The following is also reverted to add back the IntrusiveHashMap
and IntrusiveDList implementations:

Revert "libswoc: Remove legacy IntrusiveHashMap (apache#10104)"

This reverts commit 5706e3b.

Revert "libswoc: Remove legacy IntrusiveDList. (apache#10106)"

This reverts commit 5d0a549.

Co-authored-by: bneradt <bneradt@yahooinc.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ywkaras ywkaras ywkaras approved these changes

Assignees

@SolidWallOfCode SolidWallOfCode

Labels

Cleanup CPP API

Projects

None yet

Milestone

10.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@SolidWallOfCode @ywkaras @bryancall