Commit cd36159
committed
fix: Remove ef_runtime from AggregateHybridQuery - not supported by FT.AGGREGATE
AggregateHybridQuery uses FT.AGGREGATE commands which do NOT support runtime
parameters. Runtime parameters (ef_runtime for HNSW, search_window_size for
SVS-VAMANA) only work with FT.SEARCH commands (VectorQuery, VectorRangeQuery).
Changes:
- Removed ef_runtime parameter from AggregateHybridQuery.__init__()
- Removed EF_RUNTIME and EF_RUNTIME_PARAM constants
- Removed ef_runtime handling from params property and _build_query_string()
- Removed test_aggregate_hybrid_query_ef_runtime() test
- Updated docs/api/query.rst to clarify no runtime parameters are supported
- Removed ef_runtime example from docs/user_guide/11_advanced_queries.ipynb
- Updated notebook to explain runtime parameters only work with VectorQuery
This fixes CI failures where ef_runtime was causing 'Invalid option' errors
when used with FT.AGGREGATE commands.1 parent 2cfd386 commit cd36159
File tree
4 files changed
+33
-127
lines changed- docs
- api
- user_guide
- redisvl/query
- tests/unit
4 files changed
+33
-127
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
126 | 126 | | |
127 | 127 | | |
128 | 128 | | |
129 | | - | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
130 | 132 | | |
131 | | - | |
| 133 | + | |
132 | 134 | | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
| 135 | + | |
138 | 136 | | |
139 | 137 | | |
140 | 138 | | |
141 | | - | |
| 139 | + | |
142 | 140 | | |
143 | | - | |
144 | | - | |
145 | | - | |
| 141 | + | |
146 | 142 | | |
147 | 143 | | |
148 | | - | |
149 | | - | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
150 | 147 | | |
151 | 148 | | |
152 | 149 | | |
| |||
0 commit comments