auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public ECKey getKey() { <START> final ECKey key = ECKey.fromPrivate(Arrays.copyOf(bytes, 32)); <END> return isCompressed() ? key : key.decompress(); }</code><technical_language>creating a compressed key decompressing it, ECKey.fromPrivate(bytes, compressed) constructor in place</technical_language>

Target:
public ECKey getKey() { return ECKey.fromPrivate(Arrays.copyOf(bytes, 32), isCompressed()); }
Source:
<code>public ParseSpec withDimensionsSpec(DimensionsSpec spec) { <START> return new JSONParseSpec(getTimestampSpec(), spec, flattenSpec); <END> }</code><technical_language>getter</technical_language>

Target:
public ParseSpec withDimensionsSpec(DimensionsSpec spec) { return new JSONParseSpec(getTimestampSpec(), spec, getFlattenSpec()); }
Source:
<code>private TypePrinter<GraphQLEnumType> enumPrinter() { return (out, type, visibility) -> { if (isIntrospectionType(type)) { return; } GraphqlTypeComparatorEnvironment environment = GraphqlTypeComparatorEnvironment.newEnvironment() .parentType(GraphQLEnumType.class) .elementType(GraphQLEnumValueDefinition.class) .build(); Comparator<? super GraphQLType> comparator = options.comparatorRegistry.getComparator(environment); printComments(out, type, ""); out.format("enum %s%s {\n", type.getName(), directivesString(GraphQLEnumType.class, type.getDirectives())); List<GraphQLEnumValueDefinition> values = type.getValues() .stream() .sorted(comparator) .collect(toList()); for (GraphQLEnumValueDefinition enumValueDefinition : values) { printComments(out, enumValueDefinition, " "); List<GraphQLDirective> enumValueDirectives = enumValueDefinition.getDirectives(); if (enumValueDefinition.isDeprecated()) { <START> enumValueDirectives = addDeprecatedDirectiveIfNeeded(enumValueDirectives); <END> } out.format(" %s%s\n", enumValueDefinition.getName(), directivesString(GraphQLEnumValueDefinition.class, enumValueDirectives)); } out.format("}\n\n"); }; }</code><technical_language>I this private method adding deprecated directive list if exist. Is a reason this is a list instead of a set in case this private method needed</technical_language>

Target:
private TypePrinter<GraphQLEnumType> enumPrinter() { return (out, type, visibility) -> { if (isIntrospectionType(type)) { return; } GraphqlTypeComparatorEnvironment environment = GraphqlTypeComparatorEnvironment.newEnvironment() .parentType(GraphQLEnumType.class) .elementType(GraphQLEnumValueDefinition.class) .build(); Comparator<? super GraphQLSchemaElement> comparator = options.comparatorRegistry.getComparator(environment); if (shouldPrintAsAst(type.getDefinition())) { printAsAst(out, type.getDefinition(), type.getExtensionDefinitions()); } else { printComments(out, type, ""); out.format("enum %s%s {\n", type.getName(), directivesString(GraphQLEnumType.class, type.getDirectives())); List<GraphQLEnumValueDefinition> values = type.getValues() .stream() .sorted(comparator) .collect(toList()); for (GraphQLEnumValueDefinition enumValueDefinition : values) { printComments(out, enumValueDefinition, " "); List<GraphQLDirective> enumValueDirectives = enumValueDefinition.getDirectives(); if (enumValueDefinition.isDeprecated()) { enumValueDirectives = addDeprecatedDirectiveIfNeeded(enumValueDirectives); } out.format(" %s%s\n", enumValueDefinition.getName(), directivesString(GraphQLEnumValueDefinition.class, enumValueDirectives)); } out.format("}\n\n"); } }; }
Source:
<code>public OuiaComponentTypeAttribute ouiaComponentType() { <START> return new OuiaComponentTypeAttribute("tree-item"); <END> }</code><technical_language>places "tree-item", consider replacing a constant other classes</technical_language>

Target:
public OuiaComponentTypeAttribute ouiaComponentType() { return new OuiaComponentTypeAttribute(OUIA_COMPONENT_TYPE); }
Source:
<code>protected void commitWithRetries(Consumer<?, ?> consumer, Optional<Map<TopicPartition, OffsetAndMetadata>> offsets) throws DatastreamRuntimeException { preCommitHook(); AtomicReference<KafkaException> lastKafkaExceptionRef = new AtomicReference<>(null); boolean result = PollUtils.poll(() -> { try { if (offsets.isPresent()) { consumer.commitSync(offsets.get(), _commitTimeout); _kafkaTopicPartitionTracker.onOffsetsCommitted(offsets.get()); } else { consumer.commitSync(_commitTimeout); } _logger.info("Commit succeeded."); } catch (KafkaException e) { lastKafkaExceptionRef.set(e); if (_shutdown) { _logger.info("Caught KafkaException in commitWithRetries while shutting down, so exiting.", e); <START> return true; <END> } _logger.warn(String.format("Commit failed with exception. DatastreamTask = %s", _datastreamTask.getDatastreamTaskName()), e); return false; } return true; }, COMMIT_RETRY_INTERVAL_MILLIS, COMMIT_RETRY_TIMEOUT_MILLIS); postCommitHook(result, lastKafkaExceptionRef.get()); if (!result) { String msg = "Commit failed after several retries, Giving up."; _logger.error(msg); throw new DatastreamRuntimeException(msg); } }</code><technical_language>shutdown, if hits KafkaException, returns true, success. This impact postCommit hook</technical_language>

Target:
protected void commitWithRetries(Consumer<?, ?> consumer, Optional<Map<TopicPartition, OffsetAndMetadata>> offsets) throws DatastreamRuntimeException { preCommitHook(); boolean result = PollUtils.poll(() -> { try { if (offsets.isPresent()) { consumer.commitSync(offsets.get(), _commitTimeout); _kafkaTopicPartitionTracker.onOffsetsCommitted(offsets.get()); } else { consumer.commitSync(_commitTimeout); } _logger.info("Commit succeeded."); } catch (KafkaException e) { if (_shutdown) { _logger.info("Caught KafkaException in commitWithRetries while shutting down, so exiting.", e); return true; } _logger.warn(String.format("Commit failed with exception. DatastreamTask = %s", _datastreamTask.getDatastreamTaskName()), e); return false; } return true; }, COMMIT_RETRY_INTERVAL_MILLIS, COMMIT_RETRY_TIMEOUT_MILLIS); if (!result) { String msg = "Commit failed after several retries, Giving up."; _logger.error(msg); throw new DatastreamRuntimeException(msg); } }
Source:
<code>private void disposeSuccessfulTest(BaseRealm testRealm) { looperThread.postRunnable(() -> { if (subscription != null) { subscription.dispose(); } if (!testRealm.getConfiguration().equals(realm.getConfiguration())) { throw new IllegalStateException("This method only works for Realms with the same configuration as the looper Realm"); } testRealm.close(); if (!realm.equals(testRealm)) { realm.close(); } looperThread.postRunnable(new Runnable() { @Override public void run() { RealmLog.error("Global: " + Realm.getGlobalInstanceCount(testRealm.getConfiguration())); <START> RealmLog.error("Local: " + Realm.getGlobalInstanceCount(testRealm.getConfiguration())); <END> if (Realm.getGlobalInstanceCount(testRealm.getConfiguration()) == 0) { looperThread.testComplete(); } else { RealmLog.error("" + Realm.getGlobalInstanceCount(testRealm.getConfiguration())); looperThread.postRunnable(this); } } }); }); }</code><technical_language>local</technical_language>

Target:
private void disposeSuccessfulTest(BaseRealm testRealm) { looperThread.postRunnable(() -> { if (subscription != null) { subscription.dispose(); } if (!testRealm.getConfiguration().equals(realm.getConfiguration())) { throw new IllegalStateException("This method only works for Realms with the same configuration as the looper Realm"); } testRealm.close(); if (!realm.equals(testRealm)) { realm.close(); } looperThread.postRunnable(new Runnable() { @Override public void run() { if (Realm.getGlobalInstanceCount(testRealm.getConfiguration()) == 0) { looperThread.testComplete(); } else { RealmLog.error("" + Realm.getGlobalInstanceCount(testRealm.getConfiguration())); looperThread.postRunnable(this); } } }); }); }
Source:
<code>public int hashCode() { int result = cl != null ? cl.hashCode() : 0; result = 31 * result + (transport != null ? transport.hashCode() : 0); result = 31 * result + (globalJmxStatistics != null ? globalJmxStatistics.hashCode() : 0); result = 31 * result + (serialization != null ? serialization.hashCode() : 0); result = 31 * result + (evictionThreadPool != null ? evictionThreadPool.hashCode() : 0); result = 31 * result + (listenerThreadPool != null ? listenerThreadPool.hashCode() : 0); result = 31 * result + (replicationQueueThreadPool != null ? replicationQueueThreadPool.hashCode() : 0); result = 31 * result + (persistenceThreadPool != null ? persistenceThreadPool.hashCode() : 0); result = 31 * result + (shutdown != null ? shutdown.hashCode() : 0); result = 31 * result + (site != null ? site.hashCode() : 0); <START> return result; <END> }</code><technical_language>security is missing in hashCode() is present in equals()</technical_language>

Target:
public int hashCode() { int result = cl != null ? cl.hashCode() : 0; result = 31 * result + (transport.hashCode()); result = 31 * result + (globalJmxStatistics.hashCode()); result = 31 * result + (serialization.hashCode()); result = 31 * result + (evictionThreadPool.hashCode()); result = 31 * result + (listenerThreadPool.hashCode()); result = 31 * result + (replicationQueueThreadPool.hashCode()); result = 31 * result + (persistenceThreadPool.hashCode()); result = 31 * result + (shutdown.hashCode()); result = 31 * result + (site.hashCode()); result = 31 * result + (security.hashCode()); return result; }
Source:
<code>public void setMarkerCluster(GMarkerCluster markerCluster) { this.markerCluster = markerCluster; <START> add(new GMapMarkerClustererHeaderContributor()); <END> }</code><technical_language>whitespace</technical_language>

Target:
public void setMarkerCluster(GMarkerCluster markerCluster) { if(markerCluster == null) throw new IllegalArgumentException("GMarkerCluster argument should not be null."); this.markerCluster = markerCluster; if (getBehaviors(GMapMarkerClustererHeaderContributor.class).isEmpty()) { add(new GMapMarkerClustererHeaderContributor()); } }
Source:
<code><START> public String toString() { <END> return "(" + label + ", " + status + ", " + appliedBy + ")"; }</code><technical_language>missing @Override declaration</technical_language>

Target:
public String toString() { StringBuilder sb = new StringBuilder(); sb.append(label).append(": ").append(status); if (appliedBy != null) { sb.append(" by ").append(appliedBy); } return sb.toString(); }
Source:
<code>public RepositoryDirectoryValue.Builder fetch(Rule rule, Path outputDirectory, BlazeDirectories directories, Environment env, Map<String, String> markerData) throws RepositoryFunctionException, InterruptedException { SkylarkSemantics skylarkSemantics = PrecomputedValue.SKYLARK_SEMANTICS.get(env); if (skylarkSemantics == null) { return null; } if (skylarkSemantics.incompatibleRemoveNativeMavenJar()) { throw new RepositoryFunctionException( new EvalException( null, "The native maven_jar rule is deprecated." <START> + " add details about replacement here." <END> + "\nUse --incompatible_remove_native_maven_jar=false to temporarily continue" + " using the native rule."), Transience.PERSISTENT); } MavenServerValue serverValue = getServer(rule, env); if (env.valuesMissing()) { return null; } Path outputDir = getExternalRepositoryDirectory(directories).getRelative(rule.getName()); return createOutputTree(rule, outputDir, serverValue, env.getListener()); }</code><technical_language>This line a TODO</technical_language>

Target:
public RepositoryDirectoryValue.Builder fetch(Rule rule, Path outputDirectory, BlazeDirectories directories, Environment env, Map<String, String> markerData) throws RepositoryFunctionException, InterruptedException { SkylarkSemantics skylarkSemantics = PrecomputedValue.SKYLARK_SEMANTICS.get(env); if (skylarkSemantics == null) { return null; } if (skylarkSemantics.incompatibleRemoveNativeMavenJar()) { throw new RepositoryFunctionException( new EvalException( null, "The native maven_jar rule is deprecated." + " See <LINK_0>" + "backward-compatibility.html#remove-native-maven-jar for migration information." + "\nUse --incompatible_remove_native_maven_jar=false to temporarily continue" + " using the native rule."), Transience.PERSISTENT); } MavenServerValue serverValue = getServer(rule, env); if (env.valuesMissing()) { return null; } Path outputDir = getExternalRepositoryDirectory(directories).getRelative(rule.getName()); return createOutputTree(rule, outputDir, serverValue, env.getListener()); }
Source:
<code>private PackList scanPacksImpl(final PackList old) { final Map<String, PackFile> forReuse = reuseMap(old); final FileSnapshot snapshot = FileSnapshot.save(packDirectory); final Set<String> names = listPackDirectory(); final List<PackFile> list = new ArrayList<PackFile>(names.size() >> 2); boolean foundNew = false; for (final String indexName : names) { if (indexName.length() != 49 || !indexName.endsWith(".idx")) continue; final String base = indexName.substring(0, indexName.length() - 3); <START> final List<PackExt> extensions = new ArrayList<PackExt>( <END> PackExt.values().length); for (PackExt ext : PackExt.values()) { if (names.contains(base + ext.getExtension())) extensions.add(ext); } if (!extensions.contains(PACK)) { continue; } final String packName = base + PACK.getExtension(); final PackFile oldPack = forReuse.remove(packName); if (oldPack != null) { list.add(oldPack); continue; } final File packFile = new File(packDirectory, packName); list.add(new PackFile(packFile, extensions)); foundNew = true; } if (!foundNew && forReuse.isEmpty() && snapshot.equals(old.snapshot)) { old.snapshot.setClean(snapshot); return old; } for (final PackFile p : forReuse.values()) { p.close(); } if (list.isEmpty()) return new PackList(snapshot, NO_PACKS.packs); final PackFile[] r = list.toArray(new PackFile[list.size()]); Arrays.sort(r, PackFile.SORT); return new PackList(snapshot, r); }</code><technical_language>above, a list bit form</technical_language>

Target:
private PackList scanPacksImpl(final PackList old) { final Map<String, PackFile> forReuse = reuseMap(old); final FileSnapshot snapshot = FileSnapshot.save(packDirectory); final Set<String> names = listPackDirectory(); final List<PackFile> list = new ArrayList<PackFile>(names.size() >> 2); boolean foundNew = false; for (final String indexName : names) { if (indexName.length() != 49 || !indexName.endsWith(".idx")) continue; final String base = indexName.substring(0, indexName.length() - 3); int extensions = 0; for (PackExt ext : PackExt.values()) { if (names.contains(base + ext.getExtension())) extensions |= 1 << ext.getPosition(); } if ((extensions & 1 << PACK.getPosition()) == 0) { continue; } final String packName = base + PACK.getExtension(); final PackFile oldPack = forReuse.remove(packName); if (oldPack != null) { list.add(oldPack); continue; } final File packFile = new File(packDirectory, packName); list.add(new PackFile(packFile, extensions)); foundNew = true; } if (!foundNew && forReuse.isEmpty() && snapshot.equals(old.snapshot)) { old.snapshot.setClean(snapshot); return old; } for (final PackFile p : forReuse.values()) { p.close(); } if (list.isEmpty()) return new PackList(snapshot, NO_PACKS.packs); final PackFile[] r = list.toArray(new PackFile[list.size()]); Arrays.sort(r, PackFile.SORT); return new PackList(snapshot, r); }
Source:
<code>protected static Iterator<Object[]> invokeDataProvider(Object instance, Method dataProvider, ITestNGMethod method, ITestContext testContext, Object fedInstance, IAnnotationFinder annotationFinder) { List<Object> parameters = getParameters(dataProvider, method, testContext, fedInstance, annotationFinder); Object result = invokeMethodNoCheckedException(dataProvider, instance, parameters); if (result instanceof Object[][]) { return new ArrayIterator((Object[][]) result); } else if (result instanceof Object[]) { return new OneToTwoDimArrayIterator((Object[]) result); } else if (result instanceof Iterator) { Type returnType = dataProvider.getGenericReturnType(); if (returnType instanceof ParameterizedType) { ParameterizedType contentType = (ParameterizedType) returnType; Class<?> type = (Class<?>) contentType.getActualTypeArguments()[0]; if (type.isArray()) { return (Iterator<Object[]>) result; } else { return new OneToTwoDimIterator((Iterator<Object>) result); } } else { return (Iterator<Object[]>) result; } } throw new TestNGException("Data Provider " + dataProvider + " must return" <START> + " either Object[][] or Object[] or Iterator<Object>[], not " + dataProvider.getReturnType()); <END> }</code><technical_language>this addition? @DataProvider allow return Object[]</technical_language>

Target:
protected static Iterator<Object[]> invokeDataProvider(Object instance, Method dataProvider, ITestNGMethod method, ITestContext testContext, Object fedInstance, IAnnotationFinder annotationFinder) { List<Object> parameters = getParameters(dataProvider, method, testContext, fedInstance, annotationFinder); Object result = invokeMethodNoCheckedException(dataProvider, instance, parameters); if (result instanceof Object[][]) { return new ArrayIterator((Object[][]) result); } else if (result instanceof Object[]) { return new OneToTwoDimArrayIterator((Object[]) result); } else if (result instanceof Iterator) { Type returnType = dataProvider.getGenericReturnType(); if (returnType instanceof ParameterizedType) { ParameterizedType contentType = (ParameterizedType) returnType; Class<?> type = (Class<?>) contentType.getActualTypeArguments()[0]; if (type.isArray()) { return (Iterator<Object[]>) result; } else { return new OneToTwoDimIterator((Iterator<Object>) result); } } else { return (Iterator<Object[]>) result; } } throw new TestNGException("Data Provider " + dataProvider + " must return" + " either Object[][] or Object[] or Iterator<Object[]> or Iterator<Object>, not " + dataProvider.getReturnType()); }
Source:
<code>public void testKeepComment() throws Exception { IPackageFragment pack= fSourceFolder.createPackageFragment("test", false, null); StringBuffer buf= new StringBuffer(); buf.append("package test;\r\n"); buf.append("import java.util.Collection;\r\n"); buf.append("import java.util.Iterator;\r\n"); buf.append("public class A {\r\n"); buf.append(" Collection<String> c;\r\n"); buf.append(" public A() {\r\n"); buf.append(" for (final Iterator<String> iterator= c.iterator(); iterator.hasNext();) {\r\n"); buf.append(" buf.append(" String test= iterator.next();\r\n"); buf.append(" System.out.println(test);\r\n"); buf.append(" }\r\n");buf.append(" }\r\n"); buf.append("}"); ICompilationUnit unit= pack.createCompilationUnit("A.java", buf.toString(), false, null); List<IJavaCompletionProposal> proposals= fetchConvertingProposal(buf, unit); assertNotNull(fConvertLoopProposal); assertCorrectLabels(proposals); String preview= getPreviewContent(fConvertLoopProposal); buf= new StringBuffer(); buf.append("package test;\r\n"); buf.append("import java.util.Collection;\r\n"); buf.append("public class A {\r\n"); buf.append(" Collection<String> c;\r\n"); buf.append(" public A() {\r\n"); buf.append(" for (String test : c) {\r\n"); buf.append(" <START> buf.append(" System.out.println(test);\r\n"); <END> buf.append(" }\r\n"); buf.append(" }\r\n"); buf.append("}"); String expected= buf.toString(); assertEqualString(preview, expected); }</code><technical_language>replace space tabs</technical_language>

Target:
public void testKeepComment() throws Exception { IPackageFragment pack= fSourceFolder.createPackageFragment("test", false, null); StringBuffer buf= new StringBuffer(); buf.append("package test;\n"); buf.append("import java.util.Collection;\n"); buf.append("import java.util.Iterator;\n"); buf.append("public class A {\n"); buf.append(" Collection<String> c;\n"); buf.append(" public A() {\n"); buf.append(" for (final Iterator<String> iterator= c.iterator(); iterator.hasNext();) {\n"); buf.append(" buf.append(" String test= iterator.next();\n"); buf.append(" System.out.println(test);\n"); buf.append(" }\n"); buf.append(" }\n"); buf.append("}"); ICompilationUnit unit= pack.createCompilationUnit("A.java", buf.toString(), false, null); List<IJavaCompletionProposal> proposals= fetchConvertingProposal(buf, unit); assertNotNull(fConvertLoopProposal); assertCorrectLabels(proposals); String preview= getPreviewContent(fConvertLoopProposal); buf= new StringBuffer(); buf.append("package test;\n"); buf.append("import java.util.Collection;\n"); buf.append("public class A {\n"); buf.append(" Collection<String> c;\n"); buf.append(" public A() {\n"); buf.append(" for (String test : c) {\n"); buf.append(" buf.append(" System.out.println(test);\n"); buf.append(" }\n"); buf.append(" }\n"); buf.append("}"); String expected= buf.toString(); assertEqualString(preview, expected); }
Source:
<code>public void testAccessViewFsPathWithoutAuthority() throws Exception { final Path hdfsTargetPath = new Path(defaultFSURI + HDFS_USER_FOLDER); addMountLinks(defaultFSURI.getAuthority(), new String[] {HDFS_USER_FOLDER, LOCAL_FOLDER }, new String[] {hdfsTargetPath.toUri().toString(), localTargetDir.toURI().toString() }, conf); Path hdfsDir = new Path(HDFS_USER_FOLDER + "/test"); Path localDir = new Path(LOCAL_FOLDER + "/test"); try (ViewFileSystemOverloadScheme fs = (ViewFileSystemOverloadScheme) FileSystem.get(conf)) { fs.mkdirs(hdfsDir); fs.mkdirs(localDir); } FileStatus[] expectedStatus; try (FileSystem fs = FileSystem.get(conf)) { expectedStatus = fs.listStatus(new Path("/")); } Path viewFsRootPath = new Path("viewfs:/"); try { viewFsRootPath.getFileSystem(conf); Assert.fail("Mount table with authority default should not be initialized"); } catch (IOException e) { assertTrue(e.getMessage().contains("Empty Mount table in config for viewfs://default/")); } conf.set(Constants.CONFIG_VIEWFS_DEFAULT_MOUNT_TABLE_NAME_KEY, defaultFSURI.getAuthority()); try (FileSystem fs = viewFsRootPath.getFileSystem(conf)) { FileStatus[] status = fs.listStatus(viewFsRootPath); <START> List<String> expectedPaths = Arrays.stream(expectedStatus).map(s -> s.getPath().getName()).collect(Collectors.toList()); <END> List<String> paths = Arrays.stream(status).map(s -> s.getPath().getName()).collect(Collectors.toList()); assertEquals(expectedPaths, paths); } }</code><technical_language>Nit: Need formatting of lines</technical_language>

Target:
public void testAccessViewFsPathWithoutAuthority() throws Exception { final Path hdfsTargetPath = new Path(defaultFSURI + HDFS_USER_FOLDER); addMountLinks(defaultFSURI.getAuthority(), new String[] {HDFS_USER_FOLDER, LOCAL_FOLDER }, new String[] {hdfsTargetPath.toUri().toString(), localTargetDir.toURI().toString() }, conf); Path hdfsDir = new Path(HDFS_USER_FOLDER, "test"); Path localDir = new Path(LOCAL_FOLDER, "test"); FileStatus[] expectedStatus; try (FileSystem fs = FileSystem.get(conf)) { fs.mkdirs(hdfsDir); fs.mkdirs(localDir); expectedStatus = fs.listStatus(new Path("/")); } Path viewFsRootPath = new Path("viewfs:/"); try { viewFsRootPath.getFileSystem(conf); Assert.fail( "Mount table with authority default should not be initialized"); } catch (IOException e) { assertTrue(e.getMessage().contains( "Empty Mount table in config for viewfs://default/")); } conf.set(Constants.CONFIG_VIEWFS_DEFAULT_MOUNT_TABLE_NAME_KEY, defaultFSURI.getAuthority()); try (FileSystem fs = viewFsRootPath.getFileSystem(conf)) { FileStatus[] status = fs.listStatus(viewFsRootPath); List<String> expectedPaths = Arrays.stream(expectedStatus) .map(s -> s.getPath().getName()).sorted() .collect(Collectors.toList()); List<String> paths = Arrays.stream(status) .map(s -> s.getPath().getName()).sorted() .collect(Collectors.toList()); assertEquals(expectedPaths, paths); } }
Source:
<code><START> static int getUnstaged(@Nullable IndexDiffData indexDiff) { <END> if (indexDiff == null) { return 0; } int size = indexDiff.getUntracked().size(); size += indexDiff.getMissing().size(); size += indexDiff.getModified().size(); size += indexDiff.getConflicting().size(); return size; }</code><technical_language>this method declared private</technical_language>

Target:
private static int getUnstaged(@Nullable IndexDiffData indexDiff) { if (indexDiff == null) { return 0; } int size = indexDiff.getUntracked().size(); size += indexDiff.getMissing().size(); size += indexDiff.getModified().size(); size += indexDiff.getConflicting().size(); return size; }
Source:
<code>public void testUnmarshallSubprocessLevelEventWithEdgesEmptyProperties() throws Exception { Diagram<Graph, Metadata> diagram = unmarshall(marshaller, BPMN_CATCHING_INTERMEDIATE_EVENT_FILE_PATH); assertDiagram(diagram, AMOUNT_OF_NODES_IN_DIAGRAM); IntermediateMessageEventCatching emptySubprocessEvent = getCatchingIntermediateNodeById(diagram, EMPTY_WITH_OUTGOING_EDGE_SUBPROCESS_LEVEL_EVENT_ID, HAS_NO_INCOME_EDGE, HAS_OUTGOING_EDGE); assertGeneralSet(emptySubprocessEvent.getGeneral(), EMPTY_VALUE, EMPTY_VALUE); <START> assertMessageEventExecutionSet(emptySubprocessEvent.getExecutionSet(), EMPTY_VALUE, NON_CANCELLING, ""); <END> assertDataIOSet(emptySubprocessEvent.getDataIOSet(), EMPTY_VALUE); }</code><technical_language>suggestion assertMessageEventExecutionSet(emptySubprocessEvent.getExecutionSet(), EMPTY_VALUE, NON_CANCELLING, EMPTY_VALUE);</technical_language>

Target:
public void testUnmarshallSubprocessLevelEventWithEdgesEmptyProperties() throws Exception { Diagram<Graph, Metadata> diagram = unmarshall(marshaller, BPMN_CATCHING_INTERMEDIATE_EVENT_FILE_PATH); assertDiagram(diagram, AMOUNT_OF_NODES_IN_DIAGRAM); IntermediateMessageEventCatching emptySubprocessEvent = getCatchingIntermediateNodeById(diagram, EMPTY_WITH_OUTGOING_EDGE_SUBPROCESS_LEVEL_EVENT_ID, HAS_NO_INCOME_EDGE, HAS_OUTGOING_EDGE); assertGeneralSet(emptySubprocessEvent.getGeneral(), EMPTY_VALUE, EMPTY_VALUE); assertMessageEventExecutionSet(emptySubprocessEvent.getExecutionSet(), EMPTY_VALUE, NON_CANCELLING, EMPTY_VALUE); assertDataIOSet(emptySubprocessEvent.getDataIOSet(), EMPTY_VALUE); }
Source:
<code>public List<CommitValidationMessage> onCommitReceived( CommitReceivedEvent receiveEvent) throws CommitValidationException { final List<String> idList = receiveEvent.commit.getFooterLines(CHANGE_ID); List<CommitValidationMessage> messages = new LinkedList<CommitValidationMessage>(); if (idList.isEmpty()) { if (projectControl.getProjectState().isRequireChangeID()) { String shortMsg = receiveEvent.commit.getShortMessage(); String changeIdPrefix = CHANGE_ID.getName() + ":"; if (shortMsg.startsWith(changeIdPrefix) && shortMsg.substring(changeIdPrefix.length()).trim() .matches("^I[0-9a-f]{8,}.*$")) { throw new CommitValidationException( "missing <START> subject, Change-Id <END> must be in commit message footer"); } else { String errMsg = "missing Change-Id in commit message footer"; messages.add(getFixedCommitMsgWithChangeId( errMsg, receiveEvent.commit)); throw new CommitValidationException(errMsg, messages); } } } else if (idList.size() > 1) { throw new CommitValidationException( "multiple Change-Id lines in commit message footer", messages); } else { final String v = idList.get(idList.size() - 1).trim(); if (!v.matches("^I[0-9a-f]{8,}.*$")) { final String errMsg = "missing or invalid Change-Id line format in commit message footer"; messages.add( getFixedCommitMsgWithChangeId(errMsg, receiveEvent.commit)); throw new CommitValidationException(errMsg, messages); } } return Collections.emptyList(); }</code><technical_language>change this a semicolon</technical_language>

Target:
public List<CommitValidationMessage> onCommitReceived( CommitReceivedEvent receiveEvent) throws CommitValidationException { final List<String> idList = receiveEvent.commit.getFooterLines(CHANGE_ID); List<CommitValidationMessage> messages = new LinkedList<CommitValidationMessage>(); if (idList.isEmpty()) { if (projectControl.getProjectState().isRequireChangeID()) { String shortMsg = receiveEvent.commit.getShortMessage(); String changeIdPrefix = CHANGE_ID.getName() + ":"; if (shortMsg.startsWith(changeIdPrefix) && shortMsg.substring(changeIdPrefix.length()).trim() .matches("^I[0-9a-f]{8,}.*$")) { throw new CommitValidationException( "missing subject; Change-Id must be in commit message footer"); } else { String errMsg = "missing Change-Id in commit message footer"; messages.add(getFixedCommitMsgWithChangeId( errMsg, receiveEvent.commit)); throw new CommitValidationException(errMsg, messages); } } } else if (idList.size() > 1) { throw new CommitValidationException( "multiple Change-Id lines in commit message footer", messages); } else { final String v = idList.get(idList.size() - 1).trim(); if (!v.matches("^I[0-9a-f]{8,}.*$")) { final String errMsg = "missing or invalid Change-Id line format in commit message footer"; messages.add( getFixedCommitMsgWithChangeId(errMsg, receiveEvent.commit)); throw new CommitValidationException(errMsg, messages); } } return Collections.emptyList(); }
Source:
<code>public User populate(ModelTranslator modelTranslator, UserDTO source, User destination) { if (source == null) { throw new IllegalArgumentException("source is null"); } if (destination == null) { throw new IllegalArgumentException("destination is null"); } destination.setCreated(source.getCreated() != null ? new Date(source.getCreated().toInstant().toEpochMilli()) : null); destination.setUpdated(source.getUpdated() != null ? new Date(source.getUpdated().toInstant().toEpochMilli()) : null); destination.setId(source.getId()); destination.setUsername(source.getUsername()); destination.setSuperAdmin(source.getSuperAdmin()); <START> destination.setPassword(source.getPassword() != null ? Util.hash(source.getPassword()) : null); <END> destination.setHashedPassword(source.getPassword() != null ? Util.hash(source.getPassword()) : null); return destination; }</code><technical_language>User.setPassword method is hashing password, hashing putting argument, means re-hash hashed password. If check User class, setPassword setHashedPassword setting field (one hashing first, other without), is need call setters. remove setPassword here, leave next call is (setHashedPassword)</technical_language>

Target:
public User populate(ModelTranslator modelTranslator, UserDTO source, User destination) { if (source == null) { throw new IllegalArgumentException("source is null"); } if (destination == null) { throw new IllegalArgumentException("destination is null"); } destination.setCreated(source.getCreated() != null ? new Date(source.getCreated().toInstant().toEpochMilli()) : null); destination.setUpdated(source.getUpdated() != null ? new Date(source.getUpdated().toInstant().toEpochMilli()) : null); destination.setUsername(source.getUsername()); destination.setSuperAdmin(source.getSuperAdmin() == null ? false : source.getSuperAdmin()); destination.setHashedPassword(source.getPassword() != null ? Util.hash(source.getPassword()) : null); return destination; }
Source:
<code>public static RemoteConfig findRemoteConfig(Config config, String remoteName) throws URISyntaxException { <START> List<RemoteConfig> allRemoteConfigs; <END> RemoteConfig remoteConfig = null; allRemoteConfigs = RemoteConfig.getAllRemoteConfigs(config); for (RemoteConfig rc : allRemoteConfigs) { if (rc.getName().equals(remoteName)) remoteConfig = rc; } return remoteConfig; }</code><technical_language>join variable definition please</technical_language>

Target:
public static RemoteConfig findRemoteConfig(Config config, String remoteName) throws URISyntaxException { RemoteConfig remoteConfig = null; List<RemoteConfig> allRemoteConfigs = RemoteConfig .getAllRemoteConfigs(config); for (RemoteConfig rc : allRemoteConfigs) { if (rc.getName().equals(remoteName)) { remoteConfig = rc; break; } } return remoteConfig; }
Source:
<code>public Allergies setAllergies(Patient patient, Allergies allergies) { List<Allergy> dbAllergyList = getAllergies(patient); for (Allergy originalAllergy : dbAllergyList) { if (allergies.contains(originalAllergy)) { Allergy potentiallyEditedAllergy = allergies.getAllergy(originalAllergy.getAllergyId()); if (!potentiallyEditedAllergy.hasSameValues(originalAllergy)) { Allergy newAllergy = new Allergy(); try { allergies.remove(potentiallyEditedAllergy); newAllergy.copy(potentiallyEditedAllergy); allergies.add(newAllergy); voidAllergy(originalAllergy); } catch (Exception ex) { throw new APIException("Failed to copy edited values", ex); } } continue; } voidAllergy(originalAllergy); } for (Allergy allergy : allergies) { if (allergy.getAllergyId() == null && allergy.getAllergen().getCodedAllergen() == null && StringUtils.isNotBlank(allergy.getAllergen().getNonCodedAllergen())) { <START> String otherNonCodedConceptUuid = Context.getAdministrationService().getGlobalProperty(OpenmrsConstants.GP_ALLERGEN_OTHER_NON_CODED_UUID); <END> Concept otherNonCoded = Context.getConceptService().getConceptByUuid(otherNonCodedConceptUuid); if (otherNonCoded == null) { throw new APIException("Can't find concept with uuid:" + otherNonCodedConceptUuid); } allergy.getAllergen().setCodedAllergen(otherNonCoded); } } return dao.saveAllergies(patient, allergies); }</code><technical_language>set this startup, do need fetch instead of Allergen. getOtherNonCodedConceptUuid()</technical_language>

Target:
public Allergies setAllergies(Patient patient, Allergies allergies) { List<Allergy> dbAllergyList = getAllergies(patient); for (Allergy originalAllergy : dbAllergyList) { if (allergies.contains(originalAllergy)) { Allergy potentiallyEditedAllergy = allergies.getAllergy(originalAllergy.getAllergyId()); if (!potentiallyEditedAllergy.hasSameValues(originalAllergy)) { Allergy newAllergy = new Allergy(); try { allergies.remove(potentiallyEditedAllergy); newAllergy.copy(potentiallyEditedAllergy); allergies.add(newAllergy); voidAllergy(originalAllergy); } catch (Exception ex) { throw new APIException("Failed to copy edited values", ex); } } continue; } voidAllergy(originalAllergy); } for (Allergy allergy : allergies) { if (allergy.getAllergyId() == null && allergy.getAllergen().getCodedAllergen() == null && StringUtils.isNotBlank(allergy.getAllergen().getNonCodedAllergen())) { Concept otherNonCoded = Context.getConceptService().getConceptByUuid(Allergen.getOtherNonCodedConceptUuid()); if (otherNonCoded == null) { throw new APIException("Can't find concept with uuid:" + Allergen.getOtherNonCodedConceptUuid()); } allergy.getAllergen().setCodedAllergen(otherNonCoded); } } return dao.saveAllergies(patient, allergies); }
Source:
<code>public void initialize(Config newConfig) { super.initialize(newConfig); this.config = newConfig; this.connectionString = Context.stateManagerConnectionString(newConfig); this.isSchedulerService = Context.schedulerService(newConfig); this.tunnelProcesses = new ArrayList<>(); NetworkUtils.TunnelConfig tunnelConfig = NetworkUtils.TunnelConfig.build(config, NetworkUtils.HeronSystem.STATE_MANAGER); if (tunnelConfig.isTunnelNeeded()) { Pair<String, List<Process>> tunneledResults = setupZkTunnel(tunnelConfig); String newConnectionString = tunneledResults.first; if (newConnectionString.isEmpty()) { <START> throw new IllegalArgumentException("Cannot connect to tunnelHost: " + tunnelConfig.getTunnelHost() + "\nBad connectionString: " + connectionString); <END> } connectionString = newConnectionString; tunnelProcesses.addAll(tunneledResults.second); } client = getCuratorClient(); LOG.info("Starting client to: " + connectionString); client.start(); try { if (!client.blockUntilConnected(ZkContext.connectionTimeoutMs(newConfig), TimeUnit.MILLISECONDS)) { throw new RuntimeException("Failed to initialize CuratorClient"); } } catch (InterruptedException e) { throw new RuntimeException("Failed to initialize CuratorClient", e); } if (ZkContext.isInitializeTree(newConfig)) { initTree(); } }</code><technical_language>remove newline exception message. This complicates log parsing grepping loglens</technical_language>

Target:
public void initialize(Config newConfig) { super.initialize(newConfig); this.config = newConfig; this.connectionString = Context.stateManagerConnectionString(newConfig); this.isSchedulerService = Context.schedulerService(newConfig); this.tunnelProcesses = new ArrayList<>(); NetworkUtils.TunnelConfig tunnelConfig = NetworkUtils.TunnelConfig.build(config, NetworkUtils.HeronSystem.STATE_MANAGER); if (tunnelConfig.isTunnelNeeded()) { Pair<String, List<Process>> tunneledResults = setupZkTunnel(tunnelConfig); String newConnectionString = tunneledResults.first; if (newConnectionString.isEmpty()) { throw new IllegalArgumentException("Cannot connect to tunnelHost: " + tunnelConfig.getTunnelHost() + " Bad connectionString: " + connectionString); } connectionString = newConnectionString; tunnelProcesses.addAll(tunneledResults.second); } client = getCuratorClient(); LOG.info("Starting client to: " + connectionString); client.start(); try { if (!client.blockUntilConnected(ZkContext.connectionTimeoutMs(newConfig), TimeUnit.MILLISECONDS)) { throw new RuntimeException("Failed to initialize CuratorClient"); } } catch (InterruptedException e) { throw new RuntimeException("Failed to initialize CuratorClient", e); } if (ZkContext.isInitializeTree(newConfig)) { initTree(); } }
Source:
<code>private void writeToFile(URI parentName, String newFileName, String content, Properties metadata) throws Exception { File parent = new File(parentName); if (!parent.exists() && !parent.mkdirs()) { handleException("Unable to create parent directory: " + parentName); } File newFile = new File(parent, newFileName); File newMetadataFile = new File(parent, newFileName + METADATA_FILE_SUFFIX); try (BufferedWriter writer = new BufferedWriter(new FileWriter(newFile)); BufferedWriter metadataWriter = new BufferedWriter(new FileWriter(newMetadataFile))) { writer.write(content); writer.flush(); <START> metadata.store(metadataWriter, null); <END> if (log.isDebugEnabled()) { log.debug("Successfully content written to file : " + parentName + URL_SEPARATOR + newFileName); } } catch (IOException e) { handleException("Couldn't write to registry resource: " + parentName + URL_SEPARATOR + newFileName, e); } }</code><technical_language>passing null a method, overload method</technical_language>

Target:
private void writeToFile(URI parentName, String newFileName, String content, Properties metadata) { File parent = new File(parentName); if (!parent.exists() && !parent.mkdirs()) { handleException("Unable to create parent directory: " + parentName); } File newFile = new File(parent, newFileName); try (BufferedWriter writer = new BufferedWriter(new FileWriter(newFile))) { writer.write(content); writer.flush(); writeMetadata(parent, newFileName, metadata); if (log.isDebugEnabled()) { log.debug("Successfully content written to file : " + parentName + URL_SEPARATOR + newFileName); } } catch (IOException e) { handleException("Couldn't write to registry resource: " + parentName + URL_SEPARATOR + newFileName, e); } }
Source:
<code>static void setResult(ImageView target, Context context, RequestHandler.Result result, boolean noFade, boolean debugging) { Drawable placeholder = target.getDrawable(); if (placeholder instanceof Animatable) { ((Animatable) placeholder).stop(); } if (result.hasBitmap()) { Picasso.LoadedFrom loadedFrom = result.getLoadedFrom(); Bitmap bitmap = result.getBitmap(); PicassoDrawable drawable = new PicassoDrawable(context, bitmap, placeholder, loadedFrom, noFade, debugging); target.setImageDrawable(drawable); } Drawable drawable = result.getDrawable(); <START> if (drawable != null) { <END> target.setImageDrawable(drawable); if (drawable instanceof Animatable) { ((Animatable) drawable).start(); } } }</code><technical_language>need a hasDrawable() method delete hasBitmap() Result</technical_language>

Target:
static void setResult(ImageView target, Context context, RequestHandler.Result result, boolean noFade, boolean debugging) { Drawable placeholder = target.getDrawable(); if (placeholder instanceof Animatable) { ((Animatable) placeholder).stop(); } if (result.hasBitmap()) { Picasso.LoadedFrom loadedFrom = result.getLoadedFrom(); Bitmap bitmap = result.getBitmap(); PicassoDrawable drawable = new PicassoDrawable(context, bitmap, placeholder, loadedFrom, noFade, debugging); target.setImageDrawable(drawable); } else if (result.hasDrawable()) { Drawable drawable = result.getDrawable(); target.setImageDrawable(drawable); if (drawable instanceof Animatable) { ((Animatable) drawable).start(); } } }
Source:
<code><START> public AccumuloClient getConnector(String user, String passwd) <END> throws AccumuloException, AccumuloSecurityException { return impl.getAccumuloClient(user, new PasswordToken(passwd)); }</code><technical_language>this return Connector</technical_language>

Target:
public Connector getConnector(String user, String passwd) throws AccumuloException, AccumuloSecurityException { return Connector.from(impl.getAccumuloClient(user, new PasswordToken(passwd))); }
Source:
<code>public void loginFacebook(final String fbAccessToken, final MojioClient.ResponseListener<User> responseListener) { if (_oauthHelper.shouldRefreshAccessToken() || _oauthHelper.isUserToken()) { authenticateApp(new ResponseListener<UserToken>() { @Override public void onSuccess(UserToken result) { <START> loginFacebook(fbAccessToken, responseListener); <END> } @Override public void onFailure(ResponseError error) { responseListener.onFailure(error); } }); return; } String entityPath = String.format("Login/ExternalUser?accessToken=%s", fbAccessToken); this.create(UserToken.class, entityPath, new MojioClient.ResponseListener<UserToken>() { @Override public void onSuccess(UserToken result) { _oauthHelper.setAccessToken(result._id, result.ValidUntil); String userID = result.UserId; getUser(userID, responseListener); } @Override public void onFailure(ResponseError error) { responseListener.onFailure(error); } }); }</code><technical_language>handy add a Log.d message a success</technical_language>

Target:
public void loginFacebook(final String fbAccessToken, final MojioClient.ResponseListener<User> responseListener) { if (_oauthHelper.shouldRefreshAccessToken() || _oauthHelper.isUserToken()) { authenticateApp(new ResponseListener<UserToken>() { @Override public void onSuccess(UserToken result) { Log.d(TAG, "App token " + result._id + " set. Proceeding to external user login."); loginFacebook(fbAccessToken, responseListener); } @Override public void onFailure(ResponseError error) { responseListener.onFailure(error); } }); return; } String entityPath = String.format("Login/ExternalUser?accessToken=%s", fbAccessToken); this.create(UserToken.class, entityPath, new MojioClient.ResponseListener<UserToken>() { @Override public void onSuccess(UserToken result) { Log.d(TAG, "Login successful. User token: " + result._id); _oauthHelper.setAccessToken(result._id, result.ValidUntil); String userID = result.UserId; getUser(userID, responseListener); } @Override public void onFailure(ResponseError error) { responseListener.onFailure(error); } }); }
Source:
<code>public DatabaseBackupResponse createDatabaseBackup(DatabaseBackupRequest databaseBackupRequest) throws MambuApiException { <START> if (databaseBackupRequest == null || databaseBackupRequest.getCallback() == null) { <END> throw new IllegalArgumentException("Request object and callback field must not be null!!"); } createDatabaseBackup.setUrlPath(createDatabaseBackup.getEndPoint() + "/" + APIData.BACKUP); return serviceExecutor.executeJson(createDatabaseBackup, databaseBackupRequest); }</code><technical_language>I check URL is a valid one, otherwise, MBU-11020, error message returned</technical_language>

Target:
public DatabaseBackupResponse createDatabaseBackup(DatabaseBackupRequest databaseBackupRequest) throws MambuApiException { if (databaseBackupRequest != null && databaseBackupRequest.getCallback() != null) { validateCallbackUrl(databaseBackupRequest.getCallback()); } createDatabaseBackup.setUrlPath(createDatabaseBackup.getEndPoint() + "/" + APIData.BACKUP); return serviceExecutor.executeJson(createDatabaseBackup, databaseBackupRequest); }
Source:
<code>public void run() { try { <START> cloudFile.uploadFromFile(TEMP_FILE.toString()); <END> } catch (StorageException | IOException | URISyntaxException e) { throw new RuntimeException(e); } }</code><technical_language>Cache file name string</technical_language>

Target:
public void run() { try { cloudFile.uploadFromFile(TEMP_FILE_PATH); } catch (StorageException | IOException | URISyntaxException e) { throw new RuntimeException(e); } }
Source:
<code>public TypeConverter createTypeConverter(String className) throws Exception { Class<?> classInstance = objectFactory.getClassInstance(className); <START> if (TypeConverter.class.isAssignableFrom(classInstance)) { <END> Class<? extends TypeConverter> converterClass = (Class<? extends TypeConverter>) classInstance; return objectFactory.buildConverter(converterClass, null); } else { throw new IllegalArgumentException("Type converter class " + className + " doesn't implement " + TypeConverter.class.getName()); } }</code><technical_language>I this if-else replaced return createTypeConverter(classInstance)</technical_language>

Target:
public TypeConverter createTypeConverter(String className) throws Exception { Class<?> clazz = objectFactory.getClassInstance(className); return createTypeConverter(clazz); }
Source:
<code>public void workerExecutor3(Vertx vertx) { int poolSize = 10; <START> long maxExecuteTime = 120000000000L; <END> WorkerExecutor executor = vertx.createSharedWorkerExecutor("my-worker-pool", poolSize, maxExecuteTime); }</code><technical_language>I change this example call 2 minutes TimeUnit.SECONDS instead</technical_language>

Target:
public void workerExecutor3(Vertx vertx) { int poolSize = 10; long maxExecuteTime = 2; TimeUnit maxExecuteTimeUnit = TimeUnit.SECONDS; WorkerExecutor executor = vertx.createSharedWorkerExecutor("my-worker-pool", poolSize, maxExecuteTime, maxExecuteTimeUnit); }
Source:
<code>public void testSchemaWithLongName() throws Exception { deployContrib("org.nuxeo.ecm.core.storage.sql.test.tests", "OSGI-INF/test-schema-longname.xml"); applyInlineDeployments(); <START> postSetUp(); <END> Session session = repository.getConnection(); session.getRootNode(); }</code><technical_language>postSetUp called applyInlineDeployments + pushInlineDeployments</technical_language>

Target:
public void testSchemaWithLongName() throws Exception { pushInlineDeployments("org.nuxeo.ecm.core.storage.sql.test.tests:OSGI-INF/test-schema-longname.xml"); Session session = repository.getConnection(); session.getRootNode(); }
Source:
<code>public void onItemClick(AdapterView<?> parent, View view, int position, long id) { if(!currentItems.isEmpty()) { JSONObject item = (JSONObject) parent.getAdapter().getItem(position); this.selection = item.has("Text1")?item.getString("Text1"):""; this.selectionDetailText = item.has("Text2")?item.getString("Text2"):""; this.selectionIndex = itemAdapterCopy.getPosition(item)+1; } else { <START> System.out.println("Spannable Adapter/..........."); <END> Spannable item = (Spannable) parent.getAdapter().getItem(position); this.selection = item.toString(); this.selectionIndex = adapterCopy.getPosition(item) + 1; } AfterPicking(); }</code><technical_language>Wrap debugging messages a if (DEBUG) line defined DEBUG top of file. This turn this working</technical_language>

Target:
public void onItemClick(AdapterView<?> parent, View view, int position, long id) { if (!dictItems.isEmpty()) { YailDictionary item = (YailDictionary) parent.getAdapter().getItem(position); this.selection = ElementsUtil.toStringEmptyIfNull(item.get("Text1")); this.selectionDetailText = ElementsUtil.toStringEmptyIfNull(item.get("Text2")); this.selectionIndex = itemAdapterCopy.getPosition(item) + 1; } else { Spannable item = (Spannable) parent.getAdapter().getItem(position); this.selection = item.toString(); this.selectionIndex = adapterCopy.getPosition(item) + 1; } AfterPicking(); }
Source:
<code>protected void addParametersForScript( final Map<String, Object> parameters, final Set<Command> resources, final JobRequest jobRequest ) { <START> parameters.put(COMMANDS_BINDING, resources); <END> }</code><technical_language>job request is added parameters automatically, list of resources is not</technical_language>

Target:
protected void addParametersForScript( final Map<String, Object> parameters, final Set<Command> resources, final JobRequest jobRequest ) { super.addParametersForScript(parameters, resources, jobRequest); parameters.put(COMMANDS_BINDING, resources); }