auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void testVmElementDiskPluggedStatus() { DiskVmElement dvePlugged = dao.get(new VmDeviceId(PLUGGED_DISK_ID, FixturesTool.VM_RHEL5_POOL_57)); assertTrue(dvePlugged.isPlugged()); <START> DiskVmElement dveUnlugged = dao.get(new VmDeviceId(UNPLUGGED_DISK_ID, FixturesTool.VM_RHEL5_POOL_57)); assertFalse(dveUnlugged.isPlugged()); <END> }</code><technical_language>this in a test. try fetch DiskVmElement of a floating disk assert returns null</technical_language>

Target:
public void testVmElementDiskPluggedStatus() { DiskVmElement dvePlugged = dao.get(new VmDeviceId(PLUGGED_DISK_ID, FixturesTool.VM_RHEL5_POOL_57)); assertTrue(dvePlugged.isPlugged()); }
Source:
<code>OMClientResponse onFailure(OMResponse.Builder omResponse, IOException ex) { <START> LOG.error("Add acl to volume failed!", <END> getAcl(), getVolumeName(), ex); return new OMVolumeAclOpResponse(null, createErrorOMResponse(omResponse, ex)); }</code><technical_language>I this approach, code cleaner. move logging of onSuccess OnFailure. called lock held</technical_language>

Target:
OMClientResponse onFailure(OMResponse.Builder omResponse, IOException ex) { return new OMVolumeAclOpResponse(null, createErrorOMResponse(omResponse, ex)); }
Source:
<code>public void initializeCertificateClient(OzoneConfiguration config) throws IOException { LOG.info("Initializing secure Datanode."); CertificateClient.InitResponse response = dnCertClient.init(); LOG.info("Init response: {}", response); switch (response) { case SUCCESS: LOG.info("Initialization successful."); break; case GETCERT: getSCMSignedCert(dnCertClient, config); LOG.info("Successfully stored SCM signed certificate."); break; case FAILURE: LOG.error("DN security initialization failed."); throw new RuntimeException("DN security initialization failed."); case RECOVER: <START> LOG.error("DN security initialization failed. OM certificate is " + <END> "missing."); throw new RuntimeException("DN security initialization failed."); default: LOG.error("DN security initialization failed. Init response: {}", response); throw new RuntimeException("DN security initialization failed."); } }</code><technical_language>reword error message for recovery</technical_language>

Target:
public void initializeCertificateClient(OzoneConfiguration config) throws IOException { LOG.info("Initializing secure Datanode."); CertificateClient.InitResponse response = dnCertClient.init(); LOG.info("Init response: {}", response); switch (response) { case SUCCESS: LOG.info("Initialization successful, case:{}.", response); break; case GETCERT: getSCMSignedCert(config); LOG.info("Successfully stored SCM signed certificate, case:{}.", response); break; case FAILURE: LOG.error("DN security initialization failed, case:{}.", response); throw new RuntimeException("DN security initialization failed."); case RECOVER: LOG.error("DN security initialization failed, case:{}. OM certificate " + "is missing.", response); throw new RuntimeException("DN security initialization failed."); default: LOG.error("DN security initialization failed. Init response: {}", response); throw new RuntimeException("DN security initialization failed."); } }
Source:
<code>public float getExposureCompensationStep() { <START> return 0.3f; <END> }</code><technical_language>do numbers from</technical_language>

Target:
public float getExposureCompensationStep() { return 0.5f; }
Source:
<code>public static void main(String[] args) { String masterHost = null; int masterPort = RemoteSlaveConnection.DEFAULT_PORT; int slaveIndex = -1; <START> for (int i = 0; i < args.length; i++) { <END> if (args[i].equals("-master")) { String param = args[i + 1]; if (param.contains(":")) { masterHost = param.substring(0, param.indexOf(":")); try { masterPort = Integer.parseInt(param.substring(param.indexOf(":") + 1)); } catch (NumberFormatException nfe) { log.warn("Unable to parse port part of the master! Failing!"); ShutDownHook.exit(10); } } else { masterHost = param; } } else if (args[i].equals("-slaveIndex")) { try { slaveIndex = Integer.parseInt(args[i + 1]); } catch (NumberFormatException nfe) { log.warn("Unable to parse slaveIndex! Failing!"); ShutDownHook.exit(10); } } else { PluginParamHolder.parseCommandLineArg(args[i]); } } if (masterHost == null) { printUsageAndExit(); } Slave slave = new Slave(new RemoteMasterConnection(masterHost, masterPort)); try { slave.run(slaveIndex); } catch (Exception e) { e.printStackTrace(); ShutDownHook.exit(10); } }</code><technical_language>changed for loop boundaries, add a check accessing args[i + 1] below</technical_language>

Target:
public static void main(String[] args) { ArgsHolder.init(args, ArgsHolder.ArgType.SLAVE); if (ArgsHolder.getMasterHost() == null) { printUsageAndExit(); } Slave slave = new Slave(new RemoteMasterConnection(ArgsHolder.getMasterHost(), ArgsHolder.getMasterPort())); try { slave.run(ArgsHolder.getSlaveIndex()); } catch (Exception e) { e.printStackTrace(); ShutDownHook.exit(10); } }
Source:
<code>private void initialize() { project = (IProject) getElement().getAdapter(IResource.class); if (project == null) { throw new RuntimeException( <START> TasksUiPlugin.ID_PLUGIN + ": Can not get the project of " + getElement().toString()); END> } noDefaultAndApplyButton(); setDescription( Messages.ProjectTaskRepositoryPage_Select_a_task_repository_to_associate_with_this_project_below); }</code><technical_language>explicit toString() is unnecessary. Simply "+ getElement()" accomplish same, work if getElement() return null</technical_language>

Target:
private void initialize() { project = (IProject) getElement().getAdapter(IResource.class); if (project == null) { throw new RuntimeException( TasksUiPlugin.ID_PLUGIN + ": Can not get the project of " + getElement()); } noDefaultAndApplyButton(); setDescription( Messages.ProjectTaskRepositoryPage_Select_a_task_repository_to_associate_with_this_project_below); }
Source:
<code>public void handle(String target, Request baseRequest, HttpServletRequest request, HttpServletResponse response) throws IOException { if (this.delegate != null) { ConservedHeadersFilter.extractHeaders(request) .forEach((header, value) -> { if (HEADER_BLACKLIST.copyToResponse(header.getHeaderName())) { <START> response.setHeader(header.getHeaderName(), value); <END> } }); this.delegate.handle(target, baseRequest, request, response); } }</code><technical_language>I realize logic in otj-server copies conserved headers responses, in otj-conservedheaders. consolidate this logic in place</technical_language>

Target:
public void handle(String target, Request baseRequest, HttpServletRequest request, HttpServletResponse response) throws IOException { if (this.delegate != null) { ConservedHeadersFilter.extractHeaders(request) .forEach((header, value) -> { if (HEADER_BLACKLIST.canCopyToResponse(header.getHeaderName())) { response.setHeader(header.getHeaderName(), value); } }); this.delegate.handle(target, baseRequest, request, response); } }
Source:
<code>public CloseableIterator<SAMRecord> queryAlignmentStart(final String sequence, final int start) { final int referenceIndex = this.mFileHeader.getSequenceIndex(sequence); if (referenceIndex == -1) { return new CloseableIterator<SAMRecord>() { @Override public void close() { } @Override public boolean hasNext() { return false; } @Override public SAMRecord next() { <START> return null; <END> } }; } else { final HtsgetRequest req = new HtsgetRequest(this.mSource).withInterval(new Interval(sequence, start, Integer.MAX_VALUE)); final CloseableIterator<SAMRecord> queryIterator = new HtsgetBAMFileIterator(req); final BAMStartingAtIteratorFilter filter = new BAMStartingAtIteratorFilter(this.mFileHeader.getSequenceIndex(sequence), start); final CloseableIterator<SAMRecord> filteredIterator = new BAMQueryFilteringIterator(queryIterator, filter); this.iterators.add(filteredIterator); return filteredIterator; } }</code><technical_language>match iterator contract this throw NoSuchElementException</technical_language>

Target:
public CloseableIterator<SAMRecord> queryAlignmentStart(final String sequence, final int start) { final int referenceIndex = this.mFileHeader.getSequenceIndex(sequence); if (referenceIndex == -1) { return new EmptyBamIterator(); } else { final HtsgetRequest req = new HtsgetRequest(this.mSource) .withFormat(HtsgetFormat.BAM) .withInterval(new Interval(sequence, start, start + 1)); final CloseableIterator<SAMRecord> iterator = new HtsgetBAMFileIterator(req); final BAMStartingAtIteratorFilter filter = new BAMStartingAtIteratorFilter(referenceIndex, start); final CloseableIterator<SAMRecord> filteringIterator = new BAMQueryFilteringIterator(iterator, filter); final CloseableIterator<SAMRecord> queryIterator = this.mUseAsynchronousIO ? new SAMRecordPrefetchingIterator(filteringIterator, READAHEAD_LIMIT) : filteringIterator; this.iterators.add(queryIterator); return queryIterator; } }
Source:
<code>public QueueAsyncClientBuilder endpoint(String endpoint) { Objects.requireNonNull(endpoint); try { URL fullURL = new URL(endpoint); this.endpoint = new URL(fullURL.getProtocol() + "://" + fullURL.getHost()); <START> String[] pathSegments = fullURL.getPath().split("/", 2); <END> if (pathSegments.length == 2 && !ImplUtils.isNullOrEmpty(pathSegments[1])) { this.queueName = pathSegments[1]; } SASTokenCredential credential = SASTokenCredential.fromQuery(fullURL.getQuery()); if (credential != null) { this.sasTokenCredential = credential; } } catch (MalformedURLException ex) { throw new IllegalArgumentException("The Azure Storage Queue endpoint url is malformed."); } return this; }</code><technical_language>Investigate if <LINK_0> simplify stuff here</technical_language>

Target:
public QueueAsyncClientBuilder endpoint(String endpoint) { Objects.requireNonNull(endpoint); try { UrlBuilder urlBuilder = UrlBuilder.parse(endpoint); URL fullURL = new URL(endpoint); this.endpoint = new URL(fullURL.getProtocol() + "://" + fullURL.getHost()); String[] pathSegments = fullURL.getPath().split("/", 2); if (pathSegments.length == 2 && !ImplUtils.isNullOrEmpty(pathSegments[1])) { this.queueName = pathSegments[1]; } SASTokenCredential credential = SASTokenCredential.fromQuery(fullURL.getQuery()); if (credential != null) { this.sasTokenCredential = credential; } } catch (MalformedURLException ex) { throw new IllegalArgumentException("The Azure Storage Queue endpoint url is malformed."); } return this; }
Source:
<code>public void testCompareToMessageWithGreaterLineNumber() { ViolationMessage messageWithGreaterLineNumber = <START> new ViolationMessage(null, "/usr/bin/local", 12, 5, Severity.ERROR, "errMsg"); <END> int ret = this.violationMessage.compareTo(messageWithGreaterLineNumber); assertThat(ret, lessThan(0)); }</code><technical_language>I change of new ViolationMessage() calls in this file pass some rule instead of null</technical_language>

Target:
public void testCompareToMessageWithGreaterLineNumber() { ViolationMessage messageWithGreaterLineNumber = new ViolationMessage(Rules.LOWER_CAMEL_CASE, "/usr/bin/local", 12, 5, Severity.ERROR, "errMsg"); int ret = this.violationMessage.compareTo(messageWithGreaterLineNumber); assertThat(ret, lessThan(0)); }
Source:
<code>public static void setup() { WorkbenchPreferenceDialog preferenceDialog = new WorkbenchPreferenceDialog(); preferenceDialog.open(); MavenSettingsPreferencePage prefPage = new MavenSettingsPreferencePage(preferenceDialog); preferenceDialog.select(prefPage); <START> preferenceDialog.ok(); <END> new DefaultToolItem(new WorkbenchShell(), CENTRAL_LABEL).click(); new DefaultEditor(CENTRAL_LABEL); }</code><technical_language>@zcervink lines 60-64 related setting settings.xml in preferences, this code is useless, please remove it</technical_language>

Target:
public static void setup() { new DefaultToolItem(new WorkbenchShell(), CENTRAL_LABEL).click(); new DefaultEditor(CENTRAL_LABEL); }
Source:
<code>public ModelAndView editBulkSamples(@PathVariable String sampleIds, ModelMap model) throws IOException { try { List<Long> idList = getIdsFromString(sampleIds); ObjectMapper mapper = new ObjectMapper(); <START> JSONArray samplesDtos = new JSONArray(); <END> for (Sample sample : requestManager.getSamplesByIdList(idList)) { samplesDtos.add(Dtos.asDto(sample)); } model.put("title", "Bulk Edit Samples"); model.put("samplesJSON", samplesDtos); model.put("method", "Edit"); return new ModelAndView("/pages/bulkEditSamples.jsp", model); } catch (IOException ex) { if (log.isDebugEnabled()) { log.error("Failed to get bulk samples", ex); } throw ex; } }</code><technical_language>do this other sampleDtos a List<SampleDto> Jackson mapper</technical_language>

Target:
public ModelAndView editBulkSamples(@PathVariable String sampleIds, ModelMap model) throws IOException { try { List<Long> idList = getIdsFromString(sampleIds); ObjectMapper mapper = new ObjectMapper(); List<SampleDto> samplesDtos = new ArrayList<SampleDto>(); for (Sample sample : requestManager.getSamplesByIdList(idList)) { samplesDtos.add(Dtos.asDto(sample)); } model.put("title", "Bulk Edit Samples"); model.put("samplesJSON", mapper.writeValueAsString(samplesDtos)); model.put("method", "Edit"); return new ModelAndView("/pages/bulkEditSamples.jsp", model); } catch (IOException ex) { if (log.isDebugEnabled()) { log.error("Failed to get bulk samples", ex); } throw ex; } }
Source:
<code>public void addCloudConfig(String clusterName, CloudConfig cloudConfig) { <START> logger.info("Add CloudConfig to cluster {}.", clusterName); <END> if (!ZKUtil.isClusterSetup(clusterName, _zkClient)) { throw new HelixException("cluster " + clusterName + " is not setup yet"); } CloudConfig.Builder builder = new CloudConfig.Builder(cloudConfig); builder.setClusterName(clusterName); CloudConfig cloudConfigBuilder = builder.build(); ZKHelixDataAccessor accessor = new ZKHelixDataAccessor(clusterName, new ZkBaseDataAccessor<ZNRecord>(_zkClient)); Builder keyBuilder = accessor.keyBuilder(); accessor.setProperty(keyBuilder.cloudConfig(), cloudConfigBuilder); }</code><technical_language>Suggest log cloud config info</technical_language>

Target:
public void addCloudConfig(String clusterName, CloudConfig cloudConfig) { logger.info("Add CloudConfig to cluster {}, CloudConfig is {}.", clusterName, cloudConfig.toString()); if (!ZKUtil.isClusterSetup(clusterName, _zkClient)) { throw new HelixException("cluster " + clusterName + " is not setup yet"); } CloudConfig.Builder builder = new CloudConfig.Builder(cloudConfig); CloudConfig cloudConfigBuilder = builder.build(); ZKHelixDataAccessor accessor = new ZKHelixDataAccessor(clusterName, new ZkBaseDataAccessor<ZNRecord>(_zkClient)); Builder keyBuilder = accessor.keyBuilder(); accessor.setProperty(keyBuilder.cloudConfig(), cloudConfigBuilder); }
Source:
<code>public boolean isContentLoaded() { wait.forElementVisible(iframe); driver.switchTo().frame(iframe); try { wait.forElementVisible(contentInput); <START> driver.switchTo().defaultContent(); <END> return true; } catch (NoSuchElementException e) { PageObjectLogging.log("isContentLoaded", "RTE editor loaded", false, driver); driver.switchTo().defaultContent(); return false; } }</code><technical_language>finally</technical_language>

Target:
public boolean isContentLoaded() { wait.forElementVisible(iframe); driver.switchTo().frame(iframe); try { wait.forElementVisible(contentInput); return true; } catch (TimeoutException e) { PageObjectLogging.log("isContentLoaded", "RTE editor loaded", false, driver); return false; } finally { driver.switchTo().defaultContent(); } }
Source:
<code>void setValue(String param, String value) { if ("IPADDRESS".equalsIgnoreCase(param)) { m_ipAddress = value; } else if ("HOSTNAME".equalsIgnoreCase(param)) { m_hostName = value; <START> } else if ("CLIENTPORT".equalsIgnoreCase(param)) { <END> m_clientPort = Integer.parseInt(value); } }</code><technical_language>work for ADMIN client port? If a client connects a node admin port, this client try connect other nodes admin port. Right</technical_language>

Target:
void setValue(String param, String value) { if ("IPADDRESS".equalsIgnoreCase(param)) { m_ipAddress = value; } else if ("HOSTNAME".equalsIgnoreCase(param)) { m_hostName = value; } else if ("CLIENTPORT".equalsIgnoreCase(param)) { m_clientPort = Integer.parseInt(value); } else if ("ADMINPORT".equalsIgnoreCase(param)) { m_adminPort = Integer.parseInt(value); } }
Source:
<code>public void testValidateDataNotes() { this.transactionDto.setAmount(TEN); this.transactionDto.setTransactionType(DEPOSIT); <START> this.transactionDto.setNotes( <END> "notesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotesnotes"); this.transactionInputValidator.validate(this.transactionDto); }</code><technical_language>@vmaletta RandomStringUtils.randomAlphanumeric</technical_language>

Target:
public void testValidateDataNotes() { this.transactionDto.setAmount(TEN); this.transactionDto.setTransactionType(DEPOSIT); this.transactionDto.setNotes(RandomStringUtils.randomAlphabetic(256)); this.transactionInputValidator.validate(this.transactionDto); }
Source:
<code>private void waitForJoinerView(Address joiner, int joinerViewId, long timeout) throws InterruptedException { if (joinerViewId > viewId || clusterManagerStatus == ClusterManagerStatus.RECOVERING_CLUSTER) { if (trace) { if (joinerViewId > viewId) { <START> log.tracef("Waiting to install view %s before processing join request from %s", joiner, <END> joinerViewId); } else { log.tracef("Waiting to recover cluster status before processing join request from %s", joiner); } } long endTime = timeService.expectedEndTime(timeout, TimeUnit.MILLISECONDS); synchronized (clusterManagerLock) { while (viewId < joinerViewId || clusterManagerStatus == ClusterManagerStatus.RECOVERING_CLUSTER) { if (timeService.isTimeExpired(endTime) || !clusterManagerStatus.isRunning()) break; clusterManagerLock.wait(timeService.remainingTime(endTime, TimeUnit.MILLISECONDS)); } } if (timeService.isTimeExpired(endTime)) { throw new TimeoutException("Timed out waiting for view " + joinerViewId); } } }</code><technical_language>tracef parameters in wrong order (first if view, joiner address)</technical_language>

Target:
private void waitForJoinerView(Address joiner, int joinerViewId, long timeout) throws InterruptedException { if (joinerViewId > viewId || clusterManagerStatus == ClusterManagerStatus.RECOVERING_CLUSTER) { if (trace) { if (joinerViewId > viewId) { log.tracef("Waiting to install view %s before processing join request from %s", joinerViewId, joiner); } else { log.tracef("Waiting to recover cluster status before processing join request from %s", joiner); } } long endTime = timeService.expectedEndTime(timeout, TimeUnit.MILLISECONDS); synchronized (clusterManagerLock) { while (viewId < joinerViewId || clusterManagerStatus == ClusterManagerStatus.RECOVERING_CLUSTER) { if (timeService.isTimeExpired(endTime) || !clusterManagerStatus.isRunning()) break; clusterManagerLock.wait(timeService.remainingTime(endTime, TimeUnit.MILLISECONDS)); } } if (timeService.isTimeExpired(endTime)) { throw new TimeoutException("Timed out waiting for view " + joinerViewId); } } }
Source:
<code>public void test02_01_useCaseControllerHandling() { driver.findElement(By.id("useCaseControllerHandling_02_01")).click(); long retryTimes = 1L; assertThat(driver.getTitle(), is("terasoluna-gfw-functionaltest")); assertThat(dbLogProvider.countContainsMessageAndLevelsAndLogger( "\\[e.xx.9999\\] 2_1 Continue", "INFO", "org.terasoluna.gfw.common.exception.ExceptionLogger"), is(1L)); assertThat(dbLogProvider.countContainsMessageAndLevelsAndLogger( "\\[e.xx.9999\\] 2_1 Continue", "INFO", "org.terasoluna.gfw.common.exception.ExceptionLogger.Monitoring"), is(retryTimes)); assertThat(dbLogProvider.countContainsByRegexExceptionMessage(null, null, "2_1 Continue*", "..*ContinueException..*"), is( <START> retryTimes)); <END> }</code><technical_language>[x] Unnecessary changes. (HtmlUnit unique.)</technical_language>

Target:
public void test02_01_useCaseControllerHandling() { driver.findElement(By.id("useCaseControllerHandling_02_01")).click(); assertThat(driver.getTitle(), is("terasoluna-gfw-functionaltest")); assertThat(dbLogProvider.countContainsMessageAndLevelsAndLogger( "\\[e.xx.9999\\] 2_1 Continue", "INFO", "org.terasoluna.gfw.common.exception.ExceptionLogger"), is(1L)); assertThat(dbLogProvider.countContainsMessageAndLevelsAndLogger( "\\[e.xx.9999\\] 2_1 Continue", "INFO", "org.terasoluna.gfw.common.exception.ExceptionLogger.Monitoring"), is(1L)); assertThat(dbLogProvider.countContainsByRegexExceptionMessage(null, null, "2_1 Continue*", "..*ContinueException..*"), is(1L)); }
Source:
<code>public void testConfigurableStreamFileThreshold() throws Exception { byte[] data = getRng().nextBytes(300); RevBlob id = tr.blob(data); tr.branch("master").commit().add("A", id).create(); tr.packAndPrune(); assertTrue("has blob", wc.has(id)); ObjectLoader ol = wc.open(id); ObjectStream in = ol.openStream(); <START> assert(in <END> instanceof ObjectStream.SmallStream); assertEquals(300, in.available()); in.close(); wc.setStreamFileThreshold(299); ol = wc.open(id); in = ol.openStream(); assert(in instanceof ObjectStream.Filter); assertEquals(1, in.available()); }</code><technical_language>this assertTrue</technical_language>

Target:
public void testConfigurableStreamFileThreshold() throws Exception { byte[] data = getRng().nextBytes(300); RevBlob id = tr.blob(data); tr.branch("master").commit().add("A", id).create(); tr.packAndPrune(); assertTrue("has blob", wc.has(id)); ObjectLoader ol = wc.open(id); ObjectStream in = ol.openStream(); assertTrue(in instanceof ObjectStream.SmallStream); assertEquals(300, in.available()); in.close(); wc.setStreamFileThreshold(299); ol = wc.open(id); in = ol.openStream(); assertTrue(in instanceof ObjectStream.Filter); assertEquals(1, in.available()); }
Source:
<code>public void apply( WorkspaceConfigImpl workspaceConfig, Component editorComponent, FileContentProvider contentProvider) { checkArgument(workspaceConfig != null, "Workspace config must not be null"); <START> checkArgument(editorComponent != null, "Tool must not be null"); <END> checkArgument( EDITOR_COMPONENT_TYPE.equals(editorComponent.getType()), format("Plugin must have `%s` type", EDITOR_COMPONENT_TYPE)); String editorComponentName = editorComponent.getName(); String editorId = editorComponent.getId(); workspaceConfig.getAttributes().put(WORKSPACE_TOOLING_EDITOR_ATTRIBUTE, editorId); workspaceConfig .getAttributes() .put(EDITOR_COMPONENT_ALIAS_WORKSPACE_ATTRIBUTE, editorComponentName); String editorIdVersion = resolveIdAndVersion(editorComponent.getId()); workspaceConfig .getCommands() .stream() .filter( c -> c.getAttributes().get(COMPONENT_NAME_COMMAND_ATTRIBUTE).equals(editorComponentName)) .forEach(c -> c.getAttributes().put(PLUGIN_ATTRIBUTE, editorIdVersion)); }</code><technical_language>Component null</technical_language>

Target:
public void apply( WorkspaceConfigImpl workspaceConfig, Component editorComponent, FileContentProvider contentProvider) { checkArgument(workspaceConfig != null, "Workspace config must not be null"); checkArgument(editorComponent != null, "Component must not be null"); checkArgument( EDITOR_COMPONENT_TYPE.equals(editorComponent.getType()), format("Plugin must have `%s` type", EDITOR_COMPONENT_TYPE)); String editorComponentName = editorComponent.getName(); String editorId = editorComponent.getId(); workspaceConfig.getAttributes().put(WORKSPACE_TOOLING_EDITOR_ATTRIBUTE, editorId); workspaceConfig .getAttributes() .put(EDITOR_COMPONENT_ALIAS_WORKSPACE_ATTRIBUTE, editorComponentName); String editorIdVersion = resolveIdAndVersion(editorComponent.getId()); workspaceConfig .getCommands() .stream() .filter( c -> c.getAttributes().get(COMPONENT_NAME_COMMAND_ATTRIBUTE).equals(editorComponentName)) .forEach(c -> c.getAttributes().put(PLUGIN_ATTRIBUTE, editorIdVersion)); }
Source:
<code>public String setEncryptionKeyExchangeOnGMeter(final DlmsConnectionHolder conn, final DlmsDevice device, final GMeterInfoDto gMeterInfo) throws ProtocolAdapterException, FunctionalException { LOGGER.info("Device for Set Encryption Key Exchange On G-Meter is: {}", device); DlmsDevice gMeterDevice; gMeterDevice = this.domainHelperService.findDlmsDevice(gMeterInfo.getDeviceIdentification()); final ProtocolMeterInfo protocolMeterInfo = new ProtocolMeterInfo(gMeterInfo.getChannel(), gMeterInfo.getDeviceIdentification(), <START> this.getSecurityKey(gMeterDevice, SecurityKeyType.G_METER_ENCRYPTION), <END> gMeterDevice.getValidSecurityKey(SecurityKeyType.G_METER_MASTER).getKey()); this.setEncryptionKeyExchangeOnGMeterCommandExecutor.execute(conn, device, protocolMeterInfo); return "Set Encryption Key Exchange On G-Meter Result is OK for device id: " + device.getDeviceIdentification(); }</code><technical_language>I a cleaner option bring code more inline similar generate replace key functionality DLMS server authentication global unicast encryption keys (see, poorly named generateAndEncrypt method flow call). keys retrieved database configuration service code for instance, keeping dealing stored security keys close code possible</technical_language>

Target:
public String setEncryptionKeyExchangeOnGMeter(final DlmsConnectionHolder conn, final DlmsDevice device, final GMeterInfoDto gMeterInfo) throws ProtocolAdapterException, FunctionalException { LOGGER.info("Device for Set Encryption Key Exchange On G-Meter is: {}", device); DlmsDevice gMeterDevice; gMeterDevice = this.domainHelperService.findDlmsDevice(gMeterInfo.getDeviceIdentification()); final ProtocolMeterInfo protocolMeterInfo = new ProtocolMeterInfo(gMeterInfo.getChannel(), gMeterInfo.getDeviceIdentification(), this.keyHelperService.getSecurityKey(gMeterDevice, SecurityKeyType.G_METER_ENCRYPTION), gMeterDevice.getValidSecurityKey(SecurityKeyType.G_METER_MASTER).getKey()); this.setEncryptionKeyExchangeOnGMeterCommandExecutor.execute(conn, device, protocolMeterInfo); return "Set Encryption Key Exchange On G-Meter Result is OK for device id: " + device.getDeviceIdentification(); }
Source:
<code>public static boolean isTransactionFile(String file) throws IOException { try { <START> RandomAccessFileReader reader = new RandomAccessFileReader(new File(file)); <END> BinaryInputArchive logStream = new BinaryInputArchive(reader); FileHeader fhdr = new FileHeader(); fhdr.deserialize(logStream, "fileheader"); } finally { reader.close(); } return fhdr.getMagic() == FileTxnLog.TXNLOG_MAGIC; }</code><technical_language>Mixture of spaces tab this line others. Do enforce sort of consistency formatting in this project? I both</technical_language>

Target:
public static boolean isTransactionFile(String file) throws IOException { RandomAccessFileReader reader = new RandomAccessFileReader(new File(file)); try { BinaryInputArchive logStream = new BinaryInputArchive(reader); FileHeader fhdr = new FileHeader(); fhdr.deserialize(logStream, "fileheader"); } finally { reader.close(); } return fhdr.getMagic() == FileTxnLog.TXNLOG_MAGIC; }
Source:
<code>private static Map<String, Long> buildIoTune(Guid diskProfileId, QosCache cache) { if (diskProfileId == null) { return null; } Guid storageQosId = <START> cache.diskProfileStorageQosMap.get(diskProfileId); <END> if (storageQosId == null) { StorageQos qos = DbFacade.getInstance().getStorageQosDao().getQosByDiskProfileId(diskProfileId); if (qos == null) { return null; } storageQosId = qos.getId(); cache.diskProfileStorageQosMap.put(diskProfileId, storageQosId); cache.storageQosIoTuneMap.put(storageQosId, IoTuneUtils.ioTuneMapFrom(qos)); } return cache.storageQosIoTuneMap.get(storageQosId); }</code><technical_language>If call "buildIoTune" cache=null fail. previous suggestion</technical_language>

Target:
private static Map<String, Long> buildIoTune(Guid diskProfileId, QosCache cache) { if (diskProfileId == null) { return null; } if (cache == null) { cache = new QosCache(); } Guid storageQosId = cache.diskProfileStorageQosMap.get(diskProfileId); if (storageQosId == null) { StorageQos qos = DbFacade.getInstance().getStorageQosDao().getQosByDiskProfileId(diskProfileId); if (qos == null) { return null; } storageQosId = qos.getId(); cache.diskProfileStorageQosMap.put(diskProfileId, storageQosId); cache.storageQosIoTuneMap.put(storageQosId, IoTuneUtils.ioTuneMapFrom(qos)); } return cache.storageQosIoTuneMap.get(storageQosId); }
Source:
<code>final public void interrupted() { onInterrupted(); <START> if (previous != null && (!previous.isScheduled())) { <END> previous.schedule(); } }</code><technical_language>unnecessary parentheses</technical_language>

Target:
public final void interrupted() { onInterrupted(); if (previous != null && !previous.isScheduled()) { previous.schedule(); } }
Source:
<code>public void readIsDeveloper() { mDeveloper = BuildConfig.DEBUG || PreferenceManager.getDefaultSharedPreferences(getApplicationContext()) .getInt(CLICK_DEV_MENU, 0) > CLICKS_NEEDED_TO_BE_DEVELOPER; if (isDeveloper()) { String dataFolder = getDataFolder(); Log_OC.setLogDataFolder(dataFolder); Log_OC.startLogging(Environment.getExternalStorageDirectory().getAbsolutePath()); <START> Log_OC.d(BuildConfig.BUILD_TYPE, "start logging " + BuildConfig.VERSION_NAME + " " + BuildConfig.COMMIT_SHA1); <END> } }</code><technical_language>Line long</technical_language>

Target:
public void readIsDeveloper() { mDeveloper = BuildConfig.DEBUG || PreferenceManager.getDefaultSharedPreferences(getApplicationContext()) .getInt(CLICK_DEV_MENU, DEFAULT_0) > CLICKS_NEEDED_TO_BE_DEVELOPER; if (isDeveloper()) { String dataFolder = getDataFolder(); Log_OC.setLogDataFolder(dataFolder); Log_OC.startLogging(Environment.getExternalStorageDirectory().getAbsolutePath()); Log_OC.d(BuildConfig.BUILD_TYPE, "start logging " + BuildConfig.VERSION_NAME + " " + BuildConfig.COMMIT_SHA1); } }
Source:
<code>public static GetBlobOptions buildGetBlobOptions(Map<String, Object> args, SubResource subResource) <START> throws RestServiceException { <END> if (subResource == null) { return new GetBlobOptions(GetBlobOptions.OperationType.All, RestUtils.buildByteRange(args)); } if (args.containsKey(RestUtils.Headers.RANGE)) { throw new RestServiceException("Range requests not supported for subresources.", RestServiceErrorCode.InvalidArgs); } return new GetBlobOptions(GetBlobOptions.OperationType.BlobInfo, null); }</code><technical_language>Do this method cleaned a bit? check for RANGE header is done in buildByteRange(). I more if check for RANGE header is done buildByteRange() is called if passes</technical_language>

Target:
public static GetBlobOptions buildGetBlobOptions(Map<String, Object> args, SubResource subResource) throws RestServiceException { String rangeHeaderValue = getHeader(args, Headers.RANGE, false); if (subResource != null && rangeHeaderValue != null) { throw new RestServiceException("Ranges not supported for sub-resources.", RestServiceErrorCode.InvalidArgs); } return new GetBlobOptions( subResource == null ? GetBlobOptions.OperationType.All : GetBlobOptions.OperationType.BlobInfo, rangeHeaderValue != null ? RestUtils.buildByteRange(rangeHeaderValue) : null); }
Source:
<code>public void update(Host host, Statement statement, Exception exception, long newLatencyNanos) { if (exception == null) { return; } if (!errorsFilter.shouldAccountForError(exception, host)) { return; } if (!hostsMeters.containsKey(host)) { hostsMeters.putIfAbsent(host, new Meter()); } Meter hostMeter = hostsMeters.get(host); hostMeter.mark(); if (hostMeter.getOneMinuteRate() >= errorsThreshold) { logger.debug(String.format("Host : %s has reached the maximum authorized number of errors. Its error rate is : %.3f. " + "It will now be ignored by the ErrorAware load balancing policy during a period of %d milliseconds.", host, hostMeter.getOneMinuteRate(), retryPeriodMillis)); <START> exclusions.putIfAbsent(host, System.currentTimeMillis()); <END> } }</code><technical_language>If host excluded in past, map contains timestamp this call nothing. I clear entry find out timestamp is stale in newQueryPlan</technical_language>

Target:
public void update(Host host, Statement statement, Exception exception, long newLatencyNanos) { if (exception == null) { return; } if (!errorFilter.shouldConsiderError(exception, host, statement)) { return; } RollingCount hostCount = getOrCreateCount(host); hostCount.increment(); }
Source:
<code>protected void setupModuleManager(Set<Name> moduleNames) throws Exception { ModuleManager moduleManager = ModuleManagerFactory.create(); ModuleRegistry registry = moduleManager.getRegistry(); Set<Module> mods = Sets.newHashSet(); for (Name modName : moduleNames) { <START> Module module = registry.getLatestModuleVersion(modName); <END> mods.add(module); } ModuleEnvironment modEnv = moduleManager.loadEnvironment(mods, true); Log.debug("Loaded modules: " + modEnv.getModuleIdsOrderedByDependencies()); CoreRegistry.put(ModuleManager.class, moduleManager); }</code><technical_language>I recommend throwing list dependency resolver instead. This allow user full dependency chain</technical_language>

Target:
protected void setupModuleManager(Set<Name> moduleNames) throws Exception { ModuleManager moduleManager = ModuleManagerFactory.create(); ModuleRegistry registry = moduleManager.getRegistry(); DependencyResolver resolver = new DependencyResolver(registry); ResolutionResult result = resolver.resolve(moduleNames); if (result.isSuccess()) { ModuleEnvironment modEnv = moduleManager.loadEnvironment(result.getModules(), true); Log.debug("Loaded modules: " + modEnv.getModuleIdsOrderedByDependencies()); } else { Log.error("Could not resolve module dependencies for " + moduleNames); } CoreRegistry.put(ModuleManager.class, moduleManager); }
Source:
<code>public List<ToolVersionV2> getVersionsV2() { List<ToolVersion> toolVersions = this.tool.getVersions(); <START> List<ToolVersionV2> toolVersionV2s = new ArrayList<>(); <END> toolVersions.forEach(toolVersion -> { ToolVersionV2 toolVersionV2 = new ToolVersionV2(); toolVersionV2.setToolVersion(toolVersion); toolVersionV2s.add(toolVersionV2); }); return toolVersionV2s; }</code><technical_language>This V1 version for Class of version. I a template example <LINK_0></technical_language>

Target:
public List<ToolVersionV2> getVersionsV2() { List<ToolVersion> toolVersions = this.tool.getVersions(); return toolVersions.stream().map(toolVersion -> new ToolVersionV2(toolVersion)).collect(Collectors.toList()); }
Source:
<code>private List<QueryTrace> queryTracesFromRetries( final List<RetryException> retryExceptions, final List<Long> backoffTimings ) { final List<QueryTrace> traces = new ArrayList<>(); long lastTS = 0; final Iterator<Long> backoffIterator = backoffTimings.iterator(); for (final RetryException re : retryExceptions) { final long microTS = TimeUnit.MICROSECONDS.convert(re.getOffsetMillis(), TimeUnit.MILLISECONDS); long elapsed = microTS - lastTS; traces.add(QueryTrace.of(new QueryTrace.Identifier(getMessageFrom(re)), elapsed)); lastTS = microTS; if (!backoffIterator.hasNext()) { continue; } final long backoffTS = TimeUnit.MICROSECONDS.convert(backoffIterator.next(), TimeUnit.MILLISECONDS); elapsed = backoffTS - lastTS; <START> traces.add(QueryTrace.of(new QueryTrace.Identifier("retry-backoff"), elapsed)); <END> lastTS = backoffTS; } return traces; }</code><technical_language>This identifier a constant</technical_language>

Target:
private List<QueryTrace> queryTracesFromRetries( final List<RetryException> retryExceptions, final List<Long> backoffTimings ) { final List<QueryTrace> traces = new ArrayList<>(); long lastTS = 0; final Iterator<Long> backoffIterator = backoffTimings.iterator(); for (final RetryException re : retryExceptions) { final long microTS = TimeUnit.MICROSECONDS.convert(re.getOffsetMillis(), TimeUnit.MILLISECONDS); long elapsed = microTS - lastTS; traces.add(QueryTrace.of(new QueryTrace.Identifier(getMessageFrom(re)), elapsed)); lastTS = microTS; if (!backoffIterator.hasNext()) { continue; } final long backoffTS = TimeUnit.MICROSECONDS.convert(backoffIterator.next(), TimeUnit.MILLISECONDS); elapsed = backoffTS - lastTS; traces.add(QueryTrace.of(RETRY_BACKOFF, elapsed)); lastTS = backoffTS; } return traces; }
Source:
<code>protected <C extends FlagAffectedCommand & TopologyAffectedCommand> CompletableFuture<Void> remoteGetAll( InvocationContext ctx, C command, Collection<?> keys, RemoteGetAllHandler remoteGetAllHandler) { Map<Address, List<Object>> requestedKeys = getKeysByOwner(ctx, keys, checkTopologyId(command), null, null); if (requestedKeys.isEmpty()) { return CompletableFutures.completedNull(); } GlobalTransaction gtx = ctx.isInTxScope() ? ((TxInvocationContext) ctx).getGlobalTransaction() : null; ClusteredGetAllFuture allFuture = new ClusteredGetAllFuture(requestedKeys.size()); for (Map.Entry<Address, List<Object>> pair : requestedKeys.entrySet()) { ClusteredGetAllCommand clusteredGetAllCommand = cf.buildClusteredGetAllCommand(pair.getValue(), command.getFlagsBitSet(), gtx); clusteredGetAllCommand.setTopologyId(command.getTopologyId()); Address target = pair.getKey(); <START> rpcManager.invokeCommand(target, clusteredGetAllCommand, new MapResponseCollector(true, 1), <END> rpcManager.getSyncRpcOptions()) .whenComplete(new ClusteredGetAllHandler(target, allFuture, ctx, command, pair.getValue(), null, remoteGetAllHandler)); } return allFuture; }</code><technical_language>SingleResponseCollector</technical_language>

Target:
protected <C extends FlagAffectedCommand & TopologyAffectedCommand> CompletableFuture<Void> remoteGetAll( InvocationContext ctx, C command, Collection<?> keys, RemoteGetAllHandler remoteGetAllHandler) { Map<Address, List<Object>> requestedKeys = getKeysByOwner(ctx, keys, checkTopologyId(command), null, null); if (requestedKeys.isEmpty()) { return CompletableFutures.completedNull(); } GlobalTransaction gtx = ctx.isInTxScope() ? ((TxInvocationContext) ctx).getGlobalTransaction() : null; ClusteredGetAllFuture allFuture = new ClusteredGetAllFuture(requestedKeys.size()); for (Map.Entry<Address, List<Object>> pair : requestedKeys.entrySet()) { ClusteredGetAllCommand clusteredGetAllCommand = cf.buildClusteredGetAllCommand(pair.getValue(), command.getFlagsBitSet(), gtx); clusteredGetAllCommand.setTopologyId(command.getTopologyId()); Address target = pair.getKey(); rpcManager.invokeCommand(target, clusteredGetAllCommand, SingletonMapResponseCollector.ignoreLeavers(), rpcManager.getSyncRpcOptions()) .whenComplete(new ClusteredGetAllHandler(target, allFuture, ctx, command, pair.getValue(), null, remoteGetAllHandler)); } return allFuture; }
Source:
<code>protected Optional<IMetadataTable> getMetadata() { final IElement elem = getElement(); if (elem == null || !(elem instanceof Node)) { return Optional.empty(); } <START> final List<? extends IConnection> connections = NodeUtil.getIncomingConnections(((Node) elem), getContext()); <END> if (connections == null || connections.isEmpty()) { return Optional.empty(); } final IConnection connection = connections.get(0); return Optional.ofNullable(connection.getMetadataTable()); }</code><technical_language>remove unnecessary brackets leave NodeUtil.getIncomingConnections((Node) elem, getContext()); a bit easier read code get lost brackets</technical_language>

Target:
protected Optional<IMetadataTable> getMetadata() { final IElement elem = getElement(); if (elem == null || !(elem instanceof Node)) { return Optional.empty(); } final List<? extends IConnection> connections = NodeUtil.getIncomingConnections((Node) elem, getContext()); if (connections == null || connections.isEmpty()) { return Optional.empty(); } final IConnection connection = connections.get(0); return Optional.ofNullable(connection.getMetadataTable()); }
Source:
<code>public void testNewModule() { Exception exception = null; try (IAnalysisModule module = fModule.newModule(TmfTestTrace.A_TEST_10K.getTrace());) { assertNull(exception); assertNotNull(module); assertTrue(module instanceof TestAnalysis); } catch (TmfAnalysisException e) { exception = e; } try (IAnalysisModule module = fModuleOther.newModule(TmfTestTrace.A_TEST_10K.getTrace());) { } catch (TmfAnalysisException e) { exception = e; } assertNotNull(exception); assertEquals(NLS.bind(Messages.TmfAnalysisModuleHelper_AnalysisDoesNotApply, fModuleOther.getName()), exception.getMessage()); exception = null; try (IAnalysisModule module = fModuleOther.newModule(fTrace);) { assertNull(exception); assertNotNull(module); <START> assertTrue(module instanceof TestAnalysis2); <END> } catch (TmfAnalysisException e) { exception = e; } }</code><technical_language>If is exception, this code run, test pass while fail. assertNull(exception) moved try</technical_language>

Target:
public void testNewModule() { Exception exception = null; try (IAnalysisModule module = fModule.newModule(TmfTestTrace.A_TEST_10K.getTrace());) { assertNotNull(module); assertTrue(module instanceof TestAnalysis); } catch (TmfAnalysisException e) { exception = e; } assertNull(exception); try (IAnalysisModule module = fModuleOther.newModule(TmfTestTrace.A_TEST_10K.getTrace());) { } catch (TmfAnalysisException e) { exception = e; } assertNotNull(exception); assertEquals(NLS.bind(Messages.TmfAnalysisModuleHelper_AnalysisDoesNotApply, fModuleOther.getName()), exception.getMessage()); exception = null; try (IAnalysisModule module = fModuleOther.newModule(fTrace);) { assertNotNull(module); assertTrue(module instanceof TestAnalysis2); } catch (TmfAnalysisException e) { exception = e; } assertNull(exception); }
Source:
<code>public synchronized void receive(ProcessEngineEventEntity event) { <START> System.out.println(">>>> Recieved Event" + event); <END> System.out.println(">>>> \t Event Meta Data: " + event.getTimestamp() + " > " + event.getEventType()); System.out.println(">>>> \t Event Class: " + event.getClass().getCanonicalName()); eventsRepository.save(event); }</code><technical_language>@erdemedeiros remove system outs</technical_language>

Target:
public synchronized void receive(ProcessEngineEventEntity event) { eventsRepository.save(event); }
Source:
<code>public OpenshiftUIModel(ConnectionsRegistry registry) { listener = new IConnectionsRegistryListener() { @Override public void connectionRemoved(IConnection connection) { synchronized (connections) { connections.remove(connection); } fireChanged(); } @Override public void connectionChanged(IConnection c, String property, Object oldValue, Object newValue) { ConnectionWrapper connection = connections.get(c); if (connection == null) { return; } connection.connectionChanged(property, oldValue, newValue); } @Override public void connectionAdded(IConnection connection) { if (!(connection instanceof IOpenShiftConnection)) { return; } synchronized (connections) { if (connections.containsKey(connection)) { return; } connections.put((IOpenShiftConnection) connection, new ConnectionWrapper(OpenshiftUIModel.this, (IOpenShiftConnection) connection)); } fireChanged(); } }; <START> Collection<IConnection> allConnections = registry.getAll(); <END> synchronized (connections) { for (IConnection connection : allConnections) { if (connection instanceof IOpenShiftConnection) { connections.put((IOpenShiftConnection) connection, new ConnectionWrapper(OpenshiftUIModel.this, (IOpenShiftConnection) connection)); } } } registry.addListener(listener); }</code><technical_language>registry a getAll(Class) mitigate need run sync block</technical_language>

Target:
public OpenshiftUIModel(ConnectionsRegistry registry) { super(null, registry); listener = new IConnectionsRegistryListener() { @Override public void connectionRemoved(IConnection connection) { synchronized (connections) { connections.remove(connection); } fireChanged(OpenshiftUIModel.this); } @Override public void connectionChanged(IConnection c, String property, Object oldValue, Object newValue) { ConnectionWrapper connection = connections.get(c); if (connection == null) { return; } connection.connectionChanged(property, oldValue, newValue); } @Override public void connectionAdded(IConnection connection) { if (!(connection instanceof IOpenShiftConnection)) { return; } synchronized (connections) { if (connections.containsKey(connection)) { return; } connections.put((IOpenShiftConnection) connection, new ConnectionWrapper(OpenshiftUIModel.this, (IOpenShiftConnection) connection)); } fireChanged(OpenshiftUIModel.this); } }; Collection<IConnection> allConnections = registry.getAll(); synchronized (connections) { for (IConnection connection : allConnections) { if (connection instanceof IOpenShiftConnection) { connections.put((IOpenShiftConnection) connection, new ConnectionWrapper(OpenshiftUIModel.this, (IOpenShiftConnection) connection)); } } } registry.addListener(listener); }
Source:
<code>public String getId() { <START> return id; <END> }</code><technical_language>Add "this" qualifier</technical_language>

Target:
public String getId() { return this.id; }
Source:
<code>public void should_create_error_message_for_different_instances() { String errorMessage = shouldContainSame(Optional.of(new Integer(10)), new Integer(10)).create(); <START> assertThat(errorMessage).isEqualTo("\nExpecting:\n" + <END> " <Optional[10]>\n" + "to contain the instance (i.e. compared with ==):\n" + " <10>\n" + "but did not."); }</code><technical_language>though, is done done for other assertion,we String.format %n proper line separator running tests OS, like: java assertThat(message).isEqualTo(format("[Test] %n" + "Expecting:%n" + " <[\"Yoda\"]>%n" + "to contain:%n" + " <[\"Luke\", \"Yoda\"]>%n" + "but find:%n" + " <[\"Luke\"]>%n")); I mad if decide change other tests assertions messages :smile_cat:</technical_language>

Target:
public void should_create_error_message_for_different_instances() { String errorMessage = shouldContainSame(Optional.of(new Integer(10)), new Integer(10)).create(); assertThat(errorMessage).isEqualTo(format("%nExpecting:%n" + " <Optional[10]>%n" + "to contain the instance (i.e. compared with ==):%n" + " <10>%n" + "but did not.")); }
Source:
<code>protected double _calculateCost(final BinaryTransitiveClosure closure, final IConstraintEvaluationContext input) { <START> return StatisticsBasedConstraintCostFunction.DEFAULT_COST; <END> }</code><technical_language>changes related</technical_language>

Target:
protected double _calculateCost(final BinaryTransitiveClosure closure, final IConstraintEvaluationContext input) { return _calculateCost((PConstraint)closure, input); }
Source:
<code>public DefaultCompilationResponse(final Boolean successful, final List<String> mavenOutput, final Path workingDir, <START> final String requestUUID) { <END> this.successful = successful; if(mavenOutput != null) { this.mavenOutput = new ArrayList<>(mavenOutput); }else{ this.mavenOutput = Collections.emptyList(); } this.workingDir = workingDir; this.requestUUID = requestUUID; }</code><technical_language>This constructor calling more featureful constructor for consistency avoid duplication. this: this(successful, mavenOutput, workingDir, Collections.emptyList(), Collections.emptyList(), requestUUID);</technical_language>

Target:
public DefaultCompilationResponse(final Boolean successful, final List<String> mavenOutput, final Path workingDir, final String requestUUID) { this(successful,mavenOutput,workingDir, Collections.emptyList(), Collections.emptyList(), requestUUID); }