Skip to content

Commit 6ff76ff

Browse files
committed
Remove excessive MapTransform usage in the AggregateProvider
1 parent e2c24ed commit 6ff76ff

1 file changed

Lines changed: 2 additions & 28 deletions

File tree

Orm/Xtensive.Orm/Orm/Rse/Providers/Compilable/AggregateProvider.cs

Lines changed: 2 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,9 @@
55
// Created: 2008.09.18
66

77
using System;
8-
using System.Collections.Generic;
9-
using System.Linq;
10-
using Xtensive.Collections;
118
using Xtensive.Core;
129

1310
using Xtensive.Reflection;
14-
using Xtensive.Tuples;
15-
using Tuple = Xtensive.Tuples.Tuple;
16-
using Xtensive.Tuples.Transform;
1711

1812
namespace Xtensive.Orm.Rse.Providers
1913
{
@@ -30,17 +24,12 @@ public sealed class AggregateProvider : UnaryProvider
3024
/// <summary>
3125
/// Gets the aggregate columns.
3226
/// </summary>
33-
public AggregateColumn[] AggregateColumns { get; private set; }
27+
public AggregateColumn[] AggregateColumns { get; }
3428

3529
/// <summary>
3630
/// Gets column indexes to group by.
3731
/// </summary>
38-
public int[] GroupColumnIndexes { get; private set; }
39-
40-
/// <summary>
41-
/// Gets header resize transform.
42-
/// </summary>
43-
public MapTransform Transform { get; private set; }
32+
public int[] GroupColumnIndexes { get; }
4433

4534
/// <inheritdoc/>
4635
protected override RecordSetHeader BuildHeader()
@@ -69,21 +58,6 @@ protected override string ParametersToString()
6958
GroupColumnIndexes.ToCommaDelimitedString());
7059
}
7160

72-
/// <inheritdoc/>
73-
protected override void Initialize()
74-
{
75-
base.Initialize();
76-
var fieldTypes = new Type[GroupColumnIndexes.Length];
77-
var columnIndexes = new int[GroupColumnIndexes.Length];
78-
var i = 0;
79-
foreach (var index in GroupColumnIndexes) {
80-
fieldTypes[i] = Source.Header.Columns[index].Type;
81-
columnIndexes[i] = index;
82-
i++;
83-
}
84-
Transform = new MapTransform(false, TupleDescriptor.Create(fieldTypes), columnIndexes);
85-
}
86-
8761
/// <summary>
8862
/// Gets the type of the aggregate column according to a <see cref="AggregateType"/> and original column type.
8963
/// </summary>

0 commit comments

Comments
 (0)