FailedChanges

Summary

  1. Initial benchmarks module (details)
  2. Simplify HeapDAGStorageProvider.getOrCreateTree() (details)
  3. Generate less garbage when building trees (details)
  4. RefDatabase API improvements (details)
Commit 406794ccb0c645f4fbd87417f3469f53bee3a0f2 by gabriel.roldan
Initial benchmarks module
Signed-off-by: Gabriel Roldan <gabriel.roldan@gmail.com>
The file was addedsrc/benchmarks/core/.pydevproject
The file was addedsrc/benchmarks/.gitignore
The file was addedsrc/benchmarks/core/src/main/java/org/locationtech/geogig/model/RevTreeBuilderBenchmarkState.java
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/RevObjects.java (diff)
The file was modifiedsrc/pom.xml (diff)
The file was addedsrc/benchmarks/core/pom.xml
The file was addedsrc/benchmarks/core/src/main/java/org/locationtech/geogig/model/RevTreeBuilderBenchmark.java
The file was addedsrc/benchmarks/core/src/main/java/org/locationtech/geogig/plumbing/diff/PreOrderDiffWalkBenchmarkState.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/diff/PreOrderDiffWalk.java (diff)
The file was addedsrc/benchmarks/core/src/main/java/org/locationtech/geogig/plumbing/diff/PreOrderDiffWalkBenchmark.java
The file was addedsrc/benchmarks/pom.xml
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/model/impl/RevObjectTestSupport.java (diff)
Commit 4c04c6f0f8259c10d0f1e19c38ae9e9c8ca5b91e by gabriel.roldan
Simplify HeapDAGStorageProvider.getOrCreateTree()
Signed-off-by: Gabriel Roldan <gabriel.roldan@gmail.com>
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/internal/HeapDAGStorageProvider.java (diff)
Commit b9b520209df07f5f9aba835c0e6e10a596d31604 by gabriel.roldan
Generate less garbage when building trees
Signed-off-by: Gabriel Roldan <gabriel.roldan@gmail.com>
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/internal/HeapDAGStorageProvider.java (diff)
The file was modifiedsrc/storage/temporary-rocksdb/src/main/java/org/locationtech/geogig/tempstorage/rocksdb/RocksdbDAGStorageProvider.java (diff)
The file was modifiedsrc/benchmarks/core/src/main/java/org/locationtech/geogig/model/RevTreeBuilderBenchmarkState.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/internal/DAGStorageProvider.java (diff)
The file was modifiedsrc/benchmarks/core/src/main/java/org/locationtech/geogig/model/RevTreeBuilderBenchmark.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/internal/ClusteringStrategy.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/model/internal/QuadTreeTestSupport.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/internal/DAGTreeBuilder.java (diff)
The file was modifiedsrc/storage/temporary-rocksdb/src/main/java/org/locationtech/geogig/tempstorage/rocksdb/RocksdbNodeStore.java (diff)
The file was modifiedsrc/storage/temporary-rocksdb/src/main/java/org/locationtech/geogig/tempstorage/rocksdb/RocksdbDAGStore.java (diff)
Commit 0ae50001ef55d8cd44c715bdb62febe18c141739 by gabriel.roldan
RefDatabase API improvements
RefDatbase used to work with key/value pairs where the key was the ref
name and the value either an ObjectId or a String with the target ref
name for symbolic references.
With Ref being a core model object it makes sense that RefDatabase works
with Ref directly.
Some methods where added to handle atomic operations on several refs,
and a new command UpdateRefs introduced to do so at the command level.
Most commands that update several refs were modified to use UpdateRefs
instead to doing several single-ref changes.
Consequently, UpdateRef and UpdateSymRef were changed to delegeate to
UpdateRefs, so that there's a single command in charge of updating the
RefDatabase objects.
Note setting a "reason for change" property on all three ref updating
commands is temporarily mandatory in order to catch any missing but
important ref update that could be a target for the reflog, for which an
initial no-op command hook was introduced.
Signed-off-by: Gabriel Roldan <gabriel.roldan@gmail.com>
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/StagingArea.java (diff)
The file was modifiedsrc/pullrequests/src/main/java/org/geogig/commands/pr/PR.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/repository/impl/FileRepositoryResolver.java (diff)
The file was modifiedsrc/pullrequests/src/main/java/org/geogig/commands/pr/PRMergeOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/UpdateRef.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/TransactionBlobStore.java
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/internal/ClusteringStrategy.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/NamespaceRefDatabase.java
The file was modifiedsrc/geotools/datastore/src/main/java/org/locationtech/geogig/geotools/data/GeogigTransactionState.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/storage/impl/SimpleLockingRefDatabase.java
The file was addedsrc/core/src/main/java/org/locationtech/geogig/dsl/Commands.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/UpdateSymRef.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/SquashOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/TagRemoveOp.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/TransactionConflictsDatabase.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/porcelain/InitOpTest.java (diff)
The file was modifiedsrc/storage/postgres/src/test/java/org/locationtech/geogig/storage/postgresql/v9/PGBlobStoreTest.java (diff)
The file was modifiedsrc/cli/core/src/test/resources/features/porcelain/Branch.feature (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/ForEachRef.java (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/plumbing/TransactionResolve.java
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/WorkingTree.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/internal/LocalMappedRemoteRepo.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/PushOp.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/storage/ConflictsDatabase.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/storage/fs/FileRefDatabase.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/RevObjects.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/di/ContextImpl.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/RevParse.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/index/IndexTestSupport.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/pack/PushOp.java (diff)
The file was modifiedsrc/remoting/src/test/java/org/locationtech/geogig/test/integration/remoting/PullOpTest.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/internal/LocalRemoteRepo.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/repository/impl/StagingAreaImpl.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/DiffTreeTest.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/FetchOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/ResetOp.java (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/storage/impl/TransactionBlobStore.java
The file was addedsrc/api/src/main/java/org/locationtech/geogig/storage/RefChange.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/storage/decorator/ForwardingRefDatabase.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/BranchDeleteOp.java (diff)
The file was modifiedsrc/remoting/src/test/java/org/locationtech/geogig/test/integration/remoting/FetchOpWithIndexTest.java (diff)
The file was modifiedsrc/storage/rocksdb/src/test/java/org/locationtech/geogig/rocksdb/RocksdbBlobStoreTest.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/RepositoryResolver.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/BranchDeleteOpTest.java (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/repository/impl/GeogigTransaction.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/storage/memory/HeapConflictsDatabase.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/repository/impl/WorkingTreeImpl.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/TransactionStagingArea.java
The file was addedsrc/core/src/main/java/org/locationtech/geogig/dsl/Conflicts.java
The file was modifiedsrc/cli/core/src/main/java/org/locationtech/geogig/cli/plumbing/ShowRef.java (diff)
The file was modifiedsrc/pullrequests/src/test/java/org/geogig/commands/pr/PRInitOpTest.java (diff)
The file was removedsrc/core/src/test/java/org/locationtech/geogig/storage/impl/TransactionBlobStoreTest.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/RefParseTest.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/ResolveTreeish.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/TagCreateOp.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/storage/RefDatabase.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/hooks/builtin/RefLogCommandHook.java
The file was modifiedsrc/pullrequests/src/main/java/org/geogig/commands/pr/PRDeleteOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/CommitOp.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/remotes/RemoteRemoveOpTest.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/CommitOpTest.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/AddOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/CleanRefsOp.java (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/storage/impl/TransactionStagingArea.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/hooks/builtin/UpdateIndexesHook.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/FindOrCreateSubtree.java (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/storage/impl/TransactionConflictsDatabase.java
The file was modifiedsrc/cli/core/src/main/java/org/locationtech/geogig/cli/plumbing/RevList.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/UpdateSymRefTest.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/LsRemoteOp.java (diff)
The file was addedsrc/core/src/test/java/org/locationtech/geogig/transaction/NamespaceRefDatabaseTest.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/storage/memory/HeapBlobStoreTest.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/InitOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/RevertAbort.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/RemoveOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/storage/fs/FileBlobStore.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/di/RefDatabaseSnapshot.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/storage/fs/FileBlobStoreTest.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/dsl/Geogig.java (diff)
The file was modifiedsrc/geotools/datastore/src/main/java/org/locationtech/geogig/geotools/data/GeoGigDataStore.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/remotes/RemoteRemoveOp.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/RevParseIntegrationTest.java (diff)
The file was addedsrc/core/src/test/java/org/locationtech/geogig/transaction/GeogigTransactionTest.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/ResolveBranchId.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/storage/memory/MemoryRepositoryResolver.java (diff)
The file was modifiedsrc/pullrequests/src/main/java/org/geogig/commands/pr/PRPrepareOp.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/TransactionBegin.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/BranchRenameOpTest.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/BranchRenameOp.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/GeogigTransaction.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/dsl/Refs.java (diff)
The file was addedsrc/core/src/test/java/org/locationtech/geogig/transaction/TransactionBlobStoreTest.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/CleanRefsOpTest.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/repository/IndexTest.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/pack/DiffRemoteRefsOp.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/pack/FetchOp.java (diff)
The file was modifiedsrc/pullrequests/src/test/java/org/geogig/commands/pr/PRMergeOpTest.java (diff)
The file was modifiedsrc/cli/core/src/main/java/org/locationtech/geogig/cli/porcelain/Log.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/MergeOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/storage/memory/HeapBlobStore.java (diff)
The file was modifiedsrc/storage/postgres/src/main/java/org/locationtech/geogig/storage/postgresql/v9/PGConflictsDatabase.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/ResolveTree.java (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/storage/impl/TransactionRefDatabase.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/MergeOpTest.java (diff)
The file was addedsrc/storage/postgres/src/main/java/org/locationtech/geogig/storage/postgresql/v9/PGRefDatabaseWorker.java
The file was modifiedsrc/storage/postgres/src/main/java/org/locationtech/geogig/storage/postgresql/v9/PGBlobStore.java (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/plumbing/TransactionEnd.java
The file was modifiedsrc/pullrequests/src/test/java/org/geogig/commands/pr/PRHealthCheckOpTest.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/FindFeatureTypeTrees.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/TransactionBlobStoreImpl.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/WriteTree2Test.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/repository/RefDatabaseTest.java (diff)
The file was modifiedsrc/storage/rocksdb/src/main/java/org/locationtech/geogig/rocksdb/RocksdbConflictsDatabase.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/DiffTree.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/porcelain/index/UpdateIndexesOpTest.java (diff)
The file was modifiedsrc/storage/postgres/src/main/java/org/locationtech/geogig/storage/postgresql/v9/PGRefDatabase.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/merge/ReportCommitConflictsOp.java (diff)
The file was modifiedsrc/pullrequests/src/main/java/org/geogig/commands/pr/PRCloseOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/CatObject.java (diff)
The file was modifiedsrc/cli/core/src/test/java/org/locationtech/geogig/cli/test/functional/DefaultStepDefinitions.java (diff)
The file was modifiedsrc/pullrequests/src/main/java/org/geogig/commands/pr/PRHealthCheckOp.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/ParseTimestampTest.java (diff)
The file was removedsrc/core/src/test/java/org/locationtech/geogig/test/integration/GeogigTransactionTest.java
The file was removedsrc/core/src/main/java/org/locationtech/geogig/storage/impl/TransactionBlobStoreImpl.java
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/RepositoryFinder.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/repository/impl/RepositoryResolverTest.java (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/plumbing/TransactionBegin.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/RefParse.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/index/BuildIndexOp.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/FindChangedTreesTest.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/dsl/Blobs.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/LogOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/BranchCreateOp.java (diff)
The file was modifiedsrc/pullrequests/src/test/java/org/geogig/commands/pr/PRPrepareOpTest.java (diff)
The file was modifiedsrc/cli/core/src/main/java/org/locationtech/geogig/cli/porcelain/Squash.java (diff)
The file was modifiedsrc/cli/remoting/src/main/java/org/locationtech/geogig/cli/remoting/Clone.java (diff)
The file was modifiedsrc/storage/postgres/src/main/java/org/locationtech/geogig/storage/postgresql/PGRepositoryResolver.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/SymRef.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/storage/memory/HeapRefDatabase.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/integration/RepositoryTestCase.java (diff)
The file was modifiedsrc/pullrequests/src/main/java/org/geogig/commands/pr/PRInitOp.java (diff)
The file was modifiedsrc/core/src/main/resources/META-INF/services/org.locationtech.geogig.hooks.CommandHook (diff)
The file was removedsrc/core/src/main/java/org/locationtech/geogig/storage/impl/AbstractRefDatabase.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/test/TestData.java (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/model/Ref.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/plumbing/UpdateRefs.java
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/TransactionEnd.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/index/UpdateIndexesOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/storage/decorator/ForwardingConflictsDatabase.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/RevParseTest.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/CheckoutOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/CherryPickOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/RevertOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/RebaseOp.java (diff)
The file was addedsrc/core/src/main/java/org/locationtech/geogig/transaction/TransactionResolve.java
The file was modifiedsrc/storage/rocksdb/src/main/java/org/locationtech/geogig/rocksdb/RocksdbBlobStore.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/plumbing/UpdateRefTest.java (diff)
The file was modifiedsrc/remoting/src/main/java/org/locationtech/geogig/remotes/internal/IRemoteRepo.java (diff)