Skip to content
This repository was archived by the owner on Nov 7, 2025. It is now read-only.

Commit 5e41c30

Browse files
authored
Removing field caps map capabilities to be on par with elasticsearch (#1367)
Elasticsearch does not emit `.keys()` and `.values()` for maps. Definitely we should not emit `text` type as well. Below `bar` index is handled by elasticsearch. <img width="1686" alt="image" src="https://github.com/user-attachments/assets/b2480303-0a79-49f9-ae9f-2c6aa81f0afb" /> and `foo` index handled by `quesma` and `clickhouse` after fix. <img width="1689" alt="image" src="https://github.com/user-attachments/assets/3e36a022-75a8-464e-a733-2be5be1b3336" /> <!-- A note on testing your PR --> <!-- Basic unit test run is executed against each commit in the PR. If you want to run a full integration test suite, you can trigger it by commenting with '/run-integration-tests' -->
1 parent 7db7f5b commit 5e41c30

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

platform/functionality/field_capabilities/field_caps.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,10 +73,6 @@ func handleFieldCapsIndex(cfg map[string]config.IndexConfiguration, schemaRegist
7373
case "keyword", "text":
7474
addFieldCapabilityFromSchemaRegistry(fields, fmt.Sprintf("%s%s", fieldName.AsString(), types.MultifieldKeywordSuffix), schema.QuesmaTypeKeyword, resolvedIndex)
7575
addFieldCapabilityFromSchemaRegistry(fields, fmt.Sprintf("%s%s", fieldName.AsString(), types.MultifieldTextSuffix), schema.QuesmaTypeText, resolvedIndex)
76-
case "map":
77-
addFieldCapabilityFromSchemaRegistry(fields, fmt.Sprintf("%s%s", fieldName.AsString(), types.MultifieldTextSuffix), schema.QuesmaTypeText, resolvedIndex)
78-
addFieldCapabilityFromSchemaRegistry(fields, fmt.Sprintf("%s%s", fieldName.AsString(), types.MultifieldMapKeysSuffix), schema.QuesmaTypeText, resolvedIndex)
79-
addFieldCapabilityFromSchemaRegistry(fields, fmt.Sprintf("%s%s", fieldName.AsString(), types.MultifieldMapValuesSuffix), schema.QuesmaTypeText, resolvedIndex)
8076
}
8177
}
8278

0 commit comments

Comments
 (0)