...
 
Commits (2)
  • Marcus Gartner's avatar
    opt: make for loop in MapScanFilterCols more idiomatic · 59c9cd80
    Marcus Gartner authored
    This is a small clean-up to make a for loop within `MapScanFilterCols`
    of `xform/custom_funcs.go` more consistent with the style found
    elsewhere in the file. It now uses slice indexing to build a
    `FiltersExpr` rather than `append`.
    
    Release note: None
    59c9cd80
  • craig[bot]'s avatar
    Merge #48750 · 53607a3a
    craig[bot] authored
    48750: opt: make for loop in MapScanFilterCols more idiomatic r=mgartner a=mgartner
    
    This is a small clean-up to make a for loop within `MapScanFilterCols`
    of `xform/custom_funcs.go` more consistent with the style found
    elsewhere in the file. It now uses slice indexing to build a
    `FiltersExpr` rather than `append`.
    
    Release note: None
    Co-authored-by: default avatarMarcus Gartner <[email protected]>
    53607a3a
......@@ -2693,10 +2693,10 @@ func (c *CustomFuncs) MapScanFilterCols(
}
// Map the columns of each filter in the FiltersExpr.
newFilters := make([]memo.FiltersItem, 0, len(filters))
newFilters := make([]memo.FiltersItem, len(filters))
for i := range filters {
expr := c.MapFiltersItemCols(&filters[i], colMap)
newFilters = append(newFilters, c.e.f.ConstructFiltersItem(expr))
newFilters[i] = c.e.f.ConstructFiltersItem(expr)
}
return newFilters
......