auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code><START> void updateResultsTable(Collection<Persona> results) { <END> Object[][] rows = new Object[results.size()][2]; int i = 0; for (Persona result : results) { rows[i] = new String[]{String.valueOf(result.getId()), result.getName()}; i++; } DefaultTableModel updatedTableModel = new PersonaFilterTableModel( rows, new String[]{"ID", "Name"} ); filterResultsTable.setModel(updatedTableModel); filterResultsTable.setSelectionMode(ListSelectionModel.SINGLE_SELECTION); filterResultsTable.getColumnModel().getColumn(0).setMaxWidth(100); filterResultsTable.setAutoResizeMode(JTable.AUTO_RESIZE_ALL_COLUMNS); }</code><technical_language>this method private</technical_language>

Target:
private void updateResultsTable(Collection<Persona> results) { Object[][] rows = new Object[results.size()][2]; int i = 0; for (Persona result : results) { rows[i] = new String[]{String.valueOf(result.getId()), result.getName()}; i++; } DefaultTableModel updatedTableModel = new PersonaFilterTableModel( rows, new String[]{"ID", "Name"} ); filterResultsTable.setModel(updatedTableModel); filterResultsTable.setSelectionMode(ListSelectionModel.SINGLE_SELECTION); filterResultsTable.getColumnModel().getColumn(0).setMaxWidth(100); filterResultsTable.setAutoResizeMode(JTable.AUTO_RESIZE_ALL_COLUMNS); }
Source:
<code>public int hashCode() { <START> int result = this.getDevice().getDeviceIdentification().hashCode(); <END> result = 31 * result + this.index; return result; }</code><technical_language>in equals I is necessary insert getDeviceIdentification()</technical_language>

Target:
public int hashCode() { return Objects.hash(this.device, this.index); }
Source:
<code>public void placeNewOrder(Market market, String symbol, int volume, double price, OrderType orderType) throws StockMarketExpection { <START> if(orderType==OrderType.BUY && price*volume> this.cashInHand){ <END> throw new StockMarketExpection("Cannot place order for "+symbol+" since there is not enough money. Trader: "+this.name); } Order newOrder; if (orderType == OrderType.BUY) { newOrder = new BuyOrder(symbol, volume, price, this); } else if(orderType == OrderType.SELL){ newOrder = new SellOrder(symbol, volume, price, this); }else{ throw new StockMarketExpection("Type Error"); } if (OrderUtility.isAlreadyPresent(ordersPlaced, newOrder)) { throw new StockMarketExpection("The order is duplicated"); } if (orderType == OrderType.SELL && !OrderUtility.owns(position, symbol)) { throw new StockMarketExpection("The trader haven't own this stock"); } if (orderType == OrderType.SELL && volume > OrderUtility.ownedQuantity(position, symbol)) { throw new StockMarketExpection("The trader does not own enough of the stock."); } m.addOrder(newOrder); ordersPlaced.add(newOrder); }</code><technical_language>addressed in patch, need a space '{' end match formatting elsewhere</technical_language>

Target:
public void placeNewOrder(Market market, String symbol, int volume, double price, OrderType orderType) throws StockMarketExpection { if(orderType==OrderType.BUY && price*volume> this.cashInHand) { throw new StockMarketExpection("Cannot place order for "+symbol+" since there is not enough money. Trader: "+this.name); } Order newOrder; if (orderType == OrderType.BUY) { newOrder = new BuyOrder(symbol, volume, price, this); } else if(orderType == OrderType.SELL){ newOrder = new SellOrder(symbol, volume, price, this); }else{ throw new StockMarketExpection("Type Error"); } if (OrderUtility.isAlreadyPresent(ordersPlaced, newOrder)) { throw new StockMarketExpection("The order is duplicated"); } if (orderType == OrderType.SELL && !OrderUtility.owns(position, symbol)) { throw new StockMarketExpection("The trader does not own this stock"); } if (orderType == OrderType.SELL && volume > OrderUtility.ownedQuantity(position, symbol)) { throw new StockMarketExpection("The trader does not own enough of the stock."); } m.addOrder(newOrder); ordersPlaced.add(newOrder); }
Source:
<code>protected ConcreteEventStore( Clock clock, IdGenerator idGenerator, EventHasher eventHasher, MessageSender<ResourceUpdatedMessage> sender, EventPersistenceStore persistenceStore, MetricRegistry metricRegistry, String metricPrefix ) { this.clock = checkNotNull(clock); this.idGenerator = checkNotNull(idGenerator); this.sender = checkNotNull(sender); this.eventHasher = checkNotNull(eventHasher); this.persistenceStore = checkNotNull(persistenceStore); this.metricRegistry = metricRegistry; <START> write = metricPrefix + "write"; <END> }</code><technical_language>Ditto missing trailing dot. This happening well. I checked</technical_language>

Target:
protected ConcreteEventStore( Clock clock, IdGenerator idGenerator, EventHasher eventHasher, MessageSender<ResourceUpdatedMessage> sender, EventPersistenceStore persistenceStore, MetricRegistry metricRegistry, String metricPrefix ) { this.clock = checkNotNull(clock); this.idGenerator = checkNotNull(idGenerator); this.sender = checkNotNull(sender); this.eventHasher = checkNotNull(eventHasher); this.persistenceStore = checkNotNull(persistenceStore); this.metricRegistry = metricRegistry; write = metricPrefix + "write."; }
Source:
<code>public void testFileSize() throws Exception { File file = File.createTempFile("forge", "testFileSize"); file.deleteOnExit(); <START> try (FileOutputStream fos = new FileOutputStream(file)) { <END> Streams.write(new ByteArrayInputStream("Test".getBytes()), fos); FileResource<?> fileResource = factory.create(file).reify(FileResource.class); Assert.assertEquals(file.length(), fileResource.getSize()); file.delete(); } }</code><technical_language>Files.write instead</technical_language>

Target:
public void testFileSize() throws Exception { File file = File.createTempFile("forge", "testFileSize"); file.deleteOnExit(); try (FileWriter write = new FileWriter(file)) { write.append("Test"); FileResource<?> fileResource = factory.create(file).reify(FileResource.class); Assert.assertEquals(file.length(), fileResource.getSize()); } file.delete(); }
Source:
<code>public Builder tag(final String tagName, final String value) { Objects.requireNonNull(tagName, "tagName"); Objects.requireNonNull(value, "value"); <START> if (!tagName.isEmpty() <END> && !value.isEmpty() && CheckTags.isTagNameLegal(tagName) && CheckTags.isTagValueLegal(value)) { tags.put(tagName, value); } return this; }</code><technical_language>Missing test coverage for case tag is nil empty</technical_language>

