Sam Snyder

It is inefficient to have sequences of logger invocations at the same level: ```java logger.info("foo"); logger.info("bar"); ``` These would be better combined into a single invocation

Sometimes parameters to logging invocations include expensive computations which are pointless if the log level isn't set such that the message will actually be recorded. It would be useful to...

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.java.security.OwaspA01?organizationId=T3BlblJld3JpdGU%3D

Will have to take a parameter specifying what GAV coordinate to replace with what plugin id

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.staticanalysis.CommonStaticAnalysis?organizationId=T3BlblJld3JpdGU%3D

This has always been a bad thing to do. Recently it was part of why a recipe was timing out. With these changes RewriteTest will warn recipe authors before they...

Use flyweights for classes representing GAV coordinates.

2

We have seen high GC overhead / memory pressure related to these objects. They are created and discarded frequently both during parsing and recipe execution.