Giraph
  1. Giraph
  2. GIRAPH-36

Ensure that subclassing BasicVertex is possible by user apps

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Blocker 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.

      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
              Reporter:
              Jake Mannix
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development