[IGNITE-20197] .NET: Thin 3.0: Perf: Excessive dictionary lookups by GetFieldByColumnName usages

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 3.0.0-beta1
• Fix Version/s: 3.0.0-beta2
• Component/s: platforms, thin client
• Labels: .NET ignite-3
• Epic Link: AI3 Performance
• Assignee: Pavel Tupitsyn
• Reporter: Pavel Tupitsyn
• Reviewer: Igor Sapego
• Created: 11/Aug/23 10:50
• Updated: 21/Aug/23 11:53
• Resolved: 21/Aug/23 11:53

Description

All usages of ReflectionUtils.GetFieldByColumnName are within loops.

As a result, GetFieldsByColumnName(type) is called for the same type multiple times.

We should remove ReflectionUtils.GetFieldByColumnName to avoid this pitfall and perform the first lookup outside of the loop.

Issue Links

• links to: GitHub Pull Request #2467