Grokbase Groups Lucene dev June 2016
FAQ
[ https://issues.apache.org/jira/browse/SOLR-8946?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15336845#comment-15336845 ]

ASF subversion and git services commented on SOLR-8946:
-------------------------------------------------------

Commit 060902366027239832f532db4250f0d337c1cb96 in lucene-solr's branch refs/heads/branch_5_5 from [~dsmiley]
[ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=0609023 ]

SOLR-8946: fix bin/post stdin detection. faulty on Ubuntu; maybe others
(cherry picked from commit 7833f7f)

bin/post script stdin detection is faulty
-----------------------------------------

Key: SOLR-8946
URL: https://issues.apache.org/jira/browse/SOLR-8946
Project: Solr
Issue Type: Bug
Components: SimplePostTool
Affects Versions: 6.0
Environment: Ubuntu
Reporter: David Smiley
Assignee: David Smiley
Fix For: 6.0.1, 6.1

Attachments: SOLR_8946.patch


Summary: I think the {{post}} bash script tries to detect stdin the wrong way.
If I run "post -h" I see some examples, to include this:
{code}
echo '{commit: {}}' | bin/post -c my_collection -type application/json -out yes -d
{code}
However if I try it, Solr returns this error: {{Cannot parse provided JSON: JSON Parse Error: char=<,position=0 BEFORE='<' AFTER='add/>'}}
I see that {{post}} is sending an {{<add />}} in certain cases, but it's not supposed to when the mode is stdin. Looking closer, it appears to be because of this line:
{{if [[ -s /dev/stdin ]]; then}}
If I change it to this (which I figured out after googling around; I'm no bash expert!):
{{if [[ ! -t 0 ]]; then}}
Apparently '0' represents stdin. It works for me. [~thelabdude] might you take a look? I'd be happy to add a formal patch & commit for this trivial change but it really needs other eyes.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org

Search Discussions

  • ASF subversion and git services (JIRA) at Jun 17, 2016 at 8:17 pm
    [ https://issues.apache.org/jira/browse/SOLR-8946?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15336846#comment-15336846 ]

    ASF subversion and git services commented on SOLR-8946:
    -------------------------------------------------------

    Commit 0b4a81eaae07a9e271c74f59f586075d3ee00ca1 in lucene-solr's branch refs/heads/branch_5x from [~dsmiley]
    [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=0b4a81e ]

    SOLR-8946: fix bin/post stdin detection. faulty on Ubuntu; maybe others
    (cherry picked from commit 7833f7f)

    bin/post script stdin detection is faulty
    -----------------------------------------

    Key: SOLR-8946
    URL: https://issues.apache.org/jira/browse/SOLR-8946
    Project: Solr
    Issue Type: Bug
    Components: SimplePostTool
    Affects Versions: 6.0
    Environment: Ubuntu
    Reporter: David Smiley
    Assignee: David Smiley
    Fix For: 6.0.1, 6.1

    Attachments: SOLR_8946.patch


    Summary: I think the {{post}} bash script tries to detect stdin the wrong way.
    If I run "post -h" I see some examples, to include this:
    {code}
    echo '{commit: {}}' | bin/post -c my_collection -type application/json -out yes -d
    {code}
    However if I try it, Solr returns this error: {{Cannot parse provided JSON: JSON Parse Error: char=<,position=0 BEFORE='<' AFTER='add/>'}}
    I see that {{post}} is sending an {{<add />}} in certain cases, but it's not supposed to when the mode is stdin. Looking closer, it appears to be because of this line:
    {{if [[ -s /dev/stdin ]]; then}}
    If I change it to this (which I figured out after googling around; I'm no bash expert!):
    {{if [[ ! -t 0 ]]; then}}
    Apparently '0' represents stdin. It works for me. [~thelabdude] might you take a look? I'd be happy to add a formal patch & commit for this trivial change but it really needs other eyes.


    --
    This message was sent by Atlassian JIRA
    (v6.3.4#6332)

    ---------------------------------------------------------------------
    To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
    For additional commands, e-mail: dev-help@lucene.apache.org
  • Steve Rowe (JIRA) at Jun 17, 2016 at 8:17 pm
    [ https://issues.apache.org/jira/browse/SOLR-8946?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Steve Rowe reopened SOLR-8946:
    ------------------------------

    Reopening to backport to 5.6 and 5.5.2.
    bin/post script stdin detection is faulty
    -----------------------------------------

    Key: SOLR-8946
    URL: https://issues.apache.org/jira/browse/SOLR-8946
    Project: Solr
    Issue Type: Bug
    Components: SimplePostTool
    Affects Versions: 6.0
    Environment: Ubuntu
    Reporter: David Smiley
    Assignee: David Smiley
    Fix For: 6.0.1, 6.1

    Attachments: SOLR_8946.patch


    Summary: I think the {{post}} bash script tries to detect stdin the wrong way.
    If I run "post -h" I see some examples, to include this:
    {code}
    echo '{commit: {}}' | bin/post -c my_collection -type application/json -out yes -d
    {code}
    However if I try it, Solr returns this error: {{Cannot parse provided JSON: JSON Parse Error: char=<,position=0 BEFORE='<' AFTER='add/>'}}
    I see that {{post}} is sending an {{<add />}} in certain cases, but it's not supposed to when the mode is stdin. Looking closer, it appears to be because of this line:
    {{if [[ -s /dev/stdin ]]; then}}
    If I change it to this (which I figured out after googling around; I'm no bash expert!):
    {{if [[ ! -t 0 ]]; then}}
    Apparently '0' represents stdin. It works for me. [~thelabdude] might you take a look? I'd be happy to add a formal patch & commit for this trivial change but it really needs other eyes.


    --
    This message was sent by Atlassian JIRA
    (v6.3.4#6332)

    ---------------------------------------------------------------------
    To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
    For additional commands, e-mail: dev-help@lucene.apache.org
  • Steve Rowe (JIRA) at Jun 17, 2016 at 8:18 pm
    [ https://issues.apache.org/jira/browse/SOLR-8946?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

    Steve Rowe resolved SOLR-8946.
    ------------------------------
            Resolution: Fixed
         Fix Version/s: 5.5.2
                        5.6
    bin/post script stdin detection is faulty
    -----------------------------------------

    Key: SOLR-8946
    URL: https://issues.apache.org/jira/browse/SOLR-8946
    Project: Solr
    Issue Type: Bug
    Components: SimplePostTool
    Affects Versions: 6.0
    Environment: Ubuntu
    Reporter: David Smiley
    Assignee: David Smiley
    Fix For: 5.6, 5.5.2, 6.1, 6.0.1

    Attachments: SOLR_8946.patch


    Summary: I think the {{post}} bash script tries to detect stdin the wrong way.
    If I run "post -h" I see some examples, to include this:
    {code}
    echo '{commit: {}}' | bin/post -c my_collection -type application/json -out yes -d
    {code}
    However if I try it, Solr returns this error: {{Cannot parse provided JSON: JSON Parse Error: char=<,position=0 BEFORE='<' AFTER='add/>'}}
    I see that {{post}} is sending an {{<add />}} in certain cases, but it's not supposed to when the mode is stdin. Looking closer, it appears to be because of this line:
    {{if [[ -s /dev/stdin ]]; then}}
    If I change it to this (which I figured out after googling around; I'm no bash expert!):
    {{if [[ ! -t 0 ]]; then}}
    Apparently '0' represents stdin. It works for me. [~thelabdude] might you take a look? I'd be happy to add a formal patch & commit for this trivial change but it really needs other eyes.


    --
    This message was sent by Atlassian JIRA
    (v6.3.4#6332)

    ---------------------------------------------------------------------
    To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
    For additional commands, e-mail: dev-help@lucene.apache.org

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupdev @
categorieslucene
postedJun 17, '16 at 8:17p
activeJun 17, '16 at 8:18p
posts4
users1
websitelucene.apache.org

1 user in discussion

Steve Rowe (JIRA): 4 posts

People

Translate

site design / logo © 2019 Grokbase