[lttng-dev] [PATCH lttng-tool] Mi & save/load: add option to indent or not on config writer creation

2014-08-06 Thread Jonathan Rajotte Julien
The presence of tabulation and newline in mi is irrelevant and can impact client performance. The parsing & processing of a lot of data from mi with many /t and /n impact performance. This aim to fix the problem at the source. Style is useless for machine. Signed-off-by: Jonathan Rajotte Julien -

Re: [lttng-dev] [PATCH lttng-tool] Mi & save/load: add option to indent or not on config writer creation

2014-08-06 Thread Simon Marchi
On 6 August 2014 10:13, Jonathan Rajotte Julien wrote: > The presence of tabulation and newline in mi is irrelevant and > can impact client performance. The parsing & processing of a lot of data from > mi > with many /t and /n impact performance. This aim to fix the problem at the > source. Style

Re: [lttng-dev] [PATCH lttng-tool] Mi & save/load: add option to indent or not on config writer creation

2014-08-06 Thread Alexandre Montplaisir
On 08/06/2014 10:49 AM, Simon Marchi wrote: On 6 August 2014 10:13, Jonathan Rajotte Julien wrote: The presence of tabulation and newline in mi is irrelevant and can impact client performance. The parsing & processing of a lot of data from mi with many /t and /n impact performance. This aim to

Re: [lttng-dev] Bug#751337: [PATCH] tests: test_ctf_writer: replace HOST_NAME_MAX

2014-08-06 Thread Sebastian Andrzej Siewior
On 2014-08-05 16:14:12 [-0400], Jon Bernard wrote: > This may also work as well: > > diff --git a/configure.ac b/configure.ac > index 80eba33..248044e 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -50,6 +50,17 @@ AC_C_INLINE > AC_TYPE_PID_T > AC_TYPE_SIZE_T >

Re: [lttng-dev] [PATCH lttng-tool] Mi & save/load: add option to indent or not on config writer creation

2014-08-06 Thread Jonathan Rajotte-Julien
Hey, Here by client I refer about end user MI client like TMF(Trace Compass). Take the later case. Where a pretty MI output impose drastic performance problem. Some of the problems are internal to TMF but inherent to how java parse XML (in compliance with the XML specification). This patch aim