[TAJO-940] Default null config for CSV file in StoreTableExec should be moved to planning phase.

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Won't Fix
• Assignee: Hyunsik Choi
• Reporter: Hyunsik Choi
• Created: 15/Jul/14 02:14
• Updated: 16/Jan/15 06:51
• Resolved: 16/Jan/15 06:51

Description

For more clear code, the null config should be moved to the planning phase.

"StoreTableExec.java"

 if (plan instanceof InsertNode) {
      InsertNode createTableNode = (InsertNode) plan;
      appender = StorageManagerFactory.getStorageManager(context.getConf()).getAppender(meta,
          createTableNode.getTableSchema(), lastFileName);
    } else {
      String nullChar = context.getQueryContext().get(ConfVars.CSVFILE_NULL.varname, ConfVars.CSVFILE_NULL.defaultVal);
      meta.putOption(StorageConstants.CSVFILE_NULL, nullChar);
      appender = StorageManagerFactory.getStorageManager(context.getConf()).getAppender(meta, outSchema,
          lastFileName);
    }

Issue Links

• is related to: TAJO-1121 Remove the 'v2' storage package.