Thanks for the heads up. FYI, I plan to give it another pass tonight or
tmrw.
Eric Newton wrote:
I know this thing is huge. Josh has already been over it a couple of times.
I plan to push it to master very soon. If you would like to hold off for a
more intensive review, please let me know.
At this point, I've run the tests so many times, the ITs are more stable in
my branch than in master.
You can also find this at github:
https://github.com/ericnewton/accumulo-3423
-Eric
On Wed, Mar 18, 2015 at 7:58 PM, Eric Newton<[email protected]> wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32224/
-----------------------------------------------------------
Review request for accumulo.
Repository: accumulo
Description
-------
Faster WAL rollovers
Diffs
-----
core/src/main/java/org/apache/accumulo/core/client/impl/ReplicationOperationsImpl.java
6a5c74a
core/src/main/java/org/apache/accumulo/core/conf/Property.java 2403915
core/src/main/java/org/apache/accumulo/core/metadata/RootTable.java
24148b1
core/src/main/java/org/apache/accumulo/core/metadata/schema/MetadataSchema.java
534dd7f
core/src/main/java/org/apache/accumulo/core/tabletserver/log/LogEntry.java
25d0f32
core/src/test/java/org/apache/accumulo/core/metadata/MetadataTableSchemaTest.java
PRE-CREATION
server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java
fc26ca4
server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java
e5bdded
server/base/src/main/java/org/apache/accumulo/server/master/state/MetaDataStateStore.java
7ee6f0c
server/base/src/main/java/org/apache/accumulo/server/master/state/MetaDataTableScanner.java
9be5a67
server/base/src/main/java/org/apache/accumulo/server/master/state/TabletLocationState.java
b24b562
server/base/src/main/java/org/apache/accumulo/server/master/state/TabletStateStore.java
5413e31
server/base/src/main/java/org/apache/accumulo/server/master/state/ZooTabletStateStore.java
ab99396
server/base/src/main/java/org/apache/accumulo/server/replication/StatusUtil.java
898e3d4
server/base/src/main/java/org/apache/accumulo/server/util/ListVolumesUsed.java
e90d1dd
server/base/src/main/java/org/apache/accumulo/server/util/MasterMetadataUtil.java
4ca2d64
server/base/src/main/java/org/apache/accumulo/server/util/MetadataTableUtil.java
10cd749
server/base/src/main/java/org/apache/accumulo/server/util/ReplicationTableUtil.java
af02a8d
server/base/src/test/java/org/apache/accumulo/server/util/ReplicationTableUtilTest.java
b1010c2
server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectWriteAheadLogs.java
1735c0d
server/gc/src/main/java/org/apache/accumulo/gc/SimpleGarbageCollector.java
c8d5cd6
server/gc/src/main/java/org/apache/accumulo/gc/replication/CloseWriteAheadLogReferences.java
3a32727
server/gc/src/test/java/org/apache/accumulo/gc/GarbageCollectWriteAheadLogsTest.java
5801faa
server/gc/src/test/java/org/apache/accumulo/gc/replication/CloseWriteAheadLogReferencesTest.java
23db83a
server/master/src/main/java/org/apache/accumulo/master/Master.java
3762f32
server/master/src/main/java/org/apache/accumulo/master/MasterClientServiceHandler.java
f73c236
server/master/src/main/java/org/apache/accumulo/master/TabletGroupWatcher.java
d097d75
server/master/src/main/java/org/apache/accumulo/master/replication/WorkMaker.java
8532e1b
server/master/src/main/java/org/apache/accumulo/master/state/MergeStats.java
44f229e
server/master/src/test/java/org/apache/accumulo/master/ReplicationOperationsImplTest.java
a127dcd
server/master/src/test/java/org/apache/accumulo/master/TestMergeState.java
b0240f1
server/master/src/test/java/org/apache/accumulo/master/state/RootTabletStateStoreTest.java
abceae4
server/tserver/src/main/findbugs/exclude-filter.xml 47dd1f5
server/tserver/src/main/java/org/apache/accumulo/server/GarbageCollectionLogger.java
8f3785e
server/tserver/src/main/java/org/apache/accumulo/tserver/TabletLevel.java
PRE-CREATION
server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
662ee31
server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java
5acf5eb
server/tserver/src/main/java/org/apache/accumulo/tserver/log/SortedLogRecovery.java
c4d9fab
server/tserver/src/main/java/org/apache/accumulo/tserver/log/TabletServerLogger.java
711c497
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CommitSession.java
b4814e4
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/DatafileManager.java
594d9c5
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
6152500
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/TabletCommitter.java
4bc05a6
test/src/main/java/org/apache/accumulo/test/performance/thrift/NullTserver.java
b429607
test/src/test/java/org/apache/accumulo/proxy/ProxyDurabilityIT.java
404a8fd
test/src/test/java/org/apache/accumulo/test/BadDeleteMarkersCreatedIT.java
25337b2
test/src/test/java/org/apache/accumulo/test/BalanceIT.java f793925
test/src/test/java/org/apache/accumulo/test/CleanWalIT.java f553be8
test/src/test/java/org/apache/accumulo/test/ConditionalWriterIT.java
bd00f02
test/src/test/java/org/apache/accumulo/test/MissingWalHeaderCompletesRecoveryIT.java
b78a311
test/src/test/java/org/apache/accumulo/test/NoMutationRecoveryIT.java
6a9975c
test/src/test/java/org/apache/accumulo/test/ShellServerIT.java 56a6a70
test/src/test/java/org/apache/accumulo/test/functional/WALSunnyDayIT.java
PRE-CREATION
test/src/test/java/org/apache/accumulo/test/functional/WatchTheWatchCountIT.java
bd0555b
test/src/test/java/org/apache/accumulo/test/performance/RollWALPerformanceIT.java
PRE-CREATION
test/src/test/java/org/apache/accumulo/test/replication/GarbageCollectorCommunicatesWithTServersIT.java
5b89d9c
test/src/test/java/org/apache/accumulo/test/replication/MultiInstanceReplicationIT.java
9dec16e
test/src/test/java/org/apache/accumulo/test/replication/ReplicationIT.java
54348db
Diff: https://reviews.apache.org/r/32224/diff/
Testing
-------
Ran all tests, except RandomWalk.
Thanks,
Eric Newton