Add comments

chroma-core · Dec 10, 2024 · b6d727f · b6d727f
1 parent ba52dd6
commit b6d727f
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/chromadb/api/segment.py b/chromadb/api/segment.py
@@ -895,12 +895,15 @@ def _get_collection(self, collection_id: UUID) -> t.Collection:
     @trace_method("SegmentAPI._scan", OpenTelemetryGranularity.ALL)
     def _scan(self, collection_id: UUID) -> Scan:
         collection_segments = self._sysdb.get_collection_with_segments(collection_id)
+        # For now collection should have exactly one segment per scope:
+        # - Local scopes: vector, metadata
+        # - Distributed scopes: vector, metadata, record 
         scope_to_segment = {segment["scope"]: segment for segment in collection_segments["segments"]}
         return Scan(
             collection=collection_segments["collection"],
             knn=scope_to_segment[t.SegmentScope.VECTOR],
             metadata=scope_to_segment[t.SegmentScope.METADATA],
-            # Local chroma do not have record segment
+            # Local chroma do not have record segment, and this is not used by the local executor
             record=scope_to_segment.get(t.SegmentScope.RECORD, None),  # type: ignore[arg-type]
         )