hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kengo Seki (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-6964) Allow compact property description in xml
Date Sat, 27 Sep 2014 07:03:34 GMT

    [ https://issues.apache.org/jira/browse/HADOOP-6964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14150469#comment-14150469
] 

Kengo Seki commented on HADOOP-6964:
------------------------------------

Current patch implementation is:


* Attributes and Elements can be mixed. For example, 

{code:xml}
<property name="foo" final="true">
  <value>bar</value>
  <source>baz</source>
</property>
{code}

interpreted as \{"name": "foo", "value": "bar", "final": "true", "sources": \["baz"\]\}.


* If both attribute and element with the identical name are simultaneously defined, the latter
wins. But with regard to source, the attribute is added first, and the elements are appended
to the attribute.

{code:xml}
<property name="foo" source="a">
  <name>bar</name>
  <value>baz</value>
  <source>b</source>
  <source>c</source>
</property>
{code}

interpreted as \{"name": "bar", "value": "baz", "sources": \["a", "b", "c"\]\}.

Is this implementation reasonable? Can anybody review this patch?


> Allow compact property description in xml
> -----------------------------------------
>
>                 Key: HADOOP-6964
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6964
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>            Reporter: Owen O'Malley
>              Labels: newbie
>         Attachments: HADOOP-6964.patch
>
>
> We should allow users to use the more compact form of xml elements. For example, we could
allow:
> {noformat}
> <property name="mapred.local.dir" value="/disk/0/mapred,/disk/1/mapred"/>
> {noformat}
> The old format would also be supported.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message