There's a test in the mtr suite: main.mysqlpump_basic (not mysqldump) that should cover this.
But for extra testing: * Create a database test with a table and a row or two * Run: mysqlpump -uroot -p --compress-output=LZ4 test > testdb.lz * Run /usr/lib/mysql/lz4_decompress testdb.lz testdb.out * Compare testdb.out with the output of mysqlpump -uroot -p test -- Lars On 02/03/2017 06:44 PM, Robie Basak wrote: > Lars, is there a test case we can add for this that exercises > mysqldump's lz4 capability, so that we know that it still functions? > > ** Description changed: > > MySQL 5.7 source package has liblz4-dev as a build-dep, but it isn't > actually used because the build defaults to using a bundled version. > > [Impact] > No known user impact at present (other than a small difference in binary > footprint), but builds shouldn't use bundled libraries when it's available in > the system. > + > + [racb] However a future security update in liblz4-dev will not update > + MySQL. We fixed this in sid, and I think it's appropriate to backport > + given that we've tested it in Yakkety. > > Add cmake flag -DWITH_LZ4=system to use the correct version. > > [Test case] > * Build package > * Run ldd on usr/lib/lz4_decompress and see it doesn't link in liblz4 from > the system. > > [Regression potential] > This change has been in Yakkety for some time (5.7.15-0ubuntu2). > > Database dump-related utils like mysqlpump and lz4_decompress could be > impacted if any significant changes arise between the version of liblz4 > available on user systems and what is used to test upstream. > -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1631339 Title: Build with lz4 build-dep instead of bundled To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/mysql-5.7/+bug/1631339/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
