I'll close this issue as I believe that in ORM 6 we fully implemented this 2 phase loading already.

Yes I do understsnd, and don't call me shirley

Jeff

Surely you understand the difference in severity="error" and severity="warning".

Feel free to "ignore" checkstyle, but if it fails our CI jobs due to checkstyle I will not integrate it.

Hi Steve,

The first set of checkStyle were related to CR/LF, when I converted them to Windows format it worked (that was 4 files in hibernate-c3p0 project).

However, when I fixed them, the build went on to the next project (hibernate-core) and generated a slue of errors, and this one, for example, should be caught on your system.....

Note the error in the last line. If you go out to the Master, you will see the following, that line 88 does have a whitespace error..... .toUpperCase (L oc

CacheMode.java

So, not only do I have the CR/LF issue with every file, there are other errors that are not detected on the CI servers

Sorry, but I am definately going to skip the checkStyle, as it is erroneous on Windows (If I re-format every file, it will probably break your checkStyle).

Jeff

Yes, it might be the MS CR/LF issue Grrrr

I will try the import, maybe I will have better luck. I am using the head of the master, I figured that would keep my changes easier to merge. I know you did a lot of changes in Loader (Logging, remove for loops, changes in session stuff (if I remember right), etc.).

What version of Gradle are you using, the default wrapper, or another version? I know I am having mixed results depending on the verion.

Let me keep plugging away, last time I had to point the checkStyle to a temp directory to continue with the actual unit tests.

Thank you for all of your help,

Jeff