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

Posted to dev@arrow.apache.org by Andy Grove <an...@gmail.com> on 2018/04/03 04:05:07 UTC

Rust PRs

Hi,

I've been testing the Rust Arrow library with my other project and found
some issues that I've fixed.

Could someone please merge the following PRs (they are have JIRAs).

Bug fixes:

https://github.com/apache/arrow/pull/1821
https://github.com/apache/arrow/pull/1827

New functionality:

https://github.com/apache/arrow/pull/1823

Thanks,

Andy.

Re: Rust PRs

Posted by Andy Grove <an...@gmail.com>.

Thanks, Uwe.

This one is good to merge now (important bug fix):

https://github.com/apache/arrow/pull/1827

This one should be good to merge once tests finish on CI (adds JSON serde
support for data types):

https://github.com/apache/arrow/pull/1829

Thanks,

Andy.

On Tue, Apr 3, 2018 at 3:13 AM, Uwe L. Korn <uw...@xhochy.com> wrote:

> Hi Andy,
>
> I merged some of them, others have now merge conflicts and need a rebase.
> Feel free to ping me on them on github when they are ready to be merged.
> Otherwise, I try to make a pass over them after breakfast (I'm in CEST
> timezone-wise)
>
> Uwe
>
> On Tue, Apr 3, 2018, at 6:05 AM, Andy Grove wrote:
> > Hi,
> >
> > I've been testing the Rust Arrow library with my other project and found
> > some issues that I've fixed.
> >
> > Could someone please merge the following PRs (they are have JIRAs).
> >
> > Bug fixes:
> >
> > https://github.com/apache/arrow/pull/1821
> > https://github.com/apache/arrow/pull/1827
> >
> > New functionality:
> >
> > https://github.com/apache/arrow/pull/1823
> >
> > Thanks,
> >
> > Andy.
>

Re: Rust PRs

Posted by "Uwe L. Korn" <uw...@xhochy.com>.

Hi Andy, 

I merged some of them, others have now merge conflicts and need a rebase. Feel free to ping me on them on github when they are ready to be merged. Otherwise, I try to make a pass over them after breakfast (I'm in CEST timezone-wise)

Uwe

On Tue, Apr 3, 2018, at 6:05 AM, Andy Grove wrote:
> Hi,
> 
> I've been testing the Rust Arrow library with my other project and found
> some issues that I've fixed.
> 
> Could someone please merge the following PRs (they are have JIRAs).
> 
> Bug fixes:
> 
> https://github.com/apache/arrow/pull/1821
> https://github.com/apache/arrow/pull/1827
> 
> New functionality:
> 
> https://github.com/apache/arrow/pull/1823
> 
> Thanks,
> 
> Andy.