Mailing-List: contact dev-help@atlas.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@atlas.incubator.apache.org
MIME-Version: 1.0
In-Reply-To: <e94d5b44-9988-fab0-8e6f-3cbcb020a828@nanthrax.net>
References: <OFBB652156.44DE3B4A-ON002580AC.003AF6B0-802580AC.003E21AC@notes.na.collabserv.com>
 <CADz7AfwzvLJp-CR4xxCtvF_QaYFuai-hPxK_EkFP3rCANZKdGg@mail.gmail.com> <e94d5b44-9988-fab0-8e6f-3cbcb020a828@nanthrax.net>
From: =?UTF-8?B?SXNtYcOrbCBNZWrDrWE=?= <iemejia@gmail.com>
Date: Wed, 18 Jan 2017 16:07:52 +0100
Message-ID: <CAE7Uba9uSwbctT2=x_a15=Skkvc6Kw5fT7q-WgWaWtar+74bxA@mail.gmail.com>
Subject: Re: Fw: [VOTE] Release Apache Atlas 0.7.1 (incubating) - release
 candidate 2 (dev group vote)
To: dev@atlas.incubator.apache.org
Content-Type: multipart/alternative; boundary=f403045fb6700bf2cb05465fc95b
archived-at: Wed, 18 Jan 2017 15:08:24 -0000

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

+1 (non-binding)

All signatures/SHAs ok.

The UI issue (breadcrumbs) broke on 0.7.0 works well in version 0.7.1.

I had one test breaking on Atlas Web Application, and the build process was
a bit messy, for more
details:

1. I tried first:

    mvn clean package -Pdist,embedded-hbase-solr -DskipTests

2. and it broke, so I followed the instructions :)

    export MAVEN_OPTS=3D"-Xmx1536m -XX:MaxPermSize=3D512m" && mvn clean ins=
tall

It broke in this test:

Results :

Failed tests:
  EntityJerseyResourceIT.testDeleteExistentTraitNonExistentForEntity:696 =
=C2=BB
JSON ...

Tests run: 81, Failures: 1, Errors: 0, Skipped: 0

3. So I retried to produce the distribution to test the UI issue with the
same
   command in (1) and it worked perfectly.

I think that apart of the test, there is some issue with the dependencies
order that didn't let me build the binary distribution without doing
the maven install.

Note I tested on Ubuntu Linux 16.04 (x64) with Oracle Jdk 8.

Isma=C3=ABl Mej=C3=ADa


On Wed, Jan 18, 2017 at 2:12 PM, Jean-Baptiste Onofr=C3=A9 <jb@nanthrax.net=
>
wrote:

> +1
>
>
> Checked the build, legal/headers, signatures.
>
> Regards
> JB
>
>
> On 01/18/2017 05:09 AM, Keval Bhatt wrote:
>
>> Hi,
>>
>> Verified the build from apache-atlas-0.7.1-incubating-sources.tar.gz
>> <https://dist.apache.org/repos/dist/dev/incubator/atlas/0.7.
>> 1-incubating-rc2/apache-atlas-0.7.1-incubating-sources.tar.gz>
>>
>>
>> Checksum is fine.
>> Build was successful.
>> Added test data from quickstart script
>> Verified the UI and all the basic UI functionality is working fine.
>>
>> +1
>>
>> Thanks
>> Keval
>>
>>
>>
>>
>> On Wed, Jan 18, 2017 at 4:48 PM, David Radley <david_radley@uk.ibm.com>
>> wrote:
>>
>> Atlas team,
>>> I am not sure if the resdme updates have gone in; from the link below t=
he
>>> readme seems not to be correct.
>>>
>>>
>>> I have downloaded the tar.gz file.
>>>
>>>
>>> Readme feedback
>>> 1) I notice the readme starts by saying to get the code using "git clon=
e
>>> https://github.com/apache/incubator-atlas.git"
>>> This is the wrong git command - as this will put the code into git - no=
t
>>> clone it from the repository into a local folder.
>>>
>>> I look on http://atlas.incubator.apache.org/InstallationSteps.html. It
>>> says to use "git clone
>>> https://git-wip-us.apache.org/repos/asf/incubator-atlas.git atlas"
>>> We need to document a 0.7.1 equivalent for the clone in the readme.
>>>
>>> 2) I think that the readme should explicitly mention the version number
>>> 0.7.1 and also point to the latest code 0.8 and explicitly talk about t=
he
>>> need for 0.7.1, rather than using 0.8, which I assume is primarily the
>>> java level. I think all the urls should default to the latest code (0.8=
).
>>>
>>>
>>> 3) I suggest that the readme includes the versions of any libraries tha=
t
>>> need to be on a machine to be able to build. I would mention the requir=
ed
>>> levels of Java, Maven, Git and Python. I have not installed as I was no=
t
>>> sure of the levels I need of these prereqs.
>>>
>>> The vote will be open for at least 72 hours or until necessary votes ar=
e
>>> reached.
>>> [ ] +1  approve
>>> [ ] +0  no opinion
>>> [ ] -1  disapprove (and reason why)
>>>
>>> Here is my +1, as long as we address the above points.
>>>
>>> Thanks,
>>> David
>>>
>>>
>>>
>>>
>>> Unless stated otherwise above:
>>> IBM United Kingdom Limited - Registered in England and Wales with numbe=
r
>>> 741598.
>>> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6
>>> 3AU
>>>
>>>
>>
> --
> Jean-Baptiste Onofr=C3=A9
> jbonofre@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com
>

--f403045fb6700bf2cb05465fc95b--