[ARROW-7289] [C#] ListType constructor argument is redundant

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.15.1
• Fix Version/s: 0.16.0
• Component/s: C#
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/16980
• Assignee: Takashi Hashida
• Reporter: Takashi Hashida
• Created: 02/Dec/19 05:57
• Updated: 11/Jan/23 07:52
• Resolved: 04/Dec/19 00:11

Description

https://github.com/apache/arrow/blob/master/csharp/src/Apache.Arrow/Types/ListType.cs#L28

 

The ListType constructor has two arguments but 'ValueDataType' can be determined by 'Filed.DataType' and 'ValueFiled' can be created by valueDataType.

It seems to me that the constructor should be separated to "ListType(Field valueField)" and " ListType(IArrowType valueDataType)".

 

Issue Links

• links to: GitHub Pull Request #5931