Github user cestella commented on the pull request:

    https://github.com/apache/incubator-metron/pull/22#issuecomment-183613907
  
    I want to point out a couple of other things this PR provides that aren't 
strictly associated with the feature above, but are general cleanup tasks:
    
    * Removed lingering hbase-site.xml which have a bad habit of finding their 
way onto the classpath and confusing HBase in integration tests
    * The split of integration tests (defined as a test that ends with 
"IntegrationTest") into the integration-test maven lifecycle phase
    * Using the shade maven plugin to relocate our guava dependency so that we 
can use a more recent version of Guava than 12 (which is the most recent that 
HBase will allow due to google's habit of aggressive removal of deprecated 
code).  This comes up when running HBase in minicluster mode as well as in 
situations when running bolts which have to package the hbase-client.
    * General cleanup of the build to use the version properties instead of 
hard coding different versions of common components (e.g. hbase-client, 
storm-core, etc.)


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to