auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>protected void executeQueryCommand() { VDSReturnValue returnValue = runVdsCommand(VDSCommandType.GetGlusterVolumeProfileInfo, new GlusterVolumeProfileInfoVDSParameters(getParameters().getClusterId(), getUpServerId(getParameters().getClusterId()), getGlusterVolumeName(getParameters().getVolumeId()), ((GlusterVolumeProfileParameters)getParameters()).isNfs())); <START> getQueryReturnValue().setReturnValue((((GlusterVolumeProfileParameters)getParameters()).isNfs()) ? (GlusterVolumeProfileInfo) returnValue.getReturnValue() : populateBrickNames((GlusterVolumeProfileInfo) returnValue.getReturnValue())); <END> }</code><technical_language>More readable this way? GlusterVolumeProfileInfo profileInfo = (GlusterVolumeProfileInfo) returnValue.getReturnValue(); if ((GlusterVolumeProfileParameters)getParameters()).isNfs()) { populateBrickNames(profileInfo); } getQueryReturnValue().setReturnValue(profileInfo);</technical_language>

Target:
protected void executeQueryCommand() { VDSReturnValue returnValue = runVdsCommand(VDSCommandType.GetGlusterVolumeProfileInfo, new GlusterVolumeProfileInfoVDSParameters(getParameters().getClusterId(), getUpServerId(getParameters().getClusterId()), getGlusterVolumeName(getParameters().getVolumeId()), ((GlusterVolumeProfileParameters)getParameters()).isNfs())); GlusterVolumeProfileInfo profileInfo = (GlusterVolumeProfileInfo) returnValue.getReturnValue(); if (!((GlusterVolumeProfileParameters)getParameters()).isNfs()) { populateBrickNames(profileInfo); } getQueryReturnValue().setReturnValue(profileInfo); }
Source:
<code>public void convertBase64Binary(XSBinaryPointable binaryp, DataOutput dOut) throws SystemException, IOException { baaos.reset(); @SuppressWarnings("resource") Base64OutputStream b64os = new Base64OutputStream(baaos, true); b64os.write(binaryp.getByteArray(), binaryp.getStartOffset() + 2, binaryp.getLength() - 2); dOut.write(returnTag); <START> UTF8StringUtil.getNumBytesToStoreLength(baaos.size()); <END> dOut.write((byte) ((baaos.size() >>> 8) & 0xFF)); dOut.write((byte) ((baaos.size() >>> 0) & 0xFF)); dOut.write(baaos.getByteArray(), 0, baaos.size()); }</code><technical_language>result of function :)</technical_language>

Target:
public void convertBase64Binary(XSBinaryPointable binaryp, DataOutput dOut) throws SystemException, IOException { Base64OutputStream b64os = new Base64OutputStream(baaos, true); b64os.write(binaryp.getByteArray(), binaryp.getStartOffset() + 2, binaryp.getLength() - 2); startString(); sb.appendUtf8Bytes(baaos.getByteArray(), 0, baaos.size()); sendStringDataOutput(dOut); }
Source:
<code>public static void removeExternalUserId() { if (shouldLogUserPrivacyConsentErrorMessageForMethodName("removeExternalUserId()")) return; <START> setExternalUserId("", new ExternalUserIdUpdateHandler() { <END> @Override public void onSuccess(String newExternalUserId) { } @Override public void onFailure(ExternalUserIdError error) { } }); }</code><technical_language>Remove ExternalUserIdUpdateHandler, onSuccess onFailure blank</technical_language>

Target:
public static void removeExternalUserId() { if (shouldLogUserPrivacyConsentErrorMessageForMethodName("removeExternalUserId()")) return; removeExternalUserId(null); }
Source:
<code>public static String getTenantBasedDevPortalContext(String tenantDomain) throws APIManagementException { String context = null; try { Registry registry = ServiceReferenceHolder.getInstance().getRegistryService(). getGovernanceSystemRegistry(); String resourcePath = APIConstants.API_DOMAIN_MAPPINGS.replace("<tenant-id>", tenantDomain); if (registry.resourceExists(resourcePath)) { Resource resource = registry.get(resourcePath); String content = new String((byte[]) resource.getContent(), Charset.defaultCharset()); JSONParser parser = new JSONParser(); JSONObject mappings = (JSONObject) parser.parse(content); if (mappings.containsKey(APIConstants.API_DOMAIN_MAPPINGS_STORE)) { JSONObject storeMapping = (JSONObject) mappings.get(APIConstants.API_DOMAIN_MAPPINGS_STORE); if (storeMapping.containsKey(APIConstants.API_DOMAIN_MAPPINGS_CONTEXT)) { context = (String) storeMapping.get(APIConstants.API_DOMAIN_MAPPINGS_CONTEXT); } else { context = ""; } } } } catch (RegistryException e) { <START> String msg = "Error while retrieving gateway domain mappings from registry"; log.error(msg, e); throw new APIManagementException(msg, e); } catch (ClassCastException e) { String msg = "Invalid JSON found in the gateway tenant domain mappings"; log.error(msg, e); throw new APIManagementException(msg, e); } catch (ParseException e) { String msg = "Malformed JSON found in the gateway tenant domain mappings"; log.error(msg, e); <END> throw new APIManagementException(msg, e); } return context; }</code><technical_language>remove logging errors? Throwing error sufficient</technical_language>

Target:
public static String getTenantBasedDevPortalContext(String tenantDomain) throws APIManagementException { String context = null; try { Registry registry = ServiceReferenceHolder.getInstance().getRegistryService(). getGovernanceSystemRegistry(); String resourcePath = APIConstants.API_DOMAIN_MAPPINGS.replace("<tenant-id>", tenantDomain); if (registry.resourceExists(resourcePath)) { Resource resource = registry.get(resourcePath); String content = new String((byte[]) resource.getContent(), Charset.defaultCharset()); JSONParser parser = new JSONParser(); JSONObject mappings = (JSONObject) parser.parse(content); if (mappings.containsKey(APIConstants.API_DOMAIN_MAPPINGS_STORE)) { JSONObject storeMapping = (JSONObject) mappings.get(APIConstants.API_DOMAIN_MAPPINGS_STORE); if (storeMapping.containsKey(APIConstants.API_DOMAIN_MAPPINGS_CONTEXT)) { context = (String) storeMapping.get(APIConstants.API_DOMAIN_MAPPINGS_CONTEXT); } else { context = ""; } } } } catch (RegistryException e) { String msg = "Error while retrieving gateway domain mappings from registry"; throw new APIManagementException(msg, e); } catch (ClassCastException e) { String msg = "Invalid JSON found in the gateway tenant domain mappings"; throw new APIManagementException(msg, e); } catch (ParseException e) { String msg = "Malformed JSON found in the gateway tenant domain mappings"; throw new APIManagementException(msg, e); } return context; }
Source:
<code>private CopyCmdAnswer performResignature(DataObject dataObj, HostVO hostVO, boolean keepGrantedAccess) { long storagePoolId = dataObj.getDataStore().getId(); DataStore dataStore = _dataStoreMgr.getDataStore(storagePoolId, DataStoreRole.Primary); Map<String, String> details = getDetails(dataObj); ResignatureCommand command = new ResignatureCommand(details); ResignatureAnswer answer = null; try { _volumeService.grantAccess(dataObj, hostVO, dataStore); answer = (ResignatureAnswer)_agentMgr.send(hostVO.getId(), command); } catch (Exception ex) { keepGrantedAccess = false; throw new CloudRuntimeException(ex.getMessage()); } finally { if (keepGrantedAccess == false) { try { _volumeService.revokeAccess(dataObj, hostVO, dataStore); } catch (Exception ex) { <START> s_logger.debug(ex.getMessage(), ex); <END> } } } if (answer == null || !answer.getResult()) { final String errMsg; if (answer != null && answer.getDetails() != null && !answer.getDetails().isEmpty()) { errMsg = answer.getDetails(); } else { errMsg = "Unable to perform resignature operation"; } throw new CloudRuntimeException(errMsg); } VolumeObjectTO newVolume = new VolumeObjectTO(); newVolume.setSize(answer.getSize()); newVolume.setPath(answer.getPath()); newVolume.setFormat(answer.getFormat()); return new CopyCmdAnswer(newVolume); }</code><technical_language>1. log this exception WARN? operator want if failed rollback operation. 2. Please add some contextual information exception message explain failed rollback access datastore host</technical_language>

Target:
private CopyCmdAnswer performResignature(DataObject dataObj, HostVO hostVO, boolean keepGrantedAccess) { long storagePoolId = dataObj.getDataStore().getId(); DataStore dataStore = dataStoreMgr.getDataStore(storagePoolId, DataStoreRole.Primary); Map<String, String> details = getDetails(dataObj); ResignatureCommand command = new ResignatureCommand(details); ResignatureAnswer answer = null; try { _volumeService.grantAccess(dataObj, hostVO, dataStore); answer = (ResignatureAnswer)_agentMgr.send(hostVO.getId(), command); } catch (CloudRuntimeException | AgentUnavailableException | OperationTimedoutException ex) { keepGrantedAccess = false; String msg = "Failed to resign the DataObject with the following ID: " + dataObj.getId(); LOGGER.warn(msg, ex); throw new CloudRuntimeException(msg + ex.getMessage()); } finally { if (keepGrantedAccess == false) { _volumeService.revokeAccess(dataObj, hostVO, dataStore); } } if (answer == null || !answer.getResult()) { final String errMsg; if (answer != null && answer.getDetails() != null && !answer.getDetails().isEmpty()) { errMsg = answer.getDetails(); } else { errMsg = "Unable to perform resignature operation in 'StorageSystemDataMotionStrategy.performResignature'"; } throw new CloudRuntimeException(errMsg); } VolumeObjectTO newVolume = new VolumeObjectTO(); newVolume.setSize(answer.getSize()); newVolume.setPath(answer.getPath()); newVolume.setFormat(answer.getFormat()); return new CopyCmdAnswer(newVolume); }
Source:
<code>private static <T> T staticRequest( ApiResource.RequestMethod method, String url, Map<String, Object> params, Class<T> clazz, ApiResource.RequestType type, RequestOptions options) throws StripeException { <START> long requestStart = System.currentTimeMillis(); <END> StripeResponse response = rawRequest(method, url, params, type, options); long requestDurationMS = System.currentTimeMillis() - requestStart; int responseCode = response.code(); String responseBody = response.body(); String requestId = response.requestId(); if (responseCode < 200 || responseCode >= 300) { handleApiError(responseBody, responseCode, requestId); } T resource = null; try { resource = ApiResource.GSON.fromJson(responseBody, clazz); } catch (JsonSyntaxException e) { raiseMalformedJsonError(responseBody, responseCode, requestId); } if (resource instanceof StripeObject) { StripeObject obj = (StripeObject)resource; obj.setLastResponse(response); } if (Stripe.enableTelemetry && prevRequestMetrics.size() < MAX_REQUEST_METRICS_BUFFER_SIZE) { prevRequestMetrics.add(new RequestMetrics(requestId, requestDurationMS)); } return resource; }</code><technical_language>Some of spacing in this file doens't line up, spacing is correct? (if intellij standard formatter work fine)</technical_language>

Target:
private static <T> T staticRequest( ApiResource.RequestMethod method, String url, Map<String, Object> params, Class<T> clazz, ApiResource.RequestType type, RequestOptions options) throws StripeException { long requestStartMs = System.currentTimeMillis(); StripeResponse response = rawRequest(method, url, params, type, options); long requestDurationMs = System.currentTimeMillis() - requestStartMs; int responseCode = response.code(); String responseBody = response.body(); String requestId = response.requestId(); if (responseCode < 200 || responseCode >= 300) { handleApiError(responseBody, responseCode, requestId); } T resource = null; try { resource = ApiResource.GSON.fromJson(responseBody, clazz); } catch (JsonSyntaxException e) { raiseMalformedJsonError(responseBody, responseCode, requestId); } if (resource instanceof StripeObject) { StripeObject obj = (StripeObject)resource; obj.setLastResponse(response); } if (Stripe.enableTelemetry && prevRequestMetrics.size() < MAX_REQUEST_METRICS_BUFFER_SIZE) { prevRequestMetrics.add(new RequestMetrics(requestId, requestDurationMs)); } return resource; }
Source:
<code>private boolean isSinglePartitioned(RexProgram program, RexNode rexNode, List<Integer> partitionKeys) { if (rexNode instanceof RexCall) { RexCall rexCall = (RexCall) rexNode; SqlKind sqlKind = rexCall.getOperator().getKind(); switch (sqlKind) { <START> case EQUALS: <END> return isSinglePartitioned(program, rexCall.getOperands().get(0), partitionKeys) || isSinglePartitioned(program, rexCall.getOperands().get(1), partitionKeys); case AND: return isSinglePartitioned(program, rexCall.getOperands().get(0), partitionKeys) || isSinglePartitioned(program, rexCall.getOperands().get(1), partitionKeys); case OR: return isSinglePartitioned(program, rexCall.getOperands().get(0), partitionKeys) && isSinglePartitioned(program, rexCall.getOperands().get(1), partitionKeys); default: return false; } } if (rexNode instanceof RexInputRef) { return partitionKeys.contains(((RexInputRef) rexNode).getIndex()); } if (rexNode instanceof RexLocalRef) { return isSinglePartitioned(program, program.getExprList().get(((RexLocalRef) rexNode).getIndex()), partitionKeys); } return false; }</code><technical_language>if list some examples: -- PARTITION TABLE t COLUMN a; SELECT ... t a = 0; SELECT ... t a = b; SELECT ... t a = 0 b < 0; SELECT ... t a = 0 b < 0; this method static</technical_language>

Target:
private static boolean isSinglePartitioned(RexProgram program, RexNode rexNode, List<Integer> partitionKeys) { if (rexNode instanceof RexCall) { RexCall rexCall = (RexCall) rexNode; SqlKind sqlKind = rexCall.getOperator().getKind(); switch (sqlKind) { case EQUALS: return (isSinglePartitioned(program, rexCall.getOperands().get(0), partitionKeys) && !hasTableColumn(program, rexCall.getOperands().get(1))) || (isSinglePartitioned(program, rexCall.getOperands().get(1), partitionKeys) && !hasTableColumn(program, rexCall.getOperands().get(0))); case AND: return isSinglePartitioned(program, rexCall.getOperands().get(0), partitionKeys) || isSinglePartitioned(program, rexCall.getOperands().get(1), partitionKeys); case OR: return false; case NOT: return isComplementSinglePartitioned(program, rexCall.getOperands().get(0), partitionKeys); default: return false; } } else if (rexNode instanceof RexInputRef) { return partitionKeys.contains(((RexInputRef) rexNode).getIndex()); } else if (rexNode instanceof RexLocalRef) { return isSinglePartitioned(program, program.getExprList().get(((RexLocalRef) rexNode).getIndex()), partitionKeys); } else { return false; } }
Source:
<code>public void validate(ValidationContext validationContext) { <START> if (isEnabled() && !validationContext.systemEnvironment().get(SystemEnvironment.INBUILT_LDAP_PASSWORD_AUTH_ENABLED)) { <END> configErrors.add("base", "'passwordFile' tag has been deprecated in favour of bundled PasswordFile plugin. Use that instead."); } }</code><technical_language>method inbuiltLdapPasswordAuthEnabled SystemEnvironment</technical_language>

Target:
public void validate(ValidationContext validationContext) { if (isEnabled() && !validationContext.systemEnvironment().inbuiltLdapPasswordAuthEnabled()) { configErrors.add("base", "'passwordFile' tag has been deprecated in favour of bundled PasswordFile plugin. Use that instead."); } }
Source:
<code>public static DeadLetterQueueReporter createAndSetup(Map<String, Object> adminProps, ConnectorTaskId id, SinkConnectorConfig sinkConfig, Map<String, Object> producerProps, ErrorHandlingMetrics errorHandlingMetrics) { <START> KafkaProducer<byte[], byte[]> dlqProducer = setUpTopicAndProducer( adminProps, producerProps, sinkConfig, DLQ_NUM_DESIRED_PARTITIONS ); <END> return new DeadLetterQueueReporter(dlqProducer, sinkConfig, id, errorHandlingMetrics); }</code><technical_language>need this change, do we? try minimize changes existing code</technical_language>

Target:
public static DeadLetterQueueReporter createAndSetup(Map<String, Object> adminProps, ConnectorTaskId id, SinkConnectorConfig sinkConfig, Map<String, Object> producerProps, ErrorHandlingMetrics errorHandlingMetrics) { String topic = sinkConfig.dlqTopicName(); try (Admin admin = Admin.create(adminProps)) { if (!admin.listTopics().names().get().contains(topic)) { log.error("Topic doesn't exist. Will attempt to create topic.", topic); NewTopic schemaTopicRequest = new NewTopic(topic, DLQ_NUM_DESIRED_PARTITIONS, sinkConfig.dlqTopicReplicationFactor()); admin.createTopics(singleton(schemaTopicRequest)).all().get(); } } catch (InterruptedException e) { throw new ConnectException("Could not initialize dead letter queue with topic=" + topic, e); } catch (ExecutionException e) { if (!(e.getCause() instanceof TopicExistsException)) { throw new ConnectException("Could not initialize dead letter queue with topic=" + topic, e); } } KafkaProducer<byte[], byte[]> dlqProducer = new KafkaProducer<>(producerProps); return new DeadLetterQueueReporter(dlqProducer, sinkConfig, id, errorHandlingMetrics); }
Source:
<code>private void executeFirstChunkedQuery() throws SQLException { String firstQuery = _chunkedQueryManager.generateFirstQuery(_sourceQuery, new ArrayList<>(_chunkingKeys.keySet()), _chunkSize, _maxChunkIndex, _chunkIndex); _firstStmt = _connection.prepareStatement(firstQuery); _firstStmt.setFetchSize(_fetchSize); _firstStmt.setQueryTimeout(_queryTimeoutSecs); <START> long timeStart = System.currentTimeMillis(); <END> _queryResultSet = _firstStmt.executeQuery(); _metrics.updateQueryExecutionTime(System.currentTimeMillis() - timeStart); _metrics.updateQueryExecutionRate(); }</code><technical_language>nit: a private method takes a preparedStatement invokes executeQuery() it. put metrics logic in method</technical_language>

Target:
private void executeFirstChunkedQuery() throws SQLException { String firstQuery = _chunkedQueryManager.generateFirstQuery(_sourceQuery, new ArrayList<>(_chunkingKeys.keySet()), _rowCountLimit, _maxChunkIndex, _chunkIndex); _firstStmt = _connection.prepareStatement(firstQuery); _firstStmt.setFetchSize(_fetchSize); _firstStmt.setQueryTimeout(_queryTimeoutSecs); executeChunkedQuery(_firstStmt); }
Source:
<code>protected static int getLoginPort(Container container) { if (container.getNetworkSettings() != null) { Map<String, List<Map<String, String>>> ports = container.getNetworkSettings().getPorts(); if (ports != null && ports.containsKey("22/tcp")) { return Integer.parseInt(getOnlyElement(ports.get("22/tcp")).get("HostPort")); } } else if (container.getPorts() != null) { for (Port port : container.getPorts()) { if (port.getPrivatePort() == 22) { return port.getPublicPort(); } } } <START> return -1; <END> }</code><technical_language>Hmm Optional is -1 days. NoSuchElementException</technical_language>

