[PIG-3579] pig.script's deserialized version does not maintain line numbers

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Assignee: Jiaji Grace Zhang
• Reporter: Aniket Namadeo Mokashi
• Created: 16/Nov/13 01:34
• Updated: 18/Nov/13 20:03
• Resolved: 18/Nov/13 20:03

Description

If pig.script is decoded with base64, it loses line numbers because the buffered reader that adds the lines, removes '\n's.

ScriptState.java#setScript
protected void setScript(BufferedReader reader) {
        StringBuilder sb = new StringBuilder();
        try {
            String line = reader.readLine();
            while (line != null) {
                if (line.length() > 0) {
                    sb.append(line).append("\n");
                }
                line = reader.readLine();
            }
        } catch (IOException e) {
            LOG.warn("unable to parse the script", e);
        }
        setScript(sb.toString());
    }