Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1162

Add VertexProgram.getTransientComputeKeys() for removing scratch-data.

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Implemented
    • Affects Version/s: 3.1.0-incubating
    • Fix Version/s: 3.2.0-incubating
    • Component/s: process
    • Labels:

      Description

      When you run a PageRankVertexProgram or a PeerPressureVertexProgram, you have an edgeCount property which is used for determining the weight distribution. This is just some "scratch pad data." We need a way to tell the GraphComputer which properties are "scratch pad" and thus, delete them before outputting the `ComputerResult`.

      I propose:

      Set<String> VertexProgram.getTransientComputeKeys()
      

      We can have a default that returns Collections.emptySet(). This would be breaking as the GraphComputer provider will need to handle dropping such properties. This, of course, will be verified in a GraphComputerTest.shouldSupportTransientProperties().

        Activity

        Hide
        okram Marko A. Rodriguez added a comment -

        We will also want VertexProgram.getTransientMemoryKeys().

        Show
        okram Marko A. Rodriguez added a comment - We will also want VertexProgram.getTransientMemoryKeys() .
        Hide
        okram Marko A. Rodriguez added a comment -

        This was also done for memory compute keys. Also added broadcast semantics to memory compute keys (along with transience).

        Show
        okram Marko A. Rodriguez added a comment - This was also done for memory compute keys. Also added broadcast semantics to memory compute keys (along with transience).

          People

          • Assignee:
            okram Marko A. Rodriguez
            Reporter:
            okram Marko A. Rodriguez
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development