auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>protected boolean canDoAction() { boolean returnValue = isVmExist() && acquireLockInternal(); VM vm = getVm(); if (returnValue && (vm != null && vm.getstatus() == VMStatus.ImageLocked)) { returnValue = false; <START> addCanDoActionMessage(VdcBllMessages.ACTION_TYPE_FAILED_VM_IS_LOCKED); <END> } else { returnValue = returnValue && (vm == null || isDiskCanBeAddedToVm(getParameters().getDiskInfo())); if (returnValue && DiskStorageType.IMAGE == getParameters().getDiskInfo().getDiskStorageType()) { returnValue = checkIfImageDiskCanBeAdded(vm); } if (returnValue && DiskStorageType.LUN == getParameters().getDiskInfo().getDiskStorageType()) { returnValue = checkIfLunDiskCanBeAdded(); } } return returnValue; }</code><technical_language>this change disks locked provide more info user</technical_language>

Target:
protected boolean canDoAction() { boolean returnValue = isVmExist() && acquireLockInternal(); VM vm = getVm(); returnValue = returnValue && (vm == null || isDiskCanBeAddedToVm(getParameters().getDiskInfo())); if (returnValue && DiskStorageType.IMAGE == getParameters().getDiskInfo().getDiskStorageType()) { returnValue = checkIfImageDiskCanBeAdded(vm); } if (returnValue && DiskStorageType.LUN == getParameters().getDiskInfo().getDiskStorageType()) { returnValue = checkIfLunDiskCanBeAdded(); } return returnValue; }
Source:
<code>public Object getEntry(String key, String tenantDomain) throws AxisFault { <START> LocalEntryServiceProxy localEntryAdminClient = getLocalEntryAdminClient(tenantDomain); <END> return localEntryAdminClient.getEntry(key); }</code><technical_language>rename variable names replace client proxy</technical_language>

Target:
public Object getEntry(String key, String tenantDomain) throws AxisFault { LocalEntryServiceProxy localEntryServiceProxy = getLocalEntryAdminClient(tenantDomain); return localEntryServiceProxy.getEntry(key); }
Source:
<code><START> public RuntimeException transform(RuntimeException e, Response response) { <END> if (response == null) { return transformNoResponse(e); } RuntimeException result = null; if (response.getStatusInfo() instanceof Response.Status) { result = transformByResponseStatus((Response.Status) response.getStatusInfo(), e, response); } if (result == null) { result = transformByResponseStatusFamily(response.getStatusInfo().getFamily(), e, response); } return result != null ? result : defaultTransform(e, response); }</code><technical_language>Is necessary refactor this method heavily? I personally find implementation more readable. - is of transformNoResponse? If wants customize behavior, simply override transform(). - "byResponseStatus" "byResponseStatusFamily" methods? family case is a case of status case</technical_language>

Target:
public RuntimeException transform(RuntimeException e, Response response) { if (response == null) { return e; } RuntimeException result = null; if (response.getStatusInfo() instanceof Response.Status) { result = transformByResponseStatus((Response.Status) response.getStatusInfo(), e, response); } if (result == null) { result = transformByResponseStatusFamily(response.getStatusInfo().getFamily(), e, response); } return result != null ? result : defaultTransform(e, response); }
Source:
<code>private static BibEntry getBibEntryWithGivenEntryId(BibDatabaseContext database, String entryId) { for (BibEntry bibEntry : database.getEntries()) { if (bibEntry.getId().equals(entryId)) { return bibEntry; } } <START> return null; <END> }</code><technical_language>Please null Optional instead</technical_language>

Target:
private static Optional<BibEntry> getBibEntryWithGivenEntryId(BibDatabaseContext database, String entryId) { for (BibEntry bibEntry : database.getEntries()) { if (bibEntry.getId().equals(entryId)) { return Optional.of(bibEntry); } } return Optional.empty(); }
Source:
<code>public void process(ClientInvocationBuilder invocation, Object param) { <START> invocation.getInvocation().setEntity(Entity.entity(param == null? null : new GenericEntity<Object>(param, genericType), mediaType, annotations)) <END> }</code><technical_language>Missing ";" ... ;-)</technical_language>

Target:
public void process(ClientInvocationBuilder invocation, Object param) { invocation.getInvocation().setEntity(Entity.entity(param == null? null : new GenericEntity<Object>(param, genericType), mediaType, annotations)); }
Source:
<code><START> public InternalResourceGroupManager(LegacyResourceGroupConfigurationManager legacyManager, ClusterMemoryPoolManager memoryPoolManager, NodeInfo nodeInfo, MBeanExporter exporter, InternalNodeManager internalNodeManager, QueryManagerConfig queryManagerConfig, NodeSchedulerConfig nodeSchedulerConfig) <END> { this.exporter = requireNonNull(exporter, "exporter is null"); this.configurationManagerContext = new ResourceGroupConfigurationManagerContextInstance(memoryPoolManager, nodeInfo.getEnvironment()); this.legacyManager = requireNonNull(legacyManager, "legacyManager is null"); this.configurationManager = new AtomicReference<>(cast(legacyManager)); this.internalNodeManager = requireNonNull(internalNodeManager, "internalNodeManager is null"); requireNonNull(nodeSchedulerConfig, "nodeSchedulerConfig is null"); requireNonNull(queryManagerConfig, "queryManagerConfig is null"); this.isIncludeCoordinator = nodeSchedulerConfig.isIncludeCoordinator(); this.initializationRequiredWorkers = queryManagerConfig.getInitializationRequiredWorkers(); this.initializationTimeout = queryManagerConfig.getInitializationTimeout(); this.initialNanos = System.nanoTime(); this.shouldQueueQueriesWithInsufficientWorkers = queryManagerConfig.getQueueQueriesWithInsufficientWorkers(); }</code><technical_language>Put arguments line</technical_language>

Target:
public InternalResourceGroupManager( LegacyResourceGroupConfigurationManager legacyManager, ClusterMemoryPoolManager memoryPoolManager, NodeInfo nodeInfo, MBeanExporter exporter, InternalNodeManager internalNodeManager, QueryManagerConfig queryManagerConfig, NodeSchedulerConfig nodeSchedulerConfig) { this.exporter = requireNonNull(exporter, "exporter is null"); this.configurationManagerContext = new ResourceGroupConfigurationManagerContextInstance(memoryPoolManager, nodeInfo.getEnvironment()); this.legacyManager = requireNonNull(legacyManager, "legacyManager is null"); this.configurationManager = new AtomicReference<>(cast(legacyManager)); this.internalNodeManager = requireNonNull(internalNodeManager, "internalNodeManager is null"); requireNonNull(nodeSchedulerConfig, "nodeSchedulerConfig is null"); requireNonNull(queryManagerConfig, "queryManagerConfig is null"); this.isIncludeCoordinator = nodeSchedulerConfig.isIncludeCoordinator(); this.initializationRequiredWorkers = queryManagerConfig.getInitializationRequiredWorkers(); this.initializationTimeout = queryManagerConfig.getInitializationTimeout(); this.initialNanos = System.nanoTime(); this.shouldQueuePendingQueries = queryManagerConfig.getShouldQueuePendingQueries(); }
Source:
<code>public void testAccessingValuesForInvalidPath() throws Exception { FieldAccessor accessor = new FieldAccessor(); accessor.registerDecoder("TargetValue", new OafFieldDecoder()); String rawSetId = "rawset-id"; String agentId = "agent-id"; String agentName = "agent-name"; AtomicAction action = new AtomicAction(rawSetId, new Agent(agentId, agentName, Agent.AGENT_TYPE.service)); <START> accessor.getValue("$TargetValue.Unknown", action).toString(); <END> }</code><technical_language>test test? changing "Unknown" "Kind" (or totally removing it) test passes</technical_language>

Target:
public void testAccessingValuesForInvalidPath() throws Exception { FieldAccessor accessor = new FieldAccessor(); accessor.registerDecoder("targetValue", new OafFieldDecoder()); String rawSetId = "rawset-id"; String agentId = "agent-id"; String agentName = "agent-name"; AtomicAction action = new AtomicAction(rawSetId, new Agent(agentId, agentName, Agent.AGENT_TYPE.service)); accessor.getValue("agent.unknown", action); }
Source:
<code>private static byte[] toByteArray(String string) { <START> return (string.concat(System.lineSeparator())).getBytes(Charset.defaultCharset()); <END> }</code><technical_language>System.lineSeparator(): standard HTTP end-of-line marker is CRLF (<LINK_0> system dependent</technical_language>

Target:
private static byte[] toByteArray(String string) { return (string.concat(CRLF)).getBytes(StandardCharsets.UTF_8); }
Source:
<code><START> private void validateNotRemovingLabeledNetworks() { <END> Map<String, VdsNetworkInterface> nicsByName = Entities.entitiesByName(params.getInterfaces()); Map<String, VdsNetworkInterface> hostInterfacesByNetworkName = Entities.hostInterfacesByNetworkName(getExistingIfaces().values()); for (String network : removedNetworks) { VdsNetworkInterface nic = hostInterfacesByNetworkName.get(network); if (nic != null) { if (NetworkUtils.isVlan(nic)) { nic = nicsByName.get(NetworkUtils.stripVlan(nic.getName())); } if (nic.getLabels() != null && nic.getLabels().contains(getExistingClusterNetworks().get(network).getLabel())) { addViolation(VdcBllMessages.ACTION_TYPE_FAILED_CANNOT_REMOVE_LABELED_NETWORK_FROM_NIC, network); } } } }</code><technical_language>this problematic detaching network cluster this code run</technical_language>

Target:
private void validateNotRemovingLabeledNetworks() { Map<String, VdsNetworkInterface> nicsByName = Entities.entitiesByName(params.getInterfaces()); Map<String, VdsNetworkInterface> hostInterfacesByNetworkName = Entities.hostInterfacesByNetworkName(getExistingIfaces().values()); for (String network : removedNetworks) { VdsNetworkInterface nic = hostInterfacesByNetworkName.get(network); if (nic != null) { if (NetworkUtils.isVlan(nic)) { nic = nicsByName.get(NetworkUtils.stripVlan(nic.getName())); } Network removedNetwork = getExistingClusterNetworks().get(network); if (nic.getLabels() != null && removedNetwork != null && nic.getLabels().contains(removedNetwork.getLabel())) { addViolation(VdcBllMessages.ACTION_TYPE_FAILED_CANNOT_REMOVE_LABELED_NETWORK_FROM_NIC, network); } } } }
Source:
<code><START> Options(String[] args) throws CmdLineException, FileNotFoundException { <END> CmdLineParser parser = new CmdLineParser(this); parser.parseArgument(args); srcFiles.addAll(arguments); srcFiles.addAll(filesToConvert); if (srcFiles.isEmpty()) { throw new CmdLineException(parser, "No files were given"); } externsMap = getExternsMap(); }</code><technical_language>Nit/optional: nicer for code calling this convert FileNotFoundException a CmdLineException, special case catch</technical_language>

Target:
Options(String[] args) throws CmdLineException { CmdLineParser parser = new CmdLineParser(this); parser.parseArgument(args); srcFiles.addAll(arguments); srcFiles.addAll(filesToConvert); if (srcFiles.isEmpty()) { throw new CmdLineException(parser, "No files were given"); } try { externsMap = getExternsMap(); } catch (FileNotFoundException e) { throw new CmdLineException(parser, "externs file " + externsMapFile + " not found.", e); } }
Source:
<code>public List<Tool> refreshToolsByOrganization(@ApiParam(hidden = true) @Parameter(hidden = true, name = "user")@Auth User authUser, @ApiParam(value = "User ID", required = true) @PathParam("userId") Long userId, @ApiParam(value = "Organization", required = true) @PathParam("organization") String organization, @ApiParam(value = "Docker registry", required = true) @QueryParam("dockerRegistry") String dockerRegistry) { checkUser(authUser, userId); checkToolTokens(authUser, userId, organization); if (dockerRegistry == null) { <START> throw new CustomWebApplicationException("a repository is now required", HttpStatus.SC_BAD_REQUEST); <END> } dockerRepoResource.refreshToolsForUser(userId, organization, dockerRegistry); userDAO.clearCache(); authUser = userDAO.findById(authUser.getId()); authUser.updateUserMetadata(tokenDAO); List<Tool> finalTools = getTools(authUser); bulkUpsertTools(authUser); return finalTools; }</code><technical_language>I A repository is required. 1. start capitals (although always, consistent). 2. I doubt is relying current behavior</technical_language>

Target:
public List<Tool> refreshToolsByOrganization(@ApiParam(hidden = true) @Parameter(hidden = true, name = "user")@Auth User authUser, @ApiParam(value = "User ID", required = true) @PathParam("userId") Long userId, @ApiParam(value = "Organization", required = true) @PathParam("organization") String organization, @ApiParam(value = "Docker registry", required = true) @QueryParam("dockerRegistry") String dockerRegistry) { checkUser(authUser, userId); checkToolTokens(authUser, userId, organization); if (dockerRegistry == null) { throw new CustomWebApplicationException("A repository is required", HttpStatus.SC_BAD_REQUEST); } dockerRepoResource.refreshToolsForUser(userId, organization, dockerRegistry); userDAO.clearCache(); authUser = userDAO.findById(authUser.getId()); authUser.updateUserMetadata(tokenDAO); List<Tool> finalTools = getTools(authUser); bulkUpsertTools(authUser); return finalTools; }
Source:
<code>public SchedulerRestClient getRestClient() { HttpClientBuilder httpClientBuilder = new HttpClientBuilder().useSystemProperties(); if (canInsecureAccess()) { try { <START> httpClientBuilder.insecure(true); <END> } catch (Exception e) { throw new CLIException(REASON_OTHER, "Cannot disable SSL verification.", e); } } return new SchedulerRestClient(restServerUrl, new ApacheHttpClient4Engine(httpClientBuilder.build())); }</code><technical_language>httpClientBuilder.insecure(true) throw Exception, is setting boolean variables</technical_language>

Target:
public SchedulerRestClient getRestClient() { HttpClientBuilder httpClientBuilder = new HttpClientBuilder().useSystemProperties(); if (canInsecureAccess()) { httpClientBuilder.insecure(true); } return new SchedulerRestClient(restServerUrl, new ApacheHttpClient4Engine(httpClientBuilder.build())); }
Source:
<code>public SaveTaskService saveTaskService(TaskDefinitionRepository taskDefinitionRepository, <START> AuditRecordService auditRecordService, AppRegistryService registry) { <END> return new DefaultSaveTaskService(taskDefinitionRepository, auditRecordService,registry); }</code><technical_language>Tabbing a bit here</technical_language>

Target:
public TaskSaveService saveTaskService(TaskDefinitionRepository taskDefinitionRepository, AuditRecordService auditRecordService, AppRegistryService registry) { return new DefaultTaskSaveService(taskDefinitionRepository, auditRecordService, registry); }
Source:
<code><START> private static String getTestProperty(String name) { <END> String val = ImmutableMap.of( BigtableOptionsFactory.PROJECT_ID_KEY, "igorbernstein-dev", BigtableOptionsFactory.INSTANCE_ID_KEY, "instance1", GOOGLE_DATAFLOW_STAGING_LOCATION, "gs://igorbernstein-dev/dataflow-staging", CLOUD_TEST_DATA_FOLDER, "gs://igorbernstein-dev/test-dir" ).get(name); return checkNotNull(val); }</code><technical_language>This get values form system properties</technical_language>

Target:
private static String getTestProperty(String name) { return checkNotNull(System.getProperty(name), "Required property missing: " + name); }
Source:
<code>public static <T> T getDefaultValue(Class<T> classType, String preferenceName) { Object value = null; if (USER_ASK_PREFERENCES_ID.equals(preferenceName)) { value = false; } else if (SCOPE_PREFERENCES_ID.equals(preferenceName)) { value = RefactorScope.WORKSPACE.name(); } return classType.isInstance(value) ? <START> (T) value : <END> null; }</code><technical_language>Do classType.cast(value) suppress unchecked warning. :-)</technical_language>

Target:
public static <T> T getDefaultValue(Class<T> classType, String preferenceName) { Object value = null; if (USER_ASK_PREFERENCES_ID.equals(preferenceName)) { value = false; } else if (SCOPE_PREFERENCES_ID.equals(preferenceName)) { value = RefactorScope.WORKSPACE.name(); } return classType.isInstance(value) ? classType.cast(value) : null; }
Source:
<code>public void testMakeWithShallowCopy() { final GenotypeBuilder gb = new GenotypeBuilder("test"); final int[] first = new int[]{1, 2}; final int[] second = new int[]{3, 4}; final Genotype firstG = gb.attribute("first", first).makeWithShallowCopy(); final Genotype secondG = gb.attribute("second", second).makeWithShallowCopy(); Assert.assertEquals(first, firstG.getExtendedAttribute("first")); Assert.assertEquals(first, secondG.getExtendedAttribute("first")); Assert.assertEquals(null, firstG.getExtendedAttribute("second")); <START> Assert.assertEquals(second, secondG.getExtendedAttribute("second")); <END> }</code><technical_language>In test for makeWithShallowCopy(), test reference fields get shallow copied, including alleles, AD, PL, attributes</technical_language>

Target:
public void testMakeWithShallowCopy() { final GenotypeBuilder gb = new GenotypeBuilder("test"); final List<Allele> alleles = new ArrayList<>( Arrays.asList(Allele.create("A", true), Allele.create("T"))); final int[] ad = new int[]{1,5}; final int[] pl = new int[]{1,6}; final int[] first = new int[]{1, 2}; final int[] second = new int[]{3, 4}; final Genotype firstG = gb.alleles(alleles).attribute("first", first).makeWithShallowCopy(); final Genotype secondG = gb.AD(ad).PL(pl).attribute("second", second).makeWithShallowCopy(); Assert.assertEquals(first, firstG.getExtendedAttribute("first")); Assert.assertEquals(first, secondG.getExtendedAttribute("first")); Assert.assertEquals(alleles, firstG.getAlleles()); Assert.assertEquals(alleles, secondG.getAlleles()); Assert.assertNull(firstG.getAD()); Assert.assertEquals(ad, secondG.getAD()); Assert.assertNull(firstG.getPL()); Assert.assertEquals(pl, secondG.getPL()); Assert.assertNull(firstG.getExtendedAttribute("second")); Assert.assertEquals(second, secondG.getExtendedAttribute("second")); alleles.add(Allele.create("C")); Assert.assertNotEquals(alleles, firstG.getAlleles()); Assert.assertNotEquals(alleles, secondG.getAlleles()); ad[0] = 0; pl[0] = 10; Assert.assertNotEquals(ad, secondG.getAD()); Assert.assertNotEquals(pl, secondG.getPL()); }
Source:
<code>private Node processInitialLabel(String initialLabel, List<String> removedLabels, Optional<String> suggestion) { TurboLabel repoInitialLabel = TurboLabel.getFirstMatchingTurboLabel(repoLabels, initialLabel); if (!removedLabels.contains(initialLabel)) { if (suggestion.isPresent() && initialLabel.equals(suggestion.get())) { <START> return getStyledPickerLabel(repoInitialLabel, true, false, true, false, true); <END> } return getStyledPickerLabel(repoInitialLabel, false, false, false, false, true); } if (suggestion.isPresent() && initialLabel.equals(suggestion.get())) { return getStyledPickerLabel(repoInitialLabel, true, false, false, false, true); } return getStyledPickerLabel(repoInitialLabel, false, false, true, false, true); }</code><technical_language>Instead of this, do getStyledPickerLabel(repoInitialLabel).ticked().highlighted().faded()? trouble of defining getters setters, call sites more readable</technical_language>

Target:
private final Node processInitialLabel(String initialLabel, List<String> removedLabels, Optional<String> suggestion) { TurboLabel repoInitialLabel = TurboLabel.getFirstMatchingTurboLabel(allLabels, initialLabel); if (!removedLabels.contains(initialLabel)) { if (suggestion.isPresent() && initialLabel.equals(suggestion.get())) { return getPickerLabelNode( new PickerLabel(repoInitialLabel, true).faded(true).removed(true)); } return getPickerLabelNode(new PickerLabel(repoInitialLabel, true)); } if (suggestion.isPresent() && initialLabel.equals(suggestion.get())) { return getPickerLabelNode(new PickerLabel(repoInitialLabel, true).faded(true)); } return getPickerLabelNode(new PickerLabel(repoInitialLabel, true).removed(true)); }
Source:
<code>synchronized void updateSender(final String operationId, final TraceInfo traceInfo) { <START> final MigrationInfo updatedInfo = checkAndUpdateState(operationId, MOVING_PARTITION, UPDATING_SENDER); <END> final String senderId = updatedInfo.getSenderId(); sender.get().sendUpdateMsg(senderId, operationId, traceInfo); }</code><technical_language>a generic FSM implementation in Shuffle. extract out a cay common component avoid duplicate FSM code</technical_language>

Target:
synchronized void updateSender(final String operationId, final TraceInfo traceInfo) { final Migration migration = ongoingMigrations.get(operationId); if (migration == null) { LOG.log(Level.WARNING, "Migration with ID {0} was not registered, or it has already been finished.", operationId); return; } migration.checkAndUpdate(Migration.MOVING_PARTITION, Migration.UPDATING_SENDER); final String senderId = migration.getSenderId(); sender.get().sendUpdateMsg(senderId, operationId, traceInfo); }
Source:
<code>private void end(int statusCode, Throwable throwable, Context context) { for (Tracer tracer : tracers) { if (throwable != null) { tracer.setAttribute(TracerProvider.ERROR_MSG, throwable.getMessage(), context); tracer.setAttribute(TracerProvider.ERROR_TYPE, throwable.getClass().getName(), context); StringWriter errorStack = new StringWriter(); <START> throwable.printStackTrace(new PrintWriter(errorStack)); <END> tracer.setAttribute(TracerProvider.ERROR_STACK, errorStack.toString(), context); } tracer.end(statusCode, throwable, context); } }</code><technical_language>do this is a good idea? especially Throwable.printStackTrace() thread-safe give out confusing logs/statements? cc: @srnagar</technical_language>

Target:
private void end(int statusCode, Throwable throwable, Context context) { if (throwable != null) { tracer.setAttribute(TracerProvider.ERROR_MSG, throwable.getMessage(), context); tracer.setAttribute(TracerProvider.ERROR_TYPE, throwable.getClass().getName(), context); } tracer.end(statusCode, throwable, context); }
Source:
<code>public void onMessage(Message message) { if (logger.isTraceEnabled()) { logger.trace("Message received " + message); } String messageTag; if (this.correlationKey == null) { messageTag = message.getMessageProperties().getCorrelationId(); } else { messageTag = (String) message.getMessageProperties() .getHeaders().get(this.correlationKey); } if (messageTag == null) { logger.error("No correlation header in reply"); <START> throw new AmqpRejectAndDontRequeueException("No correlation header"); <END> } PendingReply pendingReply = this.replyHolder.get(messageTag); if (pendingReply == null) { if (logger.isWarnEnabled()) { logger.warn("Reply received after timeout for " + messageTag); } throw new AmqpRejectAndDontRequeueException("Reply received after timeout"); } else { restoreProperties(message, pendingReply); pendingReply.reply(message); } }</code><technical_language>I this exception need error log anymore</technical_language>

Target:
public void onMessage(Message message) { if (logger.isTraceEnabled()) { logger.trace("Message received " + message); } String messageTag; if (this.correlationKey == null) { messageTag = message.getMessageProperties().getCorrelationId(); } else { messageTag = (String) message.getMessageProperties() .getHeaders().get(this.correlationKey); } if (messageTag == null) { throw new AmqpRejectAndDontRequeueException("No correlation header in reply"); } PendingReply pendingReply = this.replyHolder.get(messageTag); if (pendingReply == null) { if (logger.isWarnEnabled()) { logger.warn("Reply received after timeout for " + messageTag); } throw new AmqpRejectAndDontRequeueException("Reply received after timeout"); } else { restoreProperties(message, pendingReply); pendingReply.reply(message); } }
Source:
<code>List<BookieSocketAddress> getCurrentEnsemble() { <START> if (newEnsemblesFromRecovery.size() > 0) { <END> return newEnsemblesFromRecovery.lastEntry().getValue(); } else { return super.getCurrentEnsemble(); } }</code><technical_language>synchronization newEnsemblesFromRecovery</technical_language>

Target:
List<BookieSocketAddress> getCurrentEnsemble() { synchronized (metadataLock) { if (!newEnsemblesFromRecovery.isEmpty()) { return newEnsemblesFromRecovery.lastEntry().getValue(); } else { return super.getCurrentEnsemble(); } } }
Source:
<code>public boolean onOptionsItemSelected(MenuItem item) { switch (item.getItemId()) { case R.id.reRun: new ConfirmDialogFragment.Builder() .withExtra("re_run") .withMessage(getString(R.string.Modal_DoYouWantToDeleteThisTest)) .withPositiveButton(getString(R.string.Modal_Delete)) <START> .build().show(getSupportFragmentManager(), null); <END> return true; default: return super.onOptionsItemSelected(item); } }</code><technical_language>I missing offering delete a test user clicks rerun</technical_language>

Target:
public boolean onOptionsItemSelected(MenuItem item) { switch (item.getItemId()) { case R.id.reRun: new ConfirmDialogFragment.Builder() .withExtra(RERUN_KEY) .withMessage(getString(R.string.Modal_ReRun_Websites_Title)) .withPositiveButton(getString(R.string.Modal_ReRun_Websites_Run)) .build().show(getSupportFragmentManager(), null); return true; default: return super.onOptionsItemSelected(item); } }
Source:
<code>public static int processLauncherAndWait(Map<String, Object> conf, String user, List<String> args, final Map<String, String> environment, final String logPreFix) throws IOException { int ret = 0; Process process = processLauncher(conf, user, null, args, environment, null, null, null); if (StringUtils.isNotBlank(logPreFix)) { Utils.readAndLogStream(logPreFix, process.getInputStream()); } try { process.waitFor(); } catch (InterruptedException e) { LOG.warn("{} interrupted.", logPreFix); Thread.currentThread().interrupt(); throw new IOException(logPreFix + " interrupted", e); <START> } <END> ret = process.exitValue(); return ret; }</code><technical_language>Need a process.destroy() throwing IOException, cleanup happen</technical_language>

Target:
public static int processLauncherAndWait(Map<String, Object> conf, String user, List<String> args, final Map<String, String> environment, final String logPreFix) throws IOException { int ret = 0; Process process = processLauncher(conf, user, null, args, environment, null, null, null); if (StringUtils.isNotBlank(logPreFix)) { Utils.readAndLogStream(logPreFix, process.getInputStream()); } try { process.waitFor(); } catch (InterruptedException e) { LOG.warn("{} interrupted.", logPreFix); Thread.currentThread().interrupt(); process.destroy(); throw new IOException(logPreFix + " interrupted", e); } ret = process.exitValue(); return ret; }
Source:
<code>private String findDevMachineName(Environment env) throws ServerException { return env.getMachines() .entrySet() .stream() .filter(entry -> entry.getValue() .getAgents() <START> .contains("org.eclipse.che.ws-agent")) <END> .findAny() .orElseThrow( () -> new ServerException("Agent 'org.eclipse.che.ws-agent' is not found in any of environment machines")) .getKey(); }</code><technical_language>Is this line version</technical_language>

Target:
private String findDevMachineName(Environment env) throws ServerException { return env.getMachines() .entrySet() .stream() .filter(entry -> entry.getValue() .getAgents() .stream() .filter(agent -> agent.contains("org.eclipse.che.ws-agent")) .findAny() .isPresent()) .findAny() .orElseThrow(() -> new ServerException("Agent 'org.eclipse.che.ws-agent' is not found in any of environment machines")) .getKey(); }
Source:
<code>private void doLoad(ExtMap input, ExtMap output) { context = input.<ExtMap> get(Base.InvokeKeys.CONTEXT); Properties configuration = context.<Properties> get(Base.ContextKeys.CONFIGURATION); context.mput( Base.ContextKeys.AUTHOR, "The oVirt Project" ).mput( Base.ContextKeys.EXTENSION_NAME, "Internal Authz (Built-in)" ).mput( Base.ContextKeys.LICENSE, "ASL 2.0" ).mput( Base.ContextKeys.HOME_URL, "<LINK_0>" ).mput( Base.ContextKeys.VERSION, "N/A" ).mput( Authz.ContextKeys.QUERY_MAX_FILTER_SIZE, Integer.parseInt( configuration.getProperty("config.query.filter.size") ) ).mput( Base.ContextKeys.BUILD_INTERFACE_VERSION, Base.INTERFACE_VERSION_CURRENT ).mput( Authz.ContextKeys.AVAILABLE_NAMESPACES, Arrays.asList(NAMESPACE) ); adminUser = new ExtMap().mput( Authz.PrincipalRecord.NAMESPACE, NAMESPACE ).mput( Authz.PrincipalRecord.NAME, configuration.getProperty("config.authz.user.name") ).mput( Authz.PrincipalRecord.ID, configuration.getProperty("config.authz.user.id") ).mput( Authz.PrincipalRecord.GROUPS, <START> Collections.emptyList() <END> ); }</code><technical_language>again, need</technical_language>

Target:
private void doLoad(ExtMap input, ExtMap output) { context = input.<ExtMap> get(Base.InvokeKeys.CONTEXT); Properties configuration = context.<Properties> get(Base.ContextKeys.CONFIGURATION); context.mput( Base.ContextKeys.AUTHOR, "The oVirt Project" ).mput( Base.ContextKeys.EXTENSION_NAME, "Internal Authz (Built-in)" ).mput( Base.ContextKeys.LICENSE, "ASL 2.0" ).mput( Base.ContextKeys.HOME_URL, "<LINK_0>" ).mput( Base.ContextKeys.VERSION, "N/A" ).mput( Authz.ContextKeys.QUERY_MAX_FILTER_SIZE, Integer.parseInt( configuration.getProperty("config.query.filter.size") ) ).mput( Base.ContextKeys.BUILD_INTERFACE_VERSION, Base.INTERFACE_VERSION_CURRENT ).mput( Authz.ContextKeys.AVAILABLE_NAMESPACES, Arrays.asList(NAMESPACE) ); adminUser = new ExtMap().mput( Authz.PrincipalRecord.NAMESPACE, NAMESPACE ).mput( Authz.PrincipalRecord.NAME, configuration.getProperty("config.authz.user.name") ).mput( Authz.PrincipalRecord.ID, configuration.getProperty("config.authz.user.id") ); }
Source:
<code>private void removeNearbyIfUnsupported() { <START> if (!DeviceUtil.supportsNearby(activity)) { <END> activity.getNavMenu().removeItem(R.id.nav_item_nearby); } }</code><technical_language>Do need a pre-production check this</technical_language>

Target:
private void removeNearbyIfUnsupported() { if (!DeviceUtil.supportsNearby(activity) && app.isPreProdRelease()) { activity.getNavMenu().removeItem(R.id.nav_item_nearby); } }
Source:
<code>private void fetchFromCache(String objectType, String layoutType, LayoutSyncCallback syncCallback, boolean fallbackOnServer) { final QuerySpec querySpec = QuerySpec.buildSmartQuerySpec(String.format(QUERY, objectType, layoutType), 1); try { final JSONArray results = smartStore.query(querySpec, 0); if (results == null || results.length() == 0) { if (fallbackOnServer) { fetchFromServer(objectType, layoutType, syncCallback); } else { onSyncComplete(objectType, syncCallback, null); } } else { final JSONArray resultArr = results.optJSONArray(0); <START> if (resultArr == null || resultArr.length() == 0) { <END> onSyncComplete(objectType, syncCallback, null); } else { onSyncComplete(objectType, syncCallback, Layout.fromJSON(resultArr.optJSONObject(0))); } } } catch (Exception e) { SmartSyncLogger.e(TAG, "Exception occurred while reading layout data from the cache", e); } }</code><technical_language>happen? check fallbackOnServer. In other words conditions move line 228</technical_language>

Target:
private void fetchFromCache(String objectType, String layoutType, LayoutSyncCallback syncCallback, boolean fallbackOnServer) { final QuerySpec querySpec = QuerySpec.buildSmartQuerySpec(String.format(QUERY, objectType, layoutType), 1); try { final JSONArray results = smartStore.query(querySpec, 0); if (results == null || results.length() == 0) { if (fallbackOnServer) { fetchFromServer(objectType, layoutType, syncCallback); } else { onSyncComplete(objectType, syncCallback, null); } } else { onSyncComplete(objectType, syncCallback, Layout.fromJSON(results.optJSONArray(0).optJSONObject(0))); } } catch (Exception e) { SmartSyncLogger.e(TAG, "Exception occurred while reading layout data from the cache", e); } }
Source:
<code>protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { Map<String, Object> ret = new HashMap<String, Object>(); if (hasParam(req, ACTION_PARAM)) { String action = getParam(req, ACTION_PARAM); if (action.equals(STATS_SET_REPORTINGINTERVAL)) { handleChangeMetricInterval(req, ret); } else if (action.equals(STATS_SET_CLEANINGINTERVAL)) { handleChangeCleaningInterval(req, ret); } else if (action.equals(STATS_SET_MAXREPORTERPOINTS)) { handleChangeEmitterPoints(req, ret); } else if (action.equals(STATS_GET_ALLMETRICSNAME)) { handleGetAllMMetricsName(req, ret); } else if (action.equals(STATS_GET_METRICHISTORY)) { handleGetMetricHistory(req, ret); } else if (action.equals(STATS_SET_ENABLEMETRICS)) { handleChangeManagerStatusRequest(req, ret, true); } else if (action.equals(STATS_SET_DISBLEMETRICS)) { handleChangeManagerStatusRequest(req, ret, false); <START> } <END> } JSONUtils.toJSON(ret, resp.getOutputStream(), true); }</code><technical_language>else put error message invalid action</technical_language>

Target:
protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { Map<String, Object> ret = new HashMap<String, Object>(); if (hasParam(req, ACTION_PARAM)) { String action = getParam(req, ACTION_PARAM); if (action.equals(STATS_SET_REPORTINGINTERVAL)) { handleChangeMetricInterval(req, ret); } else if (action.equals(STATS_SET_CLEANINGINTERVAL)) { handleChangeCleaningInterval(req, ret); } else if (action.equals(STATS_SET_MAXREPORTERPOINTS)) { handleChangeEmitterPoints(req, ret); } else if (action.equals(STATS_GET_ALLMETRICSNAME)) { handleGetAllMMetricsName(req, ret); } else if (action.equals(STATS_GET_METRICHISTORY)) { handleGetMetricHistory(req, ret); } else if (action.equals(STATS_SET_ENABLEMETRICS)) { handleChangeManagerStatusRequest(req, ret, true); } else if (action.equals(STATS_SET_DISABLEMETRICS)) { handleChangeManagerStatusRequest(req, ret, false); } else { ret.put(RESPONSE_ERROR, "Invalid action"); } } JSONUtils.toJSON(ret, resp.getOutputStream(), true); }
Source:
<code><START> private Map<ArchitectureType, Map<Version, Boolean>> getMap(ArchCapabilitiesVerb archCapabilitiesVerb) { <END> if (verbMap.containsKey(archCapabilitiesVerb)) { return verbMap.get(archCapabilitiesVerb); } else { Map<ArchitectureType, Map<Version, Boolean>> supportMap = new EnumMap<ArchitectureType, Map<Version, Boolean>>(ArchitectureType.class); for (ArchitectureType arch : ArchitectureType.values()) { Map<Version, Boolean> archMap = new HashMap<Version, Boolean>(); for (Version version : Version.ALL) { boolean isSupported = isSupported(archCapabilitiesVerb, arch, version); archMap.put(version, isSupported); } supportMap.put(arch, archMap); } verbMap.put(archCapabilitiesVerb, supportMap); return supportMap; } }</code><technical_language>Easiest solution is this method synchronized</technical_language>

Target:
private synchronized Map<ArchitectureType, Map<Version, Boolean>> getMap(ArchCapabilitiesVerb archCapabilitiesVerb) { if (verbMap.containsKey(archCapabilitiesVerb)) { return verbMap.get(archCapabilitiesVerb); } else { Map<ArchitectureType, Map<Version, Boolean>> supportMap = new EnumMap<ArchitectureType, Map<Version, Boolean>>(ArchitectureType.class); for (ArchitectureType arch : ArchitectureType.values()) { Map<Version, Boolean> archMap = new HashMap<Version, Boolean>(); for (Version version : Version.ALL) { boolean isSupported = isSupported(archCapabilitiesVerb, arch, version); archMap.put(version, isSupported); } supportMap.put(arch, archMap); } verbMap.put(archCapabilitiesVerb, supportMap); return supportMap; } }
Source:
<code>public ISYActiveBindingConfig(Map<Object, Object> config) { this.logger.info("OpenHab Binding Configuration(refresh='{}',upnp='{}',uuid='{}',ip='{}',port='{}',user='{}',password='{}')", config.get("refresh"), config.get("upnp"),config.get("uuid"),config.get("ip"),config.get("port"),config.get("user"),"xxxxxx"); <START> if (StringUtils.isNotBlank((String) config.get("refresh"))) { <END> refreshInterval = Long.parseLong((String) config.get("refresh")); } if (StringUtils.isNotBlank((String) config.get("user")) && StringUtils.isNotBlank((String) config.get("password"))) { this.user = (String) config.get("user"); this.password = (String) config.get("password"); } else { this.logger.warn("No user/password specified"); } if (StringUtils.isNotBlank((String) config.get("upnp"))) { useUpnp = Boolean.parseBoolean((String) config.get("upnp")); } if (!useUpnp && StringUtils.isNotBlank((String) config.get("uuid")) && StringUtils.isNotBlank((String) config.get("ip")) && StringUtils.isNotBlank((String) config.get("port"))) { this.uuid = (String) config.get("uuid"); this.ipAddress = (String) config.get("ip"); try { this.port = Integer.parseInt((String) config.get("port")); } catch (NumberFormatException ne) { this.logger.error("Not a valid port number"); throw new RuntimeException("Not a valid port number", ne); } if (this.port < 0 || this.port > 65535) { this.logger.error("Port out of range (0-65535)"); throw new NumberFormatException("Invalid port"); } } else { this.logger.info("Using UPNP for ISY connection."); } }</code><technical_language>java import static org.apache.commons.lang.StringUtils.isNotBlank; for easier read code</technical_language>

Target:
public ISYActiveBindingConfig(Map<Object, Object> config) { this.logger .info("OpenHab Binding Configuration(refresh='{}',upnp='{}',uuid='{}',ip='{}',port='{}',user='{}',password='{}')", config.get("refresh"), config.get("upnp"), config.get("uuid"), config.get("ip"), config.get("port"), config.get("user"), "xxxxxx"); if (isNotBlank((String) config.get("refresh"))) { refreshInterval = Long.parseLong((String) config.get("refresh")); } if (isNotBlank((String) config.get("user")) && isNotBlank((String) config.get("password"))) { this.user = (String) config.get("user"); this.password = (String) config.get("password"); } else { this.logger.warn("No user/password specified"); } if (isNotBlank((String) config.get("upnp"))) { useUpnp = Boolean.parseBoolean((String) config.get("upnp")); } if (!useUpnp && isNotBlank((String) config.get("uuid")) && isNotBlank((String) config.get("ip")) && isNotBlank((String) config.get("port"))) { this.uuid = (String) config.get("uuid"); this.ipAddress = (String) config.get("ip"); try { this.port = Integer.parseInt((String) config.get("port")); } catch (NumberFormatException ne) { this.logger.error("Not a valid port number"); throw new RuntimeException("Not a valid port number", ne); } if (this.port < 0 || this.port > 65535) { this.logger.error("Port out of range (0-65535)"); throw new NumberFormatException("Invalid port"); } } else { this.logger.info("Using UPNP for ISY connection."); } }
Source:
<code><START> public void setProjectService(KieProjectService projectService) { <END> this.projectService = projectService; }</code><technical_language>need for public setter set in constructor</technical_language>

Target:
protected void setProjectService(KieProjectService projectService) { this.projectService = projectService; }
Source:
<code>public BaseLayer(NeuralNetConfiguration conf, INDArray input) { this.input = input; <START> this.conf = conf; <END> }</code><technical_language>this(conf) behaviour: cacheMode = conf.getCacheMode(); this constructor (now) doesn't</technical_language>

Target:
public BaseLayer(NeuralNetConfiguration conf, INDArray input) { this(conf); this.input = input; }
Source:
<code>public List<FormSelectionMark> getSelectionMarks() { <START> return this.selectionMarks; <END> }</code><technical_language>return unmodifiable list user</technical_language>

Target:
public List<FormSelectionMark> getSelectionMarks() { return Collections.unmodifiableList(this.selectionMarks); }
Source:
<code>public void verifyAccessibleForCurrentUserAsInstructorOrTeamMember(AccountAttributes account, String courseId, String section, String email) { InstructorAttributes instructor = InstructorsLogic.inst().getInstructorForGoogleId(courseId, account.googleId); StudentAttributes student = StudentsLogic.inst().getStudentForCourseIdAndGoogleId(courseId, account.googleId); if (instructor == null && student == null) { throw new UnauthorizedAccessException("User is not in the course that student belongs to"); } else if (instructor != null && !instructor.isAllowedForPrivilege(section, Const.ParamsNames.INSTRUCTOR_PERMISSION_VIEW_STUDENT_IN_SECTIONS) || student != null && !StudentsLogic.inst().isStudentsInSameTeam(courseId, email, student.email)) { throw new UnauthorizedAccessException("User does not have enough privileges to view the photo"); <START> <END> } }</code><technical_language>do this way: java if (instructor != null) { if (...) { throw new UnauthorizedAccessException(...); } return; } StudentAttributes student = ...; if (student != null) { if (...) { throw new UnauthorizedAccessException(...); } return; } throw new UnauthorizedAccessException(...); This hierarchy more clearly, suppresses need query for extra student entity if instructor entity is non-null, allows for exception messages for instructor student (they same)</technical_language>

Target:
public void verifyAccessibleForCurrentUserAsInstructorOrTeamMember(AccountAttributes account, String courseId, String section, String email) { InstructorAttributes instructor = InstructorsLogic.inst().getInstructorForGoogleId(courseId, account.googleId); if (instructor != null) { if (!instructor.isAllowedForPrivilege(section, Const.ParamsNames.INSTRUCTOR_PERMISSION_VIEW_STUDENT_IN_SECTIONS)) { throw new UnauthorizedAccessException("Instructor does not have enough privileges to view the photo"); } return; } StudentAttributes student = StudentsLogic.inst().getStudentForCourseIdAndGoogleId(courseId, account.googleId); if (student != null) { if (!StudentsLogic.inst().isStudentsInSameTeam(courseId, email, student.email)) { throw new UnauthorizedAccessException("Student does not have enough privileges to view the photo"); } return; } throw new UnauthorizedAccessException("User is not in the course that student belongs to"); }
Source:
<code>public void onCacheCreatedEvent( CacheEntryEvent event ) { if ( event.getKey() == this.key ) { <START> this.lock.signal(); <END> } events.add( event ); }</code><technical_language>set a flag internally this listener. Then, if in future some code called listenToCache() already-triggered listener, return immediately</technical_language>

Target:
public void onCacheCreatedEvent( CacheEntryEvent event ) { if ( event.getKey().equals( this.key ) ) { this.lock.signal(); } events.add( event ); }
Source:
<code><START> public void checkAttributeSyntax(PerunSessionImpl perunSession, String key, Attribute attribute) throws WrongAttributeValueException { <END> if (attribute.getValue() == null) return; if (!(attribute.getValue() instanceof String)) { throw new WrongAttributeValueException(attribute, key, "value must be of type String"); } String value = attribute.valueAsString(); if (!value.contains("{link}")) { throw new WrongAttributeValueException(attribute, key, "value doesn't contain contain tag {link}"); } }</code><technical_language>skip part instanceof String</technical_language>

Target:
public void checkAttributeSyntax(PerunSessionImpl perunSession, String key, Attribute attribute) throws WrongAttributeValueException { if (attribute.getValue() == null) return; if (!attribute.valueAsString().contains("{link}")) { throw new WrongAttributeValueException(attribute, key, "value doesn't contain contain tag {link}"); } }
Source:
<code><START> public void shouldAddTransactionIdToCard() throws Exception { <END> DatabaseFixtures.TestCharge testCharge = addCharge(); Pair<Long, Long> ids = createPaymentRequest(testCharge); long cardId = nextId.getAndIncrement(); databaseTestHelper.addCard(cardId, testCharge.getChargeId(), null); worker.execute(1L); Map<String, Object> card = databaseTestHelper.getCard(cardId); assertThat(card.get("transaction_id"), is(ids.getRight())); }</code><technical_language>This test methods throw exceptions, need declare</technical_language>

Target:
public void shouldAddTransactionIdToCard() { DatabaseFixtures.TestCharge testCharge = addCharge(); Pair<Long, Long> ids = createPaymentRequest(testCharge); long cardId = nextId.getAndIncrement(); databaseTestHelper.addCard(cardId, testCharge.getChargeId(), null); worker.execute(1L); Map<String, Object> card = databaseTestHelper.getCard(cardId); assertThat(card.get("transaction_id"), is(ids.getRight())); }
Source:
<code>public void invalidTokenShouldFail() { <START> StandardSlackService service = new StandardSlackService("", "tinyspeck", false, "#general", false,"token"); <END> service.publish("message"); }</code><technical_language>suggestion StandardSlackService service = new StandardSlackService("", "tinyspeck", false, "#general", false, "token");</technical_language>

Target:
public void invalidTokenShouldFail() { StandardSlackService service = new StandardSlackService( StandardSlackService.builder() .withBaseUrl("") .withTeamDomain("tinyspeck") .withBotUser(false) .withRoomId("#general") .withReplyBroadcast(false) .withPopulatedToken("token")); service.publish("message"); }
Source:
<code>public synchronized void save() throws IOException { if(BulkChange.contains(this)) return; long start=0; if(logger.isLoggable(Level.FINE)) start = System.currentTimeMillis(); FingerprintStorage.get().save(this); if(logger.isLoggable(Level.FINE)) <START> logger.fine("Saving fingerprint took "+(System.currentTimeMillis()-start)+"ms"); <END> }</code><technical_language>needs some kind of fingerprint identifier in message, e.g. ID/hash</technical_language>

Target:
public synchronized void save() throws IOException { if(BulkChange.contains(this)) { return; } long start=0; if(logger.isLoggable(Level.FINE)) start = System.currentTimeMillis(); FingerprintStorage.get().save(this); if(logger.isLoggable(Level.FINE)) logger.fine("Saving fingerprint " + getHashString() + " took " + (System.currentTimeMillis() - start) + "ms"); }
Source:
<code>private void createNode() { if ( !isActive() ) { return; } if ( debugCreateNodeLatch != null ) { try { debugCreateNodeLatch.await(); } catch ( InterruptedException e ) { Thread.currentThread().interrupt(); return; } } try { String existingPath = nodePath.get(); String createPath = existingPath == null || (useProtection && !isSequential(mode)) ? basePath <START> : (useProtection ? basePath + existingPath.substring(existingPath.length()-10) : existingPath); <END> CreateModable<ACLBackgroundPathAndBytesable<String>> localCreateMethod = createMethod.get(); if ( localCreateMethod == null ) { CreateBuilderMain createBuilder = SafeIsTtlMode.isTtl(mode) ? client.create().withTtl(ttl) : client.create(); CreateModable<ACLBackgroundPathAndBytesable<String>> tempCreateMethod = useProtection ? createBuilder.creatingParentContainersIfNeeded().withProtection() : createBuilder.creatingParentContainersIfNeeded(); createMethod.compareAndSet(null, tempCreateMethod); localCreateMethod = createMethod.get(); } localCreateMethod.withMode(getCreateMode(existingPath != null)).inBackground(backgroundCallback).forPath(createPath, data.get()); } catch ( Exception e ) { ThreadUtils.checkInterrupted(e); throw new RuntimeException("Creating node. BasePath: " + basePath, e); } }</code><technical_language>I this cleaner some if statements. Also, 10 magic number is ideal, I a constant defining length of sequential part of node name anywhere. place be, in CuratorFramework interface. Thoughts @Randgalt</technical_language>

Target:
private void createNode() { if ( !isActive() ) { return; } if ( debugCreateNodeLatch != null ) { try { debugCreateNodeLatch.await(); } catch ( InterruptedException e ) { Thread.currentThread().interrupt(); return; } } try { String existingPath = nodePath.get(), createPath; if ( existingPath != null && !useProtection ) { createPath = existingPath; } else if ( existingPath != null && mode.isSequential() ) { createPath = basePath + ZKPaths.extractSequentialSuffix(existingPath); } else { createPath = basePath; } CreateModable<ACLBackgroundPathAndBytesable<String>> localCreateMethod = createMethod.get(); if ( localCreateMethod == null ) { CreateBuilderMain createBuilder = SafeIsTtlMode.isTtl(mode) ? client.create().withTtl(ttl) : client.create(); CreateModable<ACLBackgroundPathAndBytesable<String>> tempCreateMethod = useProtection ? createBuilder.creatingParentContainersIfNeeded().withProtection() : createBuilder.creatingParentContainersIfNeeded(); createMethod.compareAndSet(null, tempCreateMethod); localCreateMethod = createMethod.get(); } localCreateMethod.withMode(getCreateMode(existingPath != null)).inBackground(backgroundCallback).forPath(createPath, data.get()); } catch ( Exception e ) { ThreadUtils.checkInterrupted(e); throw new RuntimeException("Creating node. BasePath: " + basePath, e); } }