-
Notifications
You must be signed in to change notification settings - Fork 516
Found multiple performance optimizations for Rewrite #7259
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or request
Description
What problem are you trying to solve?
Improve speed for Rewrite by merging in optimizations we created with Codeflash
Describe the solution you'd like
Here are the Pull Requests we created and reviewed manually with the optimizations. I have ordered them roughly in the order of ease of merge and impact:
- ⚡️ Speed up method
StringUtils.formatUriForPropertiesFileby 265% HeshamHM28/rewrite#54 - Simple speed up for string operations - ⚡️ Speed up method
StringUtils.indexOfNextNonWhitespaceby 57% HeshamHM28/rewrite#53 - Speeds up string operations - ⚡️ Speed up method
LocalRemoteArtifactCache.hashUriby 697% HeshamHM28/rewrite#27 - HashURI speedup - ⚡️ Speed up method
StringUtils.indentby 161% HeshamHM28/rewrite#49 - String indent speedup - ⚡️ Speed up method
Cursor.toStringby 75% HeshamHM28/rewrite#16 - Cursor.toString speedup - ⚡️ Speed up method
Cursor.firstEnclosingby 94% HeshamHM28/rewrite#15 - Cursor firstEnclosing speedup - ⚡️ Speed up method
ExceptionUtils.containsCircularReferencesby 127% HeshamHM28/rewrite#33 - Exception circularReferences - ⚡️ Speed up method
StringUtils.isBlankby 139% HeshamHM28/rewrite#43 - String isBlank whitespace - ⚡️ Speed up method
PathUtils.getEitherOrPatternsby 209% HeshamHM28/rewrite#38 - PathUtils speedups - ⚡️ Speed up method
StringUtils.capitalizeby 15% HeshamHM28/rewrite#45 - String capitalize - ⚡️ Speed up method
StringUtils.uncapitalizeby 7% HeshamHM28/rewrite#46 - String uncapitalize - ⚡️ Speed up method
RewriteRpcProcess.getLivenessCheckby 38% HeshamHM28/rewrite#19 - RewriteRpcProcess.getLivenessCheck speedup - ⚡️ Speed up method
StringUtils.aspectjNameToPatternby 15% HeshamHM28/rewrite#51 - Speed up string operations - ⚡️ Speed up method
GitRemote.hashCodeby 114% HeshamHM28/rewrite#10 - speedup hashCode with a more optimized implementation - ⚡️ Speed up method
MetricsHelper.successTagsby 116% HeshamHM28/rewrite#28 - simple speedup - ⚡️ Speed up method
StringUtils.commonMarginby 12% HeshamHM28/rewrite#50 - speeds up string operations by reducing redundant computations
Can you let me know which ones can I create here.
Additional context
We have improved codeflash for java significantly since the runs above last month, and now we can optimize end-to-end performance benchmarks as well. Let me know if you have any performance benchmarks that you would want optimizations for?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request
Type
Projects
Status
No status