Skip to content

Conversation

@icexin
Copy link
Contributor

@icexin icexin commented Sep 15, 2025

What problem does this PR solve?

Issue Number: #2726

Problem Summary: add bazel config setting for NO_PTHREAD_MUTEX_HOOK macro.

What is changed and the side effects?

Changed:

Side effects:

  • Performance effects:

  • Breaking backward compatibility:


Check List:

@icexin icexin force-pushed the pthread_mutex_config branch from 4646e72 to d90207a Compare September 15, 2025 03:48
@wwbmmm
Copy link
Contributor

wwbmmm commented Sep 15, 2025

LGTM

@chenBright
Copy link
Contributor

Macos ci problems haven been fixed in #3093. Please update and rebase master.

@icexin icexin force-pushed the pthread_mutex_config branch from d90207a to 438b5cc Compare September 15, 2025 11:56
Copy link
Contributor

@chenBright chenBright left a comment

Choose a reason for hiding this comment

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

LGTM

@chenBright chenBright merged commit cd22e48 into apache:master Sep 15, 2025
16 checks passed
@icexin icexin deleted the pthread_mutex_config branch September 15, 2025 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants