Pig
  1. Pig
  2. PIG-2516

sh command should behave like fs command

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Not a Problem
    • Affects Version/s: 0.9.1, 0.9.2
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      Like fs command, sh command failure should result in query failure when run in batch mode. In interactive mode it should result in a warning.

      Also, sh command parsing seems to be faulty, according to Daniel, parser generated by PigScriptParser.jj would result in sh command using GetPath(), ie all sh commands might not be working.

        Activity

        Hide
        Devi Kumarappan added a comment -

        This is not an issue in the trunk version anymore.If sh command fails, then Pig exits with non zero code.

        Show
        Devi Kumarappan added a comment - This is not an issue in the trunk version anymore.If sh command fails, then Pig exits with non zero code.
        Hide
        Thejas M Nair added a comment -

        Yes, in some use cases you would not want the whole query to fail. But in such cases, you can write a shell command that does not result in failure. eg "rm -f" or "rm /nosuchfile || echo 'rm failed'".

        Show
        Thejas M Nair added a comment - Yes, in some use cases you would not want the whole query to fail. But in such cases, you can write a shell command that does not result in failure. eg "rm -f" or "rm /nosuchfile || echo 'rm failed'".
        Hide
        Dmitriy V. Ryaboy added a comment -

        Not sure I want the transient inability to clean up some local log or other trivial task to kill a whole pig job?

        Show
        Dmitriy V. Ryaboy added a comment - Not sure I want the transient inability to clean up some local log or other trivial task to kill a whole pig job?

          People

          • Assignee:
            Unassigned
            Reporter:
            Thejas M Nair
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development