Fix NullPointerException during subdoc load result collection

- WorkLoadGenerate::update_subdoc_failed_mutation_result():
  filter results to only track failures (status == false);
  previously all results were added unconditionally, causing
  get_task_result() to iterate successful entries with null err and throw NPE
- TaskRequest::doc_load(): include subdocPercent in docsPerWorker
  calculation so subdoc-only loads get correct effective worker counts
  instead of always falling back to docsPerWorker=1

Author: ashwin2002

src/main/java/RestServer/TaskRequest.java

@@ -697,7 +697,8 @@ public ResponseEntity<Map<String, Object>> doc_load() {
             (this.updatePercent > 0 ? this.updatePercent : 0) +
             (this.readPercent > 0 ? this.readPercent : 0) +
             (this.deletePercent > 0 ? this.deletePercent : 0) +
-            (this.expiryPercent > 0 ? this.expiryPercent : 0)
+            (this.expiryPercent > 0 ? this.expiryPercent : 0) +
+            (this.subdocPercent > 0 ? this.subdocPercent : 0)
         ) / 100;
 
         // Handle edge case where docsPerWorker might be 0

src/main/java/couchbase/loadgen/WorkLoadGenerate.java

@@ -82,10 +82,12 @@ private void update_subdoc_failed_mutation_result(
         }
 
         for(HashMap<String, Object> sd_res : sd_results) {
-            result_arr.add(new Result((String)sd_res.get("id"),
-                                      sd_res.get("value"),
-                                      (Throwable)sd_res.get("error"),
-                                      (boolean)sd_res.get("status")));
+            if (!(boolean)sd_res.get("status")) {
+                result_arr.add(new Result((String)sd_res.get("id"),
+                                          sd_res.get("value"),
+                                          (Throwable)sd_res.get("error"),
+                                          false));
+            }
         }
     }