Skip to content

Commit e8f9ea7

Browse files
committed
docs: Add TODO comment to prevent re-indexing if previously indexed documents have not been deleted.
1 parent 109ebb9 commit e8f9ea7

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

internal/controllers/policy/policy_controller.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -363,6 +363,8 @@ func (r *ResourceIndexPolicyReconciler) Reconcile(ctx context.Context, req ctrl.
363363

364364
// Trigger background re-indexing
365365
if needsReindexing {
366+
// TODO: Do not re-index resources if the previously indexed documents
367+
// have not been deleted. To be done here https://github.com/datum-cloud/search/issues/32
366368
logger.Info("Triggering background re-indexing", "specHash", currentHash)
367369
if err := r.publishReindexMessages(ctx, policy); err != nil {
368370
logger.Error(err, "Failed to publish re-index messages")

0 commit comments

Comments
 (0)