SuccessChanges

Summary

  1. Allow RevTreeBuilder.build() to be cancelled. (details)
  2. Make index building aware of an external abort request (details)
  3. Move max bounds acquiring from IndexInfo to a helper method (details)
  4. Fix a bug in shriking and re-expanding DAG's in ClusteringStrategy (details)
Commit 11e18649483bb78408c4bea2cd7ac14126c171f3 by groldan
Allow RevTreeBuilder.build() to be cancelled.
Overload RevTreeBuilder.build() with build(BooleanSupplier) and
implement the internal machinery to abort the tree building process
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/impl/RevTreeBuilder.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/impl/LegacyTreeBuilder.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/internal/DAGTreeBuilder.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/impl/AbstractTreeBuilder.java
Commit f79df83caa3b0075c1c54dee30591ddd5ac2ffa9 by groldan
Make index building aware of an external abort request
Check for cancellation requests when building an index and abort
accordingly
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/index/UpdateIndexesOp.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/index/UpdateIndexOp.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/index/BuildIndexOp.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/hooks/builtin/UpdateIndexesHook.java
Commit ab5b413e797f01e814e49512f53b8cdf960e0292 by groldan
Move max bounds acquiring from IndexInfo to a helper method
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/plumbing/index/BuildIndexOp.java
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/IndexInfo.java
Commit 43e21c04168f0d28a0f5ac32f3d8edaca9300796 by groldan
Fix a bug in shriking and re-expanding DAG's in ClusteringStrategy
ClusteringStrategy.shrinkIfUnderflow() didn't reset the bucket DAGs to
the empty tree, producing trees with the wrong number of nodes when
nodes where removed and then new nodes were added forcing the bucket
node to re-expand. It nows clears any DAG that had an original RevTreeId
so the old tree nodes are not rea-added.
This patch also removes unnecessary arguments to shrinkIfUnderflow() and
mergeRoot().
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/internal/ClusteringStrategy.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/model/internal/QuadTreeTestSupport.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/internal/DAG.java
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/model/internal/QuadTreeClusteringStrategy_putTest.java
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/internal/QuadTreeClusteringStrategy.java