accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubbsii <...@git.apache.org>
Subject [GitHub] accumulo issue #233: ACCUMULO-4604 Improve 'accumulo classpath' command
Date Thu, 23 Mar 2017 23:55:05 GMT
Github user ctubbsii commented on the issue:

    https://github.com/apache/accumulo/pull/233
  
    Executing the following reveals a lot of duplicates out of the box. This should be resolved
with your other PR to trim out the redundantly configured jars from the site file.
    
    ```bash
    accumulo classpath | tr ':' "\n" | less | sort | uniq -c | grep -v ' 1 '
    ```
    
    To test that, I set `general.classpaths` to `/var/tmp/` in `accumulo-site.xml`, and re-ran
this command. I verified that it removed the duplicates, but it also added an item, `$(pwd)//var/tmp/`,
to the output. I set it to empty string, and it added `$(pwd)/` to the output. These are both
incorrect. The current directory should never get added, and absolute paths should not be
resolved relative to the current directory.
    
    Also, it's not clear that the expected output of this command should include stuff from
the `AccumuloClassLoader` or the `AccumuloVFSClassLoader` by default (without specifying some
sort of depth/level).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message