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

          Jake Mannix created issue -
          Jake Mannix made changes -
          Field Original Value New Value
          Link This issue blocks GIRAPH-28 [ GIRAPH-28 ]
          Jake Mannix made changes -
          Attachment GIRAPH-36.diff [ 12501438 ]
          Jake Mannix made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Jake Mannix made changes -
          Attachment GIRAPH-36.diff [ 12501828 ]
          Jake Mannix made changes -
          Attachment GIRAPH-36.diff [ 12501863 ]
          Avery Ching made changes -
          Attachment GIRAPH-36.diff.warnings [ 12501919 ]
          Avery Ching made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development