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

Ensure that subclassing BasicVertex is possible by user apps

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.1.0
    • Fix Version/s: 0.1.0
    • Component/s: graph
    • Labels:
      None

      Description

      Original assumptions in Giraph were that all users would subclass Vertex (which extended MutableVertex extended BasicVertex). Classes which wish to have application specific data structures (ie. not a TreeMap<I, Edge<I,E>>) may need to extend either MutableVertex or BasicVertex. Unfortunately VertexRange extends ArrayList<Vertex>, and there are other places where the assumption is that vertex classes are either Vertex, or at least MutableVertex.

      Let's make sure the internal APIs allow for BasicVertex to be the base class.

        Attachments

        1. GIRAPH-36.diff
          122 kB
          Jake Mannix
        2. GIRAPH-36.diff
          124 kB
          Jake Mannix
        3. GIRAPH-36.diff
          122 kB
          Jake Mannix
        4. GIRAPH-36.diff.warnings
          121 kB
          Avery Ching

          Issue Links

            Activity

              People

              • Assignee:
                jake.mannix Jake Mannix
                Reporter:
                jake.mannix Jake Mannix
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: