You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@arrow.apache.org by Nick Bruno <ni...@hotmail.com> on 2021/01/18 04:22:25 UTC

[Java] PR review for ARROW-11173

Hi All,

I'd like to get feedback on the pull request I created a little over a week ago - https://github.com/apache/arrow/pull/9151

It adds support for Map types in the readers / writers.

Let me know how I can improve it.

Thanks,
Nick

Re: [Java] PR review for ARROW-11173

Posted by Fan Liya <li...@gmail.com>.

I will take a look in one or two days.

Best,
Liya Fan

On Wed, Jan 20, 2021 at 3:48 AM Bryan Cutler <cu...@gmail.com> wrote:

> Hi Nick,
> I left a note in the PR that I will try to review soon, thanks!
>
>
> On Sun, Jan 17, 2021 at 8:22 PM Nick Bruno <ni...@hotmail.com> wrote:
>
> > Hi All,
> >
> > I'd like to get feedback on the pull request I created a little over a
> > week ago - https://github.com/apache/arrow/pull/9151
> >
> > It adds support for Map types in the readers / writers.
> >
> > Let me know how I can improve it.
> >
> > Thanks,
> > Nick
> >
>

Re: [Java] PR review for ARROW-11173

Posted by Bryan Cutler <cu...@gmail.com>.

Hi Nick,
I left a note in the PR that I will try to review soon, thanks!

On Sun, Jan 17, 2021 at 8:22 PM Nick Bruno <ni...@hotmail.com> wrote:

> Hi All,
>
> I'd like to get feedback on the pull request I created a little over a
> week ago - https://github.com/apache/arrow/pull/9151
>
> It adds support for Map types in the readers / writers.
>
> Let me know how I can improve it.
>
> Thanks,
> Nick
>