Thanks Sidd, will wait for your response. -Atul
-------- Original message -------- From: Siddharth Teotia <[email protected]> Date: 6/15/18 8:37 PM (GMT+05:30) To: [email protected] Subject: Re: JDBC Adapter PR - 1759 Atul, I will go through it first thing today morning and assuming everything looks good, let's plan to get this merged soon. Thanks for all your work and my apologies for delay. Thanks Sidd On Thu, Jun 14, 2018, 9:58 PM Wes McKinney <[email protected]> wrote: > I commented on the PR. I think this needs a final review / approval from > Sidd > > Thanks > Wes > > On Wed, Jun 13, 2018 at 11:41 PM, Atul Dambalkar > <[email protected]> wrote: > > Hi Sid, Laurent, > > > > Any idea when you would get a chance to merge the PR - > https://github.com/apache/arrow/pull/1759/ > > > > Regards, > > -Atul > > > > -----Original Message----- > > From: Atul Dambalkar > > Sent: Friday, June 01, 2018 4:39 PM > > To: [email protected] > > Subject: RE: JDBC Adapter PR - 1759 > > > > Hi Laurent, > > > > Thanks for your review comments. We have completed the code changes and > merged as well. I have replied to few your comments. Please take a look > when you get a chance. > > > > Regards, > > -Atul > > > > -----Original Message----- > > From: Laurent Goujon [mailto:[email protected]] > > Sent: Wednesday, May 30, 2018 5:38 AM > > To: [email protected] > > Subject: Re: JDBC Adapter PR - 1759 > > > > Same here. > > > > On Tue, May 29, 2018 at 9:59 AM, Siddharth Teotia <[email protected]> > > wrote: > > > >> Hi Atul, > >> > >> I will take a look today. > >> > >> Thanks, > >> Sidd > >> > >> On Tue, May 29, 2018 at 2:45 AM, Atul Dambalkar < > >> [email protected]> > >> wrote: > >> > >> > Hi Sid, Laurent, Uwe, > >> > > >> > Any idea when can someone take a look at the PR > >> https://github.com/apache/ > >> > arrow/pull/1759/. > >> > > >> > Laurent had given bunch of comments earlier and now we have taken > >> > care of most of those. We have also added multiple test cases. It > >> > will be great > >> if > >> > someone can take a look. > >> > > >> > Regards, > >> > -Atul > >> > > >> > > >> >
