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

Make PigStorage.readField() protected

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Fixed
    • 0.8.0, 0.8.1, 0.9.0, 0.9.1, 0.9.2, 0.10.0
    • 0.12.0
    • build, internal-udfs
    • Patch Available

    Description

      for the cases when we need to extend PigStorage just to override readField. Currently, we need to copy/paste several private fields and all getNext

      I've changed readField from private to protected and added a new method: protected void addToCurrentTuple(DataByteArray data)

      Attachments

        1. PigStorage_readField.patch
          0.8 kB
          pablo martinez
        2. PIG-3057_1.patch
          2 kB
          William W. Graham Jr

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            pablomar pablo martinez
            pablomar pablo martinez
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 2h
                2h
                Remaining:
                Remaining Estimate - 2h
                2h
                Logged:
                Time Spent - Not Specified
                Not Specified

                Slack

                  Issue deployment