Uploaded image for project: 'Giraph'
  1. Giraph
  2. GIRAPH-570

Create YARN RPC Records using BuilderUtils instead of populating them by hand

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.0.0
    • Component/s: None
    • Labels:
      None

      Description

      Good newbie JIRA. Might need to check as see how far back in the Hadoop-2.0.x line the BuilderUtils exist so we know if we are cutting ourselves off from a future backport, but if we don't care, this can happen:

      Instead of creating and hand-populating each RPC record Giraph uses to request resources from YARN like:

      Record x = Records.newRecord( className );
      x.setField(blah);
      x.setOtherField(blahblah);
      // ...and so on
      

      we can use BuilderUtils:

      Record readyToSend = BuilderUtils.MakeMyNewRecord( blah, blah );
      

      anyway you get the drill.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              initialcontext Eli Reisman
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: