auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void executeThroughCli_reload() throws IOException { Assume.assumeTrue(client.version().greaterThanOrEqualTo(ServerVersion.VERSION_4_1_0) <START> || client.version().lessThan(ServerVersion.VERSION_2_0_0)); <END> super.executeThroughCli_reload(); }</code><technical_language>I thought problem is WildFly 10.0.0.Final, assumeTrue(client.version() != ServerVersion.VERSION_4_0_0) enough, no</technical_language>

Target:
public void executeThroughCli_reload() throws IOException { Assume.assumeTrue(client.version() != ServerVersion.VERSION_4_0_0); super.executeThroughCli_reload(); }
Source:
<code>public void addView(TextAnnotation textAnnotation) throws AnnotatorException { String[] tokens = textAnnotation.getTokens(); String[] tags = {"DT","NN","IN","DT","NN","VBD","IN","NN","."}; View posView = new View(ViewNames.POS,"POS-annotator",textAnnotation,1.0); textAnnotation.addView(ViewNames.POS,posView); for(int i=0;i<tokens.length;i++){ <START> posView.addConstituent(new Constituent(tags[i],ViewNames.POS,textAnnotation,i,i+1)); <END> } }</code><technical_language>I is a problem here. tokens.length (the length of loop) anything; right? size of tags is fixed here. tags[someRandomIndex], tags[i]</technical_language>

Target:
public void addView(TextAnnotation textAnnotation) throws AnnotatorException { String[] tokens = textAnnotation.getTokens(); String[] tags = {"DT","NN","IN","DT","NN","VBD","IN","NN","."}; View posView = new View(ViewNames.POS,"POS-annotator",textAnnotation,1.0); textAnnotation.addView(ViewNames.POS,posView); for(int i=0;i<tags.length;i++){ posView.addConstituent(new Constituent(tags[i],ViewNames.POS,textAnnotation,i,i+1)); } }
Source:
<code>private void copyHdfsToTd() { <START> _logger.info("Executing " + HdfsToTeradataJobRunnerMain.class.getSimpleName() + " with params: "+ _params); <END> TeradataExportTool.main(_params.toTdchParams()); }</code><technical_language>String.format</technical_language>

Target:
private void copyHdfsToTd() { _logger.info(String.format("Executing %s with params: %s", HdfsToTeradataJobRunnerMain.class.getSimpleName(), _params)); TeradataExportTool.main(_params.toTdchParams()); }
Source:
<code>public static void bind(ServiceBinder binder) { <START> binder.bind(NashornReactRenderEngine.class, NashornReactRenderEngineImplementation.class); <END> }</code><technical_language>NashornReactRenderEngineImplementation is thread-safe, bound a per-thread scope</technical_language>

Target:
public static void bind(ServiceBinder binder) { binder.bind(ReactRenderEngine.class, NashornReactRenderEngineImplementation.class); binder.bind(ModuleLoaderFactory.class, ModuleLoaderFactoryImplementation.class); }
Source:
<code>public static CharSequence generateMarkdown(Context c, String message){ Bypass bypass = new Bypass(c); CharSequence mSequence = bypass.markdownToSpannable(convertCharacterEncodings(message)); <START> System.out.println("2915 msequence: " + mSequence); <END> System.out.print("2915 msequence: " + mSequence); return MarkupUtil.trimTrailingWhitespace(mSequence); }</code><technical_language>sysout</technical_language>

Target:
public static CharSequence generateMarkdown(Context c, String message){ Bypass bypass = new Bypass(c); CharSequence mSequence = bypass.markdownToSpannable(convertCharacterEncodings(message)); return MarkupUtil.trimTrailingWhitespace(mSequence); }
Source:
<code>public NetworkFilterParameterEditor() { initWidget(WidgetUiBinder.uiBinder.createAndBindUi(this)); ViewIdHandler.idHandler.generateAndSetIds(this); driver.initialize(this); <START> ((Element) nameEditor.getElement().getChild(0)).addClassName(style.elementStyle()); <END> ((Element) nameEditor.getElement().getChild(1)).addClassName(style.elementStyle()); ((Element) valueEditor.getElement().getChild(0)).addClassName(style.elementStyle()); ((Element) valueEditor.getElement().getChild(1)).addClassName(style.elementStyle()); }</code><technical_language>I a achieve this hacking AbstractValidatedWidgetWithLabel / making assumptions child positions of 0 1 calling addClassName those. For example, methods available: addLabelStyleName addWrapperStyleName addContentWidgetStyleName addContentWidgetContainerStyleName</technical_language>

Target:
public NetworkFilterParameterEditor() { initWidget(WidgetUiBinder.uiBinder.createAndBindUi(this)); ViewIdHandler.idHandler.generateAndSetIds(this); driver.initialize(this); nameEditor.addLabelStyleName(style.elementStyle()); nameEditor.getContentWidgetContainer().getParent().addStyleName(style.elementStyle()); valueEditor.addLabelStyleName(style.elementStyle()); valueEditor.getContentWidgetContainer().getParent().addStyleName(style.elementStyle()); }
Source:
<code><START> public void testSinglePlanSingleContext() { <END> final List<String> plan = Lists.newArrayList(CONTEXT_A_ID); final int evalNum = 3; final List<EventHandler<ActiveContext>> activeContextHandlerList = new ArrayList<>(); for (int i = 1; i < plan.size(); i++) { activeContextHandlerList.add(new SubmitContextToACHandler(plan.get(i))); } activeContextHandlerList.add(new LastACHandler()); finishedCounter = new CountDownLatch(evalNum); evaluatorManager.allocateEvaluators(evalNum, new SubmitContextToAEHandler(plan.get(0), plan), activeContextHandlerList); try { finishedCounter.await(); } catch (final InterruptedException e) { throw new RuntimeException(e); } for (final String evalId : evalIdToActualContextIdStack.keySet()) { final List<String> expectedContextIdStack = evalIdToExpectedContextIdStack.get(evalId); final List<String> actualContextIdStack = evalIdToActualContextIdStack.get(evalId); assertEquals(expectedContextIdStack, actualContextIdStack); } }</code><technical_language>test self-contained. Please change shared (mutable) variables local variables in test</technical_language>

Target:
public void testSinglePlanSingleContext() { final List<String> plan = Lists.newArrayList(CONTEXT_A_ID); final int evalNum = 3; final Tuple2<EventHandler<AllocatedEvaluator>, List<EventHandler<ActiveContext>>> handlers = getHandlersFromPlan(plan); finishedCounter = new CountDownLatch(evalNum); evaluatorManager.allocateEvaluators(evalNum, handlers.getT1(), handlers.getT2()); try { finishedCounter.await(TEST_TIMEOUT_MILLIS, TimeUnit.MILLISECONDS); } catch (final InterruptedException e) { throw new RuntimeException(e); } for (final String evalId : evalIdToActualContextIdStack.keySet()) { final List<String> expectedContextIdStack = evalIdToExpectedContextIdStack.get(evalId); final List<String> actualContextIdStack = evalIdToActualContextIdStack.get(evalId); assertEquals(expectedContextIdStack, actualContextIdStack); } }
Source:
<code>public void hideAll() { Collection<Phantom> phantoms = fPhantomRegistry.values(); Iterable<Phantom> visiblePhantoms = Iterables.filter(phantoms, new Predicate<Phantom>() { public boolean apply(Phantom phantom) { return phantom.getFigure().getParent() != null; } }); <START> for (Phantom phantom : visiblePhantoms) { <END> handleDecorator(phantom, false, true); } }</code><technical_language>this is typical case Predicate is wise. code more readable filtering in foreach loop: for (Phantom phantom : fPhantomRegistry.values()) { if (phantom.getFigures.getParent() != null) { handleDecorator(phantom, false, true); } }</technical_language>

Target:
public void hideAll() { for (Phantom phantom : fPhantomRegistry.values()) { if (phantom.getFigure().getParent() != null) { handleDecorator(phantom, false, true); } } }
Source:
<code>private void setWhereClause(ParseQuery query) { <START> if (city != null && !city.equalsIgnoreCase("Any") && !city.equalsIgnoreCase("")) <END> query.whereMatches("cityName", city, "i"); if (group != null && !group.equalsIgnoreCase("Any") && !group.equalsIgnoreCase("")) query.whereMatches("groupType", group, "i"); if (season != null && !season.equalsIgnoreCase("Any") && !season.equalsIgnoreCase("")) query.whereMatches("travelSeason", season, "i"); }</code><technical_language>This simplified this method for fields: <LINK_0></technical_language>

Target:
private void setWhereClause(ParseQuery query) { if (!TextUtils.isEmpty(city) && !city.equalsIgnoreCase("Any")) query.whereMatches("cityName", city, "i"); if (!TextUtils.isEmpty(group) && !group.equalsIgnoreCase("Any")) query.whereMatches("groupType", group, "i"); if (!TextUtils.isEmpty(season) && !season.equalsIgnoreCase("Any")) query.whereMatches("travelSeason", season, "i"); }
Source:
<code>public Map<Long, V> getAll() { rwLock.readLock().lock(); try { <START> return (Map<Long, V>) ((TreeMap) subDataMap).clone(); <END> } finally { rwLock.readLock().unlock(); } }</code><technical_language>TreeMap is a raw type. a critical hole all, compiler complains it</technical_language>

Target:
public Map<Long, V> getAll() { rwLock.readLock().lock(); try { return (Map<Long, V>) ((TreeMap<Long, V>) subDataMap).clone(); } finally { rwLock.readLock().unlock(); } }
Source:
<code>protected static void assertSqlSimpleData(Ignite node, String sql, int expSize) { try { SqlQuery qry = new SqlQuery(typeName(ValueClass.class), sql).setArgs(SQL_ARG_1); List<Cache.Entry<BinaryObject, BinaryObject>> res = node.cache(CACHE_NAME).withKeepBinary().query(qry).getAll(); Set<Long> ids = new HashSet<>(); for (Cache.Entry<BinaryObject, BinaryObject> entry : res) { long id = entry.getKey().field(FIELD_KEY); long field1 = entry.getValue().field(FIELD_NAME_1_ESCAPED); long field2 = entry.getValue().field(FIELD_NAME_2_ESCAPED); assertTrue(field1 >= SQL_ARG_1); assertEquals(id, field1); assertEquals(id, field2); assertTrue(ids.add(id)); } assertEquals("Size mismatch [node=" + node.name() + ", exp=" + expSize + ", actual=" + res.size() + ", ids=" + ids + ']', expSize, res.size()); } catch (Exception e) { for (Throwable th = e; th != null; th = th.getCause()) { if (th.getMessage().contains(CONC_DESTROY_MSG)) <START> return; <END> } if (X.cause(e, QueryRetryException.class) == null) throw e; } }</code><technical_language>Is related a problem in JIRA ticket</technical_language>

Target:
protected static void assertSqlSimpleData(Ignite node, String sql, int expSize) { try { SqlQuery qry = new SqlQuery(typeName(ValueClass.class), sql).setArgs(SQL_ARG_1); List<Cache.Entry<BinaryObject, BinaryObject>> res = node.cache(CACHE_NAME).withKeepBinary().query(qry).getAll(); Set<Long> ids = new HashSet<>(); for (Cache.Entry<BinaryObject, BinaryObject> entry : res) { long id = entry.getKey().field(FIELD_KEY); long field1 = entry.getValue().field(FIELD_NAME_1_ESCAPED); long field2 = entry.getValue().field(FIELD_NAME_2_ESCAPED); assertTrue(field1 >= SQL_ARG_1); assertEquals(id, field1); assertEquals(id, field2); assertTrue(ids.add(id)); } assertEquals("Size mismatch [node=" + node.name() + ", exp=" + expSize + ", actual=" + res.size() + ", ids=" + ids + ']', expSize, res.size()); } catch (Exception e) { if (X.cause(e, QueryRetryException.class) == null) throw e; } }
Source:
<code>public void testDisputeList() throws StripeException, InterruptedException { createDisputedCharge(100).getDispute(); createDisputedCharge(150).getDispute(); <START> createDisputedCharge(300).getDispute(); <END> Map<String, Object> listParams = new HashMap<String, Object>(); listParams.put("count", 3); List<Dispute> disputes = Dispute.all(listParams).getData(); assertEquals(3, disputes.size()); }</code><technical_language>I safe remove</technical_language>

Target:
public void testDisputeList() throws StripeException, InterruptedException { Map<String, Object> listParams = new HashMap<String, Object>(); listParams.put("count", 3); List<Dispute> disputes = Dispute.all(listParams).getData(); assertEquals(3, disputes.size()); }
Source:
<code>public String getUrl() { return url; <START> } <END></code><technical_language>deprecate of getURL getUrl, deprecated call non-deprecated one</technical_language>

Target:
public String getUrl() { return getURL(); }
Source:
<code>public String add(Network network) { <START> if (isReadOnly()){ throw new EngineException(EngineError.NO_IMPLEMENTATION); } <END> return super.add(network); }</code><technical_language>please extract method testProviderIsNotReadOnly remove duplicates</technical_language>

Target:
public String add(Network network) { testProviderIsNotReadOnly(); return super.add(network); }
Source:
<code>public boolean execute() { <START> for (int i = 0; i < maxAttempts; i++) { final Boolean result = taskRunner.get(); if (result != null && result) { return true; } final boolean wasLastAttempt = i + 1 == maxAttempts; if (!wasLastAttempt) { threadSleeper.accept(fixedBackOff); } } return false; <END> }</code><technical_language>suggestion for (int i = 1; i < maxAttempts; i++) { if (taskRunner.get()) { return true; } threadSleeper.accept(fixedBackOff); } return taskRunner.get(); Is nicer IMO</technical_language>

Target:
public boolean execute() { for (int i = 1; i < maxAttempts; i++) { if (taskRunner.getAsBoolean()) { return true; } threadSleeper.accept(fixedBackOff); } return taskRunner.getAsBoolean(); }
Source:
<code>public Map<String, String> convertProperties(String properties) { Map<String, String> map = new LinkedHashMap<String, String>(); if (StringUtils.isNotEmpty(properties)) { String keyValuePairs[] = SEMICOLON_PATTERN.split(properties); for (String keyValuePairStr : keyValuePairs) { String[] pairParts = keyValuePairStr.split(KEY_VALUE_DELIMETER, 2); <START> String key = pairParts[0]; <END> String value = pairParts[1]; map.put(key, value); } } return map; }</code><technical_language>need for additional check if propertyParts.length is ok</technical_language>

Target:
public Map<String, String> convertProperties(String properties) { if (syntaxErrorInProperties(properties)) { throw new IllegalArgumentException("Invalid properties syntax!"); } Map<String, String> map = new LinkedHashMap<String, String>(); if (StringUtils.isNotEmpty(properties)) { String keyValuePairs[] = SEMICOLON_PATTERN.split(properties); for (String keyValuePairStr : keyValuePairs) { String[] pairParts = keyValuePairStr.split(KEY_VALUE_DELIMETER, 2); String key = pairParts[0]; String value = pairParts[1]; map.put(key, value); } } return map; }
Source:
<code>public boolean processOwnsAnchorXML(Process process) throws IOException { <START> URI yearFile = FileService.this.createAnchorFile(FileService.this.getMetadataFilePath(process, false, false)); <END> return FileService.this.fileExist(yearFile); }</code><technical_language>suggestion URI yearFile = createAnchorFile(FileService.this.getMetadataFilePath(process, false, false));</technical_language>

Target:
public boolean processOwnsAnchorXML(Process process) throws IOException { URI yearFile = createAnchorFile(getMetadataFilePath(process, false, false)); return fileExist(yearFile); }
Source:
<code>protected void executeVmCommand() { lockImageInDb(); List<UpdateVmDiskParameters.Phase> phaseList = new ArrayList<>(); ImagesHandler.setDiskAlias(getParameters().getDiskInfo(), getVm()); if (resizeDiskImageRequested()) { phaseList.add(UpdateVmDiskParameters.Phase.EXTEND_DISK); } <START> if (Objects.equals(getOldDisk().getDiskStorageType(), DiskStorageType.IMAGE) && amendDiskRequested()) { <END> phaseList.add(UpdateVmDiskParameters.Phase.AMEND_DISK); } phaseList.add(UpdateVmDiskParameters.Phase.UPDATE_DISK); getParameters().setDiskUpdatePhases(phaseList); setSucceeded(true); }</code><technical_language>for comparison enum ==</technical_language>

Target:
protected void executeVmCommand() { lockImageInDb(); List<UpdateVmDiskParameters.Phase> phaseList = new ArrayList<>(); ImagesHandler.setDiskAlias(getParameters().getDiskInfo(), getVm()); if (resizeDiskImageRequested()) { phaseList.add(UpdateVmDiskParameters.Phase.EXTEND_DISK); } if (getOldDisk().getDiskStorageType() == DiskStorageType.IMAGE && amendDiskRequested()) { phaseList.add(UpdateVmDiskParameters.Phase.AMEND_DISK); } phaseList.add(UpdateVmDiskParameters.Phase.UPDATE_DISK); getParameters().setDiskUpdatePhases(phaseList); setSucceeded(true); }
Source:
<code>public String getDescription() { <START> return de.danoeh.antennapod.core.util.TextUtils.getNormalizedText(description); <END> }</code><technical_language>Parsing html done of Feed.java. Feed is model represents actual feed. display feed belong feed itself, in opinion</technical_language>

Target:
public String getDescription() { return description; }
Source:
<code>public Set<String> getSegmentVersions(final String dataSource) { ArrayList<DataSegment> segments; try { StatusResponseHolder response = makeRequest(HttpMethod.GET, getFullSegmentsURL(dataSource)); segments = jsonMapper.readValue( response.getContent(), new TypeReference<List<DataSegment>>() { } ); } catch (Exception e) { <START> throw Throwables.propagate(e); <END> } return segments.stream().map(s -> s.getVersion()).collect(Collectors.toSet()); }</code><technical_language>nit: this is deprecated. please throw new RuntimeException(e) instead</technical_language>

Target:
public Set<String> getSegmentVersions(final String dataSource) { ArrayList<DataSegment> segments; try { StatusResponseHolder response = makeRequest(HttpMethod.GET, getFullSegmentsURL(dataSource)); segments = jsonMapper.readValue( response.getContent(), new TypeReference<List<DataSegment>>() { } ); } catch (Exception e) { throw new RuntimeException(e); } return segments.stream().map(s -> s.getVersion()).collect(Collectors.toSet()); }
Source:
<code>public ExistsPredicateBuilder<LuceneSearchPredicateBuilder> exists(String absoluteFieldPath) { <START> LuceneScopedIndexObjectComponent<LuceneFieldPredicateBuilderFactory> schemaObjectNodeComponent = scopeModel.getSchemaObjectNodeComponent( <END> absoluteFieldPath, PREDICATE_BUILDER_FACTORY_RETRIEVAL_STRATEGY ); if ( schemaObjectNodeComponent != null ) { LuceneExistsCompositePredicateBuilder objectPredicateBuilder = new LuceneExistsCompositePredicateBuilder(); for ( Map.Entry<String, LuceneFieldPredicateBuilderFactory> fieldComponentEntry : schemaObjectNodeComponent.getFieldComponents().entrySet() ) { ExistsPredicateBuilder<LuceneSearchPredicateBuilder> existsPredicateBuilder = fieldComponentEntry.getValue().createExistsPredicateBuilder( fieldComponentEntry.getKey() ); objectPredicateBuilder.addChild( existsPredicateBuilder ); } return objectPredicateBuilder; } return scopeModel .getSchemaNodeComponent( absoluteFieldPath, PREDICATE_BUILDER_FACTORY_RETRIEVAL_STRATEGY ) .getComponent().createExistsPredicateBuilder( absoluteFieldPath ); }</code><technical_language>I get feeling this is simple. if index contains object field, other a non-object field path? fail, execute predicate for object field</technical_language>

Target:
public ExistsPredicateBuilder<LuceneSearchPredicateBuilder> exists(String absoluteFieldPath) { LuceneObjectPredicateBuilderFactory objectPredicateBuilderFactory = scopeModel.getObjectPredicateBuilderFactory( absoluteFieldPath ); if ( objectPredicateBuilderFactory != null ) { return objectPredicateBuilderFactory.createExistsPredicateBuilder(); } return scopeModel .getSchemaNodeComponent( absoluteFieldPath, PREDICATE_BUILDER_FACTORY_RETRIEVAL_STRATEGY ) .getComponent().createExistsPredicateBuilder( absoluteFieldPath ); }
Source:
<code>public String getCertificate (final KeyStore keyStore, final String alias, final boolean <START> canAttachSelfSign) throws <END> ScriptSignatureException { if (keyStore!=null && alias!=null) { try { if (!keyStore.containsAlias(alias)) throw new ScriptSignatureException("Alias can't be found", null); if (!keyStore.isCertificateEntry(alias) && !canAttachSelfSign) throw new ScriptSignatureException("This certificate is self-signed certificate. Chose another trusted certificate.", null); Certificate certificate = keyStore.getCertificate(alias); byte[] certByte = certificate.getEncoded(); if (certByte == null) throw new ScriptSignatureException("Certificate not retrieved. Try again.", null); return fHelper.convertBytesToBase64(certByte); } catch (KeyStoreException e) { throw new ScriptSignatureException("Keystore not initialized properly. Try again.", e); } catch (CertificateEncodingException e) { throw new ScriptSignatureException("Encoding of certificate is improper. Please try again or chose another alias.", e); } } throw new ScriptSignatureException("Either keystore is not initialized properly or alias is not provided. Try again.", null); }</code><technical_language>explain flag? Do present this in some UI</technical_language>

Target:
public String getCertificate(final KeyStore keyStore, final String alias, final boolean canAttachSelfSign) throws ScriptSignatureException { if (keyStore != null && alias != null) try { if (!keyStore.containsAlias(alias)) throw new ScriptSignatureException("Alias can't be found"); if (!keyStore.isCertificateEntry(alias) && !canAttachSelfSign) throw new ScriptSignatureException("This certificate is self-signed certificate. Chose another trusted certificate."); Certificate certificate = keyStore.getCertificate(alias); byte[] certByte = certificate.getEncoded(); if (certByte == null) throw new ScriptSignatureException("Certificate not retrieved. Try again."); return fHelper.convertBytesToBase64(certByte); } catch (KeyStoreException e) { throw new ScriptSignatureException("Keystore not initialized properly. Try again.", e); } catch (CertificateEncodingException e) { throw new ScriptSignatureException("Encoding of certificate is improper. Please try again or chose another alias.", e); } throw new ScriptSignatureException("Either keystore is not initialized properly or alias is not provided. Try again."); }
Source:
<code>private List<Allele> buildNewAllelesFromZCall(final String zCallPedFileAlleles, final Map<String, Object> newAttributes) { <START> final char allele1 = zCallPedFileAlleles.charAt(0); <END> final char allele2 = zCallPedFileAlleles.charAt(1); final List<Allele> newAlleles = new ArrayList<>(); final String alleleA = String.valueOf(newAttributes.get("ALLELE_A")); final String alleleB = String.valueOf(newAttributes.get("ALLELE_B")); newAlleles.add(translateAllele(alleleA, alleleB, allele1)); newAlleles.add(translateAllele(alleleA, alleleB, allele2)); return newAlleles; }</code><technical_language>extract 0 1 constants 193 above</technical_language>

