Uploaded image for project: 'Pig'
  1. Pig
  2. PIG-218

parameter substitution broken with some commands

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.1.0
    • Component/s: None
    • Labels:
      None

      Description

      If parameter value is generated via a command, it would fail for some commands. Example:

      %declare cmd `/usr/local/bin/perl -e 'print "studenttab10k"'`

      This command fails because currently the command is executed via call to Runtime.exec(cmd). In this case, default tokenizer is used to split the command and it does not respect the quotes.

      1. PIG-218_v2.patch
        4 kB
        Olga Natkovich
      2. PIG-218.patch
        4 kB
        Olga Natkovich

        Activity

        Hide
        olgan Olga Natkovich added a comment -

        patch committed

        Show
        olgan Olga Natkovich added a comment - patch committed
        Hide
        alangates Alan Gates added a comment -

        New tests look good.

        Show
        alangates Alan Gates added a comment - New tests look good.
        Hide
        olgan Olga Natkovich added a comment -

        The new patch has a modified test that includes pipe

        Show
        olgan Olga Natkovich added a comment - The new patch has a modified test that includes pipe
        Hide
        alangates Alan Gates added a comment -

        Will this change handle commands that have a unix pipe in them? If so, can we add a test for that?

        Show
        alangates Alan Gates added a comment - Will this change handle commands that have a unix pipe in them? If so, can we add a test for that?

          People

          • Assignee:
            olgan Olga Natkovich
            Reporter:
            olgan Olga Natkovich
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development