[GORA-465] Remove @Deprecated logic for hbase-mapping.xml

• Type: Test
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 0.6.1
• Fix Version/s: 0.7
• Component/s: gora-hbase
• Assignee: Cihad Guzel
• Reporter: Lewis John McGibbney
• Created: 27/Jan/16 06:19
• Updated: 24/May/16 13:24
• Resolved: 24/May/16 13:02

Description

We have some useless checking and log statements in HBaseStore which relate to a very old hbase-mapping.xml.
These should just be removed.

  @Deprecated
  private static final String DEPRECATED_MAPPING_FILE = "hbase-mapping.xml";
...
      try {
        mapping = readMapping(getConf().get(PARSE_MAPPING_FILE_KEY, DEPRECATED_MAPPING_FILE));
        LOG.warn("{} is deprecated, please rename the file to {}", DEPRECATED_MAPPING_FILE, DEFAULT_MAPPING_FILE);
      } catch (FileNotFoundException ex1) {
        LOG.error(ex1.getMessage(), ex1);
        //throw (ex1); //throw the original exception
      } catch (Exception ex1) {
        LOG.warn("{} is deprecated, please rename the file to {}", DEPRECATED_MAPPING_FILE, DEFAULT_MAPPING_FILE);
        throw new RuntimeException(ex1);
      } 

etc

Issue Links

• links to: GitHub Pull Request #67