auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>protected ILogData removeFromQueue(NavigableSet<Long> queue) { boolean readNext; ILogData ld = null; Long currentRead; do { currentRead = queue.pollFirst(); if (currentRead == null) { return null; } try { ld = read(currentRead, queue); if (queue == getCurrentContext().readQueue) { if (ld.getType() == DataType.HOLE || (ld.containsStream(this.id) && <START> ld.getType() == DataType.DATA)) { <END> if (ld.isHole()) { ld = new LogData(DataType.DATA, SMREntry.getNOP(), ld.getMetadataMap()); } addToResolvedQueue(getCurrentContext(), currentRead); readNext = false; } else { log.trace("getNextEntry[{}]: the data for address does not belong to this stream. Skip.", this, currentRead); readNext = true; } } else { readNext = false; } } catch (TrimmedException te) { if (!getReadOptions().isIgnoreTrim()) { throw te; } log.debug("removeFromQueue[{}]: ignoring trimmed addresses {}", this, te.getTrimmedAddresses()); te.getTrimmedAddresses().forEach(address -> queue.remove(address)); readNext = true; } } while (readNext); return ld; }</code><technical_language>This layer understands ILogData move this conversion StreamViewSMRAdapter maps ILogData SMREntry</technical_language>

Target:
protected ILogData removeFromQueue(NavigableSet<Long> queue) { boolean readNext; ILogData ld = null; Long currentRead; do { currentRead = queue.pollFirst(); if (currentRead == null) { return null; } try { ld = read(currentRead, queue); if (queue == getCurrentContext().readQueue) { if ((ld.containsStream(this.id) && ld.isData()) || (ld.isHole() && (ld.getBackpointerMap().isEmpty() || ld.containsStream(this.id))) ) { addToResolvedQueue(getCurrentContext(), currentRead); readNext = false; } else { log.trace("getNextEntry[{}]: the data for address does not belong to this stream. Skip.", this, currentRead); readNext = true; } } else { readNext = false; } } catch (TrimmedException te) { if (!getReadOptions().isIgnoreTrim()) { throw te; } log.debug("removeFromQueue[{}]: ignoring trimmed addresses {}", this, te.getTrimmedAddresses()); te.getTrimmedAddresses().forEach(address -> queue.remove(address)); readNext = true; } } while (readNext); return ld; }
Source:
<code>private void validate() { ArgChecker.inOrderNotEqual(startDate, endDate, "startDate", "endDate"); <START> if (indexInterpolated != null && indexInterpolated.equals(index)) { <END> throw new IllegalArgumentException("Interpolation requires two different indices"); } }</code><technical_language>index is guaranteed non-null, index.equals(indexInterpolated) is sufficient</technical_language>

Target:
private void validate() { ArgChecker.inOrderNotEqual(startDate, endDate, "startDate", "endDate"); if (index.equals(indexInterpolated)) { throw new IllegalArgumentException("Interpolation requires two different indices"); } }
Source:
<code>public void calculate(float dataMin, float dataMax) { float min = dataMin; float max = dataMax; if( mCustomAxisMin ) { min = mAxisMinimum; <START> } else if( mUseAutoScaleRestrictionMin ) { <END> min = Math.min( min, mAutoScaleMinRestriction ); } if( mCustomAxisMax ) { max = mAxisMaximum; } else if( mUseAutoScaleRestrictionMax ) { max = Math.max( max, mAutoScaleMaxRestriction ); } float range = Math.abs(max - min); if (range == 0f) { max = max + 1f; min = min - 1f; } if (!mCustomAxisMin) { float bottomSpace = range / 100f * getSpaceBottom(); this.mAxisMinimum = (min - bottomSpace); } if (!mCustomAxisMax) { float topSpace = range / 100f * getSpaceTop(); this.mAxisMaximum = (max + topSpace); } this.mAxisRange = Math.abs(this.mAxisMaximum - this.mAxisMinimum); }</code><technical_language>Indent this, code checks for mUseAutoScale???Restriction if mCustomAxisMin is true, mAxisMinum variable, ways</technical_language>

Target:
public void calculate(float dataMin, float dataMax) { float min = dataMin; float max = dataMax; if( mCustomAxisMin ) { if( mUseAutoScaleRestrictionMin ) { min = Math.min( dataMin, mAxisMinimum ); } else { min = mAxisMinimum; } } if( mCustomAxisMax ) { if( mUseAutoScaleRestrictionMax ) { max = Math.max( max, mAxisMaximum ); } else { max = mAxisMaximum; } } float range = Math.abs(max - min); if (range == 0f) { max = max + 1f; min = min - 1f; } if (!mCustomAxisMin) { float bottomSpace = range / 100f * getSpaceBottom(); this.mAxisMinimum = (min - bottomSpace); } if (!mCustomAxisMax) { float topSpace = range / 100f * getSpaceTop(); this.mAxisMaximum = (max + topSpace); } this.mAxisRange = Math.abs(this.mAxisMaximum - this.mAxisMinimum); }
Source:
<code>public int hashCode() { <START> if (path != null) { <END> return path.hashCode(); } return super.hashCode(); }</code><technical_language>style-nit; braces for simple single stament constructs</technical_language>

Target:
public int hashCode() { if (path != null) return path.hashCode(); return super.hashCode(); }
Source:
<code>public void updateVisibility() { super.updateVisibility(); messageLabel.setVisible(false); <START> subnetTab.setVisible(exportEditor.asCheckBox().getValue()); <END> }</code><technical_language>Again, I understand this is needed? in AbstractNetworkPopupPresenterWidget</technical_language>

Target:
public void updateVisibility() { super.updateVisibility(); messageLabel.setVisible(false); }
Source:
<code>protected boolean canAddVm(ArrayList<String> reasons, Collection<StorageDomain> destStorages) { VmStatic vmStaticFromParams = getParameters().getVmStaticData(); boolean canAddVm = <START> canAddVm(reasons, vmStaticFromParams.getName(), getStoragePoolId(), vmStaticFromParams.getPriority()); <END> if (canAddVm) { List<ValidationError> validationErrors = validateCustomProperties(vmStaticFromParams); if (!validationErrors.isEmpty()) { VmPropertiesUtils.getInstance().handleCustomPropertiesError(validationErrors, reasons); return false; } } if (shouldCheckSpaceInStorageDomains()) { if (!getStoragePoolId().equals(getStoragePoolIdFromSourceImageContainer())) { reasons.add(VdcBllMessages.ACTION_TYPE_FAILED_STORAGE_POOL_NOT_MATCH.toString()); return false; } for (StorageDomain domain : destStorages) { StorageDomainValidator storageDomainValidator = new StorageDomainValidator(domain); if (!validate(storageDomainValidator.isDomainExistAndActive())) { return false; } } if (!validateSpaceRequirements()) { return false; } } return isDedicatedVdsOnSameCluster(vmStaticFromParams); }</code><technical_language>If early return pattern, code: if (!canAddVm(...)) { return false; }</technical_language>

Target:
protected boolean canAddVm(ArrayList<String> reasons, Collection<StorageDomain> destStorages) { VmStatic vmStaticFromParams = getParameters().getVmStaticData(); if (!canAddVm(reasons, vmStaticFromParams.getName(), getStoragePoolId(), vmStaticFromParams.getPriority())) { return false; } List<ValidationError> validationErrors = validateCustomProperties(vmStaticFromParams); if (!validationErrors.isEmpty()) { VmPropertiesUtils.getInstance().handleCustomPropertiesError(validationErrors, reasons); return false; } if (shouldCheckSpaceInStorageDomains()) { if (!getStoragePoolId().equals(getStoragePoolIdFromSourceImageContainer())) { reasons.add(VdcBllMessages.ACTION_TYPE_FAILED_STORAGE_POOL_NOT_MATCH.toString()); return false; } for (StorageDomain domain : destStorages) { StorageDomainValidator storageDomainValidator = new StorageDomainValidator(domain); if (!validate(storageDomainValidator.isDomainExistAndActive())) { return false; } } if (!validateSpaceRequirements()) { return false; } } return isDedicatedVdsOnSameCluster(vmStaticFromParams); }
Source:
<code>public boolean equals(Object o) { if (this == o) { return true; } if (o == null || getClass() != o.getClass()) { return false; } ServerTemplateSelected that = (ServerTemplateSelected) o; <START> return serverTemplate.getId().equals(that.serverTemplate.getId()); <END> }</code><technical_language>In getServerTemplateId() work possibility of serverTemplate null. calling serverTemplate.getId() null check - NPE here? for hashCode below.</technical_language>

Target:
public boolean equals(Object o) { if (this == o) { return true; } if (!(o instanceof ServerTemplateSelected)) { return false; } ServerTemplateSelected that = (ServerTemplateSelected) o; if (getServerTemplate() == null && that.getServerTemplate() == null) { return true; } if (getServerTemplate() != null && that.getServerTemplate() == null) { return false; } if (getServerTemplate() == null && that.getServerTemplate() != null) { return false; } return getServerTemplate().getId().equals(that.getServerTemplate().getId()); }
Source:
<code>public SqmMultiTableMutationStrategy getFallbackSqmMutationStrategy( EntityMappingType rootEntityDescriptor, RuntimeModelCreationContext runtimeModelCreationContext) { return new LocalTemporaryTableStrategy( new IdTable( rootEntityDescriptor, name -> "temp." + name ), <START> () -> new PhysicalIdTableExporter() { <END> @Override protected String getCreateOptions() { return "ignore rollback"; } }, AfterUseAction.DROP, null, runtimeModelCreationContext.getSessionFactory() ); }</code><technical_language>No, TempIdTableExporter</technical_language>

Target:
public SqmMultiTableMutationStrategy getFallbackSqmMutationStrategy( EntityMappingType rootEntityDescriptor, RuntimeModelCreationContext runtimeModelCreationContext) { return new LocalTemporaryTableStrategy( new IdTable( rootEntityDescriptor, name -> "temp." + name ), () -> new TempIdTableExporter( true, this::getTypeName ) { @Override protected String getCreateOptions() { return "ignore rollback"; } }, AfterUseAction.DROP, null, runtimeModelCreationContext.getSessionFactory() ); }
Source:
<code>private String buildMessageForCompositeName(String clusterName, String serviceName) { <START> return String.format("cluster name <%s>, name <%s>", <END> clusterName, serviceName); }</code><technical_language>check this prints messages decent grammar? sure, "cluster name ... service name ..."</technical_language>

Target:
private String buildMessageForCompositeName(String clusterName, String serviceName) { return String.format("cluster name <%s>, service name <%s>", clusterName, serviceName); }
Source:
<code>public void setup() { metacard1 = new MetacardImpl(); results = new ArrayList<>(); results.add(new ResultImpl(metacard1)); catalogFramework = mock(CatalogFramework.class); helper = mock(AdminSourceHelper.class); <START> poller = new AdminPollerServiceBeanTest().new MockedAdminPoller(null, null); <END> poller.setOperationActionProviders(new ArrayList<>()); poller.setReportActionProviders(new ArrayList<>()); poller.setCatalogFramework(catalogFramework); }</code><technical_language>this poller = new AdminPollerServiceBean(null, null);</technical_language>

Target:
public void setup() { metacard1 = new MetacardImpl(); results = new ArrayList<>(); results.add(new ResultImpl(metacard1)); catalogFramework = mock(CatalogFramework.class); helper = mock(AdminSourceHelper.class); poller = new AdminPollerServiceBeanTest().new MockedAdminPoller(null, null, catalogFramework); poller.setOperationActionProviders(new ArrayList<>()); poller.setReportActionProviders(new ArrayList<>()); }
Source:
<code>private void addToBackupList(String latestFilename, BigInteger maxNumBackup) { <START> int noOfBackups = (null != maxNumBackup ? maxNumBackup.intValue() : 0); <END> String filenameToDelete = null; if (backupFileList.size() >= noOfBackups) { filenameToDelete = backupFileList.remove(0); } backupFileList.add(latestFilename); if (filenameToDelete != null) { try { File fileToDelete = new File(filenameToDelete); if (fileToDelete.exists()) { fileToDelete.delete(); } } catch (Exception e) { LOG.warn("Failed to delete backup file {}: {}", filenameToDelete, e.getLocalizedMessage()); LOG.trace("Deletion exception: {}", e.getLocalizedMessage(), e); } } }</code><technical_language>need check in maxNumBackup > 0 based caller logic</technical_language>

