[ARROW-13128] [C#] TimestampArray conversion logic for nano and micro is wrong

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0.1
• Fix Version/s: 5.0.0
• Component/s: C#
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/28829
• Assignee: Xavier Lacroze
• Reporter: Xavier Lacroze
• Created: 19/Jun/21 21:32
• Updated: 11/Jan/23 08:31
• Resolved: 30/Jun/21 19:53

Description

The logic in this line looks wrong

case TimeUnit.Microsecond:
    return ticks / TimeSpan.TicksPerMillisecond / 1000;

It should read

case TimeUnit.Microsecond:
    return (ticks / TimeSpan.TicksPerMillisecond) * 1000;

Issue Links

• links to: GitHub Pull Request #10561