Target:
public Builder tag(final String tagName, final String value) { Objects.requireNonNull(tagName, "tagName"); Objects.requireNonNull(value, "value"); if (CheckTags.isLegalFullCheck(tagName, value)) { tags.put(tagName, value); } return this; }
Source:
<code>public final byte[] call(final byte[] memento) throws Exception { LOG.log(Level.INFO, String.format("%s starting...", taskId)); userControllerTask.initialize(); try (final MetricTrackerManager metricTrackerManager = this.metricTrackerManager;) { metricTrackerManager.registerTrackers(metricTrackerSet); int iteration = 0; while(!userControllerTask.isTerminated(iteration)) { metricTrackerManager.start(); ctrlMessageBroadcast.send(CtrlMessage.RUN); sendData(iteration); receiveData(iteration); <START> userControllerTask.run(iteration); <END> metricTrackerManager.stop(); updateTopology(); iteration++; } ctrlMessageBroadcast.send(CtrlMessage.TERMINATE); userControllerTask.cleanup(); } return null; }</code><technical_language>ComputeTask is measuring interval of UserComputeTask.run(), ControllerTask for UserControllerTask.run(). hurt measure statistics ComputeTask ControllerTask</technical_language>

Target:
public final byte[] call(final byte[] memento) throws Exception { LOG.log(Level.INFO, String.format("%s starting...", taskId)); userControllerTask.initialize(); try (final MetricsCollector metricsCollector = this.metricsCollector;) { metricsCollector.registerTrackers(metricTrackerSet); int iteration = 0; while(!userControllerTask.isTerminated(iteration)) { metricsCollector.start(); ctrlMessageBroadcast.send(CtrlMessage.RUN); sendData(iteration); receiveData(iteration); runUserControllerTask(iteration); metricsCollector.stop(); updateTopology(); iteration++; } ctrlMessageBroadcast.send(CtrlMessage.TERMINATE); userControllerTask.cleanup(); } return null; }
Source:
<code>private void getModel(String modelId, Consumer consumer, ActionListener<Model> modelActionListener) { ModelAndConsumer cachedModel = localModelCache.get(modelId); if (cachedModel != null) { cachedModel.consumers.add(consumer); <START> modelActionListener.onResponse(cachedModel.model); <END> logger.trace(() -> new ParameterizedMessage("[{}] loaded from cache", modelId)); return; } if (loadModelIfNecessary(modelId, consumer, modelActionListener) == false) { logger.trace(() -> new ParameterizedMessage("[{}] is loading or loaded, added new listener to queue", modelId)); } }</code><technical_language>is a pretty frustrating race condition here, unsure if is worth fixing. - ModelAndConsumer cachedModel = localModelCache.get(modelId); returns cache successfully - Model is removed cache is referenced a pipeline (updated cluster state change) - Search adds a consume late is evicted referenced model get call around. next call for model load index :/. I of fixing this is locking (which defeats purpose of fast get). cost is low, OK this is recoverable later</technical_language>

Target:
private void getModel(String modelId, Consumer consumer, ActionListener<Model> modelActionListener) { ModelAndConsumer cachedModel = localModelCache.get(modelId); if (cachedModel != null) { cachedModel.consumers.add(consumer); modelActionListener.onResponse(cachedModel.model); logger.trace(() -> new ParameterizedMessage("[{}] loaded from cache", modelId)); return; } if (loadModelIfNecessary(modelId, consumer, modelActionListener)) { logger.trace(() -> new ParameterizedMessage("[{}] is loading or loaded, added new listener to queue", modelId)); } }
Source:
<code>public void testUpdatingDefaultVersionHostedWorkflow() { ApiClient webClient = getWebClient(); WorkflowsApi workflowsApi = new WorkflowsApi(getWebClient()); HostedApi hostedApi = new HostedApi(webClient); Workflow hostedWorkflow = hostedApi.createHostedWorkflow("awesomeTool", "cwl", null, null); SourceFile file = new SourceFile(); file.setContent("cwlVersion: v1.0\\nclass: Workflow\\ninputs:\\ninp: File\\nex: string\\noutputs:\\nclassout:\\ntype: File\\noutputSource: compile/classfile\\nsteps:\\nuntar:\\nrun: tar-param.cwl\\nin:\\ntarfile: inp\\nextractfile: ex\\nout: [example_out]\\ncompile:\\nrun: arguments.cwl\\nin:\\nsrc: untar/example_out\\nout: [classfile]"); file.setType(SourceFile.TypeEnum.DOCKSTORE_CWL); file.setPath("/Dockstore.cwl"); Workflow dockstoreWorkflow = hostedApi.editHostedWorkflow(hostedWorkflow.getId(), Lists.newArrayList(file)); Optional<WorkflowVersion> first = dockstoreWorkflow.getWorkflowVersions().stream().max(Comparator.comparingInt((WorkflowVersion t) -> Integer.parseInt(t.getName()))); Assert.assertEquals("correct number of source files", 1, first.get().getSourceFiles().size()); <START> if (first.isPresent()) { <END> workflowsApi.updateDefaultVersion(hostedWorkflow.getId(), first.get().getName()); } }</code><technical_language>here, practice in normal code, a test ok (or do assertTrue(first.isPresent()))</technical_language>

Target:
public void testUpdatingDefaultVersionHostedWorkflow() { ApiClient webClient = getWebClient(); WorkflowsApi workflowsApi = new WorkflowsApi(getWebClient()); HostedApi hostedApi = new HostedApi(webClient); Workflow hostedWorkflow = hostedApi.createHostedWorkflow("awesomeTool", "cwl", null, null); SourceFile file = new SourceFile(); file.setContent("cwlVersion: v1.0\\nclass: Workflow\\ninputs:\\ninp: File\\nex: string\\noutputs:\\nclassout:\\ntype: File\\noutputSource: compile/classfile\\nsteps:\\nuntar:\\nrun: tar-param.cwl\\nin:\\ntarfile: inp\\nextractfile: ex\\nout: [example_out]\\ncompile:\\nrun: arguments.cwl\\nin:\\nsrc: untar/example_out\\nout: [classfile]"); file.setType(SourceFile.TypeEnum.DOCKSTORE_CWL); file.setPath("/Dockstore.cwl"); Workflow dockstoreWorkflow = hostedApi.editHostedWorkflow(hostedWorkflow.getId(), Lists.newArrayList(file)); Optional<WorkflowVersion> first = dockstoreWorkflow.getWorkflowVersions().stream().max(Comparator.comparingInt((WorkflowVersion t) -> Integer.parseInt(t.getName()))); Assert.assertEquals("correct number of source files", 1, first.get().getSourceFiles().size()); assertTrue(first.isPresent()); workflowsApi.updateWorkflowDefaultVersion(hostedWorkflow.getId(), first.get().getName()); }
Source:
<code>public void testLocal() throws Exception { final String OUTPUT = TEST_OUTPUT_GCS_FOLDER + "CountReadsITCase-testLocal-output.txt"; String[] ARGS = { "--apiKey=" + API_KEY, <START> "--project=genomics-pipelines", <END> "--output=" + OUTPUT, "--references=1:550000:560000", "--BAMFilePath=" + TEST_BAM_FNAME }; final long EXPECTED = 685; GenomicsOptions popts = PipelineOptionsFactory.create().as(GenomicsOptions.class); popts.setApiKey(API_KEY); GcsUtil gcsUtil = new GcsUtil.GcsUtilFactory().create(popts); touchOutput(gcsUtil, OUTPUT); CountReads.main(ARGS); BufferedReader reader = new BufferedReader(Channels.newReader(gcsUtil.open(GcsPath.fromUri(OUTPUT)), "UTF-8")); long got = Long.parseLong(reader.readLine()); Assert.assertEquals(EXPECTED, got); }</code><technical_language>project in environment variable, I notice time</technical_language>

Target:
public void testLocal() throws Exception { final String OUTPUT = TEST_OUTPUT_GCS_FOLDER + "CountReadsITCase-testLocal-output.txt"; String[] ARGS = { "--apiKey=" + API_KEY, "--project=" + TEST_PROJECT, "--output=" + OUTPUT, "--references=1:550000:560000", "--BAMFilePath=" + TEST_BAM_FNAME }; final long EXPECTED = 685; GenomicsOptions popts = PipelineOptionsFactory.create().as(GenomicsOptions.class); popts.setApiKey(API_KEY); GcsUtil gcsUtil = new GcsUtil.GcsUtilFactory().create(popts); touchOutput(gcsUtil, OUTPUT); CountReads.main(ARGS); BufferedReader reader = new BufferedReader(Channels.newReader(gcsUtil.open(GcsPath.fromUri(OUTPUT)), "UTF-8")); long got = Long.parseLong(reader.readLine()); Assert.assertEquals(EXPECTED, got); }
Source:
<code>public void setUp() { serviceRead = new Retrofit.Builder() .baseUrl(APIUtils.GET_API) .addConverterFactory(JacksonConverterFactory.create()) .addCallAdapterFactory(RxJava2CallAdapterFactory.create()) .client(HttpClientBuilder()) .build() .create(OpenFoodAPIService.class); OkHttpClient httpClientWithAuth = new OkHttpClient.Builder() .addInterceptor(new HttpLoggingInterceptor().setLevel(HttpLoggingInterceptor.Level.BODY)) .addInterceptor(chain -> { Request original = chain.request(); Request.Builder requestBuilder = original.newBuilder() .header("Authorization", "Basic b2ZmOm9mZg==") .header("Accept", "application/json") .method(original.method(), original.body()); Request request = requestBuilder.build(); return chain.proceed(request); }) .build(); serviceWrite = new Retrofit.Builder() <START> .baseUrl(APIUtils.GET_API) <END> .addConverterFactory(JacksonConverterFactory.create()) .addCallAdapterFactory(RxJava2CallAdapterFactory.create()) .client(httpClientWithAuth) .build() .create(OpenFoodAPIService.class); }</code><technical_language>testing URL i.e. <LINK_0> host? Current implementation leads creation of real product in prod db. <LINK_1> This product is created post_product() test. This is unrelated changes, touched this part of code I pointed out opinion</technical_language>

Target:
public void setUp() { serviceRead = new Retrofit.Builder() .baseUrl(APIUtils.GET_API) .addConverterFactory(JacksonConverterFactory.create()) .addCallAdapterFactory(RxJava2CallAdapterFactory.create()) .client(HttpClientBuilder()) .build() .create(OpenFoodAPIService.class); OkHttpClient httpClientWithAuth = new OkHttpClient.Builder() .addInterceptor(new HttpLoggingInterceptor().setLevel(HttpLoggingInterceptor.Level.BODY)) .addInterceptor(chain -> { Request original = chain.request(); Request.Builder requestBuilder = original.newBuilder() .header("Authorization", "Basic b2ZmOm9mZg==") .header("Accept", "application/json") .method(original.method(), original.body()); Request request = requestBuilder.build(); return chain.proceed(request); }) .build(); serviceWrite = new Retrofit.Builder() .baseUrl(APIUtils.DEV_API) .addConverterFactory(JacksonConverterFactory.create()) .addCallAdapterFactory(RxJava2CallAdapterFactory.create()) .client(httpClientWithAuth) .build() .create(OpenFoodAPIService.class); }
Source:
<code>private List<GroupInfo> list(ListRequest listParameter) throws RestApiException { List<GroupInfo> list = null; String query = ""; if (listParameter.getLimit() > 0) { query = UrlUtils.appendToUrlQuery(query, "n=" + listParameter.getLimit()); } String url = GroupApiRestClient.BASE_URL + "/"; if (!Strings.isNullOrEmpty(query)) { url += '?' + query; } JsonElement result = gerritRestClient.getRequest(url); if (result == null) { <START> list = new ArrayList<GroupInfo>(); <END> } else { list = groupsParser.parseGroupInfos(result); } return list; }</code><technical_language>Collections.emptyList()</technical_language>

