eerhardt commented on a change in pull request #7654: URL: https://github.com/apache/arrow/pull/7654#discussion_r457569636
########## File path: csharp/src/Apache.Arrow/Arrays/DelegatingArrayBuilder.cs ########## @@ -0,0 +1,96 @@ +//------------------------------------------------------------------------------ +// <copyright file="DelegatingArrayBuilder.cs" company="Jetstone Asset Management LLP"> +// Copyright (C) Jetstone Asset Management LLP. All rights reserved. +// Unauthorised copying of this file, via any medium, is strictly prohibited. +// Proprietary and confidential +// </copyright> +// <author>Adam Szmigin</author> +//------------------------------------------------------------------------------ + +using System; +using Apache.Arrow.Memory; + +namespace Apache.Arrow +{ + /// <summary> + /// The <see cref="DelegatingArrayBuilder{T,TArray,TBuilder}"/> class can be used as the base for any array builder + /// that needs to delegate most of its functionality to an inner array builder. + /// </summary> + /// <remarks> + /// The typical use case is when an array builder may accept a number of different types as input, but which are + /// all internally converted to a single type for assembly into an array. + /// </remarks> + /// <typeparam name="T">Type of item accepted by inner array builder.</typeparam> + /// <typeparam name="TArray">Type of array produced by this (and the inner) builder.</typeparam> + /// <typeparam name="TBuilder">Type of builder (see Curiously-Recurring Template Pattern).</typeparam> + public abstract class DelegatingArrayBuilder<T, TArray, TBuilder> : IArrowArrayBuilder<TArray, TBuilder> Review comment: > and probably NodaTime.Instant if we use NodaTime for the TZDB Just a FYI, see this [previous discussion](https://github.com/apache/arrow/pull/5965#discussion_r356233161) where we discussed NodaTime. In short, I don't believe the Arrow library should have a dependency on NodaTime. But instead, the Arrow API should allow for a consumer to use NodaTime and Arrow together. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org