Target:
private void addToBackupList(String latestFilename, BigInteger maxNumBackup) { int noOfBackups = maxNumBackup.intValue(); String filenameToDelete = null; if (backupFileList.size() >= noOfBackups) { filenameToDelete = backupFileList.remove(0); } backupFileList.add(latestFilename); if (filenameToDelete != null) { try { File fileToDelete = new File(filenameToDelete); if (fileToDelete.exists()) { fileToDelete.delete(); } } catch (Exception e) { LOG.warn("Failed to delete backup file {}: {}", filenameToDelete, e.getLocalizedMessage()); } } }
Source:
<code>void write(ByteBuffer buf) throws Exception { int toRead = buf.remaining(); if (writableChannel == null || toRead + readSoFar > size) { throw new IllegalStateException("Cannot push data"); } if (beBad) { Exception exception = new Exception("Channel encountered an error"); returnedFuture.done(null, exception); callback.onCompletion(null, exception); return; } writableChannel.write(buf, null).get(); readSoFar += toRead; if (readSoFar == size) { if (sendZeroSizedLastChunk) { sendZeroAndWait(); } returnedFuture.done(size, null); callback.onCompletion(size, null); } else { <START> if (sendZeroSizedLastChunk) { <END> if (random.nextInt(2) % 2 == 0) { sendZeroAndWait(); } } } }</code><technical_language>minor: if (sendZeroSizedLastChunk && random.nextInt(2) % 2 == 0) { sendZeroAndWait(); }</technical_language>

Target:
void write(ByteBuffer buf) throws Exception { int toRead = buf.remaining(); if (writableChannel == null || toRead + readSoFar > size) { throw new IllegalStateException("Cannot push data"); } if (beBad) { Exception exception = new Exception("Channel encountered an error"); returnedFuture.done(null, exception); callback.onCompletion(null, exception); return; } writableChannel.write(buf, null).get(); readSoFar += toRead; if (readSoFar == size) { if (sendZeroSizedBuffers) { sendZeroAndWait(); } returnedFuture.done(size, null); callback.onCompletion(size, null); } else { if (sendZeroSizedBuffers && random.nextInt(2) % 2 == 0) { sendZeroAndWait(); } } }
Source:
<code>private WorkingSetDescriptor getSelectedWorkingSet() { IStructuredSelection selection = typesListViewer.getStructuredSelection(); WorkingSetDescriptor descriptor = null; if (!selection.isEmpty()) { descriptor = (WorkingSetDescriptor) <START> selection.getFirstElement(); <END> } return descriptor; }</code><technical_language>I code if contract od sselection is return null getFirstElement() if sselection is empty, code changed simply return element, if</technical_language>

Target:
private WorkingSetDescriptor getSelectedWorkingSet() { return (WorkingSetDescriptor) typesListViewer.getStructuredSelection().getFirstElement(); }
Source:
<code>public void shouldThrowExceptionOnNullTransformFunction() { <START> final Option<Integer> option = Option.some(1); <END> option.transform(null); }</code><technical_language>inline this</technical_language>

Target:
public void shouldThrowExceptionOnNullTransformFunction() { Option.some(1).transform(null); }
Source:
<code>public IPath getAbsolutePath() { final File repoDir; if (db.isBare()) <START> repoDir = db.getDirectory(); <END> else repoDir = db.getWorkTree(); return new Path(repoDir.getAbsolutePath() + File.separatorChar + path); }</code><technical_language>this sense ? If repository is bare returned path is non-existing Eclipse resources mapping path project imported. do I miss</technical_language>

Target:
public IPath getAbsolutePath() { return new Path(db.getWorkTree().getAbsolutePath() + File.separatorChar + path); }
Source:
<code>public DroolsCorrelationEngine(final String name, final MetricRegistry metricRegistry) { <START> super(); <END> this.m_name = name; final Gauge<Integer> size = () -> { return getMemorySize(); }; metricRegistry.register(MetricRegistry.name(name, "working-memory-size"), size); m_eventsMeter = metricRegistry.meter(MetricRegistry.name(name, "events")); }</code><technical_language>Nit: explicit call super() is needed</technical_language>

Target:
public DroolsCorrelationEngine(final String name, final MetricRegistry metricRegistry) { this.m_name = name; final Gauge<Long> factCount = () -> { return getWorkingMemory().getFactCount(); }; metricRegistry.register(MetricRegistry.name(name, "fact-count"), factCount); final Gauge<Integer> pendingTasksCount = () -> { return getPendingTasksCount(); }; metricRegistry.register(MetricRegistry.name(name, "pending-tasks-count"), pendingTasksCount); m_eventsMeter = metricRegistry.meter(MetricRegistry.name(name, "events")); }
Source:
<code>public Builder addField(FieldSpec fieldSpec) { checkState(kind != Kind.ANNOTATION, "%s %s cannot have fields", kind, name); if (kind == Kind.INTERFACE) { requireExactlyOneOf(EnumSet.of(Modifier.PUBLIC, Modifier.PRIVATE), fieldSpec.modifiers); <START> if (kind == Kind.INTERFACE) { <END> Set<Modifier> check = EnumSet.of(Modifier.STATIC, Modifier.FINAL); checkState(fieldSpec.modifiers.containsAll(check), "%s %s.%s requires modifiers %s", kind, name, fieldSpec.name, check); } } fieldSpecs.add(fieldSpec); return this; }</code><technical_language>This if removed. above</technical_language>

Target:
public Builder addField(FieldSpec fieldSpec) { checkState(kind != Kind.ANNOTATION, "%s %s cannot have fields", kind, name); if (kind == Kind.INTERFACE) { requireExactlyOneOf(EnumSet.of(Modifier.PUBLIC, Modifier.PRIVATE), fieldSpec.modifiers); Set<Modifier> check = EnumSet.of(Modifier.STATIC, Modifier.FINAL); checkState(fieldSpec.modifiers.containsAll(check), "%s %s.%s requires modifiers %s", kind, name, fieldSpec.name, check); } fieldSpecs.add(fieldSpec); return this; }
Source:
<code>I0ItecIZkStateListenerHelixImpl(IZkStateListener listener) { <START> this.listener = listener; <END> }</code><technical_language>nit, _listener = listener is code convention</technical_language>

Target:
I0ItecIZkStateListenerHelixImpl(IZkStateListener listener) { _listener = listener; }
Source:
<code>private void processPendingChannelFlags() { if (!pendingFlagChannels.isEmpty()) { for (int fd = 0; (fd = pendingFlagChannels.nextSetBit(fd)) >= 0; pendingFlagChannels.clear(fd)) { AbstractEpollChannel ch = channels.get(fd); if (ch != null) { try { ch.modifyEvents(); } catch (IOException e) { ch.pipeline().fireExceptionCaught(e); <START> ch.close(ch.voidPromise()); <END> } } } } }</code><technical_language>voidPromise() is allowed for ch.close(...). call ch.close()</technical_language>