Target:
protected static int getLoginPort(Container container) { if (container.getNetworkSettings() != null) { Map<String, List<Map<String, String>>> ports = container.getNetworkSettings().getPorts(); if (ports != null && ports.containsKey("22/tcp")) { return Integer.parseInt(getOnlyElement(ports.get("22/tcp")).get("HostPort")); } } else if (container.getPorts() != null) { for (Port port : container.getPorts()) { if (port.getPrivatePort() == 22) { return port.getPublicPort(); } } } return NO_LOGIN_PORT; }
Source:
<code>public Aggregator factorize(final ColumnSelectorFactory columnFactory) { <START> getCompiledScript(); <END> return new JavaScriptAggregator( fieldNames.stream().map(columnFactory::makeColumnValueSelector).collect(Collectors.toList()), compiledScript ); }</code><technical_language>This method instead of compiledScript a lines below, point. in other places getCompiledScript() is called</technical_language>

Target:
public Aggregator factorize(final ColumnSelectorFactory columnFactory) { JavaScriptAggregator.ScriptAggregator compiledScript = getCompiledScript(); return new JavaScriptAggregator( fieldNames.stream().map(columnFactory::makeColumnValueSelector).collect(Collectors.toList()), compiledScript ); }
Source:
<code>public void testShouldReturnEnvironmentFromLocalBranchInCaseRemoteDeleted() throws Exception { JGitConfigServerTestData testData = JGitConfigServerTestData .prepareClonedGitRepository(TestConfiguration.class); String branchToDelete = "branchToDelete"; testData.getServerGit().getGit().branchCreate().setName(branchToDelete).call(); Environment environment = testData.getRepository().findOne("bar", "staging", "branchToDelete"); assertNotNull(environment); testData.getServerGit().getGit().branchDelete().setBranchNames(branchToDelete).call(); <START> testData.getRepository().findOne("bar", "staging", "branchToDelete"); <END> assertNotNull(environment); }</code><technical_language>result set environment</technical_language>

Target:
public void testShouldReturnEnvironmentFromLocalBranchInCaseRemoteDeleted() throws Exception { JGitConfigServerTestData testData = JGitConfigServerTestData .prepareClonedGitRepository(TestConfiguration.class); String branchToDelete = "branchToDelete"; testData.getServerGit().getGit().branchCreate().setName(branchToDelete).call(); Environment environment = testData.getRepository().findOne("bar", "staging", "branchToDelete"); assertNotNull(environment); testData.getServerGit().getGit().branchDelete().setBranchNames(branchToDelete).call(); environment = testData.getRepository().findOne("bar", "staging", "branchToDelete"); assertNotNull(environment); }
Source:
<code>private void hotPlugUnplugCpus() { if (canHotPlugUnplugCpu()) { VDSReturnValue vdsReturnValue = runVdsCommand(VDSCommandType.SetNumberOfCpus, new SetNumberOfCpusVDSCommand.Params( getVm().getRunOnVds(), getVm().getId(), <START> getParameters().getVm().getNumOfCpus())); <END> AuditLogableBase logable = new AuditLogableBase(); logable.setVmId(getVmId()); logable.addCustomValue("numberOfCpus", String.valueOf(getParameters().getVm().getNumOfCpus())); logable.addCustomValue("ErrorMessage", vdsReturnValue.getVdsError().getMessage()); boolean hotPlugUnplugSucceeded = vdsReturnValue.getSucceeded() && vdsReturnValue.getVdsError().getCode() == VdcBllErrors.Done; AuditLogDirector.log(logable, hotPlugUnplugSucceeded ? AuditLogType.SET_NUMBER_OF_CPUS : AuditLogType.FAILED_SET_NUMBER_OF_CPUS); if (!hotPlugUnplugSucceeded) { newVmStatic.setNumOfSockets(oldVm.getNumOfSockets()); } } }</code><technical_language>i guess newVmStatic (and well)</technical_language>

Target:
private void hotPlugUnplugCpus() { if (getVm().getNumOfCpus() != getParameters().getVm().getNumOfCpus()) { VDSReturnValue returnValue = runVdsCommand(VDSCommandType.SetNumberOfCpus, new SetNumberOfCpusVDSCommand.Params( getVm().getRunOnVds(), getVm().getId(), getParameters().getVm().getNumOfCpus())); AuditLogableBase logable = new AuditLogableBase(); logable.setVmId(getVmId()); logable.addCustomValue("numberOfCpus", String.valueOf(getParameters().getVm().getNumOfCpus())); logable.addCustomValue("ErrorMessage", returnValue.getExceptionString() + " " + returnValue.getVdsError().getMessage()); AuditLogDirector.log(logable, returnValue.getSucceeded() ? AuditLogType.SET_NUMBER_OF_CPUS : AuditLogType.FAILED_SET_NUMBER_OF_CPUS); if (!returnValue.getSucceeded()) { newVmStatic.setNumOfSockets(oldVm.getNumOfSockets()); } } }
Source:
<code>private void checkLeaderAndTimelockBlocks() { if (leader().isPresent()) { Preconditions.checkState(areTimeAndLockConfigsAbsent(), "If the leader block is present, then the lock and timestamp server blocks must both be absent."); Preconditions.checkState(!timelock().isPresent(), "If the leader block is present, then the timelock block must be absent."); } <START> checkLeaderBlockHasAtLeastOneServerIfPresent(leader()); <END> if (timelock().isPresent()) { Preconditions.checkState(areTimeAndLockConfigsAbsent(), "If the timelock block is present, then the lock and timestamp blocks must both be absent."); } }</code><technical_language>This method appears once, instead of adding method, simply add body of if block above: Preconditions.checkState(!leaderConfig.leaders().isEmpty(), "Leader config server."));</technical_language>

