Pig
  1. Pig
  2. PIG-227

Minor irritants in parsing input/output specs for streaming command in the define clause

    Details

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

      Description

      Currently the following throw syntax errors:

      1. define CMD `blah` input(stdin using PigStorage)
      -> Expects PigStorage()

      2. define CMD `blah` input(stdin)
      -> Expects a 'using' clause.

      Should be straight-forward fix to the parser...

      1. PIG-227_1_20080502.patch
        3 kB
        Arun C Murthy
      2. PIG-227_0_20080501.patch
        3 kB
        Arun C Murthy

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        4h 39m 1 Arun C Murthy 02/May/08 06:30
        Patch Available Patch Available Resolved Resolved
        15h 14m 1 Olga Natkovich 02/May/08 21:45
        Resolved Resolved Closed Closed
        691d 15m 1 Alan Gates 24/Mar/10 22:01
        Alan Gates made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Olga Natkovich made changes -
        Fix Version/s 0.1.0 [ 12312848 ]
        Olga Natkovich made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Olga Natkovich added a comment -

        committed the fix, thanks Arun

        Show
        Olga Natkovich added a comment - committed the fix, thanks Arun
        Arun C Murthy made changes -
        Attachment PIG-227_1_20080502.patch [ 12381342 ]
        Hide
        Arun C Murthy added a comment -

        Updated patch to reflect recent changes to trunk ...

        Show
        Arun C Murthy added a comment - Updated patch to reflect recent changes to trunk ...
        Arun C Murthy made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Arun C Murthy added a comment -

        Marking it PA, passes all unit tests locally...

        Show
        Arun C Murthy added a comment - Marking it PA, passes all unit tests locally...
        Arun C Murthy made changes -
        Field Original Value New Value
        Attachment PIG-227_0_20080501.patch [ 12381299 ]
        Hide
        Arun C Murthy added a comment -

        Straight-forward fix to the QueryParser to handle the aforementioned cases...

        Show
        Arun C Murthy added a comment - Straight-forward fix to the QueryParser to handle the aforementioned cases...
        Arun C Murthy created issue -

          People

          • Assignee:
            Arun C Murthy
            Reporter:
            Arun C Murthy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development