Target:
private void processPendingChannelFlags() { if (!pendingFlagChannels.isEmpty()) { for (int fd = 0; (fd = pendingFlagChannels.nextSetBit(fd)) >= 0; pendingFlagChannels.clear(fd)) { AbstractEpollChannel ch = channels.get(fd); if (ch != null) { try { ch.modifyEvents(); } catch (IOException e) { ch.pipeline().fireExceptionCaught(e); ch.close(); } } } } }
Source:
<code>protected void patch(Context context, HttpServletRequest request, String apiCategory, String model, UUID id, Patch patch) throws AuthorizeException, SQLException { final Group group = dsoService.find(context, id); if (group != null && isPatchUnprocessable(context, group)) { <START> throw new UnprocessableEntityException("This group cannot be patched."); <END> } patchDSpaceObject(apiCategory, model, id, patch); }</code><technical_language>more specific error messages here, a scenarios aware of: * A permanent group's name *never* patched. But, description updated. * a Community/Collection group's name (not editable) & description (editable), detached. In case patched. So, a minimum, need errors & checks here. Permanent groups *never* name updates, description updates. Community/Collection groups updated if Community/Collection is deleted, considered "permanent"</technical_language>

Target:
protected void patch(Context context, HttpServletRequest request, String apiCategory, String model, UUID id, Patch patch) throws AuthorizeException, SQLException { patchDSpaceObject(apiCategory, model, id, patch); }
Source:
<code><START> AdapterPatientDiscoveryProxyWebServiceHelper() { <END> }</code><technical_language>follow singleton pattern. add private prevent others call new AdapterPatientDiscoveryProxyWebServiceHelper()</technical_language>

Target:
private AdapterPatientDiscoveryProxyWebServiceHelper() { }
Source:
<code>public String generate() { <START> ThreadLocalRandom random = ThreadLocalRandom.current(); <END> char[] chars = new char[32]; Random64Generator.long2hex(chars, 0, random.nextLong()); Random64Generator.long2hex(chars, 16, random.nextLong()); return new String(chars); }</code><technical_language>final</technical_language>

Target:
public String generate() { ThreadLocalRandom random = ThreadLocalRandom.current(); char[] chars = new char[32]; Chars.toLowerHex(random.nextLong(), chars, 0); Chars.toLowerHex(random.nextLong(), chars, 16); return new String(chars); }
Source:
<code>private void deleteIndexes(List<Index> indexes) { <START> LOG.trace("Deleting index(es) '{}' ...", toString(indexes)); <END> String[] indexNames = toIndexNames(indexes); DeleteIndexRequestBuilder deleteIndexRequest = client.admin().indices().prepareDelete(indexNames); DeleteIndexResponse deleteIndexResponse; try { deleteIndexResponse = deleteIndexRequest.get(); } catch (ResourceNotFoundException e) { LOG.debug("", e); throw new UnknownIndexException(toString(indexes)); } catch (ElasticsearchException e) { LOG.error("", e); throw new IndexException(format("Error deleting index(es) '%s'.", toString(indexes))); } if (!deleteIndexResponse.isAcknowledged()) { throw new IndexException(format("Error deleting index(es) '%s'", toString(indexes))); } LOG.debug("Deleted index(es) '{}'.", toString(indexes)); }</code><technical_language>Add Log level check, toString invoked if log level is trace. (Couple more times in this file)</technical_language>

Target:
private void deleteIndexes(List<Index> indexes) { if (LOG.isTraceEnabled()) { LOG.trace("Deleting index(es) '{}' ...", toString(indexes)); } String[] indexNames = toIndexNames(indexes); DeleteIndexRequestBuilder deleteIndexRequest = client.admin().indices().prepareDelete(indexNames); DeleteIndexResponse deleteIndexResponse; try { deleteIndexResponse = deleteIndexRequest.get(); } catch (ResourceNotFoundException e) { LOG.debug("", e); throw new UnknownIndexException(toString(indexes)); } catch (ElasticsearchException e) { LOG.error("", e); throw new IndexException(format("Error deleting index(es) '%s'.", toString(indexes))); } if (!deleteIndexResponse.isAcknowledged()) { throw new IndexException(format("Error deleting index(es) '%s'", toString(indexes))); } if (LOG.isDebugEnabled()) { LOG.debug("Deleted index(es) '{}'.", toString(indexes)); } }
Source:
<code>public void execute(Context context, ContentProviderClient contentProviderClient, RowDataSnapshot<TaskContract.Instances> rowSnapshot, Uri taskUri) { AppWidgetManager appWidgetManager = AppWidgetManager.getInstance(context); <START> final ComponentName componentName = new ComponentName(context, TaskListWidgetProvider.class); <END> final int[] appWidgetIds = appWidgetManager.getAppWidgetIds(componentName); appWidgetManager.notifyAppWidgetViewDataChanged(appWidgetIds, R.id.task_list_widget_lv); }</code><technical_language>Currently widget providers (mostly for compatibility). Please ensure update both</technical_language>

Target:
public void execute(Context context, ContentProviderClient contentProviderClient, RowDataSnapshot<TaskContract.Instances> rowSnapshot, Uri taskUri) { AppWidgetManager appWidgetManager = AppWidgetManager.getInstance(context); notifyTaskListDataChanged(appWidgetManager, new ComponentName(context, TaskListWidgetProvider.class)); notifyTaskListDataChanged(appWidgetManager, new ComponentName(context, TaskListWidgetProviderLarge.class)); }
Source:
<code>public Response setDynamicConfigs(final CoordinatorDynamicConfig.Builder dynamicConfigBuilder, @HeaderParam(AuditManager.X_DRUID_AUTHOR) @DefaultValue("") final String author, @HeaderParam(AuditManager.X_DRUID_COMMENT) @DefaultValue("") final String comment, @Context HttpServletRequest req ) { try { CoordinatorDynamicConfig current = manager.watch( CoordinatorDynamicConfig.CONFIG_KEY, CoordinatorDynamicConfig.class ).get(); final SetResult setResult = manager.set( CoordinatorDynamicConfig.CONFIG_KEY, current == null ? dynamicConfigBuilder.build() : dynamicConfigBuilder.build(current), new AuditInfo(author, comment, req.getRemoteAddr()) ); if (setResult.isOk()) { return Response.ok().build(); } else { return Response.status(Response.Status.BAD_REQUEST) .entity(ImmutableMap.of("error", setResult.getException())) .build(); } } catch (IllegalArgumentException e) { return Response.status(Response.Status.BAD_REQUEST) <START> .entity(ImmutableMap.<String, Object>of("error", e.getMessage())) <END> .build(); } }</code><technical_language>I a Jetty util for cleaning exceptions floating</technical_language>

Target:
public Response setDynamicConfigs( final CoordinatorDynamicConfig.Builder dynamicConfigBuilder, @HeaderParam(AuditManager.X_DRUID_AUTHOR) @DefaultValue("") final String author, @HeaderParam(AuditManager.X_DRUID_COMMENT) @DefaultValue("") final String comment, @Context HttpServletRequest req ) { try { CoordinatorDynamicConfig current = CoordinatorDynamicConfig.current(manager); final SetResult setResult = manager.set( CoordinatorDynamicConfig.CONFIG_KEY, dynamicConfigBuilder.build(current), new AuditInfo(author, comment, req.getRemoteAddr()) ); if (setResult.isOk()) { return Response.ok().build(); } else { return Response.status(Response.Status.BAD_REQUEST) .entity(ServletResourceUtils.sanitizeException(setResult.getException())) .build(); } } catch (IllegalArgumentException e) { return Response.status(Response.Status.BAD_REQUEST) .entity(ServletResourceUtils.sanitizeException(e)) .build(); } }
Source:
<code>public void encodes() throws IOException { <START> final Identity identity = new Identity.Simple("urn:test:3"); <END> MatcherAssert.assertThat( new String(new CcBase64(new CcPlain()).encode(identity)), Matchers.equalTo("dXJuJTNBdGVzdCUzQTM=") ); }</code><technical_language>@ikhvostenkov this variable inlined</technical_language>

Target:
public void encodes() throws IOException { MatcherAssert.assertThat( new String( new CcBase64(new CcPlain()).encode( new Identity.Simple("urn:test:3") ) ), Matchers.equalTo("dXJuJTNBdGVzdCUzQTM=") ); }
Source:
<code>public List<DetailedQcStatus> getDetailedQcStatuses() throws IOException { List<DetailedQcStatus> list = new ArrayList<>(detailedQcStatusService.getAll()); Collections.sort(list, new Comparator<DetailedQcStatus>() { @Override public int compare(DetailedQcStatus o1, DetailedQcStatus o2) { <START> return o1.getId().compareTo(o2.getId()); <END> } }); return list; }</code><technical_language>sort status (order: true > null > false), noteRequired (false > true) description (A > Z)</technical_language>

Target:
public List<DetailedQcStatus> getDetailedQcStatuses() throws IOException { List<DetailedQcStatus> list = new ArrayList<>(detailedQcStatusService.getAll()); Collections.sort(list, new Comparator<DetailedQcStatus>() { @Override public int compare(DetailedQcStatus o1, DetailedQcStatus o2) { if (o1.getStatus() == null) { return (o2.getStatus() == null ? 0 : -1); } else if (o2.getStatus() == null) { return 1; } else { return o1.getStatus().compareTo(o2.getStatus()); } } }); return list; }
Source:
<code>public void testGetTenantIDFunction() { String[] invalidUrls = new String[] { null, "", "jdbc:phoenix", "jdbc:phoenix;test=true", "jdbc:phoenix:localhost", "localhost:123:/foo-bar", "localhost:123:/foo-bar;TenantId", "localhost:123:/foo-bar;TenantId=" }; for (int i = 0; i < invalidUrls.length; i++) { try { <START> assertEquals( ConnectionInfo.getTenantID(invalidUrls[i]), null); <END> } catch (AssertionError e) { throw new AssertionError("For \"" + invalidUrls[i] + "\": " + e.getMessage()); } } String[] validUrls = new String[] { null, "TenantId=1", "jdbc:phoenix;TenantId=1", "jdbc:phoenix;test=true;TenantId=1", "jdbc:phoenix:localhost;TenantId=1", "localhost:123:/foo-bar;TenantId=1" }; for (int i = 0; i < validUrls.length; i++) { try { assertEquals( ConnectionInfo.getTenantID(validUrls[i]), null); } catch (AssertionError e) { throw new AssertionError("For \"" + invalidUrls[i] + "\": " + e.getMessage()); } } }</code><technical_language>nit: expected is</technical_language>

Target:
public void testGetTenantIDFunction() { String[] invalidUrls = new String[] { null, "", "jdbc:phoenix", "jdbc:phoenix;test=true", "jdbc:phoenix:localhost", "localhost:123:/foo-bar", "localhost:123:/foo-bar;TenantId", "localhost:123:/foo-bar;TenantId=" }; for (int i = 0; i < invalidUrls.length; i++) { assertEquals(null, PhoenixEmbeddedDriver.getTenantIDParser(invalidUrls[i])); } String[] validUrls = new String[] { "TenantId=1", "jdbc:phoenix;TenantId=1", "jdbc:phoenix;test=true;TenantId=1", "jdbc:phoenix:localhost;TenantId=1", "localhost:123:/foo-bar;TenantId=1" }; for (int i = 0; i < validUrls.length; i++) { assertEquals("1", PhoenixEmbeddedDriver.getTenantIDParser(validUrls[i])); } assertEquals("TenantId=", PhoenixEmbeddedDriver.getTenantIDParser("TenantId=TenantId=")); assertEquals("XY", PhoenixEmbeddedDriver.getTenantIDParser("TenantIdXY")); }
Source:
<code>public NIOServerCnxnFactory() { String serverCnxnClassName = System.getProperty(ServerCnxnFactory.ZOOKEEPER_SERVER_CNXN); <START> if ( serverCnxnClassName != null ) { <END> try { Class<? extends NIOServerCnxn> serverCnxnClass = Class.forName(serverCnxnClassName).asSubclass(NIOServerCnxn.class); serverCnxnClassCtr = serverCnxnClass.getConstructor(ZooKeeperServer.class, SocketChannel.class, SelectionKey.class, NIOServerCnxnFactory.class, SelectorThread.class); } catch (ClassNotFoundException e) { LOG.debug("Can not find class for . Using NIOServerCnxn", serverCnxnClassName); } catch (NoSuchMethodException e) { LOG.debug("Can not find constructor for . Using NIOServerCnxn", serverCnxnClassName); } catch (Throwable t) { LOG.debug("Unexpected Exception for dealing with ", serverCnxnClassName); } } }</code><technical_language>nit: extra spaces</technical_language>

Target:
public NIOServerCnxnFactory() { String serverCnxnClassName = System.getProperty(ServerCnxnFactory.ZOOKEEPER_SERVER_CNXN, "org.apache.zookeeper.server.NIOServerCnxn"); try { Class<? extends NIOServerCnxn> serverCnxnClass = Class.forName(serverCnxnClassName).asSubclass(NIOServerCnxn.class); serverCnxnClassCtr = serverCnxnClass.getConstructor(ZooKeeperServer.class, SocketChannel.class, SelectionKey.class, NIOServerCnxnFactory.class, SelectorThread.class); } catch (Throwable t) { throw new RuntimeException("Exception when trying to get constructor for " + serverCnxnClassName, t); } }
Source:
<code>public GetTaskListCommandParameters(Guid vdsId) { super(vdsId); storagePoolId = null; <START> compatbilityVersion = null; <END> }</code><technical_language>here, call this(,,)</technical_language>

Target:
public GetTaskListCommandParameters(Guid vdsId) { super(vdsId); }
Source:
<code>private boolean rowsEqual(List<Object> expectedRow, List<Object> actualRow) { for (int i = 0; i < expectedRow.size(); ++i) { List<Object> expectedValues = expectedRow.get(i) instanceof MultiValue ? ((MultiValue) expectedRow.get(i)).getValues() : ImmutableList.of(expectedRow.get(i)); Object actualValue = actualRow.get(i); <START> if (!valuesEqual(i, expectedValues, actualValue)) { <END> return false; } } return true; }</code><technical_language>isAnyValueEqual</technical_language>

