SuccessChanges

Summary

  1. Allow RevTreeBuilder.build() to be cancelled. (details)
  2. Make index building aware of an external abort request (details)
  3. Give CLI running commands a chance to abort cleanly before closing the (details)
  4. Move max bounds acquiring from IndexInfo to a helper method (details)
  5. Fix a bug in shriking and re-expanding DAG's in ClusteringStrategy (details)
  6. Use the correct tree for export-diff. (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/AbstractTreeBuilder.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/internal/DAGTreeBuilder.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/impl/RevTreeBuilder.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/impl/LegacyTreeBuilder.java (diff)
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/UpdateIndexOp.java (diff)
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/index/BuildIndexOp.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/porcelain/index/UpdateIndexesOp.java (diff)
Commit 5d626f0e9b7534e190515c28373c26150d1f4715 by groldan
Give CLI running commands a chance to abort cleanly before closing the
repository on CTRL+C
Signed-off-by: Gabriel Roldan <groldan@boundlessgeo.com>
The file was modifiedsrc/cli-app/src/main/java/org/locationtech/geogig/cli/app/CLI.java (diff)
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 (diff)
The file was modifiedsrc/api/src/main/java/org/locationtech/geogig/repository/IndexInfo.java (diff)
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 (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/internal/DAG.java (diff)
The file was modifiedsrc/core/src/main/java/org/locationtech/geogig/model/internal/QuadTreeClusteringStrategy.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/model/internal/QuadTreeTestSupport.java (diff)
The file was modifiedsrc/core/src/test/java/org/locationtech/geogig/model/internal/QuadTreeClusteringStrategy_putTest.java (diff)
Commit 5e0aa9f1f0e112ecdfe6a85feb325ccee9ed194d by gabriel.roldan
Use the correct tree for export-diff.
It was always using the HEAD commit regardless of which commits were
involved in the diff.
Signed-off-by: Johnathan Garrett <jd@prominentedge.com>
The file was modifiedsrc/geotools/src/main/java/org/locationtech/geogig/geotools/geopkg/GeopkgDataStoreExportDiffOp.java (diff)
The file was modifiedsrc/geotools/src/main/java/org/locationtech/geogig/geotools/plumbing/DataStoreExportOp.java (diff)
The file was modifiedsrc/web/api/src/test/java/org/locationtech/geogig/rest/geopkg/GeoPackageExportDiffIntegrationTest.java (diff)