Mailing-List: contact dev-help@hive.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hive.apache.org
Date: Sun, 16 Dec 2012 12:10:12 +0000 (UTC)
From: "qiangwang (JIRA)" <jira@apache.org>
To: hive-dev@hadoop.apache.org
Message-ID: <JIRA.12624184.1355659806950.13856.1355659812179@arcas>
In-Reply-To: <JIRA.12624184.1355659806950@arcas>
References: <JIRA.12624184.1355659806950@arcas>
Subject: [jira] [Created] (HIVE-3810) HiveHistory need replace '\r' with
 space before writing Entry.value to historyfile
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

qiangwang created HIVE-3810:
-------------------------------

             Summary: HiveHistory need replace '\r' with space before writing Entry.value to historyfile
                 Key: HIVE-3810
                 URL: https://issues.apache.org/jira/browse/HIVE-3810
             Project: Hive
          Issue Type: Bug
          Components: Logging
            Reporter: qiangwang
            Priority: Minor


HiveHistory.log will replace '\n' with space before writing Entry.value to history file:

    val = val.replace('\n', ' ');

but HiveHistoryViewer use BufferedReader.readLine which takes '\n', '\r', '\r\n'  as line delimiter to parse history file

if val contains '\r', there is a high possibility that HiveHistoryViewer.parseLine will fail, in which case usually RecordTypes.valueOf(recType) will throw exception 'java.lang.IllegalArgumentException'

HiveHistory.log need to replace '\r' with space as well:

- val = val.replace('\n', ' ');
+ val = val.replaceAll("\r|\n", " ");
or
- val = val.replace('\n', ' ');
+ val = val.replace('\r', ' ').replace('\n', ' ');


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira