)]}'
{
  "commit": "aed4160787ba0ec272dad8d31642fee27a16344d",
  "tree": "e5d16a0f2d6c1142dc30dfbee975d222b8c2c6e7",
  "parents": [
    "61a9f57d595492e0cd68758ca63fc24832198554"
  ],
  "author": {
    "name": "Googler",
    "email": "noreply@google.com",
    "time": "Tue Jun 02 12:22:53 2020 -0700"
  },
  "committer": {
    "name": "Copybara-Service",
    "email": "copybara-worker@google.com",
    "time": "Tue Jun 02 12:24:52 2020 -0700"
  },
  "message": "Clean up artifact retrieval methods in ActionExecutionValue.\n\nAll callers of getArtifactValue were doing a checkNotNull on the return value, which is exactly what getExistingFileArtifactValue does. Migrate them so that we only need one retrieval method. Loosen the parameter to Artifact and do the instanceof check in one place, so any bugs get an appropriate error message.\n\nAdditionally, use the recently added isChildOfDeclaredDirectory method to determine which map to search for the artifact, instead of unnecessarily checking both.\n\nRELNOTES: None.\nPiperOrigin-RevId: 314380367\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "3ea28f6abacf79975146ccf68fa4274747ff9896",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/ActionExecutionFunction.java",
      "new_id": "50f1c359cdd14de090b94ee95ed09344fa4568f6",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/ActionExecutionFunction.java"
    },
    {
      "type": "modify",
      "old_id": "1c63e3ed231ca863cbeb68b3548d6b70f313ee51",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/ActionExecutionValue.java",
      "new_id": "b097fe1ec55f024b986147055cb80fafda21d5ec",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/ActionExecutionValue.java"
    },
    {
      "type": "modify",
      "old_id": "03fed1ac3f9ef29a394b91750fd4dbd908cf376d",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/ActionInputMapHelper.java",
      "new_id": "5f1d89ede95ffd82c73f66d8c40e3f63e69f5899",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/ActionInputMapHelper.java"
    },
    {
      "type": "modify",
      "old_id": "d9f50b4361ad69d3e6f5d30891acbb8ddc4adb2a",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java",
      "new_id": "9f83aa2536fc7df1845f45136c383a5953263e4c",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java"
    },
    {
      "type": "modify",
      "old_id": "76914f44001941ffb4bc002db7ecda62099d38e9",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/FilesystemValueChecker.java",
      "new_id": "9e3932eb8cc55c0d9938329c786b8d55f93a1da8",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/FilesystemValueChecker.java"
    },
    {
      "type": "modify",
      "old_id": "f260515976c1063bd313a76f6c6392bea2bc5d14",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/RecursiveFilesystemTraversalFunction.java",
      "new_id": "c6117000f3574f08185accebb3823c202f152e87",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/RecursiveFilesystemTraversalFunction.java"
    },
    {
      "type": "modify",
      "old_id": "70e9ebacda61b2988ee48aa2dbdb36fbf267c456",
      "old_mode": 33188,
      "old_path": "src/test/java/com/google/devtools/build/lib/skyframe/ActionMetadataHandlerTest.java",
      "new_id": "e77a62c390c5b10bf7263f968e69fd93731dc08c",
      "new_mode": 33188,
      "new_path": "src/test/java/com/google/devtools/build/lib/skyframe/ActionMetadataHandlerTest.java"
    },
    {
      "type": "modify",
      "old_id": "5607782f7ee1ac1ff030ae40a020b5cf7d028705",
      "old_mode": 33188,
      "old_path": "src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java",
      "new_id": "67924f13a7382b874e3e7e3e89352f3d4122e547",
      "new_mode": 33188,
      "new_path": "src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java"
    }
  ]
}