accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-3338) Update example classpath configuration
Date Sat, 15 Nov 2014 21:50:34 GMT


Christopher Tubbs commented on ACCUMULO-3338:

Why did this commit strip useful comments from the template? Especially since the {{}}
script explicitly looks for some of them, but was not also modified? Minimally, those comments
should be re-added, unless there is sufficient reason to justify removing them.

Also, under what circumstances are these {{/usr/hdp/current/\*}} paths appropriate? I'm not
familiar with that specific packaging, and I think there's possible risks for including too
broad a classpath (as is, I tend to recommend people modify the example configs, and avoid
the kinds of wildcards in the classpaths that we use in the example). (For that matter, I'm
not necessarily comfortable with the {{/usr/lib/hadoop\*}} ones either, because they seem
to map to a very specific 3rd party packaging (and I don't want to get into the business of
trying to support all possible 3rd party packaging schemes), but I'm not sure when those were

> Update example classpath configuration
> --------------------------------------
>                 Key: ACCUMULO-3338
>                 URL:
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: docs
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>            Priority: Trivial
>             Fix For: 1.6.2, 1.7.0
>          Time Spent: 20m
>  Remaining Estimate: 0h
> Make sure the example configuration general.classpaths contain some extra paths for HDFS
jars which make them more likely to generally work.

This message was sent by Atlassian JIRA

View raw message