You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@calcite.apache.org by Atri Sharma <at...@gmail.com> on 2017/06/27 18:03:38 UTC
PR for Calcite 873
Hi All,
Please find PR for Calcite 873 at:
https://github.com/apache/calcite/pull/485
Please let me know if there are any comments or feedback.
Regards,
Atri
--
Regards,
Atri
l'apprenant
Re: PR for Calcite 873
Posted by Atri Sharma <at...@gmail.com>.
Sure, thanks, will do
On Thu, Jun 29, 2017 at 10:50 PM, Julian Hyde <jh...@apache.org> wrote:
> OK, I will review.
>
> Next time, please put these comments in the JIRA case, not as an email the dev list. It reduces traffic on the dev list, and it makes the history clear to whoever is reading the JIRA. Anyone who is interested in that case will already have subscribed to it as a “watcher”.
>
>> On Jun 29, 2017, at 10:11 AM, Atri Sharma <at...@gmail.com> wrote:
>>
>> Hi Julian,
>>
>> I have fixed your comments and updated. Please take a look and let me know.
>>
>> Regards,
>>
>> Atri
>>
>> On Wed, Jun 28, 2017 at 1:29 AM, Julian Hyde <jh...@apache.org> wrote:
>>> Thanks for the PR! See my review comments in
>>> https://issues.apache.org/jira/browse/CALCITE-873.
>>>
>>> On Tue, Jun 27, 2017 at 11:18 AM, Atri Sharma <at...@gmail.com> wrote:
>>>> Just noticed that my editor decided to fold the imports automatically. Will
>>>> fix that tomorrow
>>>>
>>>> On Jun 27, 2017 11:33 PM, "Atri Sharma" <at...@gmail.com> wrote:
>>>>
>>>>> Hi All,
>>>>>
>>>>> Please find PR for Calcite 873 at:
>>>>>
>>>>> https://github.com/apache/calcite/pull/485
>>>>>
>>>>> Please let me know if there are any comments or feedback.
>>>>>
>>>>> Regards,
>>>>>
>>>>> Atri
>>>>>
>>>>> --
>>>>> Regards,
>>>>>
>>>>> Atri
>>>>> l'apprenant
>>>>>
>>
>>
>>
>> --
>> Regards,
>>
>> Atri
>> l'apprenant
>
--
Regards,
Atri
l'apprenant
Re: PR for Calcite 873
Posted by Julian Hyde <jh...@apache.org>.
OK, I will review.
Next time, please put these comments in the JIRA case, not as an email the dev list. It reduces traffic on the dev list, and it makes the history clear to whoever is reading the JIRA. Anyone who is interested in that case will already have subscribed to it as a “watcher”.
> On Jun 29, 2017, at 10:11 AM, Atri Sharma <at...@gmail.com> wrote:
>
> Hi Julian,
>
> I have fixed your comments and updated. Please take a look and let me know.
>
> Regards,
>
> Atri
>
> On Wed, Jun 28, 2017 at 1:29 AM, Julian Hyde <jh...@apache.org> wrote:
>> Thanks for the PR! See my review comments in
>> https://issues.apache.org/jira/browse/CALCITE-873.
>>
>> On Tue, Jun 27, 2017 at 11:18 AM, Atri Sharma <at...@gmail.com> wrote:
>>> Just noticed that my editor decided to fold the imports automatically. Will
>>> fix that tomorrow
>>>
>>> On Jun 27, 2017 11:33 PM, "Atri Sharma" <at...@gmail.com> wrote:
>>>
>>>> Hi All,
>>>>
>>>> Please find PR for Calcite 873 at:
>>>>
>>>> https://github.com/apache/calcite/pull/485
>>>>
>>>> Please let me know if there are any comments or feedback.
>>>>
>>>> Regards,
>>>>
>>>> Atri
>>>>
>>>> --
>>>> Regards,
>>>>
>>>> Atri
>>>> l'apprenant
>>>>
>
>
>
> --
> Regards,
>
> Atri
> l'apprenant
Re: PR for Calcite 873
Posted by Atri Sharma <at...@gmail.com>.
Hi Julian,
I have fixed your comments and updated. Please take a look and let me know.
Regards,
Atri
On Wed, Jun 28, 2017 at 1:29 AM, Julian Hyde <jh...@apache.org> wrote:
> Thanks for the PR! See my review comments in
> https://issues.apache.org/jira/browse/CALCITE-873.
>
> On Tue, Jun 27, 2017 at 11:18 AM, Atri Sharma <at...@gmail.com> wrote:
>> Just noticed that my editor decided to fold the imports automatically. Will
>> fix that tomorrow
>>
>> On Jun 27, 2017 11:33 PM, "Atri Sharma" <at...@gmail.com> wrote:
>>
>>> Hi All,
>>>
>>> Please find PR for Calcite 873 at:
>>>
>>> https://github.com/apache/calcite/pull/485
>>>
>>> Please let me know if there are any comments or feedback.
>>>
>>> Regards,
>>>
>>> Atri
>>>
>>> --
>>> Regards,
>>>
>>> Atri
>>> l'apprenant
>>>
--
Regards,
Atri
l'apprenant
Re: PR for Calcite 873
Posted by Julian Hyde <jh...@apache.org>.
Thanks for the PR! See my review comments in
https://issues.apache.org/jira/browse/CALCITE-873.
On Tue, Jun 27, 2017 at 11:18 AM, Atri Sharma <at...@gmail.com> wrote:
> Just noticed that my editor decided to fold the imports automatically. Will
> fix that tomorrow
>
> On Jun 27, 2017 11:33 PM, "Atri Sharma" <at...@gmail.com> wrote:
>
>> Hi All,
>>
>> Please find PR for Calcite 873 at:
>>
>> https://github.com/apache/calcite/pull/485
>>
>> Please let me know if there are any comments or feedback.
>>
>> Regards,
>>
>> Atri
>>
>> --
>> Regards,
>>
>> Atri
>> l'apprenant
>>
Re: PR for Calcite 873
Posted by Atri Sharma <at...@gmail.com>.
Just noticed that my editor decided to fold the imports automatically. Will
fix that tomorrow
On Jun 27, 2017 11:33 PM, "Atri Sharma" <at...@gmail.com> wrote:
> Hi All,
>
> Please find PR for Calcite 873 at:
>
> https://github.com/apache/calcite/pull/485
>
> Please let me know if there are any comments or feedback.
>
> Regards,
>
> Atri
>
> --
> Regards,
>
> Atri
> l'apprenant
>