Re: [hibernate-dev] Byte-buddy: default enhancer in 5.3 ? [HHH-11253]

An update on this; I've found multiple issues, yet when digging none seem to be a strong reason to withold switching to Byte Buddy as default. The main problem seems to be that running the testsuite will somehow load 2.867 different Class instances all having the same name "org.hibernate.test.bytecode.enhancement.access.MixedAccessTest", and a matching amount of "org.hibernate.test.bytecode.enhancement.access.MixedAccessTest$NoDirtyCheckingContext" classes. I'm not really understanding how this happens yet, the ClassLoader chain is huge and complex: there's a "root" classloader by Gradle which applies some custom filtering, while on our test side this is followed by two levels of enhancers implemented by two different copies of byte-buddy: Mockito is including a shaded copy of byte-buddy and has its own additional map with weak keys to cache more classes and classloaders. So we're using Byte Buddy both for entity enhancement, and to enhance our tests which I guess could include re-enhancing the test entities in some situations. We're also spending about half a gigabyte in empty HashMap instances, and silly things such as 40MB in repeated copies of the same String or same Integer instances. I could fix some little waste caused by Hibernate ORM so the good news is that we'll be consuming a bit less at runtime. Most of the duplication though is caused by overhead by the huge amount of independent ClassLoaders: even class names are repeated, package filtering and so on are repeated over and over into the various loading caches of each classloader, repeated with similar layers of Vector and LinkedHashMaps composing the repeated metadata structures. Now the good news: the test "MixedAccessTest" is marked with @Ignored. If I delete it, everything works fine. I don't like deleting things but I'm not fully understanding what this test is meant to accomplish.. what shall I do? I propose we delete it, anyone who's willing to get back to it can restore it from history. I see three options: 1) delete MixedAccessTest 2) ignore it for now and raise the memory setting of the testsuite from 2G to 8G 3) keep working on this and see if I can fix the tests, hopefully with some help Thanks, Sanne On 28 February 2018 at 14:13, Steve Ebersole <steve(a)hibernate.org&gt; wrote: