Mailing-List: contact dev-help@hive.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hive.apache.org
Date: Thu, 8 Dec 2011 00:50:41 +0000 (UTC)
From: "Phabricator (Updated) (JIRA)" <jira@apache.org>
To: hive-dev@hadoop.apache.org
Message-ID: 
 <571323089.51651.1323305441359.JavaMail.tomcat@hel.zones.apache.org>
Subject: [jira] [Updated] (HIVE-1040) use sed rather than diff for masking
 out noise in diff-based tests
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/HIVE-1040?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Phabricator updated HIVE-1040:
------------------------------

    Attachment: HIVE-1040.D597.3.patch

mareksapotafb updated the revision "HIVE-1040 [jira] use sed rather than diff for masking out noise in diff-based tests".
Reviewers: JIRA, jsichi

  Don't load .q.out files to memory.  This patch masks both result and reference
  during the run, so I don't have to attach all the .q.out files=)

REVISION DETAIL
  https://reviews.facebook.net/D597

AFFECTED FILES
  ql/src/test/org/apache/hadoop/hive/ql/QTestUtil.java

                
> use sed rather than diff for masking out noise in diff-based tests
> ------------------------------------------------------------------
>
>                 Key: HIVE-1040
>                 URL: https://issues.apache.org/jira/browse/HIVE-1040
>             Project: Hive
>          Issue Type: Improvement
>          Components: Testing Infrastructure
>    Affects Versions: 0.4.1
>            Reporter: John Sichi
>            Assignee: Marek Sapota
>            Priority: Minor
>         Attachments: HIVE-1040-code-patch.patch, HIVE-1040.1.patch, HIVE-1040.2.patch, HIVE-1040.D597.1.patch, HIVE-1040.D597.2.patch, HIVE-1040.D597.3.patch
>
>
> The current diff -I approach has two problems:  (1) it does not allow resolution finer than line-level, so it's impossible to mask out pattern occurrences within a line, and (2) it produces unmasked files, so if you run diff on the command line to compare the result .q.out with the checked-in file, you see the noise.
> My suggestion is to first run sed to replace noise patterns with an unlikely-to-occur string like ZYZZYZVA, and then diff the pre-masked files without using any -I.
> This would require a one-time hit to update all existing .q.out files so that they would contain the pre-masked results.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira