Sure, I will. Thanks for fixing these Julian.

-Jesús




On 9/8/17, 1:44 PM, "Julian Hyde" <jh...@apache.org> wrote:

>And I discovered two more breakages from that change.
>
>SqlTypeFamily references java.sql.Types.TIMESTAMP_WITH_TIMEZONE, which doesn’t 
>exist in JDK 1.7, which breaks the build under JDK 1.7.
>
>And Long(long) constructor is deprecated in JDK 1.9, which generates a 
>deprecation warning (which we treat as a build error) under JDK 1.9.
>
>I’ll fix them.
>
>But this close to the release, please run under JDK 1.7, 1.8 and 1.9, 
>including javadoc and checking for deprecation warnings, before every commit.
>
>Julian
>
>
>> On Sep 7, 2017, at 1:50 PM, Jesus Camacho Rodriguez <jcama...@apache.org> 
>> wrote:
>> 
>> Julian,
>> 
>> I cannot repro in my environment. Could you share the errors that you are 
>> seeing?
>> 
>> Thanks,
>> 
>> -Jesús
>> 
>> 
>> 
>> On 9/7/17, 1:12 PM, "Jesus Camacho Rodriguez" 
>> <jcamachorodrig...@hortonworks.com on behalf of jcama...@apache.org> wrote:
>> 
>>> Sure, let me take a look.
>>> 
>>> -Jesús
>>> 
>>> 
>>> 
>>> 
>>> On 9/7/17, 10:44 AM, "Julian Hyde" <jh...@apache.org> wrote:
>>> 
>>>> Jesus,
>>>> 
>>>> I am seeing javadoc errors when I run "mvn site" (under JDK 1.8 on
>>>> Windows, as it happens) that are very likely from your CALCITE-1947
>>>> commit. Can you fix ASAP.
>>>> 
>>>> Julian
>>>> 
>>> 
>> 
>

Reply via email to