Target:
private boolean rowsEqual(List<Object> expectedRow, List<Object> actualRow) { for (int i = 0; i < expectedRow.size(); ++i) { List<Object> acceptableValues = expectedRow.get(i) instanceof AcceptableValues ? ((AcceptableValues) expectedRow.get(i)).getValues() : singletonList(expectedRow.get(i)); Object actualValue = actualRow.get(i); if (!isAnyValueEqual(i, acceptableValues, actualValue)) { return false; } } return true; }
Source:
<code>public void shouldReturnFactoryFromRepositoryWithDevfile() throws Exception { String githubUrl = "<LINK_0>"; FactoryDto computedFactory = generateDevfileFactory(); when(urlFactoryBuilder.createFactoryFromDevfile(any(RemoteFactoryUrl.class), any(), anyMap())) .thenReturn(Optional.of(computedFactory)); Map<String, String> params = new HashMap<>(); params.put(URL_PARAMETER_NAME, githubUrl); <START> <END> FactoryDto factory = githubFactoryParametersResolver.createFactory(params); assertNotNull(factory.getDevfile()); assertNull(factory.getWorkspace()); verify(urlFactoryBuilder) .createFactoryFromDevfile(factoryUrlArgumentCaptor.capture(), any(), anyMap()); verify(urlFactoryBuilder, never()).buildDefaultDevfile(eq("che")); assertEquals( factoryUrlArgumentCaptor.getValue().devfileFileLocation(), "<LINK_1>"); assertEquals(factoryUrlArgumentCaptor.getValue().getDevfileFilename(), "devfile.yaml"); }</code><technical_language>ImmutableMap.of(</technical_language>

Target:
public void shouldReturnFactoryFromRepositoryWithDevfile() throws Exception { String githubUrl = "<LINK_0>"; FactoryDto computedFactory = generateDevfileFactory(); when(urlFactoryBuilder.createFactoryFromDevfile(any(RemoteFactoryUrl.class), any(), anyMap())) .thenReturn(Optional.of(computedFactory)); Map<String, String> params = ImmutableMap.of(URL_PARAMETER_NAME, githubUrl); FactoryDto factory = githubFactoryParametersResolver.createFactory(params); assertNotNull(factory.getDevfile()); assertNull(factory.getWorkspace()); verify(urlFactoryBuilder) .createFactoryFromDevfile(factoryUrlArgumentCaptor.capture(), any(), anyMap()); verify(urlFactoryBuilder, never()).buildDefaultDevfile(eq("che")); assertEquals( factoryUrlArgumentCaptor.getValue().devfileFileLocation(), "<LINK_1>"); assertEquals(factoryUrlArgumentCaptor.getValue().getDevfileFilename(), "devfile.yaml"); }
Source:
<code>public float getFloat() { if (floats[index] != null) { return floats[index]; } else { <START> return NullHandling.ZERO_FLOAT; <END> } }</code><technical_language>please fix too</technical_language>

Target:
public float getFloat() { if (floats[index] != null) { return floats[index]; } else if (NullHandling.replaceWithDefault()) { return NullHandling.ZERO_FLOAT; } else { throw new IllegalStateException("Should never be invoked when current value is null && SQL-compatible null handling is enabled!"); } }
Source:
<code>public void removeItemWithoutTemplateTest() { NavTreeBuilder builder = new NavTreeBuilder(); LayoutTemplate lt = new LayoutTemplate("lt1"); <START> final String ITEM = "item1"; final String ITEM_TO_REMOVE = "item2"; final String GROUP = "group"; <END> builder.group(GROUP, GROUP, "", false); builder.item(ITEM, ITEM, "", false, NavWorkbenchCtx.perspective(lt.getName())); builder.item(ITEM_TO_REMOVE, ITEM_TO_REMOVE, "", false); builder.endGroup(); NavTree originalTree = builder.build(); assertNotNull(originalTree.getItemById(ITEM_TO_REMOVE)); List<LayoutTemplate> templates = new ArrayList<>(); templates.add(lt); NavTree runtimeTree = runtimeNavigationBuilder.buildRuntimeTree(originalTree, templates); assertEquals(1, runtimeTree.getRootItems().size()); NavGroup group = (NavGroup) runtimeTree.getItemById(GROUP); assertNotNull(group); assertEquals(1, group.getChildren().size()); assertNull(runtimeTree.getItemById(ITEM_TO_REMOVE)); assertNotNull(runtimeTree.getItemById(ITEM)); }</code><technical_language>Consider declaring variables class-scoped constants. I repeatedly declared/used other methods</technical_language>

Target:
public void removeItemWithoutTemplateTest() { NavTreeBuilder builder = new NavTreeBuilder(); LayoutTemplate lt = new LayoutTemplate("lt1"); final String ITEM_TO_REMOVE = "item2"; builder.group(GROUP, GROUP, "", false); builder.item(ITEM, ITEM, "", false, NavWorkbenchCtx.perspective(lt.getName())); builder.item(ITEM_TO_REMOVE, ITEM_TO_REMOVE, "", false); builder.endGroup(); NavTree originalTree = builder.build(); assertNotNull(originalTree.getItemById(ITEM_TO_REMOVE)); List<LayoutTemplate> templates = new ArrayList<>(); templates.add(lt); NavTree runtimeTree = runtimeNavigationBuilder.buildRuntimeTree(originalTree, templates); assertEquals(1, runtimeTree.getRootItems().size()); NavGroup group = (NavGroup) runtimeTree.getItemById(GROUP); assertNotNull(group); assertEquals(1, group.getChildren().size()); assertNull(runtimeTree.getItemById(ITEM_TO_REMOVE)); assertNotNull(runtimeTree.getItemById(ITEM)); }
Source:
<code><START> private void createRasterElevationSource() { <END> ArrayList<String> filePaths = new ArrayList<>(); filePaths.add(Environment.getExternalStorageDirectory() + getString(R.string.raster_package_location)); try { RasterElevationSource rasterElevationSource = new RasterElevationSource(filePaths); rasterElevationSource.addLoadStatusChangedListener(loadStatusChangedEvent -> { if (loadStatusChangedEvent.getNewLoadStatus() == LoadStatus.LOADED) { mSceneView.getScene().getBaseSurface().getElevationSources().add(rasterElevationSource); } else if (loadStatusChangedEvent.getNewLoadStatus() == LoadStatus.FAILED_TO_LOAD) { logErrorToUser(getString(R.string.error_raster_elevation_source_load_failure_message, "")); } }); rasterElevationSource.loadAsync(); } catch (IllegalArgumentException e) { logErrorToUser(getString(R.string.error_raster_elevation_source_load_failure_message, e.getMessage())); } }</code><technical_language>float this permissions methods</technical_language>

Target:
private void createRasterElevationSource() { ArrayList<String> filePaths = new ArrayList<>(); filePaths.add(Environment.getExternalStorageDirectory() + getString(R.string.raster_package_location)); try { RasterElevationSource rasterElevationSource = new RasterElevationSource(filePaths); rasterElevationSource.addLoadStatusChangedListener(loadStatusChangedEvent -> { if (loadStatusChangedEvent.getNewLoadStatus() == LoadStatus.LOADED) { mSceneView.getScene().getBaseSurface().getElevationSources().add(rasterElevationSource); } else if (loadStatusChangedEvent.getNewLoadStatus() == LoadStatus.FAILED_TO_LOAD) { logErrorToUser(getString(R.string.error_raster_elevation_source_load_failure_message, rasterElevationSource.getLoadError())); } }); rasterElevationSource.loadAsync(); } catch (IllegalArgumentException e) { logErrorToUser(getString(R.string.error_raster_elevation_source_load_failure_message, e.getMessage())); } }
Source:
<code>public boolean isClientHealthy(RawClient client) { try { client.ping(); <START> return !client.listBuckets().isEmpty(); <END> } catch (IOException ioe) { return false; } }</code><technical_language>listBuckets() operation kill cluster in production needs trough keys. is a check for health</technical_language>

Target:
public boolean isClientHealthy(RawClient client) { try { client.ping(); return true; } catch (IOException ioe) { return false; } }
Source:
<code>void triggerWindowProcess(List<RowData> inputs, int i, int index) throws Exception { if (windowData.isEmpty()) { if (i >= lowerBoundary) { for (int j = (int) (i - lowerBoundary); j <= i; j++) { windowData.add(inputs.get(j)); } currentBatchCount += lowerBoundary; } else { for (int j = 0; j <= i; j++) { RowData rowData = inputs.get(j); windowData.add(rowData); currentBatchCount++; } Long previousTimestamp; List<RowData> previousData; int length; long remainingDataCount = lowerBoundary - i; ListIterator<Long> iter = sortedTimestamps.listIterator(index); while (remainingDataCount > 0 && iter.hasPrevious()) { previousTimestamp = iter.previous(); previousData = inputState.get(previousTimestamp); length = previousData.size(); ListIterator<RowData> previousDataIter = previousData.listIterator(length); while (previousDataIter.hasPrevious() && remainingDataCount > 0) { windowData.addFirst(previousDataIter.previous()); remainingDataCount--; currentBatchCount++; } } } } else { if (windowData.size() > lowerBoundary) { windowData.pop(); } <START> windowData.add(inputs.get(i)); <END> currentBatchCount += windowData.size(); } for (RowData rowData : windowData) { arrowSerializer.write(getFunctionInput(rowData)); } invokeCurrentBatch(); }</code><technical_language>logic clear outdated state missing in latest PR</technical_language>

Target:
void triggerWindowProcess(List<RowData> inputs, int i, int index) throws Exception { if (windowData.isEmpty()) { if (i >= lowerBoundary) { for (int j = (int) (i - lowerBoundary); j <= i; j++) { windowData.add(inputs.get(j)); } currentBatchCount += lowerBoundary; } else { for (int j = 0; j <= i; j++) { RowData rowData = inputs.get(j); windowData.add(rowData); currentBatchCount++; } Long previousTimestamp; List<RowData> previousData; int length; long remainingDataCount = lowerBoundary - i; ListIterator<Long> iter = sortedTimestamps.listIterator(index); while (remainingDataCount > 0 && iter.hasPrevious()) { previousTimestamp = iter.previous(); previousData = inputState.get(previousTimestamp); length = previousData.size(); ListIterator<RowData> previousDataIter = previousData.listIterator(length); while (previousDataIter.hasPrevious() && remainingDataCount > 0) { windowData.addFirst(previousDataIter.previous()); remainingDataCount--; currentBatchCount++; } } while (iter.hasPrevious()) { previousTimestamp = iter.previous(); inputState.remove(previousTimestamp); } } } else { if (windowData.size() > lowerBoundary) { windowData.pop(); } windowData.add(inputs.get(i)); currentBatchCount += windowData.size(); } for (RowData rowData : windowData) { arrowSerializer.write(getFunctionInput(rowData)); } invokeCurrentBatch(); }
Source:
<code>public Collection<CheckMessage> getIssues(String relativePath, JavaScriptCheck check) { File file = new File(relativePath); List<Issue> issues = new ArrayList<>(); try { JavaScriptCheckContext context = TestUtils.createContext(file); issues = check.scanFile(context); <START> } catch (RecognitionException e) { <END> if ("ParsingErrorCheck".equals(check.getClass().getSimpleName())) { issues .add(new LineIssue(null, e.getLine(), e.getMessage())); } } return getCheckMessages(issues); }</code><technical_language>![CRITICAL](<LINK_1> log rethrow this exception. [![rule](<LINK_2>](<LINK_0></technical_language>

Target:
public Collection<CheckMessage> getIssues(String relativePath, JavaScriptCheck check) { File file = new File(relativePath); List<Issue> issues = new ArrayList<>(); try { JavaScriptCheckContext context = TestUtils.createContext(file); issues = check.scanFile(context); } catch (RecognitionException e) { if (check instanceof ParsingErrorCheck) { issues.add(new LineIssue(check, e.getLine(), e.getMessage())); } } return getCheckMessages(issues); }
Source:
<code>public String getPortalUrlForDeployment(String projectId, String releaseVersion, String environmentId, String tenantId) throws IllegalArgumentException, IOException { <START> String urlFilter = "?take=10" + "&projects=" + projectId + "&environments=" + environmentId; <END> if (tenantId != null && !tenantId.isEmpty()) { urlFilter += "&tenants=" + tenantId; } AuthenticatedWebClient.WebResponse response = webClient.get("deployments" + urlFilter); if (response.isErrorCode()) { throw new IOException(String.format("Code %s - %n%s", response.getCode(), response.getContent())); } JSONObject json = (JSONObject)JSONSerializer.toJSON(response.getContent()); for (Object obj : json.getJSONArray("Items")) { JSONObject jsonObj = (JSONObject)obj; for (Object change : jsonObj.getJSONArray("Changes")) { String version = ((JSONObject)change).getString("Version"); if(version.equals(releaseVersion)) { JSONObject links = jsonObj.getJSONObject("Links"); return links.getString("Web"); } } } return null; }</code><technical_language>this bigger, safe, 25..</technical_language>

Target:
public String getPortalUrlForDeployment(String projectId, String releaseVersion, String environmentId, String tenantId) throws IllegalArgumentException, IOException { String urlFilter = "?take=25" + "&projects=" + projectId + "&environments=" + environmentId; if (tenantId != null && !tenantId.isEmpty()) { urlFilter += "&tenants=" + tenantId; } AuthenticatedWebClient.WebResponse response = webClient.get("deployments" + urlFilter); if (response.isErrorCode()) { throw new IOException(String.format("Code %s - %n%s", response.getCode(), response.getContent())); } JSONObject json = (JSONObject)JSONSerializer.toJSON(response.getContent()); for (Object obj : json.getJSONArray("Items")) { JSONObject jsonObj = (JSONObject)obj; for (Object change : jsonObj.getJSONArray("Changes")) { String version = ((JSONObject)change).getString("Version"); if(version.equals(releaseVersion)) { JSONObject links = jsonObj.getJSONObject("Links"); return links.getString("Web"); } } } return null; }
Source:
<code>public void testTrustExtensionsIsHonored() throws PathNotFoundException { final Configuration conf = new Configuration(); <START> assertEquals(getSamFormat(conf, "misnamedBam.sam"), SAMFormat.SAM); <END> conf.set(AnySAMInputFormat.TRUST_EXTS_PROPERTY, "false"); final SAMFormat result = getSamFormat(conf, "misnamedBam.sam"); assertEquals(result, SAMFormat.BAM); }</code><technical_language>Flip</technical_language>

Target:
public void testTrustExtensionsIsHonored() throws PathNotFoundException { final Configuration conf = new Configuration(); assertEquals(SAMFormat.SAM, getSamFormat(conf, "misnamedBam.sam")); conf.set(AnySAMInputFormat.TRUST_EXTS_PROPERTY, "false"); final SAMFormat result = getSamFormat(conf, "misnamedBam.sam"); assertEquals(SAMFormat.BAM, result); }
Source:
<code>public void buildAndOptimizeTopology(final Properties props) { maybePerformOptimizations(props); final PriorityQueue<StreamsGraphNode> graphNodePriorityQueue = new PriorityQueue<>(5, Comparator.comparing(StreamsGraphNode::buildPriority)); graphNodePriorityQueue.offer(root); while (!graphNodePriorityQueue.isEmpty()) { final StreamsGraphNode streamGraphNode = graphNodePriorityQueue.remove(); if (LOG.isDebugEnabled()) { LOG.debug("Adding nodes to topology child nodes {}", streamGraphNode, streamGraphNode.children()); } if (streamGraphNode.allParentsWrittenToTopology() && !streamGraphNode.hasWrittenToTopology()) { streamGraphNode.writeToTopology(internalTopologyBuilder); streamGraphNode.setHasWrittenToTopology(true); } for (final StreamsGraphNode graphNode : streamGraphNode.children()) { graphNodePriorityQueue.offer(graphNode); } } <START> internalTopologyBuilder.validateCoPartition(); <END> }</code><technical_language>question, do verification in topology builder, is validation code in stage removed</technical_language>

Target:
public void buildAndOptimizeTopology(final Properties props) { maybePerformOptimizations(props); final PriorityQueue<StreamsGraphNode> graphNodePriorityQueue = new PriorityQueue<>(5, Comparator.comparing(StreamsGraphNode::buildPriority)); graphNodePriorityQueue.offer(root); while (!graphNodePriorityQueue.isEmpty()) { final StreamsGraphNode streamGraphNode = graphNodePriorityQueue.remove(); if (LOG.isDebugEnabled()) { LOG.debug("Adding nodes to topology child nodes {}", streamGraphNode, streamGraphNode.children()); } if (streamGraphNode.allParentsWrittenToTopology() && !streamGraphNode.hasWrittenToTopology()) { streamGraphNode.writeToTopology(internalTopologyBuilder); streamGraphNode.setHasWrittenToTopology(true); } for (final StreamsGraphNode graphNode : streamGraphNode.children()) { graphNodePriorityQueue.offer(graphNode); } } internalTopologyBuilder.validateCopartition(); }