Target:
private List<GroupInfo> list(ListRequest listParameter) throws RestApiException { List<GroupInfo> list = null; String query = ""; if (listParameter.getLimit() > 0) { query = UrlUtils.appendToUrlQuery(query, "n=" + listParameter.getLimit()); } if (listParameter.getStart() > 0) { query = UrlUtils.appendToUrlQuery(query, "S=" + listParameter.getLimit()); } if (listParameter.getOwned()) { query = UrlUtils.appendToUrlQuery(query, "owned"); } if ((listParameter.getSuggest() != null) && (listParameter.getSuggest().length() > 0)) { query = UrlUtils.appendToUrlQuery(query, "suggest=" + listParameter.getSuggest()); } if ((listParameter.getOptions() != null) && (listParameter.getOptions().size() > 0)) { throw new NotImplementedException(); } if ((listParameter.getProjects() != null) && (listParameter.getProjects().size() > 0)) { throw new NotImplementedException(); } if ((listParameter.getGroups() != null) && (listParameter.getGroups().size() > 0)) { throw new NotImplementedException(); } if (listParameter.getUser() != null) { throw new NotImplementedException(); } if (listParameter.getSubstring() != null) { throw new NotImplementedException(); } String url = GroupApiRestClient.getBaseRequestPath() + "/"; if (!Strings.isNullOrEmpty(query)) { url += '?' + query; } JsonElement result = gerritRestClient.getRequest(url); if (result == null) { list = Collections.emptyList(); } else { list = groupsParser.parseGroupInfos(result); } return list; }
Source:
<code>public List<AbstractWorkflowDefinition<? extends WorkflowState>> getWorkflowDefinitions() { <START> synchronized (workflowDefinitions) { <END> return new ArrayList<>(workflowDefinitions.values()); } }</code><technical_language>Is synchronized needed here? workflowDefinitions is a synchronizedMap</technical_language>

Target:
public List<AbstractWorkflowDefinition<? extends WorkflowState>> getWorkflowDefinitions() { return new ArrayList<>(workflowDefinitions.values()); }
Source:
<code>protected void delete(LogEntry delegate, String reason, RequestContext context) throws ResourceDoesNotSupportOperationException { <START> throw new UnsupportedOperationException("delete not supported"); <END> }</code><technical_language>ResourceDoesNotSupportOperationException</technical_language>

Target:
protected void delete(LogEntry delegate, String reason, RequestContext context) throws ResourceDoesNotSupportOperationException { throw new ResourceDoesNotSupportOperationException(); }
Source:
<code>public void testTransformAdhocQueryToCheckPolicy_InValidDirection() { AdhocQueryRequestEventType eventA = new AdhocQueryRequestEventType(); eventA.setDirection("in_outbound"); CheckPolicyRequestType result = instance.transformAdhocQueryToCheckPolicy(eventA); <START> assertNull("We should expect null checkPolicyEngine ", result); <END> eventA.setDirection(null); assertNull(instance.transformAdhocQueryToCheckPolicy(eventA)); assertNull(instance.transformAdhocQueryToCheckPolicy(null)); }</code><technical_language>do here. remove lines 212 213</technical_language>

Target:
public void testTransformAdhocQueryToCheckPolicy_InValidDirection() { AdhocQueryRequestEventType eventA = new AdhocQueryRequestEventType(); eventA.setDirection("in_outbound"); CheckPolicyRequestType result = adhocQueryTransformHelp.transformAdhocQueryToCheckPolicy(eventA); assertNull("We should expect null checkPolicyEngine ", result); eventA.setDirection(null); assertNull(adhocQueryTransformHelp.transformAdhocQueryToCheckPolicy(eventA)); }
Source:
<code>public IBinder onBind(Intent intent) { Log.d(TAG, "Received onBind event"); <START> if(intent.getAction() != null && intent.getAction().equals("android.media.browse.MediaBrowserService")) { <END> return super.onBind(intent); } else { return mBinder; } }</code><technical_language>hm, "...".equals(intent.getAction()) TextUtils.equals(intent.getAction(), "...")? [fyi, commit amend + force push fine]</technical_language>

Target:
public IBinder onBind(Intent intent) { Log.d(TAG, "Received onBind event"); if(intent.getAction() != null && TextUtils.equals(intent.getAction(), MediaBrowserServiceCompat.SERVICE_INTERFACE)) { return super.onBind(intent); } else { return mBinder; } }
Source:
<code>protected void processNoPrimaryReference(APMSpanBuilder builder, TraceReporter reporter) { initTopLevelState(this, reporter); <START> List<String> traceIds = builder.references.stream().map(ref -> { <END> if (ref.getReferredTo() instanceof APMSpan) { return ((APMSpan)ref.getReferredTo()).getTraceContext().getTraceId(); } else if (ref.getReferredTo() instanceof APMSpanBuilder) { return ((APMSpanBuilder)ref.getReferredTo()).getState().get(Constants.HAWKULAR_APM_TRACEID).toString(); } return null; }).distinct().collect(Collectors.toList()); if (traceIds.size() > 0) { if (traceIds.size() > 1) { log.warning("References should all belong to the same 'trace' instance"); } if (builder.references.get(0).getReferredTo() instanceof APMSpan) { traceContext.initTraceState(((APMSpan)builder.references.get(0).getReferredTo()).getTraceContext()); } else if (builder.references.get(0).getReferredTo() instanceof APMSpanBuilder) { traceContext.initTraceState(((APMSpanBuilder)builder.references.get(0).getReferredTo()).getState()); } } processRemainingReferences(builder, null); makeInternalLink(builder); }</code><technical_language>a Set worry values</technical_language>

Target:
protected void processNoPrimaryReference(APMSpanBuilder builder, TraceReporter reporter) { initTopLevelState(this, reporter); Set<String> traceIds = builder.references.stream().map(ref -> { if (ref.getReferredTo() instanceof APMSpan) { return ((APMSpan)ref.getReferredTo()).getTraceContext().getTraceId(); } else if (ref.getReferredTo() instanceof APMSpanBuilder) { return ((APMSpanBuilder)ref.getReferredTo()).getState().get(Constants.HAWKULAR_APM_TRACEID).toString(); } log.warning("Reference refers to an unsupported SpanContext implementation: " + ref.getReferredTo()); return null; }).collect(Collectors.toSet()); if (traceIds.size() > 0) { if (traceIds.size() > 1) { log.warning("References should all belong to the same 'trace' instance"); } if (builder.references.get(0).getReferredTo() instanceof APMSpan) { traceContext.initTraceState(((APMSpan)builder.references.get(0).getReferredTo()).getTraceContext()); } else if (builder.references.get(0).getReferredTo() instanceof APMSpanBuilder) { traceContext.initTraceState(((APMSpanBuilder)builder.references.get(0).getReferredTo()).getState()); } } processRemainingReferences(builder, null); makeInternalLink(builder); }
Source:
<code>public final boolean mightBeRacilyClean(final int smudge_s, final int smudge_ns) { final int base = infoOffset + P_MTIME; final int mtime = NB.decodeInt32(info, base); <START> if (smudge_s < mtime) <END> return true; if (smudge_s == mtime) return smudge_ns <= NB.decodeInt32(info, base + 4); return false; }</code><technical_language>delete this rule? IIRC reason ensure smudged entries stayed smudged. If change smudge logic length based this longer sense, is a source of bugs selecting modified files potentially racily clean</technical_language>

