[ 
https://issues.apache.org/jira/browse/SOLR-1933?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12873286#action_12873286
 ] 

Yonik Seeley commented on SOLR-1933:
------------------------------------

Background: the very first versions of Solr *always* indented... and hence only 
supported indenting up to 7 levels, and was very frugal in it's use of 
bandwidth (most indenting levels consisted of just 2 characters in total, not 
including the newline).  It did this through a mix of tabs and spaces, which 
lead to varying width for each indentation level.

{code}
 private static final String[] indentArr = new String[] {
    "\n",
    "\n ",
    "\n  ",
    "\n\t",
    "\n\t ",
    "\n\t  ",  // could skip this one (the only 3 char seq)
    "\n\t\t" };
{code}

Given that there is now an intent=true/false parameter that defaults to false 
(this has been there since Solr was open-sourced), we should perhaps improve 
the indenting at the cost of bandwidth.


> improve solr indenting
> ----------------------
>
>                 Key: SOLR-1933
>                 URL: https://issues.apache.org/jira/browse/SOLR-1933
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: Yonik Seeley
>             Fix For: 3.1
>
>
> Improve the uniformity and scale of solr indenting (solr only supported 7 
> levels of indenting previously)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to