Skip to content

Conversation

@doudou
Copy link
Member

@doudou doudou commented Dec 17, 2025

Depends on:

On top of #539

Micro but still giving ~10% improvements on deployment times on our systems.

@doudou doudou requested review from jhonasiv and wvmcastro December 17, 2025 16:22
@doudou doudou changed the title chore: micro-optimizations chore: micro-optimizations (on top of #539) Dec 17, 2025
@doudou doudou force-pushed the micro-opt branch 2 times, most recently from ccf840a to 9dd7c0d Compare December 17, 2025 17:58
Base automatically changed from lazy_deploy to transition-to-runkit December 30, 2025 13:22
Copy link

@wvmcastro wvmcastro left a comment

Choose a reason for hiding this comment

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

Looks like that only 783c731 is the one that impacts performance, the other commits are "just" refactors aiming better readability and less repetition. If I am mistaken, can you point the other bits that do impact on perf?

@doudou
Copy link
Member Author

doudou commented Jan 22, 2026

Looks like that only 783c731 is the one that impacts performance, the other commits are "just" refactors aiming better readability and less repetition. If I am mistaken, can you point the other bits that do impact on perf?

That's right, actually. Only the through_method_missing one has a small impact.

The other ones are performance-related (they helped a lot with the process), but are not optimizations.

@doudou doudou merged commit 2cadffd into transition-to-runkit Jan 22, 2026
@doudou doudou deleted the micro-opt branch January 22, 2026 20:05
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.

4 participants