{"push_id":874024370,"size":12,"distinct_size":12,"ref":"refs/heads/master","head":"1f6a059b6fafaf3c22e7fba45f330e4983f0d1cf","before":"10f46112c1c52fe182a951636323b0fbf70ed723","commits":[{"sha":"c8da401fe64145a78884de05b5c28eae7ac85de3","author":{"email":"bedb734a6a6f189ca744b0ea449a43edc5a478cd@gmail.com","name":"Fred Hart"},"message":"Fix spacing issue in log statement","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/c8da401fe64145a78884de05b5c28eae7ac85de3"},{"sha":"a6409f9356a4871e2f57052b6a84620525795113","author":{"email":"c7c805cfdc011ca1e14ebab2aa81f978ac1fdd95@yahoo.com","name":"Paul Bemowski"},"message":"Merge branch 'PerfTrack_Formatting' into 'master'\n\nPerf track formatting\n\nAss space after method name before Exception information for failed calls logged by PerfTrack","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/a6409f9356a4871e2f57052b6a84620525795113"},{"sha":"3559ff8eff697792f76ce0f0aca5ba951f7c98d4","author":{"email":"bedb734a6a6f189ca744b0ea449a43edc5a478cd@gmail.com","name":"Fred Hart"},"message":"Added debugging code to try to diagnose apparent re-ordering of log\nmessages on linux compiled tests\nRemoved some line numbers and compiler version information from stack\ntraces when comparing test output to try to make saved test output less\ndependent on the system on which it was compiled","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/3559ff8eff697792f76ce0f0aca5ba951f7c98d4"},{"sha":"1317f18c9174d73d4c220fa3830cda2a882df828","author":{"email":"bedb734a6a6f189ca744b0ea449a43edc5a478cd@gmail.com","name":"Fred Hart"},"message":"Log the PerfTracked method ending message in favor of the Logged method\nending message when both annotations are present on a method. This will\nprevent apparent our of order log statements and will contain additional\ninformation.\n\nRemove uneeded temporary debugging information from PerfTrackedAspect","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/1317f18c9174d73d4c220fa3830cda2a882df828"},{"sha":"19445c650c4892031a65b7aa64cfabc85bec0a8b","author":{"email":"bedb734a6a6f189ca744b0ea449a43edc5a478cd@gmail.com","name":"Fred Hart"},"message":"Updated unit test output","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/19445c650c4892031a65b7aa64cfabc85bec0a8b"},{"sha":"abcb56e79cdb07b80f3b6c75e9d3b32725e39fed","author":{"email":"bedb734a6a6f189ca744b0ea449a43edc5a478cd@gmail.com","name":"Fred Hart"},"message":"Fixed misnamed variable","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/abcb56e79cdb07b80f3b6c75e9d3b32725e39fed"},{"sha":"c1f04171604e5ba57dffb05fe962b60fbfb166e6","author":{"email":"bedb734a6a6f189ca744b0ea449a43edc5a478cd@gmail.com","name":"Fred Hart"},"message":"Added verbose arg to @PerfTracked annotation to indicate whether or not\nto include method signature in the PerfTrack summary\n\nUpdated AnnotationTest to test for proper behavior of verbose arg to\nPerfTracked","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/c1f04171604e5ba57dffb05fe962b60fbfb166e6"},{"sha":"0899ab34c2d4a4903ccdd3faa0c7fcfdf0f76221","author":{"email":"bedb734a6a6f189ca744b0ea449a43edc5a478cd@gmail.com","name":"Fred Hart"},"message":"Added Javadoc for the @PerfTracked threshold and verbose args","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/0899ab34c2d4a4903ccdd3faa0c7fcfdf0f76221"},{"sha":"eb2f29f6bc18096e7c006148a226f460a9a37a52","author":{"email":"bedb734a6a6f189ca744b0ea449a43edc5a478cd@gmail.com","name":"Fred Hart"},"message":"Deleted rogue character from \"verbose\" arg to PerfTracked","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/eb2f29f6bc18096e7c006148a226f460a9a37a52"},{"sha":"3aa45b7dad582b9ef3ff04895d097d38902f12e6","author":{"email":"c7c805cfdc011ca1e14ebab2aa81f978ac1fdd95@yahoo.com","name":"Paul Bemowski"},"message":"1.0.3","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/3aa45b7dad582b9ef3ff04895d097d38902f12e6"},{"sha":"80a85664dd742b68fd3a76d844df75a397bca37c","author":{"email":"c7c805cfdc011ca1e14ebab2aa81f978ac1fdd95@yahoo.com","name":"Paul Bemowski"},"message":"Merge branch 'PerfTrack_Formatting' into 'master'\n\nPerf track formatting - fix\n\nThe apparent re-ordering of the method exit log statements and the PerfTrack summary on some test executions was due to the fact that the method exit log messages on the method in question was not generated by the @PerfTracked annotation. Rather it was generated by the @Logged annotation on the same method. Both annotations were present on the same ```AnnotationTest.format1``` method. In this case the @Logged annotations _were_ being used for the method entry and exit messages and the @PerfTracked annotation only printed the PerTrack summary. The reason for allowing both annotations is to allow verbose parameter logging on method entry without having to have those parameters printed in the PerfTrack summary since they can be quite large if the data in the parameters is large, making the PerfTracked summary pretty hard to read. So the decision was made in the code to prefer the @Logged method entry and exit messages if both annotations were present. The problem this caused is that the @Logged method exit message and the PerfTrack summary are apparently not guaranteed to be in the same order on different platforms or possibly on different executions of the tests since they are generated by different annotations. With this PR, we now prefer the method entry of the @Logged annotation if @PerfTracked is also present on the method and we prefer the @PerfTracked method exit message if both annotations are present. This guarantees that the method exit and PerfTracked summary are in the proper order and the @PerfTracked method exit message contains the method execution time which is not on the @Logged version of the method exit message, an additional benefit of this change.\n\nSome System.out temporary debugging was also removed and the test case output used to compare against future executions of the test was updated.","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/80a85664dd742b68fd3a76d844df75a397bca37c"},{"sha":"1f6a059b6fafaf3c22e7fba45f330e4983f0d1cf","author":{"email":"c7c805cfdc011ca1e14ebab2aa81f978ac1fdd95@yahoo.com","name":"Paul Bemowski"},"message":"1.0.4","distinct":true,"url":"https://api.github.com/repos/bemowski/jmutils/commits/1f6a059b6fafaf3c22e7fba45f330e4983f0d1cf"}]}