In that case will stick with <opErrstr/> for all the null elements.
On 08/05/2015 04:10 PM, Prashanth Pai wrote:
Having (null) is not common in xml convention. Usually, it's either
<opErrstr/>
or
<opErrstr></opErrstr>
Regards,
-Prashanth Pai
----- Original Message -----
From: "Avra Sengupta" <aseng...@redhat.com>
To: "Atin Mukherjee" <amukh...@redhat.com>, "Gluster Devel"
<gluster-devel@gluster.org>, gluster-us...@gluster.org,
kauff...@cs.uchicago.edu, "Dusmant Kumar Pati" <dp...@redhat.com>, "Shubhendu
Tripathi" <shtri...@redhat.com>,
msvb...@redhat.com
Sent: Wednesday, August 5, 2015 3:16:10 PM
Subject: Re: [Gluster-devel] Non-Uniform opErrstr output in all gluster --xml
commands
On 08/05/2015 03:06 PM, Atin Mukherjee wrote:
On 08/05/2015 02:58 PM, Avra Sengupta wrote:
Hi,
As reported in https://bugzilla.redhat.com/show_bug.cgi?id=1218732, in
the event where there is no opErrstr, some gluster commands'(like
snapshot status, volume status etc.) xml output shows
"<opErrstr>(null)</opErrstr>", while other commands show just
"<opErrstr/>". This non-uniform output is troublesome for people who
parse this.
IMHO showing "<opErrstr>(null)</opErrstr>" is much more descriptive. I
would like to propose making sure all commands follow this uniform
approach, instead of displaying just "<opErrstr/>".
Makes sense to me as long as its uniform. However the existing ones
which follow "<opErrstr/>" need change in parsing logic :
To attain uniformity, one or the other parsing logic will break. we
might as well get it right for once.
Regards,
Avra
_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
http://www.gluster.org/mailman/listinfo/gluster-devel
_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
http://www.gluster.org/mailman/listinfo/gluster-devel
_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
http://www.gluster.org/mailman/listinfo/gluster-devel
