Repository navigation
Commit 7b9fad2
fix: Use group_by in Polars backend aggregation (#6965)
polars removed DataFrame.groupby in 1.0 in favour of group_by, so
PolarsBackend.groupby_agg raised AttributeError and any aggregation run
by the local compute engine with backend=polars failed.
Signed-off-by: Yihang Chen <[email protected]>
Co-authored-by: Hao Xu <[email protected]>1 parent 5e7ef3e commit 7b9fad2
2 files changed
Lines changed: 30 additions & 1 deletion
File tree
- sdk/python
- feast/infra/compute_engines/backends
- tests/unit/infra/compute_engines/local
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
29 | | - | |
| 29 | + | |
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
| 6 | + | |
6 | 7 | | |
7 | 8 | | |
8 | 9 | | |
| |||
104 | 105 | | |
105 | 106 | | |
106 | 107 | | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
107 | 136 | | |
108 | 137 | | |
109 | 138 | | |
| |||
0 commit comments