Pig
  1. Pig
  2. PIG-2525

Support pluggable PigProgressNotifcationListeners on the command line

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.11
    • Component/s: None
    • Labels:
      None

      Description

      It would be convenient to be able to implement PigProcessNotifcationListener and to wire it into a pig script. This jira is to support setting a listener and its constructor args on the command, perhaps like this:

      noformat
      pig -j MyListener -J "foo,bar,bat" -f my_script.pig
      noformat

      where MyListener takes a single string in the constructor which will get passed foo,bar,bat.

      1. PIG-2525.patch
        6 kB
        Dmitriy V. Ryaboy

        Activity

        Hide
        Dmitriy V. Ryaboy added a comment -

        Why -j? I was thinking we could use properties for that:

        pig -Dpig.progress.listener=com.foo.PigListenerImpl -Dpig.progress.listener.args="foo,bar" -f my_script.pig

        Show
        Dmitriy V. Ryaboy added a comment - Why -j? I was thinking we could use properties for that: pig -Dpig.progress.listener=com.foo.PigListenerImpl -Dpig.progress.listener.args="foo,bar" -f my_script.pig
        Hide
        Dmitriy V. Ryaboy added a comment -

        Well this turned out pretty darn simple..

        Show
        Dmitriy V. Ryaboy added a comment - Well this turned out pretty darn simple..
        Hide
        Dmitriy V. Ryaboy added a comment -

        Re-attaching, forgot to rename "args" to "arg"...

        Show
        Dmitriy V. Ryaboy added a comment - Re-attaching, forgot to rename "args" to "arg"...
        Hide
        Daniel Dai added a comment -

        +1

        Show
        Daniel Dai added a comment - +1
        Hide
        Dmitriy V. Ryaboy added a comment -

        Committed to trunk.

        In case anyone wants it in earlier versions – patch applies cleanly to 0.9.2 and up.

        Show
        Dmitriy V. Ryaboy added a comment - Committed to trunk. In case anyone wants it in earlier versions – patch applies cleanly to 0.9.2 and up.

          People

          • Assignee:
            Dmitriy V. Ryaboy
            Reporter:
            Bill Graham
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development