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

Ensure that subclassing BasicVertex is possible by user apps

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Blocker
    • Resolution: Fixed
    • 0.1.0
    • 0.1.0
    • graph
    • 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.warnings
          121 kB
          Avery Ching
        2. GIRAPH-36.diff
          122 kB
          Jake Mannix
        3. GIRAPH-36.diff
          124 kB
          Jake Mannix
        4. GIRAPH-36.diff
          122 kB
          Jake Mannix

        Issue Links

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved: