Sentry-jdk-1.8 - Build # 209 - Fixed
The Apache Jenkins build system has built Sentry-jdk-1.8 (build #209) Status: Fixed Check console output at https://builds.apache.org/job/Sentry-jdk-1.8/209/ to view the results.
Re: Review Request 69941: SENTRY-2494: Fix TestRollingFileWithoutDeleteAppender test case testFileNamePattern
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69941/#review212703 --- Ship it! Ship It! - kalyan kumar kalvagadda On Feb. 11, 2019, 5:30 p.m., Arjun Mishra wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69941/ > --- > > (Updated Feb. 11, 2019, 5:30 p.m.) > > > Review request for sentry and kalyan kumar kalvagadda. > > > Bugs: SENTRY-2494 > https://issues.apache.org/jira/browse/SENTRY-2494 > > > Repository: sentry > > > Description > --- > > The log size is set to 10 bytes. However if the message size is 15 bytes, it > creates a 15, 15 and 0 byte file ( which is sometimes flaky) > > Explanation: > Before we logged a string that was at 15 bytes each. The assumption was > Logger would split that across 2 files but it never did that. It would put 15 > bytes of line on one file. > > Previously we had 2 log statements: > debug."123456789012345"; > debug."123456789012345"; > > The file being created was "123456789012345", "123456789012345", "" (LAST ONE > empty) > > as opposed to "1234567890", "1234512345", "6789012345" > > The above output would be flaky because LOGGER.appender did not handle a LONG > string properly. It would sometimes generate two files with > "123456789012345", "" (LAST ONE empty) > > > Diffs > - > > > sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/log/appender/TestRollingFileWithoutDeleteAppender.java > 5db15a3d0e78a071ec10c9fba4048c95f2f0cc89 > > > Diff: https://reviews.apache.org/r/69941/diff/1/ > > > Testing > --- > > > Thanks, > > Arjun Mishra > >
Review Request 69941: SENTRY-2494: Fix TestRollingFileWithoutDeleteAppender test case testFileNamePattern
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69941/ --- Review request for sentry and kalyan kumar kalvagadda. Bugs: SENTRY-2494 https://issues.apache.org/jira/browse/SENTRY-2494 Repository: sentry Description --- The log size is set to 10 bytes. However if the message size is 15 bytes, it creates a 15, 15 and 0 byte file ( which is sometimes flaky) Explanation: Before we logged a string that was at 15 bytes each. The assumption was Logger would split that across 2 files but it never did that. It would put 15 bytes of line on one file. Previously we had 2 log statements: debug."123456789012345"; debug."123456789012345"; The file being created was "123456789012345", "123456789012345", "" (LAST ONE empty) as opposed to "1234567890", "1234512345", "6789012345" The above output would be flaky because LOGGER.appender did not handle a LONG string properly. It would sometimes generate two files with "123456789012345", "" (LAST ONE empty) Diffs - sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/log/appender/TestRollingFileWithoutDeleteAppender.java 5db15a3d0e78a071ec10c9fba4048c95f2f0cc89 Diff: https://reviews.apache.org/r/69941/diff/1/ Testing --- Thanks, Arjun Mishra