Target:
private List<Allele> buildNewAllelesFromZCall(final String zCallPedFileAlleles, final Map<String, Object> newAttributes) { final char allele1 = zCallPedFileAlleles.charAt(0); final char allele2 = zCallPedFileAlleles.charAt(1); final List<Allele> newAlleles = new ArrayList<>(); final String alleleA = String.valueOf(newAttributes.get(ALLELE_A)); final String alleleB = String.valueOf(newAttributes.get(ALLELE_B)); newAlleles.add(translateAllele(alleleA, alleleB, allele1)); newAlleles.add(translateAllele(alleleA, alleleB, allele2)); return newAlleles; }
Source:
<code>public static void flatGroups(ExtMap principal) { <START> HashMap<String, ExtMap> accumulator = new HashMap<>(); <END> flatGroups(principal, PrincipalRecord.GROUPS, accumulator); principal.put(PrincipalRecord.GROUPS, new ArrayList<>(accumulator.values())); }</code><technical_language>Right, I missed :-(</technical_language>

Target:
public static void flatGroups(ExtMap principal) { Map<String, ExtMap> accumulator = new HashMap<>(); flatGroups(principal, PrincipalRecord.GROUPS, accumulator); principal.put(PrincipalRecord.GROUPS, new ArrayList<>(accumulator.values())); }
Source:
<code>private void testBloomFilterForPrimitives(long seed) { final int numValues = 1024 * 1024; <START> final int numBytes = BlockSplitBloomFilter.optimalNumOfBits(numValues , 0.01) / 8; <END> BloomFilter bloomFilter = new BlockSplitBloomFilter(numBytes); Random random = new Random(seed); Set<Object> values = new HashSet<>(); for (int i = 0; i < numValues; i++) { int type = random.nextInt(4); Object v; switch (type) { case 0: { v = random.nextInt(); break; } case 1: { v = random.nextLong(); break; } case 2: { v = random.nextFloat(); break; } default: { v = random.nextDouble(); } } values.add(v); bloomFilter.insertHash(bloomFilter.hash(v)); } for (Object v : values) { assertTrue(String.format("the value %s should not be filtered, seed = %d", v, seed), bloomFilter.findHash(bloomFilter.hash(v))); } }</code><technical_language>invoke this 4 times do varying size of bloom filter well? generate false negatives if bloom filter size is optimal</technical_language>

Target:
private void testBloomFilterForPrimitives(long seed) { Random random = new Random(seed); final int numValues = 1024 * 1024; final int numBytes = BlockSplitBloomFilter.optimalNumOfBits(numValues, random.nextDouble() / 10) / 8; BloomFilter bloomFilter = new BlockSplitBloomFilter(numBytes); Set<Object> values = new HashSet<>(); for (int i = 0; i < numValues; i++) { int type = random.nextInt(4); Object v; switch (type) { case 0: { v = random.nextInt(); break; } case 1: { v = random.nextLong(); break; } case 2: { v = random.nextFloat(); break; } default: { v = random.nextDouble(); } } values.add(v); bloomFilter.insertHash(bloomFilter.hash(v)); } for (Object v : values) { assertTrue(String.format("the value %s should not be filtered, seed = %d", v, seed), bloomFilter.findHash(bloomFilter.hash(v))); } }
Source:
<code>public void publishEvent(Object event) { <START> publishEvent(new PayloadApplicationEvent<Object>(null, event)); <END> }</code><technical_language>null source odd</technical_language>

Target:
public void publishEvent(Object event) { publishEvent(new PayloadApplicationEvent<>(this, event)); }
Source:
<code>public void testUserSpecifiedRule() throws Exception { Style multipleRulesStyle = getCatalog().getStyleByName(MockData.ROAD_SEGMENTS.getLocalPart()).getStyle(); assertNotNull(multipleRulesStyle); @SuppressWarnings("deprecation") <START> Rule rule = multipleRulesStyle.getFeatureTypeStyles()[0].getRules()[0]; <END> LOGGER.info( "testing single rule " + rule.getName() + " from style " + multipleRulesStyle.getName()); FeatureTypeInfo ftInfo = getCatalog() .getFeatureTypeByName( MockData.ROAD_SEGMENTS.getNamespaceURI(), MockData.ROAD_SEGMENTS.getLocalPart()); GetLegendGraphicRequest req = getRequest(ftInfo.getFeatureType(), multipleRulesStyle); req.setRule(rule.getName()); req.setLegendOptions(new HashMap<String, String>()); req.setFormat(JSONFormat); final int HEIGHT_HINT = 30; req.setHeight(HEIGHT_HINT); JSONObject result = (JSONObject) this.legendProducer.buildLegendGraphic(req); JSONArray rules = result.getJSONArray(JSONLegendGraphicBuilder.LEGEND); assertEquals(1, rules.size()); assertEquals( rule.getDescription().getTitle().toString(), rules.getJSONObject(0) .getJSONArray(JSONLegendGraphicBuilder.RULES) .getJSONObject(0) .get(JSONLegendGraphicBuilder.TITLE)); }</code><technical_language>deprecated API, multipleRulesStyle.featureTypeStyles().get(0).rules().get(0) instead</technical_language>

Target:
public void testUserSpecifiedRule() throws Exception { Style multipleRulesStyle = getCatalog().getStyleByName(MockData.ROAD_SEGMENTS.getLocalPart()).getStyle(); assertNotNull(multipleRulesStyle); Rule rule = multipleRulesStyle.featureTypeStyles().get(0).rules().get(0); LOGGER.info( "testing single rule " + rule.getName() + " from style " + multipleRulesStyle.getName()); FeatureTypeInfo ftInfo = getCatalog() .getFeatureTypeByName( MockData.ROAD_SEGMENTS.getNamespaceURI(), MockData.ROAD_SEGMENTS.getLocalPart()); GetLegendGraphicRequest req = getRequest(ftInfo.getFeatureType(), multipleRulesStyle); req.setRule(rule.getName()); req.setLegendOptions(new HashMap<String, String>()); req.setFormat(JSONFormat); final int HEIGHT_HINT = 30; req.setHeight(HEIGHT_HINT); JSONObject result = this.legendProducer.buildLegendGraphic(req); JSONArray rules = result.getJSONArray(JSONLegendGraphicBuilder.LEGEND); assertEquals(1, rules.size()); assertEquals( rule.getDescription().getTitle().toString(), rules.getJSONObject(0) .getJSONArray(JSONLegendGraphicBuilder.RULES) .getJSONObject(0) .get(JSONLegendGraphicBuilder.TITLE)); }
Source:
<code>public ASTNode visit(Sentence node, Void _) { boolean change = false; Set<Variable> vars = node.getBody().variables(); KList ands = new KList(); Term condition = node.getRequires(); if (null != condition) { ands.getContents().add(condition); } for (Variable var : vars) { if (var.isSyntactic()) continue; <START> if ((var.getExpectedSort()==null && MetaK.isKSort(var.getSort())) <END> || (var.getExpectedSort()!=null && MetaK.isKSort(var.getExpectedSort()))) continue; change = true; ands.getContents().add(getPredicateTerm(var)); } if (!change) return node; if (ands.getContents().size() > 1) { condition = new KApp(KLabelConstant.ANDBOOL_KLABEL, ands); } else { condition = ands.getContents().get(0); } node = node.shallowCopy(); node.setRequires(condition); return node; }</code><technical_language>I need speak @radumereuta this. odd expecting a variable this null. strange need change actual logic of this step is modified FlattenTerms. problem ran solve I suggested? route abandoned</technical_language>

Target:
public ASTNode visit(Sentence node, Void _) { boolean change = false; Set<Variable> vars = node.getBody().variables(); KList ands = new KList(); Term condition = node.getRequires(); if (null != condition) { ands.getContents().add(condition); } for (Variable var : vars) { if (var.isSyntactic()) continue; if (MetaK.isKSort(var.getSort())) continue; change = true; ands.getContents().add(getPredicateTerm(var)); } if (!change) return node; if (ands.getContents().size() > 1) { condition = new KApp(KLabelConstant.ANDBOOL_KLABEL, ands); } else { condition = ands.getContents().get(0); } node = node.shallowCopy(); node.setRequires(condition); return node; }
Source:
<code>public long getNotificationCount() { return adapter.map(ad -> <START> ad.getNotificationCount()).orElse(0l); <END> }</code><technical_language>0l? a legitimate notification count. Or, fonts, misread 01 (zero one) Instead, return a public static final long NOT_ATTACHED = -1l;</technical_language>

Target:
public long getNotificationCount() { return adapter.getNotificationCount(); }
Source:
<code>public void initTunedProfiles() { this.startProgress(); if (getVersion() == null || getVersion().getSelectedItem() == null) { return; } Version version = getVersion().getSelectedItem(); Frontend.getInstance().runQuery(VdcQueryType.GetGlusterTunedProfiles, new IdAndNameQueryParameters(null, version.getValue()), new AsyncQuery(new INewAsyncCallback() { @Override public void onSuccess(Object model, Object returnValue) { ClusterModel.this.stopProgress(); ClusterModel clusterModel = (ClusterModel)model; List<String> glusterTunedProfiles = new ArrayList<>(); if (((VdcQueryReturnValue) returnValue).getSucceeded()) { glusterTunedProfiles.addAll((List<String>)(((VdcQueryReturnValue) returnValue).getReturnValue())); } final String oldSelectedProfile = <START> clusterModel != null && <END> clusterModel.getGlusterTunedProfile() != null ? clusterModel.getGlusterTunedProfile().getSelectedItem() : getEntity() != null ? getEntity().getGlusterTunedProfile() : null; glusterTunedProfile.setItems(glusterTunedProfiles); glusterTunedProfile.setIsAvailable(glusterTunedProfile.getItems().size() > 0); if (oldSelectedProfile != null) { String newSelectedItem = Linq.firstOrDefault(glusterTunedProfiles, new IPredicate<String>() { @Override public boolean match(String item) { return item.equals(oldSelectedProfile); } }); if (newSelectedItem != null) { glusterTunedProfile.setSelectedItem(newSelectedItem); } } if (glusterTunedProfile.getSelectedItem() == null) { glusterTunedProfile.setSelectedItem(Linq.firstOrDefault(glusterTunedProfiles)); } } })); }</code><technical_language>I clusterModel cost null this stage. I feel simply return I I of null if user closes dialog query is executed this check.So return instead</technical_language>

Target:
public void initTunedProfiles() { this.startProgress(); if (getVersion() == null || getVersion().getSelectedItem() == null) { return; } Version version = getVersion().getSelectedItem(); Frontend.getInstance().runQuery(VdcQueryType.GetGlusterTunedProfiles, new IdAndNameQueryParameters(null, version.getValue()), new AsyncQuery(new INewAsyncCallback() { @Override public void onSuccess(Object model, Object returnValue) { ClusterModel.this.stopProgress(); List<String> glusterTunedProfiles = new ArrayList<>(); if (((VdcQueryReturnValue) returnValue).getSucceeded()) { glusterTunedProfiles.addAll((List<String>)(((VdcQueryReturnValue) returnValue).getReturnValue())); } final String oldSelectedProfile = glusterTunedProfile.getSelectedItem(); glusterTunedProfile.setItems(glusterTunedProfiles); glusterTunedProfile.setIsAvailable(glusterTunedProfile.getItems().size() > 0); if (oldSelectedProfile != null) { String newSelectedItem = Linq.firstOrNull(glusterTunedProfiles, new Linq.EqualsPredicate(oldSelectedProfile)); if (newSelectedItem != null) { glusterTunedProfile.setSelectedItem(newSelectedItem); } else if (getIsEdit()) { glusterTunedProfile.setSelectedItem(Linq.firstOrNull(glusterTunedProfiles, new Linq.EqualsPredicate(getEntity().getGlusterTunedProfile()))); } } } })); }
Source:
<code>public List<URIStatus> listStatus(AlluxioURI path, ListStatusPOptions options) throws FileDoesNotExistException, IOException, AlluxioException { checkUri(path); <START> List<URIStatus> statuses = mMetadataCache.listStatus(path); <END> if (statuses == null) { statuses = super.listStatus(path, options); mMetadataCache.put(path, statuses); } return statuses; }</code><technical_language>@apc999 updated, PTAL</technical_language>

Target:
public List<URIStatus> listStatus(AlluxioURI path, ListStatusPOptions options) throws FileDoesNotExistException, IOException, AlluxioException { checkUri(path); if (options.getRecursive()) { return super.listStatus(path, options); } List<URIStatus> statuses = mMetadataCache.listStatus(path); if (statuses == null) { statuses = super.listStatus(path, options); mMetadataCache.put(path, statuses); } return statuses; }
Source:
<code>public void testBigUsage() { <START> final long maxMemory = Math.min(Runtime.getRuntime().maxMemory(), 10*1024*1024*1024L); <END> List<Maybe<byte[]>> references = MutableList.of(); long created = 0; while (created < 2*maxMemory) { byte d[] = new byte[10*1000*1000]; references.add(Maybe.soft(d)); MemoryUsageTracker.SOFT_REFERENCES.track(d, d.length); created += d.length; long totalMemory = Runtime.getRuntime().totalMemory(); long freeMemory = Runtime.getRuntime().freeMemory(); LOG.info("created "+Strings.makeSizeString(created) + " ... in use: "+Strings.makeSizeString(totalMemory - freeMemory)+" / " + Strings.makeSizeString(totalMemory) + " ... reclaimable: "+Strings.makeSizeString(MemoryUsageTracker.SOFT_REFERENCES.getBytesUsed()) + " ... live refs: "+Strings.makeSizeString(sizeOfActiveReferences(references)) + " ... maxMem="+maxMemory+"; totalMem="+totalMemory+"; usedMem="+(totalMemory-freeMemory)); } Asserts.succeedsEventually(new Runnable() { public void run() { long totalMemory = Runtime.getRuntime().totalMemory(); long freeMemory = Runtime.getRuntime().freeMemory(); assertLessThan(MemoryUsageTracker.SOFT_REFERENCES.getBytesUsed(), maxMemory); assertLessThan(MemoryUsageTracker.SOFT_REFERENCES.getBytesUsed(), totalMemory); assertLessThan(MemoryUsageTracker.SOFT_REFERENCES.getBytesUsed(), totalMemory - freeMemory); }}); }</code><technical_language>Nitpick: a named constant nice here; MAX_MEMORY_CAP</technical_language>

Target:
public void testBigUsage() { final int ALLOCATION_CHUNK_SIZE = 10*1000*1000; final long MAX_MEMORY_CAP = 10*1024*1024*1024L; final long maxMemory = Math.min(Runtime.getRuntime().maxMemory(), MAX_MEMORY_CAP); List<Maybe<byte[]>> references = MutableList.of(); long created = 0; while (created < 2*maxMemory) { byte d[] = new byte[ALLOCATION_CHUNK_SIZE]; references.add(Maybe.soft(d)); MemoryUsageTracker.SOFT_REFERENCES.track(d, d.length); created += d.length; long totalMemory = Runtime.getRuntime().totalMemory(); long freeMemory = Runtime.getRuntime().freeMemory(); LOG.info("created "+Strings.makeSizeString(created) + " ... in use: "+Strings.makeSizeString(totalMemory - freeMemory)+" / " + Strings.makeSizeString(totalMemory) + " ... reclaimable: "+Strings.makeSizeString(MemoryUsageTracker.SOFT_REFERENCES.getBytesUsed()) + " ... live refs: "+Strings.makeSizeString(sizeOfActiveReferences(references)) + " ... maxMem="+maxMemory+"; totalMem="+totalMemory+"; usedMem="+(totalMemory-freeMemory)); } Asserts.succeedsEventually(new Runnable() { public void run() { long totalMemory = Runtime.getRuntime().totalMemory(); long freeMemory = Runtime.getRuntime().freeMemory(); assertLessThan(MemoryUsageTracker.SOFT_REFERENCES.getBytesUsed(), maxMemory); assertLessThan(MemoryUsageTracker.SOFT_REFERENCES.getBytesUsed(), totalMemory); assertLessThan(MemoryUsageTracker.SOFT_REFERENCES.getBytesUsed(), totalMemory - freeMemory); }}); }
Source:
<code>public void getProblemFactsWithNullField() { TestdataAbstractSolutionBasedSolution solution = new TestdataAbstractSolutionBasedSolution(); solution.setSingleValue(null); TestdataValue v1 = new TestdataValue("v1"); TestdataValue v2 = new TestdataValue("v2"); TestdataValue v3 = new TestdataValue("v3"); List<TestdataValue> valueList = Arrays.asList(v1, v2, v3); solution.setValueList(valueList); solution.setEntityList(null); solution.setSingleEntity(null); solution.setScore(null); <START> assertThat((Collection<Object>) solution.getProblemFactList()).containsExactly(v1, v2, v3); <END> }</code><technical_language>Redundant cast</technical_language>

Target:
public void getProblemFactsWithNullField() { TestdataAbstractSolutionBasedSolution solution = new TestdataAbstractSolutionBasedSolution(); solution.setSingleValue(null); TestdataValue v1 = new TestdataValue("v1"); TestdataValue v2 = new TestdataValue("v2"); TestdataValue v3 = new TestdataValue("v3"); List<TestdataValue> valueList = Arrays.asList(v1, v2, v3); solution.setValueList(valueList); solution.setEntityList(null); solution.setSingleEntity(null); solution.setScore(null); assertThat(solution.getProblemFactList()).containsExactly(v1, v2, v3); }
Source:
<code>protected boolean canDoAction() { VmHandler.updateDisksFromDb(getVm()); Collection<DiskImage> diskImages = ImagesHandler.filterDiskBasedOnImages(getVm().getDiskMap().values()); DiskImage vmDisk = LinqUtils.first(diskImages); boolean result = true; Snapshot snapshot = getSnapshotDao().get(getParameters().getDstSnapshotId()); SnapshotsValidator snapshotsValidator = new SnapshotsValidator(); result = result && validate(new SnapshotsValidator().vmNotDuringSnapshot(getVmId())) <START> && validate(snapshotsValidator.snapshotNotBroken(snapshot)); <END> if (vmDisk != null) { result = result && ImagesHandler.PerformImagesChecks(getVm(), getReturnValue().getCanDoActionMessages(), getVm().getstorage_pool_id(), Guid.Empty, true, true, false, false, true, true, true, true, new ArrayList<DiskImage>(diskImages)); } if (result && LinqUtils.foreach(diskImages, new Function<DiskImage, Guid>() { @Override public Guid eval(DiskImage disk) { return disk.getvm_snapshot_id().getValue(); } }).contains(getParameters().getDstSnapshotId())) { result = false; addCanDoActionMessage(VdcBllMessages.CANNOT_PREIEW_CURRENT_IMAGE); } if (!result) { addCanDoActionMessage(VdcBllMessages.VAR__ACTION__PREVIEW); addCanDoActionMessage(VdcBllMessages.VAR__TYPE__SNAPSHOT); } return result; }</code><technical_language>snap. validator twice</technical_language>

Target:
protected boolean canDoAction() { VmHandler.updateDisksFromDb(getVm()); Collection<DiskImage> diskImages = ImagesHandler.filterDiskBasedOnImages(getVm().getDiskMap().values()); DiskImage vmDisk = LinqUtils.first(diskImages); boolean result = true; Snapshot snapshot = getSnapshotDao().get(getParameters().getDstSnapshotId()); SnapshotsValidator snapshotsValidator = new SnapshotsValidator(); result = result && validate(new SnapshotsValidator().vmNotDuringSnapshot(getVmId())) && validate(snapshotsValidator.snapshotNotBroken(snapshot)); if (vmDisk != null) { result = result && ImagesHandler.PerformImagesChecks(getVm(), getReturnValue().getCanDoActionMessages(), getVm().getstorage_pool_id(), Guid.Empty, true, true, false, false, true, true, true, true, getVm().getDiskMap().values()); } if (result && LinqUtils.foreach(diskImages, new Function<DiskImage, Guid>() { @Override public Guid eval(DiskImage disk) { return disk.getvm_snapshot_id().getValue(); } }).contains(getParameters().getDstSnapshotId())) { result = false; addCanDoActionMessage(VdcBllMessages.CANNOT_PREIEW_CURRENT_IMAGE); } if (!result) { addCanDoActionMessage(VdcBllMessages.VAR__ACTION__PREVIEW); addCanDoActionMessage(VdcBllMessages.VAR__TYPE__SNAPSHOT); } return result; }
Source:
<code>private void initProperties() { properties.put(ClusterPropertyTypes.TITLE, null); List<String> nodeids = cluster.getNodes().stream() .map(e -> e.getId()) .collect(Collectors.toList()); if (cluster.getNodes().size() > 1) { properties.put(ClusterPropertyTypes.ID, nodeids.toString()); properties.put(ClusterPropertyTypes.STARTRANK, Integer.toString(cluster.getStartRank())); <START> } else if (cluster.getNodes().size() == 1) { <END> SequenceNode sn = cluster.getNodes().iterator().next(); properties.put(ClusterPropertyTypes.ID, sn.getId()); properties.put(ClusterPropertyTypes.STARTREF, Integer.toString(sn.getStartRef())); properties.put(ClusterPropertyTypes.ENDREF, Integer.toString(sn.getEndRef())); properties.put(ClusterPropertyTypes.SOURCES, sn.getSources().toString()); properties.put(ClusterPropertyTypes.SEQUENCE, sn.getSequence()); } }</code><technical_language>a Cluster 0 nodes, this else</technical_language>

Target:
private void initProperties() { properties.put(ClusterPropertyTypes.TITLE, null); List<String> nodeIds = cluster.getNodes().stream() .map(e -> e.getId()) .collect(Collectors.toList()); if (cluster.getNodes().size() > 1) { properties.put(ClusterPropertyTypes.ID, nodeIds.toString()); properties.put(ClusterPropertyTypes.STARTRANK, Integer.toString(cluster.getStartRank())); } else { SequenceNode sn = cluster.getNodes().iterator().next(); properties.put(ClusterPropertyTypes.ID, sn.getId()); properties.put(ClusterPropertyTypes.STARTREF, Integer.toString(sn.getStartRef())); properties.put(ClusterPropertyTypes.ENDREF, Integer.toString(sn.getEndRef())); properties.put(ClusterPropertyTypes.SOURCES, sn.getSources().toString()); properties.put(ClusterPropertyTypes.SEQUENCE, sn.getSequence()); } }
Source:
<code>private State getItemState(String itemId) throws ServiceException { ItemRegistry itemRegistry = getItemRegistry(); State itemState = null; try { Item item = itemRegistry.getItem(itemId); itemState = item.getState(); <START> logger.debug("{}: State: {}", itemId, itemState.toString()); <END> } catch (ItemNotFoundException ex) { logger.info("{} not found {}", itemId, ex); } return itemState; }</code><technical_language>Omit .toString()</technical_language>

Target:
private State getItemState(String itemId) throws ServiceException { ItemRegistry itemRegistry = getItemRegistry(); State itemState = null; try { Item item = itemRegistry.getItem(itemId); itemState = item.getState(); logger.debug("{}: State: {}", itemId, itemState); } catch (ItemNotFoundException ex) { logger.info("{} not found {}", itemId, ex.getMessage()); } return itemState; }
Source:
<code>ReviewedPanel(PatchSet.Id id, String path, boolean bottom) { initWidget(uiBinder.createAndBindUi(this)); patchId = id; fileId = path; if (bottom) { <START> fileName.setVisible(false); <END> } else { fileName.setText(path); } nextLink.setHTML(PatchUtil.C.next() + Util.C.nextPatchLinkIcon()); }</code><technical_language>If Element this is UIObject.setVisible(fileName, false);</technical_language>

Target:
ReviewedPanel(PatchSet.Id id, String path, boolean bottom) { initWidget(uiBinder.createAndBindUi(this)); patchId = id; fileId = path; if (bottom) { UIObject.setVisible(fileName, false); } else { fileName.setInnerText(path); } nextLink.setHTML(PatchUtil.C.next() + Util.C.nextPatchLinkIcon()); }
Source:
<code>public void readsByte() throws IOException { final String data = "{test}"; final InputStream stream = new RqLengthAware( new RqFake( Arrays.asList( "GET /test1", "Host: b.example.com", <START> "Content-type: text/json" <END> ), data ) ).body(); MatcherAssert.assertThat( stream.read(), Matchers.equalTo((int) data.getBytes()[0]) ); MatcherAssert.assertThat( stream.available(), Matchers.equalTo(data.length() - 1) ); }</code><technical_language>do need content type here</technical_language>

Target:
public void readsByte() throws IOException { final String data = "test"; final InputStream stream = new RqLengthAware( new RqFake( Arrays.asList( "GET /test1", "Host: b.example.com" ), data ) ).body(); MatcherAssert.assertThat( stream.read(), Matchers.equalTo((int) data.getBytes()[0]) ); MatcherAssert.assertThat( stream.available(), Matchers.equalTo(data.length() - 1) ); }
Source:
<code>protected void performFirstChildNegotiation( List<IkePayload> childReqList, List<IkePayload> childRespList) { childReqList.add(mIkeInitNoncePayload); childRespList.add(mIkeRespNoncePayload); deferMessage( obtainMessage( CMD_HANDLE_FIRST_CHILD_NEGOTIATION, new FirstChildNegotiationData( mFirstChildSessionParams, mFirstChildCallbacks, childReqList, childRespList))); <START> mUserCbExecutor.execute( <END> () -> { IkeSessionConfiguration ikeSessionConfiguration = null; for (IkePayload payload : childRespList) { if (payload instanceof IkeConfigPayload) { ikeSessionConfiguration = new IkeSessionConfiguration((IkeConfigPayload) payload); } } mIkeSessionCallback.onOpened(ikeSessionConfiguration); }); transitionTo(mChildProcedureOngoing); }</code><technical_language>for readability. - confusing build IkeSessionConfiguration child payload list. is weird IKE extracts payload needs a child payload list. pass more information IkeSessionConfiguration underlying network addresses. #buildIkeSessionConfiguration IKE related parameters. - hard find a place valite configType. If do in extractChildPayloadsFromMessage, need add more input if this is a request response message expect a CONFIG_TYPE_REQUEST in a request message a CONFIG_TYPE_REPLY in a response message</technical_language>

Target:
protected void performFirstChildNegotiation( List<IkePayload> childReqList, List<IkePayload> childRespList) { childReqList.add(mIkeInitNoncePayload); childRespList.add(mIkeRespNoncePayload); deferMessage( obtainMessage( CMD_HANDLE_FIRST_CHILD_NEGOTIATION, new FirstChildNegotiationData( mFirstChildSessionParams, mFirstChildCallbacks, childReqList, childRespList))); mUserCbExecutor.execute( () -> { mIkeSessionCallback.onOpened(mIkeSessionConfiguration); }); transitionTo(mChildProcedureOngoing); }