Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Sqoop, mail # dev - Review Request: SQOOP-528: Remove references to incubator from sources


Copy link to this message
-
Re: Review Request: SQOOP-528: Remove references to incubator from sources
Jarek Cecho 2012-08-01, 15:15


> On Aug. 1, 2012, 1:50 p.m., Abhijeet Gaikwad wrote:
> > /pom-old.xml, line 41
> > <https://reviews.apache.org/r/6248/diff/1/?file=131391#file131391line41>
> >
> >     Not related to your change here, but do we need to update this version?

This pom file is there only for couple of developers (notice the "old" suffix in the name), so I let it there originally. I'll update it in next version.
> On Aug. 1, 2012, 1:50 p.m., Abhijeet Gaikwad wrote:
> > /DISCLAIMER.txt, line 11
> > <https://reviews.apache.org/r/6248/diff/1/?file=131388#file131388line11>
> >
> >     As Apache Sqoop is a Top Level Project now, this whole paragraph talking about on-going incubation of the project seems old.
> >     Do we require this file any more?

I've obviously only corrected the link without reading the purpose of the file. I'm sorry for that. I've looked into some other TLP projects and they do not have DISCLAMER file. Therefore I'll agree with the removal.
- Jarek
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6248/#review9688
-----------------------------------------------------------
On July 31, 2012, 7:35 p.m., Jarek Cecho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6248/
> -----------------------------------------------------------
>
> (Updated July 31, 2012, 7:35 p.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> I've removed references to the incubator.
>
>
> This addresses bug SQOOP-528.
>     https://issues.apache.org/jira/browse/SQOOP-528
>
>
> Diffs
> -----
>
>   /DISCLAIMER.txt 1367605
>   /ivy/sqoop-test.xml 1367605
>   /ivy/sqoop.xml 1367605
>   /pom-old.xml 1367605
>   /src/docs/dev/compiling.txt 1367605
>   /src/docs/dev/preface.txt 1367605
>   /src/docs/user/preface.txt 1367605
>   /src/docs/user/support.txt 1367605
>   /src/docs/web/footer.xsl 1367605
>
> Diff: https://reviews.apache.org/r/6248/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Jarek Cecho
>
>