[ https://issues.apache.org/jira/browse/SOLR-3434?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13268506#comment-13268506 ]
Yonik Seeley commented on SOLR-3434: ------------------------------------ bq. My opinion is: 'trim' should be true by default, and certainly apply to both data and header, although I understand it would break backward-compatibility. IMO, we should always trim the headers - the CSV loader certainly never intended to support fieldnames with leading or trailing whitespace.... this was simply just a case of lacking any examples that added whitespace. As for your other problem, it seems like we also need a skipLinesAfterHeader option? > CSVRequestHandler does not trim header when using header=true&trim=true > ----------------------------------------------------------------------- > > Key: SOLR-3434 > URL: https://issues.apache.org/jira/browse/SOLR-3434 > Project: Solr > Issue Type: Improvement > Affects Versions: 3.6 > Environment: Linux > Reporter: david babits > Labels: CSV,, header, separator > > when using {{header=true&trim=true}} the field names in the header row are > not trimmed. > this is consistent with the documentation, but that doesn't mean it makes > sense. > would be good to change this so trim=true also applied to the header row (at > least by default) -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org