Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Threaded View
Accumulo >> mail # dev >> [VOTE] Accumulo Instamo Archetype 1.4.4


Copy link to this message
-
Re: [VOTE] Accumulo Instamo Archetype 1.4.4 RC2
Good to know. Thanks for that link, especially.

--
Christopher L Tubbs II
http://gravatar.com/ctubbsii
On Wed, Sep 25, 2013 at 5:24 PM, Bill Havanki <[EMAIL PROTECTED]> wrote:
> The #set directives define escapes for special characters (I just found
> this out). See the section "Escape special characters" at:
>
> https://community.jboss.org/wiki/MavenArchetypeCreationGuidelines?_sscc=t
>
> The escapes only need to be there if they are used. It looks like
> MapReduceExample.java uses only symbol_escape, and the rest are not needed
> - but harmless to remain.
>
>
> On Wed, Sep 25, 2013 at 3:45 PM, Christopher <[EMAIL PROTECTED]> wrote:
>
>> #5, #6, and #7 need to be fixed. #1, #2, and #8 are trivial to fix
>> while addressing the others.
>>
>> #3 only needs to be fixed if the missing ones are important.. but I
>> don't really know what they're for.
>>
>>
>> --
>> Christopher L Tubbs II
>> http://gravatar.com/ctubbsii
>>
>>
>> On Wed, Sep 25, 2013 at 3:37 PM, Keith Turner <[EMAIL PROTECTED]> wrote:
>> > On Wed, Sep 25, 2013 at 3:10 PM, Christopher <[EMAIL PROTECTED]>
>> wrote:
>> >
>> >> MD5s and SHA1s look good, and so does the GPG signatures.
>> >>
>> >> I saw the following issues:
>> >>
>> >> 1) No javadoc jar (should be defined in the "apache-release" profile).
>> >> 2) Tarball looks like it isn't using gnu tar format. This could cause
>> >> problems with long filenames on some systems. You need the following
>> >> in your pluginManagement section:
>> >>         <plugin>
>> >>           <groupId>org.apache.maven.plugins</groupId>
>> >>           <artifactId>maven-assembly-plugin</artifactId>
>> >>           <configuration>
>> >>             <tarLongFileMode>gnu</tarLongFileMode>
>> >>           </configuration>
>> >>         </plugin>
>> >> (I really don't know why that isn't the default. This might be
>> >> addressed with the "apache-release" profile)
>> >> 3) What's the deal with the #set lines at the top of the file? What do
>> >> these do? Are these needed? Because ShellExample.java is missing them.
>> >> Is that expected?
>> >> 4) There appears to be two distinct READMEs. Would it be better to
>> >> consolidate them?
>> >>
>> >
>> > I think not.  One is directed to someone building the archetype and one
>> to
>> > someone using the archetype. I think it saves end users time to split
>> them.
>> >
>> >
>> >> 5) There appears to be a discrepancy between the LICENSE and NOTICE
>> >> files that are in the project and the archetype-resources. I'd expect
>> >> them to be the same.
>> >> 6) I can't see the commit you referenced above, because it hasn't been
>> >> pushed. You should push it in another branch, if you don't want to tag
>> >> it, so I can compare the source in that commit with the contents of
>> >> the tarball, before it's actually tagged as "1.4.4" upon vote success.
>> >> 7) It looks like the release plugin put in the wrong tag name
>> >> ("1.4.4-RC2" instead of "1.4.4").
>> >> 8) I think the attaching sources is redundant, because it's configured
>> >> in the parent POM's "apache-release" profile.
>> >>
>> >> Overall, -1, due to the above issues (though they are all minor, they
>> add
>> >> up).
>> >>
>> >
>> > Which issues do you think must be fixed?
>> >
>> >
>> >>
>> >> --
>> >> Christopher L Tubbs II
>> >> http://gravatar.com/ctubbsii
>> >>
>> >>
>> >> On Sun, Sep 22, 2013 at 10:42 PM, Josh Elser <[EMAIL PROTECTED]>
>> wrote:
>> >> > Devs,
>> >> >
>> >> > Please consider the following candidate project as a "sister" release
>> to
>> >> > Apache Accumulo 1.4.4: Apache Accumulo Instamo Archetype 1.4.4
>> >> >
>> >> > Staging repo:
>> >> >
>> >>
>> https://repository.apache.org/content/repositories/orgapacheaccumulo-089/
>> >> > Source tarball:
>> >> >
>> >>
>> https://repository.apache.org/content/repositories/orgapacheaccumulo-089/org/apache/accumulo/instamo-archetype/1.4.4/instamo-archetype-1.4.4-source-release.tar.gz
>> >> > Git tag: 56e3c674674dc2521a76dce2466360b1f82ca0ea (1.4.4-RC2)
>> >> >
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB