Add additional ignores

[mar-kolya]

This seems to be improving pet clinic startup by ~3 seconds.

[trask]

Hi @mar-kolya! How are you checking these packages for Runnable or Callable classes that may need to be instrumented?

Also, curious if there's been discussion already about how to decide which libraries (out of the thousands of millions of Java libraries 😆) to include in the ignore list. Thanks!

[mar-kolya]

Hi @mar-kolya! How are you checking these packages for Runnable or Callable classes that may need to be instrumented?

We now have a test that essentially disables this library matcher - and makes sure that no classes are modified that would have been excluded by this matcher. So since all tests pass this means we didn't try to modify excluded classes in our tests.

Also, curious if there's been discussion already about how to decide which libraries (out of the thousands of millions of Java libraries 😆) to include in the ignore list. Thanks!

There was no specific discussion, purely subjective choice - mainly based on what libraries we pull to run tracer's tests

[trask]

We now have a test that essentially disables this library matcher - and makes sure that now classes are modified that would have been excluded by this matcher. So since all tests pass this means we didn't try to modify excluded classes in our tests.

Nice!

There was no specific discussion, purely subjective choice - mainly based on what libraries we pull to run tracer's tests

Makes sense, thx for your responses!