Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-2442

[C++] Disambiguate Builder::Append overloads

    Details

      Description

      See discussion in https://github.com/apache/arrow/pull/1852#discussion_r179919627

      There are various Append() overloads in Builder and subclasses, some of which append one value, some of which append multiple values at once.

      The API might be clearer and less error-prone if multiple-append variants were named differently, for example AppendValues(). Especially with the pointer-taking variants, it's probably easy to call the wrong overload by mistake.

      The existing methods would have to go through a deprecation cycle.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                pitrou Antoine Pitrou
                Reporter:
                pitrou Antoine Pitrou
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: