hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Mackrory (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HADOOP-13660) Upgrade commons-configuration version
Date Tue, 25 Oct 2016 15:34:58 GMT

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

Sean Mackrory updated HADOOP-13660:
    Attachment: HADOOP-13660.004.patch

{quote}IOException is imported but not used in MetricsConfig.java{quote}

Fixed. Also, we're able to use a helper class that makes the new code in this class a little
more concise.

{quote}Depending on exception message is generally risky. Would it be possible to add a regression
test? Or is it already covered by existing unit tests?{quote}

This is already covered by existing unit tests: TestMetricsConfig#testLoadFirst and TestMetricsConfig#testMissingFiles
will both fail if the code doesn't gracefully detect that one of the files it attempts to
load are missing and proceeds to try elsewhere.

{quote}Want to call this out so we can study it further.{quote}

You mean like a release note? Can definitely add one. I believe the concern is very minor
here, though. I've compared the logic for reading in the configuration, and the code is functionally
equivalent (it's actually mostly identical). The documented definition of how things get escaped
in a list also remains the same. Where things differ is exactly how things get escaped when
writing back out the configuration, which appears to mainly be done for logging / debugging.
Even then, it should only be different for some edge cases that as far as I can tell should
come up exceptionally rarely (perhaps never) in the Metrics2 use case (values that end with
backslashes, etc.) Tough to be sure I've looked at all the use cases exhaustively, though.

> Upgrade commons-configuration version
> -------------------------------------
>                 Key: HADOOP-13660
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13660
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: build
>    Affects Versions: 3.0.0-alpha2
>            Reporter: Sean Mackrory
>            Assignee: Sean Mackrory
>         Attachments: HADOOP-13660-configuration2.001.patch, HADOOP-13660.001.patch, HADOOP-13660.002.patch,
HADOOP-13660.003.patch, HADOOP-13660.004.patch
> We're currently pulling in version 1.6 - I think we should upgrade to the latest 1.10.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org

View raw message