[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16433084#comment-16433084 ] Hudson commented on HADOOP-15357: - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13960 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/13960/]) HADOOP-15357. Configuration.getPropsWithPrefix no longer does variable (jlowe: rev e81397545a273cf9a090010eb644b836e0ef8c7b) * (edit) hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/conf/Configuration.java * (edit) hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Affects Versions: 2.9.0, 3.0.0 >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Fix For: 2.10.0, 3.2.0, 3.1.1, 2.9.2, 3.0.3 > > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch, > HADOOP-15357.003.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16432419#comment-16432419 ] Jim Brennan commented on HADOOP-15357: -- [~jlowe], [~lmccay], let me know if you would prefer that I add another test to cover the deprecation feature via this interface. Otherwise, I think this may be ready to be committed. > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch, > HADOOP-15357.003.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16430573#comment-16430573 ] Jim Brennan commented on HADOOP-15357: -- [~lmccay], yeah, I think that functionality is covered by existing tests. Thanks for the review! > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch, > HADOOP-15357.003.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16428702#comment-16428702 ] Larry McCay commented on HADOOP-15357: -- I supposed the fact that get() is already tested for deprecated keys is likely enough coverage for this as well. LGTM +1 > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch, > HADOOP-15357.003.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16426962#comment-16426962 ] Jim Brennan commented on HADOOP-15357: -- [~lmccay], looks like this is ready for review again. Only change is to switch to using get(). > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch, > HADOOP-15357.003.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16426321#comment-16426321 ] genericqa commented on HADOOP-15357: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 32s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 32m 20s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 33m 10s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 11s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 12s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 30s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 54s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 26m 46s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 26m 46s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 51s{color} | {color:green} hadoop-common-project/hadoop-common: The patch generated 0 new + 243 unchanged - 1 fixed = 243 total (was 244) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 3s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 9m 56s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 37s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 55s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 8m 47s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 35s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}133m 24s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:8620d2b | | JIRA Issue | HADOOP-15357 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12917611/HADOOP-15357.003.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 291b683cc295 3.13.0-137-generic #186-Ubuntu SMP Mon Dec 4 19:09:19 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 3087e89 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_162 | | findbugs | v3.1.0-RC1 | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/14435/testReport/ | | Max. process+thread count | 1718 (vs. ulimit of 1) | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/14435/console | | Powered by | Apache Yetus 0.8.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Configuration.getPropsWithPrefix no longer does variable
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16426094#comment-16426094 ] Larry McCay commented on HADOOP-15357: -- I don't disagree - that is not to say that I know that we need it but it is certainly safer to support it. > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16426069#comment-16426069 ] Jim Brennan commented on HADOOP-15357: -- [~lmccay], thanks for the review! {quote}I notice that you decided to not use the get() method itself was this due to thinking that we don't need the handleDeprecation stuff? If so, why do you think we don't need it? {quote} I was thinking that the variables that this function finds are not likely to be known constants, and as such cannot be in the deprecated map. Only the prefixed portion is necessarily a known constant. So it seemed like wasted effort to do the deprecated handling. e.g.: with {{my_prefix[var]=[value]}}, would we ever find {{my_prefix[var]}} in the deprecated map? Looking at it again, I'm not convinced by my own reasoning - I was thinking of the MAPREDUCE-7069 use-case, but if this function is used to find a set of known properties with a common prefix, it is certainly possible that they could require deprecation handling. I'm inclined to put up a new patch changing this to use get(). Let me know if you disagree. > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16425973#comment-16425973 ] Larry McCay commented on HADOOP-15357: -- Hi [~Jim_Brennan] - the patch looks good. I notice that you decided to not use the get() method itself was this due to thinking that we don't need the handleDeprecation stuff? If so, why do you think we don't need it? I'm not saying that we do but would like to understand why you think we don't. > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16424565#comment-16424565 ] Jim Brennan commented on HADOOP-15357: -- [~lmccay], [~asuresh], I believe this patch is ready for review. cc: [~jlowe] > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16424553#comment-16424553 ] genericqa commented on HADOOP-15357: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 9m 36s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 25m 24s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 29m 39s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 13s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 24s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 32s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 56s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 26m 19s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 26m 19s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 53s{color} | {color:green} hadoop-common-project/hadoop-common: The patch generated 0 new + 243 unchanged - 1 fixed = 243 total (was 244) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 5s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 7s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 39s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 56s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 8m 7s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 36s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}131m 36s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:8620d2b | | JIRA Issue | HADOOP-15357 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12917415/HADOOP-15357.002.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 4777a3250d9f 3.13.0-139-generic #188-Ubuntu SMP Tue Jan 9 14:43:09 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 5a174f8 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_162 | | findbugs | v3.1.0-RC1 | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/14429/testReport/ | | Max. process+thread count | 1512 (vs. ulimit of 1) | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/14429/console | | Powered by | Apache Yetus 0.8.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Configuration.getPropsWithPrefix no longer does variable
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16424387#comment-16424387 ] Jim Brennan commented on HADOOP-15357: -- Renamed local variable to fix the check-style issue and submitted a new patch. > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Assignee: Jim Brennan >Priority: Major > Attachments: HADOOP-15357.001.patch, HADOOP-15357.002.patch > > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16424316#comment-16424316 ] genericqa commented on HADOOP-15357: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 15m 32s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 25m 34s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 27m 57s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 52s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 6s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 2s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 29s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 56s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 45s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 26m 43s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 26m 43s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 51s{color} | {color:orange} hadoop-common-project/hadoop-common: The patch generated 1 new + 243 unchanged - 1 fixed = 244 total (was 244) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 3s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 4s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 38s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 54s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 8m 49s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 35s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}136m 25s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:8620d2b | | JIRA Issue | HADOOP-15357 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12917380/HADOOP-15357.001.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 513dc5057496 3.13.0-137-generic #186-Ubuntu SMP Mon Dec 4 19:09:19 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 93d47a0 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_162 | | findbugs | v3.1.0-RC1 | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/14428/artifact/out/diff-checkstyle-hadoop-common-project_hadoop-common.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/14428/testReport/ | | Max. process+thread count | 1717 (vs. ulimit of 1) | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/14428/console | | Powered by | Apache Yetus
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16424056#comment-16424056 ] Jim Brennan commented on HADOOP-15357: -- [~lmccay], thanks for the prompt replies. I will happy to put up a patch later today! > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Priority: Major > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16423282#comment-16423282 ] Larry McCay commented on HADOOP-15357: -- [~Jim_Brennan] - I think that both of your assertions are correct. Will you be providing a patch for this? > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Priority: Major > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16423054#comment-16423054 ] Jim Brennan commented on HADOOP-15357: -- One additional question: shouldn't this code be using {{props.stringPropertyNames()}} instead of {{props.propertyNames()}}? > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Priority: Major > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16423004#comment-16423004 ] Jim Brennan commented on HADOOP-15357: -- I think you can just replace this: {code:java} String value = props.getProperty(name); {code} with this: {code:java} String value = get(name); {code} I'm not certain if we need to worry about the deprecation stuff in Configuration.get() for this. If it's not needed, then I think this would be all we'd need: {code:java} String value = substituteVars(props.getProperty(name)); {code} > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Priority: Major > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16422962#comment-16422962 ] Larry McCay commented on HADOOP-15357: -- Hi [~Jim_Brennan] - this definitely is an oversight in the replacement of the iterator based implementation and was not done intentionally. Is it possible to add the missing functionality without reverting the thread safety added by this change? > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Priority: Major > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15357) Configuration.getPropsWithPrefix no longer does variable substitution
[ https://issues.apache.org/jira/browse/HADOOP-15357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16422925#comment-16422925 ] Jim Brennan commented on HADOOP-15357: -- [~lmc...@apache.org], [~asuresh], can you please comment on this? cc: [~jlowe] > Configuration.getPropsWithPrefix no longer does variable substitution > - > > Key: HADOOP-15357 > URL: https://issues.apache.org/jira/browse/HADOOP-15357 > Project: Hadoop Common > Issue Type: Bug >Reporter: Jim Brennan >Priority: Major > > Before [HADOOP-13556], Configuration.getPropsWithPrefix() used the > Configuration.get() method to get the value of the variables. After > [HADOOP-13556], it now uses props.getProperty(). > The difference is that Configuration.get() does deprecation handling and more > importantly variable substitution on the value. So if a property has a > variable specified with ${variable_name}, it will no longer be expanded when > retrieved via getPropsWithPrefix(). > Was this change in behavior intentional? I am using this function in the fix > for [MAPREDUCE-7069], but we do want variable expansion to happen. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org