Re: Review Request 33273: Add a specific storage routine for bulk loading data.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33273/#review81064 --- Ship it! Ship It! - Maxim Khutornenko On April 16, 2015, 9:50 p.m., Bill Farner wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33273/ > --- > > (Updated April 16, 2015, 9:50 p.m.) > > > Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. > > > Repository: aurora > > > Description > --- > > This behavior is primarily for DbStorage, which will benefit from a signal > that a mutate transaction is a bulk data load, so that it can relax > transactional guarantees in the interest of recovery speed. > > > Diffs > - > > > src/main/java/org/apache/aurora/scheduler/storage/CallOrderEnforcingStorage.java > 07d81e428342b35be019aa2046c7f1554393527f > src/main/java/org/apache/aurora/scheduler/storage/Storage.java > 972a3c1f094dc62ebf1d35074485441181d7fe0d > src/main/java/org/apache/aurora/scheduler/storage/db/DbStorage.java > 526df103883fd720cb6c00f8dff1bb3cf7cead38 > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java > 63b5b1f87ba12edbc5ad2546189dff420409d645 > src/main/java/org/apache/aurora/scheduler/storage/mem/MemStorage.java > dafe1c4f7c184c41dba0360f876dd8381f4aeb59 > > src/test/java/org/apache/aurora/scheduler/app/local/FakeNonVolatileStorage.java > 3336f8cddb2a7a6e8c9e4bc6664708342ab97979 > src/test/java/org/apache/aurora/scheduler/storage/db/DbStorageTest.java > PRE-CREATION > src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java > cb6ba25c18878b751fcaffe15977b1ca10c74d65 > > Diff: https://reviews.apache.org/r/33273/diff/ > > > Testing > --- > > > Thanks, > > Bill Farner > >
Re: Review Request 33273: Add a specific storage routine for bulk loading data.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33273/#review80412 --- Ship it! Ship It! - Kevin Sweeney On April 16, 2015, 2:50 p.m., Bill Farner wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33273/ > --- > > (Updated April 16, 2015, 2:50 p.m.) > > > Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. > > > Repository: aurora > > > Description > --- > > This behavior is primarily for DbStorage, which will benefit from a signal > that a mutate transaction is a bulk data load, so that it can relax > transactional guarantees in the interest of recovery speed. > > > Diffs > - > > > src/main/java/org/apache/aurora/scheduler/storage/CallOrderEnforcingStorage.java > 07d81e428342b35be019aa2046c7f1554393527f > src/main/java/org/apache/aurora/scheduler/storage/Storage.java > 972a3c1f094dc62ebf1d35074485441181d7fe0d > src/main/java/org/apache/aurora/scheduler/storage/db/DbStorage.java > 526df103883fd720cb6c00f8dff1bb3cf7cead38 > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java > 63b5b1f87ba12edbc5ad2546189dff420409d645 > src/main/java/org/apache/aurora/scheduler/storage/mem/MemStorage.java > dafe1c4f7c184c41dba0360f876dd8381f4aeb59 > > src/test/java/org/apache/aurora/scheduler/app/local/FakeNonVolatileStorage.java > 3336f8cddb2a7a6e8c9e4bc6664708342ab97979 > src/test/java/org/apache/aurora/scheduler/storage/db/DbStorageTest.java > PRE-CREATION > src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java > cb6ba25c18878b751fcaffe15977b1ca10c74d65 > > Diff: https://reviews.apache.org/r/33273/diff/ > > > Testing > --- > > > Thanks, > > Bill Farner > >
Re: Review Request 33273: Add a specific storage routine for bulk loading data.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33273/#review80386 --- Ship it! Master (b18dc44) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On April 16, 2015, 9:50 p.m., Bill Farner wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33273/ > --- > > (Updated April 16, 2015, 9:50 p.m.) > > > Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. > > > Repository: aurora > > > Description > --- > > This behavior is primarily for DbStorage, which will benefit from a signal > that a mutate transaction is a bulk data load, so that it can relax > transactional guarantees in the interest of recovery speed. > > > Diffs > - > > > src/main/java/org/apache/aurora/scheduler/storage/CallOrderEnforcingStorage.java > 07d81e428342b35be019aa2046c7f1554393527f > src/main/java/org/apache/aurora/scheduler/storage/Storage.java > 972a3c1f094dc62ebf1d35074485441181d7fe0d > src/main/java/org/apache/aurora/scheduler/storage/db/DbStorage.java > 526df103883fd720cb6c00f8dff1bb3cf7cead38 > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java > 63b5b1f87ba12edbc5ad2546189dff420409d645 > src/main/java/org/apache/aurora/scheduler/storage/mem/MemStorage.java > dafe1c4f7c184c41dba0360f876dd8381f4aeb59 > > src/test/java/org/apache/aurora/scheduler/app/local/FakeNonVolatileStorage.java > 3336f8cddb2a7a6e8c9e4bc6664708342ab97979 > src/test/java/org/apache/aurora/scheduler/storage/db/DbStorageTest.java > PRE-CREATION > src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java > cb6ba25c18878b751fcaffe15977b1ca10c74d65 > > Diff: https://reviews.apache.org/r/33273/diff/ > > > Testing > --- > > > Thanks, > > Bill Farner > >
Review Request 33273: Add a specific storage routine for bulk loading data.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33273/ --- Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Repository: aurora Description --- This behavior is primarily for DbStorage, which will benefit from a signal that a mutate transaction is a bulk data load, so that it can relax transactional guarantees in the interest of recovery speed. Diffs - src/main/java/org/apache/aurora/scheduler/storage/CallOrderEnforcingStorage.java 07d81e428342b35be019aa2046c7f1554393527f src/main/java/org/apache/aurora/scheduler/storage/Storage.java 972a3c1f094dc62ebf1d35074485441181d7fe0d src/main/java/org/apache/aurora/scheduler/storage/db/DbStorage.java 526df103883fd720cb6c00f8dff1bb3cf7cead38 src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java 63b5b1f87ba12edbc5ad2546189dff420409d645 src/main/java/org/apache/aurora/scheduler/storage/mem/MemStorage.java dafe1c4f7c184c41dba0360f876dd8381f4aeb59 src/test/java/org/apache/aurora/scheduler/app/local/FakeNonVolatileStorage.java 3336f8cddb2a7a6e8c9e4bc6664708342ab97979 src/test/java/org/apache/aurora/scheduler/storage/db/DbStorageTest.java PRE-CREATION src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java cb6ba25c18878b751fcaffe15977b1ca10c74d65 Diff: https://reviews.apache.org/r/33273/diff/ Testing --- Thanks, Bill Farner