Target:
public final boolean mightBeRacilyClean(final int smudge_s, final int smudge_ns) { final int base = infoOffset + P_MTIME; final int mtime = NB.decodeInt32(info, base); if (smudge_s == mtime) return smudge_ns <= NB.decodeInt32(info, base + 4); return false; }
Source:
<code>public TaskLockHelper getTaskLockHelper() { <START> return Preconditions.checkNotNull(taskLockHelper, "segmentLockHelper is not initialized yet"); <END> }</code><technical_language>Update error message "taskLockHelper" instead of "segmentLockHelper"</technical_language>

Target:
public TaskLockHelper getTaskLockHelper() { return Preconditions.checkNotNull(taskLockHelper, "taskLockHelper is not initialized yet"); }
Source:
<code>public void onUrlClick(@NonNull String href, @Nullable String titleString) { if (href.startsWith(" href = "https:" + href; } L.d("Link clicked was " + href); if (href.startsWith("/wiki/")) { PageTitle title = TextUtils.isEmpty(titleString) ? getWikiSite().titleForInternalLink(href) : PageTitle.withSeparateFragment(titleString, UriUtil.getFragment(href), getWikiSite()); onInternalLinkClicked(title); } else if (href.startsWith("#")) { onPageLinkClicked(href.substring(1)); } else if (fragment != null && fragment.getTitle() != null) { String parsoidFragmentLinkPrefix = "./" + fragment.getTitle().getText() + "#"; if (href.startsWith(parsoidFragmentLinkPrefix)) { onPageLinkClicked(href.substring(parsoidFragmentLinkPrefix.length())); } <START> } else { <END> Uri uri = Uri.parse(href); String authority = uri.getAuthority(); if (authority != null && WikiSite.supportedAuthority(authority) && uri.getPath().startsWith("/wiki/")) { WikiSite wiki = new WikiSite(authority, getWikiSite().languageCode()); PageTitle title = wiki.titleForUri(uri); onInternalLinkClicked(title); } else { if (context == null) { return; } if (href.startsWith("/w/")) { href = String.format("%1$s://%2$s", getWikiSite().scheme(), getWikiSite().authority()) + href; } handleExternalLink(context, Uri.parse(href)); } } }</code><technical_language>this change, this else clause reached, longer handing external links properly</technical_language>

Target:
public void onUrlClick(@NonNull String href, @Nullable String titleString) { if (href.startsWith(" href = "https:" + href; } L.d("Link clicked was " + href); if (href.startsWith("/wiki/")) { PageTitle title = TextUtils.isEmpty(titleString) ? getWikiSite().titleForInternalLink(href) : PageTitle.withSeparateFragment(titleString, UriUtil.getFragment(href), getWikiSite()); onInternalLinkClicked(title); } else if (href.startsWith("#")) { onPageLinkClicked(href.substring(1)); } else if (href.startsWith("./") && fragment != null && fragment.getTitle() != null) { String parsoidFragmentLinkPrefix = "./" + fragment.getTitle().getText() + "#"; if (href.startsWith(parsoidFragmentLinkPrefix)) { onPageLinkClicked(href.substring(parsoidFragmentLinkPrefix.length())); } } else { Uri uri = Uri.parse(href); String authority = uri.getAuthority(); if (authority != null && WikiSite.supportedAuthority(authority) && uri.getPath().startsWith("/wiki/")) { WikiSite wiki = new WikiSite(authority, getWikiSite().languageCode()); PageTitle title = wiki.titleForUri(uri); onInternalLinkClicked(title); } else { if (context == null) { return; } if (href.startsWith("/w/")) { href = String.format("%1$s://%2$s", getWikiSite().scheme(), getWikiSite().authority()) + href; } handleExternalLink(context, Uri.parse(href)); } } }
Source:
<code>private void removeDirectMember(PerunSession sess, Group group, Member member) throws InternalErrorException, NotGroupMemberException, GroupNotExistsException, WrongAttributeValueException, WrongReferenceAttributeValueException { lockGroupMembership(group, Collections.singletonList(member)); member.setSourceGroupId(group.getId()); getGroupsManagerImpl().removeMember(sess, group, member); if (this.getGroupsManagerImpl().isGroupMember(sess, group, member)) { getPerunBl().getAuditer().log(sess, new DirectMemberRemovedFromGroup(member, group)); return; } else { notifyMemberRemovalFromGroup(sess, group, member); getPerunBl().getAttributesManagerBl().removeAllAttributes(sess, member, group); getPerunBl().getAuditer().log(sess, new MemberRemovedFromGroupTotally(member, group)); } List<Integer> relations = groupsManagerImpl.getResultGroupsIds(sess, group.getId()); for (Integer groupId : relations) { removeRelationMembers(sess, groupsManagerImpl.getGroupById(sess, groupId), Collections.singletonList(member), group.getId()); } if (!VosManager.MEMBERS_GROUP.equals(group.getName())) { recalculateMemberGroupStatusRecursively(sess, member, group); } <START> addMemberToGroupsFromTriggerAttribute(sess, group, member); <END> }</code><technical_language>call this if member removed totally (line 1086/87 in this file)</technical_language>

Target:
private void removeDirectMember(PerunSession sess, Group group, Member member) throws InternalErrorException, NotGroupMemberException, GroupNotExistsException, WrongAttributeValueException, WrongReferenceAttributeValueException { lockGroupMembership(group, Collections.singletonList(member)); member.setSourceGroupId(group.getId()); getGroupsManagerImpl().removeMember(sess, group, member); if (this.getGroupsManagerImpl().isGroupMember(sess, group, member)) { getPerunBl().getAuditer().log(sess, new DirectMemberRemovedFromGroup(member, group)); return; } else { notifyMemberRemovalFromGroup(sess, group, member); getPerunBl().getAttributesManagerBl().removeAllAttributes(sess, member, group); getPerunBl().getAuditer().log(sess, new MemberRemovedFromGroupTotally(member, group)); } List<Integer> relations = groupsManagerImpl.getResultGroupsIds(sess, group.getId()); for (Integer groupId : relations) { removeRelationMembers(sess, groupsManagerImpl.getGroupById(sess, groupId), Collections.singletonList(member), group.getId()); } if (!VosManager.MEMBERS_GROUP.equals(group.getName())) { recalculateMemberGroupStatusRecursively(sess, member, group); } if (!getGroupsManagerImpl().isGroupMember(sess, group, member)) { addMemberToGroupsFromTriggerAttribute(sess, group, member); } }
Source:
<code><START> public void triggerOnScrollSlots() { <END> JavascriptExecutor js = driver; js.executeScript("window.scrollTo(0, document.body.scrollHeight);"); }</code><technical_language>[JavascriptActions.scrollToBottom()](<LINK_0></technical_language>

Target:
public void triggerOnScrollSlots() { jsActions.scrollToBottom(driver); }
Source:
<code>protected void setColumnOrder(int size) { int[] order = new int[size]; Arrays.setAll(order, <START> i -> i); <END> Table table = (Table) getControl(); table.setColumnOrder(order); }</code><technical_language>IntUnaryOperator.identity</technical_language>

Target:
protected void setColumnOrder(int size) { int[] order = new int[size]; Arrays.setAll(order, IntUnaryOperator.identity()); Table table = (Table) getControl(); table.setColumnOrder(order); }
Source:
<code>protected long getNextBackoff() { if (currentBackoff == null) { <START> currentBackoff = createBackoffBuilder().build(); <END> } try { return currentBackoff.nextBackOffMillis(); } catch (IOException e) { return BackOff.STOP; } }</code><technical_language>createBackoff()? modification</technical_language>

Target:
protected long getNextBackoff() { if (currentBackoff == null) { currentBackoff = createBackoff(); } try { return currentBackoff.nextBackOffMillis(); } catch (IOException e) { return BackOff.STOP; } }
Source:
<code><START> public Builder tag(@Nullable String tag) { <END> this.tag = tag; return this; }</code><technical_language>I allow null. This @NonNull, for field well</technical_language>

Target:
@NonNull public Builder tag(@Nullable String tag) { this.tag = tag; return this; }
Source:
<code>public String getPrefix(String namespaceURI) { for (Map.Entry<String, String> stringStringEntry : prefixToUri.entrySet()) { <START> if (stringStringEntry.getValue().equals(namespaceURI)) <END> return stringStringEntry.getKey(); } return null; }</code><technical_language>Please add curly brackets if statement</technical_language>

Target:
public String getPrefix(String namespaceURI) { for (Map.Entry<String, String> prexiToUriEntry : prefixToUri.entrySet()) { if (prexiToUriEntry.getValue().equals(namespaceURI)) { return prexiToUriEntry.getKey(); } } return null; }
Source:
<code>private void cacheWriterDeleteAllCall(Iterable<? extends Map.Entry<? extends K, ? extends V>> entries, Set<K> successes, Map<K, Exception> failures) { if (cacheWriter != null) { KeysIterable entriesKeys = new KeysIterable(entries); Set<K> toDelete = new HashSet<K>(); for (Map.Entry<? extends K, ? extends V> entry: entries) { toDelete.add(entry.getKey()); } try { <START> cacheWriter.deleteAll(entriesKeys); <END> successes.addAll(toDelete); } catch (Exception e) { for (Map.Entry<? extends K, ? extends V> entry: entries) { failures.put(entry.getKey(), e); } } } }</code><technical_language>I missing something, call deleteAll toDelete create a separate KeysIterable temp var? Actually, this method need accept iterable entries, keys</technical_language>

Target:
private void cacheWriterDeleteAllCall(Iterable<? extends Map.Entry<? extends K, ? extends V>> entries, Set<K> successes, Map<K, Exception> failures) { if (cacheWriter != null) { Set<K> toDelete = new HashSet<K>(); for (Map.Entry<? extends K, ? extends V> entry: entries) { toDelete.add(entry.getKey()); } try { cacheWriter.deleteAll(toDelete); successes.addAll(toDelete); } catch (Exception e) { for (Map.Entry<? extends K, ? extends V> entry: entries) { failures.put(entry.getKey(), e); } } } }
Source:
<code>private void checkObjectCollision(PackedObjectInfo obj) throws IOException { ObjectTypeAndSize info = openDatabase(obj, new ObjectTypeAndSize()); final byte[] readBuffer = buffer(); final byte[] curBuffer = new byte[readBuffer.length]; long sz = info.size; try (ObjectStream cur = readCurs.open(obj, info.type).openStream()) { if (cur.getSize() != sz) { throw new IOException(MessageFormat.format( JGitText.get().collisionOn, obj.name())); } try (InputStream pck = inflate(Source.DATABASE, sz)) { while (0 < sz) { int n = (int) Math.min(readBuffer.length, sz); IO.readFully(cur, curBuffer, 0, n); IO.readFully(pck, readBuffer, 0, n); for (int i = 0; i < n; i++) { if (curBuffer[i] != readBuffer[i]) { throw new IOException(MessageFormat.format( JGitText.get().collisionOn, obj.name())); } } sz -= n; } <START> stats.addNumBytesDuplicated(sz); <END> stats.incrementObjectsDuplicated(); } } catch (MissingObjectException notLocal) { } }</code><technical_language>this try block</technical_language>

Target:
private void checkObjectCollision(PackedObjectInfo obj) throws IOException { ObjectTypeAndSize info = openDatabase(obj, new ObjectTypeAndSize()); final byte[] readBuffer = buffer(); final byte[] curBuffer = new byte[readBuffer.length]; long sz = info.size; try (ObjectStream cur = readCurs.open(obj, info.type).openStream()) { if (cur.getSize() != sz) { throw new IOException(MessageFormat.format( JGitText.get().collisionOn, obj.name())); } try (InputStream pck = inflate(Source.DATABASE, sz)) { while (0 < sz) { int n = (int) Math.min(readBuffer.length, sz); IO.readFully(cur, curBuffer, 0, n); IO.readFully(pck, readBuffer, 0, n); for (int i = 0; i < n; i++) { if (curBuffer[i] != readBuffer[i]) { throw new IOException(MessageFormat.format( JGitText.get().collisionOn, obj.name())); } } sz -= n; } stats.increaseNumBytesDuplicated(sz); stats.incrementObjectsDuplicated(); } } catch (MissingObjectException notLocal) { } }
Source:
<code>private boolean isBoardNameEmpty(String newName) { <START> if (newName.equals("")) { <END> return true; } return false; }</code><technical_language>This replaced java return newName.isEmpty();</technical_language>

Target:
private boolean isBoardNameEmpty(String newName) { return newName.isEmpty(); }
Source:
<code>RepeatableIterator(TransactionMap<K, ?> transactionMap, K from, K to, boolean forEntries) { super(forEntries); keyType = transactionMap.map.getKeyType(); Transaction transaction = transactionMap.getTransaction(); this.transactionId = transaction.transactionId; BitSet committingTransactions; RootReference mapRootReference; do { committingTransactions = transactionMap.getCommittingTransactions(); mapRootReference = transactionMap.getRootReference(); } while (committingTransactions != transactionMap.getCommittingTransactions()); this.snapshotCursor = new Cursor<>(mapRootReference.root, from, to); this.snapshotCommittingTransactions = committingTransactions; <START> this.uncommittedCursor = new Cursor<>(transactionMap.map.flushAndGetRoot().root, from, to); <END> fetchNext(); }</code><technical_language>This is a snapshot beginning of current command, current snapshot, is same. changes this transaction current command started, not</technical_language>

Target:
RepeatableIterator(TransactionMap<K, ?> transactionMap, K from, K to, boolean forEntries) { super(forEntries); keyType = transactionMap.map.getKeyType(); Transaction transaction = transactionMap.getTransaction(); this.transactionId = transaction.transactionId; Snapshot snapshot = transactionMap.getRootSnapshot(); this.snapshotCursor = new Cursor<>(snapshot.root.root, from, to); this.snapshotCommittingTransactions = snapshot.committingTransactions; snapshot = transactionMap.getCurrentRootSnapshot(); this.uncommittedCursor = new Cursor<>(snapshot.root.root, from, to); fetchNext(); }
Source:
<code>protected boolean batchValidate(EObject eObject, DiagnosticChain diagnostics, Map<Object, Object> context) { IStatus status = Status.OK_STATUS; if (diagnostics != null) { if (!hasProcessed(eObject, context)) { <START> if (canCompareUpperGeLower(eObject)) { <END> status = batchValidator.validate( eObject, new NullProgressMonitor()); processed(eObject, context, status); appendDiagnostics(status, diagnostics); } } } return status.isOK(); }</code><technical_language>Lower/Upper constraint disabled. In this case, unrelated constraints element</technical_language>

Target:
protected boolean batchValidate(EObject eObject, DiagnosticChain diagnostics, Map<Object, Object> context) { IStatus status = Status.OK_STATUS; if (diagnostics != null) { if (!hasProcessed(eObject, context)) { status = batchValidator.validate( eObject, new NullProgressMonitor()); processed(eObject, context, status); appendDiagnostics(status, diagnostics); } } return status.isOK(); }
Source:
<code>protected abstract InputStream getInputStreamInternal(RepositoryPath repositoryPath) throws IOException; protected RepositoryInputStream decorate(RepositoryPath repositoryPath, InputStream is) throws IOException { if (is == null || is instanceof RepositoryInputStream) { return (RepositoryInputStream) is; } try { is = new MultipleDigestInputStream(is); } catch (NoSuchAlgorithmException e) { <START> throw new IOException(); <END> } return new RepositoryInputStream(repositoryPath, is); }</code><technical_language>lost e</technical_language>

Target:
protected abstract InputStream getInputStreamInternal(RepositoryPath repositoryPath) throws IOException; protected RepositoryInputStream decorate(RepositoryPath repositoryPath, InputStream is) throws IOException { if (is == null || is instanceof RepositoryInputStream) { return (RepositoryInputStream) is; } return new RepositoryInputStream(repositoryPath, is); }
Source:
<code><START> List<?> handle(SendMessage command) { <END> return Collections.emptyList(); }</code><technical_language>This method weird, especially in a context of ValidAggregate. discuss it</technical_language>

Target:
List<? extends EventMessage> handle(SendMessage command) { return Collections.emptyList(); }
Source:
<code>public static DefaultDruidTeslaAether getAetherClient(ExtensionsConfig config) { List<String> remoteUriList = config.getRemoteRepositories(); List<Repository> remoteRepositories = new ArrayList<Repository>(); for (String uri : remoteUriList) { try { URI u = new URI(uri); Repository r = new Repository(uri); if(u.getUserInfo() != null) { String[] auth = u.getUserInfo().split(":", 2); if(auth.length == 2) { r.setUsername(auth[0]); r.setPassword(auth[1]); <START> } <END> } remoteRepositories.add(r); } catch(URISyntaxException e) { throw Throwables.propagate(e); } } if (log.isTraceEnabled() || log.isDebugEnabled()) { return new DefaultDruidTeslaAether(config.getLocalRepository(), remoteRepositories); } PrintStream oldOut = System.out; try { System.setOut(new PrintStream( new OutputStream() { @Override public void write(int b) throws IOException { } @Override public void write(byte[] b) throws IOException { } @Override public void write(byte[] b, int off, int len) throws IOException { } } )); return new DefaultDruidTeslaAether(config.getLocalRepository(), remoteRepositories); } finally { System.setOut(oldOut); } }</code><technical_language>alert if auth length match</technical_language>

Target:
public static DefaultTeslaAether getAetherClient(ExtensionsConfig config) { List<String> remoteUriList = config.getRemoteRepositories(); List<Repository> remoteRepositories = Lists.newArrayList(); for (String uri : remoteUriList) { try { URI u = new URI(uri); Repository r = new Repository(uri); if(u.getUserInfo() != null) { String[] auth = u.getUserInfo().split(":", 2); if(auth.length == 2) { r.setUsername(auth[0]); r.setPassword(auth[1]); } else { log.warn( "Invalid credentials in repository URI, expecting [<user>:<password>], got [%s] for [%s]", u.getUserInfo(), uri ); } } remoteRepositories.add(r); } catch(URISyntaxException e) { throw Throwables.propagate(e); } } if (log.isTraceEnabled() || log.isDebugEnabled()) { return new DefaultTeslaAether( config.getLocalRepository(), remoteRepositories.toArray(new Repository[remoteRepositories.size()]) ); } PrintStream oldOut = System.out; try { System.setOut(new PrintStream( new OutputStream() { @Override public void write(int b) throws IOException { } @Override public void write(byte[] b) throws IOException { } @Override public void write(byte[] b, int off, int len) throws IOException { } } )); return new DefaultTeslaAether( config.getLocalRepository(), remoteRepositories.toArray(new Repository[remoteRepositories.size()]) ); } finally { System.setOut(oldOut); } }
Source:
<code>public void simplePlainSeleniumTest() { RemoteWebDriver driver = chrome.getWebDriver(); <START> driver.get("http://host.testcontainers.internal:" + 8080 + "/foo"); <END> List<WebElement> hElement = driver.findElementsByTagName("h"); assertTrue("The h element is found", hElement != null && hElement.size() > 0); }</code><technical_language>suggestion driver.get("http://host.testcontainers.internal:" + port + "/foo");</technical_language>

Target:
public void simplePlainSeleniumTest() { RemoteWebDriver driver = chrome.getWebDriver(); driver.get("http://host.testcontainers.internal:" + port + "/foo.html"); List<WebElement> hElement = driver.findElementsByTagName("h"); assertTrue("The h element is found", hElement != null && hElement.size() > 0); }
Source:
<code>protected List<Map<String, Object>> read(final List<String> names, final String key, final String value) { logger.debug("Reading key / value: / {}", key, value); final List<Map<String, Object>> listAttributes = new ArrayList<>(); if (key.equals(getIdAttribute())) { <START> final Map<String,Object> profile = profiles.get(value); <END> if (profile != null) { listAttributes.add(populateAttributes(profile, names)); } } else { for (Map<String,Object> profile : profiles.values()) { if (profile.get(key).equals(value)) { listAttributes.add(populateAttributes(profile, names)); } } } logger.debug("Found: {}", listAttributes); return listAttributes; }</code><technical_language>stream api too? this time, is reason for null values present in profiles :)</technical_language>

Target:
protected List<Map<String, Object>> read(final List<String> names, final String key, final String value) { logger.debug("Reading key / value: / {}", key, value); final List<Map<String, Object>> listAttributes; if (key.equals(getIdAttribute())) { listAttributes = new ArrayList<>(); final Map<String,Object> profile = profiles.get(value); if (profile != null) { listAttributes.add(populateAttributes(profile, names)); } } else { listAttributes = profiles.entrySet().stream() .filter(p -> p.getValue().get(key).equals(value)) .map(p -> populateAttributes(p.getValue(), names)) .collect(Collectors.toList()); } logger.debug("Found: {}", listAttributes); return listAttributes; }
Source:
<code>private void addIndexOptions(JsonObject field, EntityIndexBinding binding, String fieldName, String fieldType, Field.Index index, AnalyzerReference analyzerReference) { String elasticsearchIndex; switch ( index ) { case ANALYZED: case ANALYZED_NO_NORMS: elasticsearchIndex = canTypeBeAnalyzed( fieldType ) ? ANALYZED : NOT_ANALYZED; break; case NOT_ANALYZED: case NOT_ANALYZED_NO_NORMS: elasticsearchIndex = NOT_ANALYZED; break; case NO: elasticsearchIndex = "no"; default: throw new AssertionFailure( "Unexpected index type: " + index ); } field.addProperty( "index", elasticsearchIndex ); <START> if ( "no".equals( elasticsearchIndex ) && FieldHelper.isSortableField( binding, fieldName ) ) { <END> field.addProperty( "doc_values", true ); } if ( ANALYZED.equals( elasticsearchIndex ) && analyzerReference != null ) { String analyzerName = analyzerName( binding.getDocumentBuilder().getBeanClass(), fieldName, analyzerReference ); if ( analyzerName != null ) { field.addProperty( "analyzer", analyzerName ); } } }</code><technical_language>add a constant for "no"</technical_language>

Target:
private void addIndexOptions(JsonObject field, EntityIndexBinding binding, String fieldName, String fieldType, Field.Index index, AnalyzerReference analyzerReference) { String elasticsearchIndex; switch ( index ) { case ANALYZED: case ANALYZED_NO_NORMS: elasticsearchIndex = canTypeBeAnalyzed( fieldType ) ? ANALYZED : NOT_ANALYZED; break; case NOT_ANALYZED: case NOT_ANALYZED_NO_NORMS: elasticsearchIndex = NOT_ANALYZED; break; case NO: elasticsearchIndex = NOT_INDEXED; break; default: throw new AssertionFailure( "Unexpected index type: " + index ); } field.addProperty( "index", elasticsearchIndex ); if ( NOT_INDEXED.equals( elasticsearchIndex ) && FieldHelper.isSortableField( binding, fieldName ) ) { field.addProperty( "doc_values", true ); } if ( ANALYZED.equals( elasticsearchIndex ) && analyzerReference != null ) { String analyzerName = analyzerName( binding.getDocumentBuilder().getBeanClass(), fieldName, analyzerReference ); if ( analyzerName != null ) { field.addProperty( "analyzer", analyzerName ); } } }
Source:
<code>public void start() { this.executor = Executors.newSingleThreadExecutor(createThreadFactory()); if (this.recorder != null) { this.recorder.reset(); } this.identifierFactory.start(); final List<Scheduler> schedulers = new ArrayList<>(2); schedulers.add(new SessionExpirationScheduler(this.batcher, new ExpiredSessionRemover<>(this.factory, this.expirationListener))); if (this.maxActiveSessions >= 0) { schedulers.add(new SessionEvictionScheduler(this.cache.getName() + ".eviction", this.factory, this.batcher, this.dispatcherFactory, this.maxActiveSessions)); } this.scheduler = new Scheduler() { @Override public void schedule(String sessionId, ImmutableSessionMetaData metaData) { for (Scheduler scheduler1 : schedulers) { scheduler1.schedule(sessionId, metaData); } } @Override public void cancel(String sessionId) { for (Scheduler scheduler1 : schedulers) { scheduler1.cancel(sessionId); } } @Override public void cancel(Locality locality) { for (Scheduler scheduler1 : schedulers) { scheduler1.cancel(locality); } } @Override public void close() { <START> for (Scheduler scheduler1 : schedulers) { <END> scheduler1.close(); } } }; this.dispatcher = this.dispatcherFactory.createCommandDispatcher(this.cache.getName() + ".schedulers", this.scheduler); this.cache.addListener(this, this.filter); this.schedule(new SimpleLocality(false), new CacheLocality(this.cache)); }</code><technical_language>Rename "scheduler"</technical_language>

Target:
public void start() { if (this.recorder != null) { this.recorder.reset(); } this.identifierFactory.start(); this.cache.addListener(this, this.filter); this.expirationRegistration = this.expirationRegistrar.register(this.expirationListener); }