Target:
private void checkLeaderAndTimelockBlocks() { if (leader().isPresent()) { Preconditions.checkState(areTimeAndLockConfigsAbsent(), "If the leader block is present, then the lock and timestamp server blocks must both be absent."); Preconditions.checkState(!timelock().isPresent(), "If the leader block is present, then the timelock block must be absent."); Preconditions.checkState(!leader().get().leaders().isEmpty(), "Leader config must have at least one server."); } if (timelock().isPresent()) { Preconditions.checkState(areTimeAndLockConfigsAbsent(), "If the timelock block is present, then the lock and timestamp blocks must both be absent."); } }
Source:
<code>public Object doWork() { final AllelicPanelOfNormals allelicPoN = allelicPoNFile != null ? AllelicPanelOfNormals.read(allelicPoNFile) : AllelicPanelOfNormals.EMPTY_PON; final AllelicCountCollection acc = new AllelicCountCollection(snpCountsFile); final AlleleFractionSegmenter alleleFractionSegmenter = new AlleleFractionSegmenter(initialNumAFStates, acc, allelicPoN); <START> final ReadCountCollection counts; <END> try { counts = ReadCountCollectionUtils.parse(new File(coverageFile)); } catch (final IOException ex) { throw new UserException.BadInput("could not read input file"); } final List<Double> coverage = Doubles.asList(counts.counts().getColumn(0)); final List<SimpleInterval> targets = counts.targets().stream().map(Target::getInterval).collect(Collectors.toList()); final CopyRatioSegmenter copyRatioSegmenter = new CopyRatioSegmenter(initialNumCRStates, targets, coverage); final JointAFCRSegmenter jointSegmenter = JointAFCRSegmenter.createJointSegmenter(copyRatioSegmenter, alleleFractionSegmenter); final List<Pair<SimpleInterval, AFCRHiddenState>> segmentation = jointSegmenter.findSegments(); final List<ACNVModeledSegment> segments = segmentation.stream().map(pair -> new ACNVModeledSegment(pair.getLeft(), errorlessPosterior(pair.getRight().getLog2CopyRatio()), errorlessPosterior(pair.getRight().getMinorAlleleFraction()))) .collect(Collectors.toList()); SegmentUtils.writeACNVModeledSegmentFile(outputSegmentsFile, segments, new Genome(counts, acc.getCounts())); return "SUCCESS"; }</code><technical_language>I this is load a ReadCountCollection file...why ReadCountCollection a constructor takes a File this exception handling for us? @asmirnov239 @mbabadi consider in future refactorings</technical_language>

Target:
public Object doWork() { ParamUtils.isPositive(initialNumAFStates, "Must have at least one allele-fraction state."); ParamUtils.isPositive(initialNumCRStates, "Must have at least one copy-ratio state."); final AllelicPanelOfNormals allelicPoN = allelicPoNFile != null ? AllelicPanelOfNormals.read(allelicPoNFile) : AllelicPanelOfNormals.EMPTY_PON; final AllelicCountCollection acc = new AllelicCountCollection(snpCountsFile); final ReadCountCollection rcc; try { rcc = ReadCountCollectionUtils.parse(new File(coverageFile)); } catch (final IOException ex) { throw new UserException.BadInput("could not read input file"); } final JointAFCRSegmenter jointSegmenter = JointAFCRSegmenter.createJointSegmenter(initialNumCRStates, rcc, initialNumAFStates, acc, allelicPoN); final List<Pair<SimpleInterval, AFCRHiddenState>> segmentation = jointSegmenter.findSegments(); final List<ACNVModeledSegment> segments = segmentation.stream().map(pair -> new ACNVModeledSegment(pair.getLeft(), errorlessPosterior(pair.getRight().getLog2CopyRatio()), errorlessPosterior(pair.getRight().getMinorAlleleFraction()))) .collect(Collectors.toList()); SegmentUtils.writeACNVModeledSegmentFile(outputSegmentsFile, segments, new Genome(rcc, acc.getCounts())); return "SUCCESS"; }
Source:
<code>private void parse() throws IOException { stream.mark(0); try { final byte[] curIdentifier = new byte[IDENTIFIER_LENGTH]; for (int i = 0; i < curIdentifier.length; i++) { curIdentifier[i] = parseByte(); } final String identifier = new String(curIdentifier); setIdentifier(identifier); if (!identifier.equals(GTC_IDENTIFIER)) { throw new PicardException("Invalid identifier '" + identifier + "' for GTC file"); } setFileVersion(parseByte()); setNumberOfEntries(parseInt()); for (InfiniumFileTOC toc : getTableOfContents()) { stream.reset(); readData(stream, toc); } if (numCalls == 0) { numCalls = aaCalls + abCalls + bbCalls; } } finally { stream.close(); } <START> if (allNormalizationIds != null) { <END> normalizeIntensities(); } }</code><technical_language>true</technical_language>

Target:
private void parse() throws IOException { stream.mark(0); try { final byte[] curIdentifier = new byte[IDENTIFIER_LENGTH]; for (int i = 0; i < curIdentifier.length; i++) { curIdentifier[i] = parseByte(); } final String identifier = new String(curIdentifier); setIdentifier(identifier); if (!identifier.equals(GTC_IDENTIFIER)) { throw new PicardException("Invalid identifier '" + identifier + "' for GTC file"); } setFileVersion(parseByte()); setNumberOfEntries(parseInt()); for (InfiniumFileTOC toc : getTableOfContents()) { stream.reset(); readData(stream, toc); } if (numCalls == 0) { numCalls = aaCalls + abCalls + bbCalls; } } finally { stream.close(); } normalizeIntensities(); }
Source:
<code>protected void updateOSValue(final Integer selectedOsId) { Cluster cluster = getModel().getSelectedCluster(); <START> if (cluster != null) { <END> List<Integer> vmOsValues = AsyncDataProvider.getInstance().getOsIds(cluster.getArchitecture()); if (selectedOsId == null || !vmOsValues.contains(selectedOsId)) { updateOSValues(); return; } getModel().getOSType().setSelectedItem(selectedOsId); } }</code><technical_language>Please, change this statement early return form. standard code. objection I currently. If fix now, able merge code branching 4.3</technical_language>

Target:
protected void updateOSValue(final Integer selectedOsId) { Cluster cluster = getModel().getSelectedCluster(); if (cluster == null) { return; } List<Integer> vmOsValues = AsyncDataProvider.getInstance().getOsIds(cluster.getArchitecture()); if (selectedOsId == null || !vmOsValues.contains(selectedOsId)) { updateOSValues(); return; } getModel().getOSType().setSelectedItem(selectedOsId); }
Source:
<code>public boolean isClosed() { <START> return m_closed == true; <END> }</code><technical_language>return m_closed</technical_language>

