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

stack commented on HBASE-8657:
------------------------------

bq. Why remove []-s around toStringBinary, it makes it harder to read (or sed) 
from the log

Because it is out of form w/ the rest of the logging.  No where else do we have 
this square bracket delimiting.  If you want to go there, lets do it 
everywhere... else lets not introduce yet another new delimiter.  For example:

We used to delimit name and value as:

<name>':' <value>

We then moved to:

<name>'='<value>

... it is easy enough to grep for and we were pretty good at putting it 
everywhere.  Lists were delimited by a ','.

Then, along came ruby with it s '=>' delimiter and its bracketing for lists and 
maps.

An intern at facebook made some progress formatting the slow query detail as 
JSON.

Then there was protobuf which is like the original ':' only there is no 
delimiter between attributes -- no ',' -- just a space.

Then you came along w/ the square bracket which I suppose is a ruby 'list' only 
we don't really do ruby lists in our logs.

I would love to do a standard.  We wouldn't have to go there all in one go but 
we could write it up and then aim for it.

Good on you Sergey.
                
> Miscellaneous log fixups for hbase-it; tidier logging, fix a few NPEs
> ---------------------------------------------------------------------
>
>                 Key: HBASE-8657
>                 URL: https://issues.apache.org/jira/browse/HBASE-8657
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: stack
>            Assignee: stack
>             Fix For: 0.98.0, 0.95.1
>
>         Attachments: fixup2.txt, fixups.txt
>
>
> This is a miscellaneous set of fixups that come of my staring at hbase-it 
> logs trying to follow what is going on.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to