Mailing-List: contact dev-help@hawq.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hawq.incubator.apache.org
MIME-Version: 1.0
References: <CAH=zMbqovYp30WMSVokpUFO-Ayxcr5QjU39p5RMnLSy-dOY=DA@mail.gmail.com>
 <CAPgpW0ZdQctBi59EH9XRnXLpJFBoz=Tc8KSRmaatOuucXL0wMA@mail.gmail.com>
 <CAPgpW0ZoOUf9s+rQhR1scCnofWwar5r0mRuQMWx8Q5yNp4=KCA@mail.gmail.com>
 <CAOPnHqdwQgyT=-+6ZE4DzjLCZkhGj9bfAgGzhvk9NhaducVEhg@mail.gmail.com>
 <CAH=zMbrjZZGSz5GALFu1AaoDcrM2BT-18N9Z5sO-KH4-dYmLBA@mail.gmail.com>
 <CAOPnHqdQ88ZLgyCZMkhe+rS6=cv_HRTdiZ95z-6p1YB=Mzktkw@mail.gmail.com>
 <CAH=zMbqWoU=j+nC-OFokdG2k0Yo353MUANJnnUQS=H33V72ZaA@mail.gmail.com> <CAPgpW0bfy1+de_8tjOH9XSvEZ=QCpmUQmPGfcG2yjjBes=zORw@mail.gmail.com>
In-Reply-To: <CAPgpW0bfy1+de_8tjOH9XSvEZ=QCpmUQmPGfcG2yjjBes=zORw@mail.gmail.com>
From: Goden Yao <godenyao@apache.org>
Date: Tue, 19 Jul 2016 20:53:39 +0000
Message-ID: <CAPgpW0bseRbZe+_w2FKS-Jxp_FTd64ZaCfxdECygLZiLuiC2JA@mail.gmail.com>
Subject: Re: HAWQ 2.0.0-incubating Release
To: dev@hawq.incubator.apache.org
Content-Type: multipart/alternative; boundary=001a113f0730294deb0538034835
archived-at: Tue, 19 Jul 2016 20:54:02 -0000

--001a113f0730294deb0538034835
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

I compiled latest content again in the release branch. build was successful
and installed.
found JavaDoc 100 Warnings, and filed JIRA:
https://issues.apache.org/jira/browse/HAWQ-941
But I don=E2=80=99t think it=E2=80=99s a blocker though.

There=E2=80=99s a few other warnings , e.g. dependencies

[WARNING] /Users/tyao/source/asf/incubator-hawq/contrib/hawq-hadoop/hawq-ma=
preduce-parquet/src/test/java/com/pivotal/hawq/mapreduce/parquet/HAWQParque=
tOutputDriver.java:
/Users/tyao/source/asf/incubator-hawq/contrib/hawq-hadoop/hawq-mapreduce-pa=
rquet/src/test/java/com/pivotal/hawq/mapreduce/parquet/HAWQParquetOutputDri=
ver.java
uses or overrides a deprecated API.
[WARNING] /Users/tyao/source/asf/incubator-hawq/contrib/hawq-hadoop/hawq-ma=
preduce-parquet/src/test/java/com/pivotal/hawq/mapreduce/parquet/HAWQParque=
tOutputDriver.java:
Recompile with -Xlint:deprecation for details.

I=E2=80=99m not sure if these matter, please let me know if you see issues =
or
concerns during compilation.

-Goden

On Tue, Jul 19, 2016 at 10:26 AM Goden Yao godenyao@apache.org
<http://mailto:godenyao@apache.org> wrote:

As release manager for this release, I'll check the branch status again. As
> this release only targets IP clean source tarball, I think these commits
> might be OK to sit in the branch to avoid churns.
>
> But please make sure we only cherry-pick "2.0.0.0-incubating" JIRAs into
> the branch next time. This is a process we should enforce in the future
> releases.
>
> -Goden
>
> On Mon, Jul 18, 2016 at 8:24 PM Radar Da lei <rlei@pivotal.io> wrote:
>
>> 'git push -f' kind of dangerous, re-create the branch should be a safe
>> way.
>> Thanks.
>>
>> Regards,
>> Radar
>>
>> On Tue, Jul 19, 2016 at 11:21 AM, Hong Wu <xunzhangthu@gmail.com> wrote:
>>
>> > Why not use `push -f` instead?
>> >
>> > xunzhang
>> >
>> > 2016-07-19 11:19 GMT+08:00 Radar Da lei <rlei@pivotal.io>:
>> >
>> > > When I was merging Hong's commit 'HAWQ-933' to branch
>> > '2.0.0.0-incubating',
>> > > below  7 commits are merged by accidents.
>> > >
>> > > HAWQ-894. Add feature test for polymorphism with new test framework
>> > > HAWQ-808. Refactor feature test for external_oid with new framework.
>> > > HAWQ-894. Add missed answer file for polymorphism case
>> > > HAWQ-924. Refactor feature test for querycontext with new test
>> framework
>> > > HAWQ-918. Fix memtuple forming bug when null-saved size is larger th=
an
>> > > 32763 bytes
>> > > HAWQ-917. Refactor feature tests for data type check with new
>> googletest
>> > > framework
>> > > HAWQ-896. Add feature test for create table with new test framework
>> > >
>> > > Sorry for the wrong merge, now I'm trying to revert these commits. B=
ut
>> > > revert so many commits looks ugly and will be confused, so I proposa=
l
>> let
>> > > me re-create the branch. The new created branch will be the same as
>> > before,
>> > > then the branch can keep clean.
>> > >
>> > > Any comments? Thanks.
>> > >
>> > > Regards,
>> > > Radar
>> > >
>> > > On Tue, Jul 19, 2016 at 10:35 AM, Hong Wu <xunzhangthu@gmail.com>
>> wrote:
>> > >
>> > > > HAWQ-933 is fixed(69f3c9026892bc9c6937ed132fa45716d7912a8f
>> > > > <
>> > > >
>> > >
>> >
>> https://github.com/apache/incubator-hawq/commit/69f3c9026892bc9c6937ed13=
2fa45716d7912a8f
>> > > > >),
>> > > > thank you for pointing this out. I will cherry-pick it to
>> > > > 2.0.0.0-incubating soon.
>> > > >
>> > > > Best
>> > > > xunzhang
>> > > >
>> > > > 2016-07-19 4:47 GMT+08:00 Goden Yao <godenyao@apache.org>:
>> > > >
>> > > > > Filed a JIRA for these warnings.
>> > > > >
>> > > > > https://issues.apache.org/jira/browse/HAWQ-933
>> > > > > Can someone who's familiar with gpnetbench* code, take a look an=
d
>> fix
>> > > > them
>> > > > > ?
>> > > > >
>> > > > > -Goden
>> > > > >
>> > > > > On Mon, Jul 18, 2016 at 11:04 AM Goden Yao <godenyao@apache.org>
>> > > wrote:
>> > > > >
>> > > > > > just compiled the latest from git pull.
>> > > > > >
>> > > > > > Build was successful with a few warnings, we probably want to
>> make
>> > > them
>> > > > > > clean.
>> > > > > >
>> > > > > > gpnetbenchClient.c:36:6: warning: no previous prototype for
>> > function
>> > > > > 'usage' [-Wmissing-prototypes]
>> > > > > > void usage()
>> > > > > >      ^
>> > > > > > gpnetbenchServer.c:34:6: warning: no previous prototype for
>> > function
>> > > > > 'usage' [-Wmissing-prototypes]
>> > > > > > void usage()
>> > > > > >      ^
>> > > > > > gpnetbenchClient.c:198:14: warning: comparison of unsigned
>> > > expression <
>> > > > > 0 is always false [-Wtautological-compare]
>> > > > > >                 if (retval < 0)
>> > > > > >                     ~~~~~~ ^ ~
>> > > > > > gpnetbenchClient.c:231:6: warning: no previous prototype for
>> > function
>> > > > > 'print_headers' [-Wmissing-prototypes]
>> > > > > > void print_headers()
>> > > > > >      ^
>> > > > > > gpnetbenchClient.c:33:6: note: this declaration is not a
>> prototype;
>> > > add
>> > > > > 'void' to make it a prototype for a zero-parameter function
>> > > > > > void print_headers();
>> > > > > >      ^
>> > > > > >                    void
>> > > > > > 1 warning generated.
>> > > > > > 3 warnings generated.
>> > > > > >
>> > > > > > On Mon, Jul 18, 2016 at 3:28 AM Radar Da lei rlei@pivotal.io
>> > > > > > <http://mailto:rlei@pivotal.io> wrote:
>> > > > > >
>> > > > > > I have merged fixes for 'HAWQ-622', 'HAWQ-926', 'HAWQ-915' int=
o
>> > > > > >> '2.0.0.0-incubating' branch.
>> > > > > >>
>> > > > > >> There seems no blocker issues left.
>> > > > > >>
>> > > > > >>
>> > > > > >> Regards,
>> > > > > >> Radar
>> > > > > >>
>> > > > > >> On Sat, Jul 16, 2016 at 5:05 AM, Goden Yao <
>> godenyao@apache.org>
>> > > > wrote:
>> > > > > >>
>> > > > > >> > Thanks Alan.
>> > > > > >> >
>> > > > > >> > Just a reminder - when you do commits in master, please che=
ck
>> > the
>> > > > JIRA
>> > > > > >> > associated with it and if it's labeled "2.0.0.0-incubating"
>> in
>> > fix
>> > > > > >> version,
>> > > > > >> > please port your commits to the release branch and close th=
e
>> > JIRA.
>> > > > > >> >
>> > > > > >> > I can see in the past a couple days,
>> > > > > >> >
>> > > > > >> >    - HAWQ-915 <
>> https://issues.apache.org/jira/browse/HAWQ-915>
>> > > RAT
>> > > > is
>> > > > > >> >    failing on the latest maste
>> > > > > >> >    - HAWQ-926 <
>> https://issues.apache.org/jira/browse/HAWQ-926>
>> > > > Remove
>> > > > > >> >    pycrypto from source code
>> > > > > >> >
>> > > > > >> > They're both needed in this release but I don't see them
>> being
>> > > > ported
>> > > > > to
>> > > > > >> > the release branch and as the JIRAs stay open, I'm not sure
>> if
>> > the
>> > > > > >> > assignees plan to augment the commits or not.
>> > > > > >> >
>> > > > > >> > Please check. thank you.
>> > > > > >> >
>> > > > > >> > -Goden
>> > > > > >>
>> > > > > > =E2=80=8B
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>>
> =E2=80=8B

--001a113f0730294deb0538034835--