Target:
public boolean isClosed() { return m_closed; }
Source:
<code>private static void appendAllGroups(Matcher matcher, int findIndex, StringBuilder capture) { for (int i = 1; i <= matcher.groupCount(); i++) { capture.append("") .append(findIndex) .append(".") .append(i) .append(" ") .append(buildGroup(matcher.start(i), matcher.end(i))) <START> .append("\t") <END> .append(matcher.group(i)) .append("\n"); } }</code><technical_language>This moved method well, need pass matcher index (for appendFind index 0)</technical_language>

Target:
private static void appendAllGroups(Matcher matcher, int findIndex, StringBuilder capture) { for (int i = 1; i <= matcher.groupCount(); i++) { capture.append("").append(findIndex).append(".").append(i).append(" "); buildGroup(capture, matcher, i); } }
Source:
<code>public static void sortDetailedCoursesByCreationDate(List<CourseDetailsBundle> courses) { <START> courses.sort(Comparator.comparing((CourseDetailsBundle obj) -> obj.course.createdAt)); <END> }</code><technical_language>careful, this is reversed</technical_language>

Target:
public static void sortDetailedCoursesByCreationDate(List<CourseDetailsBundle> courses) { courses.sort(Comparator.comparing((CourseDetailsBundle obj) -> obj.course.createdAt).reversed()); }
Source:
<code>public void execute(JobExecutionContext arg0) throws JobExecutionException { <START> LOGGER.info("*****Chart Data Generator is startin now.*****"); <END> SpringBeanAutowiringSupport.processInjectionBasedOnCurrentContext(this); List<CertifiedProductFlatSearchResult> certifiedProducts = certifiedProductSearchDAO.getAllCertifiedProducts(); LOGGER.info("Certified Product Count: " + certifiedProducts.size()); analyzeSed(certifiedProducts); analyzeProducts(certifiedProducts); analyzeDevelopers(certifiedProducts); analyzeListingCounts(certifiedProducts); analyzeNonconformity(); LOGGER.info("*****Chart Data Generator is done running.*****"); }</code><technical_language>startin -> starting</technical_language>

Target:
public void execute(JobExecutionContext arg0) throws JobExecutionException { LOGGER.info("*****Chart Data Generator is starting now.*****"); SpringBeanAutowiringSupport.processInjectionBasedOnCurrentContext(this); List<CertifiedProductFlatSearchResult> certifiedProducts = certifiedProductSearchDAO.getAllCertifiedProducts(); LOGGER.info("Certified Product Count: " + certifiedProducts.size()); analyzeSed(certifiedProducts); analyzeProducts(certifiedProducts); analyzeDevelopers(certifiedProducts); analyzeListingCounts(certifiedProducts); analyzeNonconformity(); LOGGER.info("*****Chart Data Generator is done running.*****"); }
Source:
<code>private static void createStream(String streamName, int shardCount, AmazonKinesisClient kinesisClient) { CreateStreamRequest createStreamRequest = new CreateStreamRequest(); createStreamRequest.setStreamName(streamName); createStreamRequest.setShardCount(shardCount); kinesisClient.createStream(createStreamRequest); <START> LOG.info("Stream " + streamName + " created"); <END> }</code><technical_language>string format syntax Logger instead of concatenation</technical_language>

Target:
private static void createStream(String streamName, int shardCount, AmazonKinesisClient kinesisClient) { CreateStreamRequest createStreamRequest = new CreateStreamRequest(); createStreamRequest.setStreamName(streamName); createStreamRequest.setShardCount(shardCount); kinesisClient.createStream(createStreamRequest); LOG.info("Stream is being created", streamName); }
Source:
<code>protected void publishHelp() { printHelpHeader(); out("Usage: dockstore " + getEntryType().toLowerCase() + " publish --help"); out(" dockstore " + getEntryType().toLowerCase() + " publish"); out(" dockstore " + getEntryType().toLowerCase() + " publish [parameters]"); out(" dockstore " + getEntryType().toLowerCase() + " publish --unpub [parameters]"); out(""); out("Description:"); out(" Publish/unpublish a registered " + getEntryType() + "."); out(" No arguments will list the current and potential " + getEntryType() + "s to share."); out("Required Parameter(s):"); out(" --entry <entry> Complete " + getEntryType() + " path in the Dockstore (ex. quay.io/collaboratory/seqware-bwa-workflow)"); out("Optional Parameter(s):"); out(" --entryname <New" + getEntryType() + "name> " + "New name to give the tool specified by --entry. " <START> + "This will register and publish a new 'copy' of the tool with the given name."); <END> printHelpFooter(); }</code><technical_language>is "copy" in quotes? I be</technical_language>

Target:
protected void publishHelp() { printHelpHeader(); out("Usage: dockstore " + getEntryType().toLowerCase() + " publish --help"); out(" dockstore " + getEntryType().toLowerCase() + " publish"); out(" dockstore " + getEntryType().toLowerCase() + " publish [parameters]"); out(" dockstore " + getEntryType().toLowerCase() + " publish --unpub [parameters]"); out(""); out("Description:"); out(" Publish/unpublish a registered " + getEntryType() + "."); out(" No arguments will list the current and potential " + getEntryType() + "s to share."); out("Required Parameter(s):"); out(" --entry <entry> Complete " + getEntryType() + " path in the Dockstore (ex. quay.io/collaboratory/seqware-bwa-workflow)"); out("Optional Parameter(s):"); out(" --entryname <New" + getEntryType() + "name> " + "New name to give the tool specified by --entry. " + "This will register and publish a new copy of the tool with the given name."); printHelpFooter(); }
Source:
<code>public void testSimpleTransactionInAnotherThreadContinued() throws IgniteCheckedException { withAllIsolationsAndConcurrencies(new IgniteClosure<Object, Void>() { @Override public Void apply(Object o) { try { simpleTransactionInAnotherThreadContinued(); <START> } <END> catch (Exception ignore) { assert false : "Impossible situation"; } return null; } }); }</code><technical_language>rename "ignore" "ignored" Please fail() <LINK_0></technical_language>

Target:
public void testSimpleTransactionInAnotherThreadContinued() throws Exception { runWithAllIsolations(new IgniteCallable<Void>() { @Override public Void call() throws Exception { simpleTransactionInAnotherThreadContinued(); return null; } }); }
Source:
<code>public static void showMessage(Activity activity, int resId, int duration) { <START> showMessage(findBestView(activity), activity.getString(resId), duration); <END> }</code><technical_language>call function instead</technical_language>

Target:
public static void showMessage(Activity activity, int resId, int duration) { showMessage(activity, activity.getString(resId), duration); }
Source:
<code>public FrenchSouthWestProvider(final String authorization) { <START> super(NetworkId.ALPC, authorization); <END> setTimeZone("Europe/Paris"); }</code><technical_language>ALPC? match new id in NetworkId.java</technical_language>

Target:
public FrenchSouthWestProvider(final String authorization) { super(NetworkId.FRENCHSOUTHWEST, authorization); setTimeZone("Europe/Paris"); }
Source:
<code>protected List<String> getCodeFields() { ParameterizedType p = (ParameterizedType) this.getClass().getGenericSuperclass(); <START> System.out.println(p.getActualTypeArguments()[0]); <END> return getProperties(); }</code><technical_language>is this printing stdout</technical_language>

Target:
protected List<String> getCodeFields() { return getProperties(); }
Source:
<code>protected void ExecuteIrsBrokerCommand() { <START> P params = getParameters(); <END> String spUUIDstr = params.getStoragePoolId().toString(); String targetVersion = params.getCompatibilityVersion(); log.info("-- UpgradeStoragePoolVDSCommand::ExecuteIrsBrokerCommand: calling 'upgradeStoragePool' with two new parameters: poolVersion and UUID"); log.infoFormat("-- upgradeStoragePool parameters:\r\n" + "spUUID={0}\r\n" + "version={1}\r\n", spUUIDstr, targetVersion); status = getIrsProxy().upgradeStoragePool(spUUIDstr, targetVersion); ProceedProxyReturnValue(); }</code><technical_language>Consider adding "final" modifier local variables: final P params = ... final String spUUIDstr = ... difference in this case, catch some subtle errors in general</technical_language>

Target:
protected void ExecuteIrsBrokerCommand() { final P params = getParameters(); final String storagePoolId = params.getStoragePoolId().toString(); final String targetVersion = params.getCompatibilityVersion(); log.info("Upgrading storage pool \"" + storagePoolId + "\" to version \"" + targetVersion + "\""); status = getIrsProxy().upgradeStoragePool(storagePoolId, targetVersion); ProceedProxyReturnValue(); }
Source:
<code>private void parseCommentGeneric(String comment) { if (comment == null) return; String[] lines = comment.split("\n"); if (lines.length > 1) { StringBuilder buf = new StringBuilder(comment.length()); for (String line : lines) { int eq = line.indexOf('='); if (eq != -1) { String key = line.substring(0, eq).trim(); String value = line.substring(eq + 1).trim(); addGlobalMeta(key, value); } else if (!line.startsWith("[")) { buf.append(line); <START> buf.append("\n"); <END> } } description = buf.toString(); addGlobalMeta("Comment", description); } }</code><technical_language>If performance is issue buf.append('\n'); here</technical_language>

Target:
private void parseCommentGeneric(String comment) { if (comment == null) return; String[] lines = comment.split("\n"); if (lines.length > 1) { StringBuilder buf = new StringBuilder(comment.length()); for (String line : lines) { int eq = line.indexOf('='); if (eq != -1) { String key = line.substring(0, eq).trim(); String value = line.substring(eq + 1).trim(); addGlobalMeta(key, value); } else if (!line.startsWith("[")) { buf.append(line); buf.append('\n'); } } description = buf.toString(); addGlobalMeta("Comment", description); } }
Source:
<code>public void testEvtOpenPublisherEnum() throws Exception { Winevt.EVT_RPC_LOGIN login = new Winevt.EVT_RPC_LOGIN("localhost", null, null, null, Winevt.EVT_RPC_LOGIN_FLAGS.EvtRpcLoginAuthDefault); EVT_HANDLE session = null; EVT_HANDLE publisherEnumHandle = null; List<String> publisherList = new ArrayList<String>(); try { session = Wevtapi.INSTANCE.EvtOpenSession(Winevt.EVT_LOGIN_CLASS.EvtRpcLogin, login, 0, 0); if (session == null) { throw new Win32Exception(Kernel32.INSTANCE.GetLastError()); } publisherEnumHandle = Wevtapi.INSTANCE.EvtOpenPublisherEnum(session, 0); if (publisherEnumHandle == null) { throw new Win32Exception(Kernel32.INSTANCE.GetLastError()); } <START> Memory buff; <END> while (true) { try { buff = WevtapiUtil.EvtNextPublisherId(publisherEnumHandle); } catch (Win32Exception e) { if (e.getErrorCode() == WinError.ERROR_NO_MORE_ITEMS) { break; } else { throw e; } } publisherList.add(buff.getWideString(0)); } assertThat(publisherList.size() > 0, is(true)); } finally { if (publisherEnumHandle != null) { Wevtapi.INSTANCE.EvtClose(publisherEnumHandle); } if (session != null) { Wevtapi.INSTANCE.EvtClose(session); } } }</code><technical_language>suggested changes WevtapiUtil get a String directly, in case this needs adjusting</technical_language>

Target:
public void testEvtOpenPublisherEnum() throws Exception { Winevt.EVT_RPC_LOGIN login = new Winevt.EVT_RPC_LOGIN("localhost", null, null, null, Winevt.EVT_RPC_LOGIN_FLAGS.EvtRpcLoginAuthDefault); EVT_HANDLE session = null; EVT_HANDLE publisherEnumHandle = null; List<String> publisherList = new ArrayList<String>(); try { session = Wevtapi.INSTANCE.EvtOpenSession(Winevt.EVT_LOGIN_CLASS.EvtRpcLogin, login, 0, 0); if (session == null) { throw new Win32Exception(Kernel32.INSTANCE.GetLastError()); } publisherEnumHandle = Wevtapi.INSTANCE.EvtOpenPublisherEnum(session, 0); if (publisherEnumHandle == null) { throw new Win32Exception(Kernel32.INSTANCE.GetLastError()); } String providerName; while (true) { try { providerName = WevtapiUtil.EvtNextPublisherId(publisherEnumHandle); } catch (Win32Exception e) { if (e.getErrorCode() == WinError.ERROR_NO_MORE_ITEMS) { break; } else { throw e; } } publisherList.add(providerName); } assertThat(publisherList.size() > 0, is(true)); } finally { if (publisherEnumHandle != null) { Wevtapi.INSTANCE.EvtClose(publisherEnumHandle); } if (session != null) { Wevtapi.INSTANCE.EvtClose(session); } } }
Source:
<code>private static Map<String, String> getMergedEnvironment(LaunchConfiguration configuration) throws IOException { <START> Map<String, String> currentEnv = new HashMap<String, String>(EnvironmentUtils.getProcEnvironment()); <END> currentEnv.putAll(configuration.getEnvironment()); return currentEnv; }</code><technical_language>order matter here</technical_language>

Target:
private static Map<String, String> getMergedEnvironment(LaunchConfiguration configuration) throws IOException { Map<String, String> currentEnv = EnvironmentUtils.getProcEnvironment(); currentEnv.putAll(configuration.getEnvironment()); return currentEnv; }
Source:
<code>void upgradeTo(ChannelHandlerContext ctx, FullHttpResponse upgradeResponse) throws Exception; } private final SourceCodec sourceCodec; private final UpgradeCodec upgradeCodec; private boolean upgradeRequested; public HttpClientUpgradeHandler(SourceCodec sourceCodec, UpgradeCodec upgradeCodec, int maxContentLength) { super(maxContentLength); ObjectUtil.checkNotNull(sourceCodec, "sourceCodec"); ObjectUtil.checkNotNull(upgradeCodec, "upgradeCodec"); this.sourceCodec = sourceCodec; <START> this.upgradeCodec = upgradeCodec; <END> }</code><technical_language>nit: merge lines checkNotNull return argument</technical_language>

Target:
void upgradeTo(ChannelHandlerContext ctx, FullHttpResponse upgradeResponse) throws Exception; } private final SourceCodec sourceCodec; private final UpgradeCodec upgradeCodec; private boolean upgradeRequested; public HttpClientUpgradeHandler(SourceCodec sourceCodec, UpgradeCodec upgradeCodec, int maxContentLength) { super(maxContentLength); this.sourceCodec = ObjectUtil.checkNotNull(sourceCodec, "sourceCodec"); this.upgradeCodec = ObjectUtil.checkNotNull(upgradeCodec, "upgradeCodec"); }
Source:
<code>public TreacherousUrge(UUID ownerId, CardSetInfo setInfo) { super(ownerId,setInfo,new CardType[]{CardType.INSTANT},"{4}{B}"); <START> this.subtype.add(SubType.ARCANE); <END> this.getSpellAbility().addEffect(new TreacherousUrgeEffect()); this.getSpellAbility().addTarget(new TargetOpponent()); }</code><technical_language>This needs removed</technical_language>

Target:
public TreacherousUrge(UUID ownerId, CardSetInfo setInfo) { super(ownerId,setInfo,new CardType[]{CardType.INSTANT},"{4}{B}"); this.getSpellAbility().addEffect(new TreacherousUrgeEffect()); this.getSpellAbility().addTarget(new TargetOpponent()); }
Source:
<code>public HeapKeyedStateBackend<K> build() throws BackendBuildingException { Map<String, StateTable<K, ?, ?>> registeredKVStates = new HashMap<>(); Map<String, HeapPriorityQueueSnapshotRestoreWrapper> registeredPQStates = new HashMap<>(); CloseableRegistry cancelStreamRegistryForBackend = new CloseableRegistry(); HeapSnapshotStrategy<K> snapshotStrategy = initSnapshotStrategy( asynchronousSnapshots, registeredKVStates, registeredPQStates, cancelStreamRegistryForBackend); InternalKeyContext<K> keyContext = new InternalKeyContextImpl<>( keyGroupRange, numberOfKeyGroups ); HeapRestoreOperation<K> restoreOperation = new HeapRestoreOperation<>( restoreStateHandles, keySerializerProvider, userCodeClassLoader, registeredKVStates, registeredPQStates, cancelStreamRegistry, priorityQueueSetFactory, keyGroupRange, numberOfKeyGroups, snapshotStrategy, keyContext); try { restoreOperation.restore(); <START> logger.info("Finish to build heap keyed state-backend."); <END> } catch (Exception e) { throw new BackendBuildingException("Failed when trying to restore heap backend", e); } return new HeapKeyedStateBackend<>( kvStateRegistry, keySerializerProvider.currentSchemaSerializer(), userCodeClassLoader, executionConfig, ttlTimeProvider, cancelStreamRegistryForBackend, keyGroupCompressionDecorator, registeredKVStates, registeredPQStates, localRecoveryConfig, priorityQueueSetFactory, snapshotStrategy, keyContext); }</code><technical_language>suggestion logger.info("Finished build heap keyed state-backend.");</technical_language>

Target:
public HeapKeyedStateBackend<K> build() throws BackendBuildingException { Map<String, StateTable<K, ?, ?>> registeredKVStates = new HashMap<>(); Map<String, HeapPriorityQueueSnapshotRestoreWrapper> registeredPQStates = new HashMap<>(); CloseableRegistry cancelStreamRegistryForBackend = new CloseableRegistry(); HeapSnapshotStrategy<K> snapshotStrategy = initSnapshotStrategy( asynchronousSnapshots, registeredKVStates, registeredPQStates, cancelStreamRegistryForBackend); InternalKeyContext<K> keyContext = new InternalKeyContextImpl<>( keyGroupRange, numberOfKeyGroups ); HeapRestoreOperation<K> restoreOperation = new HeapRestoreOperation<>( restoreStateHandles, keySerializerProvider, userCodeClassLoader, registeredKVStates, registeredPQStates, cancelStreamRegistry, priorityQueueSetFactory, keyGroupRange, numberOfKeyGroups, snapshotStrategy, keyContext); try { restoreOperation.restore(); logger.info("Finished to build heap keyed state-backend."); } catch (Exception e) { throw new BackendBuildingException("Failed when trying to restore heap backend", e); } return new HeapKeyedStateBackend<>( kvStateRegistry, keySerializerProvider.currentSchemaSerializer(), userCodeClassLoader, executionConfig, ttlTimeProvider, cancelStreamRegistryForBackend, keyGroupCompressionDecorator, registeredKVStates, registeredPQStates, localRecoveryConfig, priorityQueueSetFactory, snapshotStrategy, keyContext); }