auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void testNoneStrategyWithStreams() { <START> String fakedStream1 = "fakedstream1"; <END> String fakedStream2 = "fakedstream2"; List<String> streams = new ArrayList<>(); streams.add(fakedStream1); streams.add(fakedStream2); Properties testConfig = TestUtils.getStandardProperties(); testConfig.setProperty(ConsumerConfigConstants.RECORD_PUBLISHER_TYPE, RecordPublisherType.EFO.toString()); testConfig.setProperty(ConsumerConfigConstants.EFO_REGISTRATION_TYPE, EFORegistrationType.NONE.toString()); streams.forEach( stream -> testConfig.setProperty(ConsumerConfigConstants.EFO_CONSUMER_ARN_PREFIX + "." + stream, stream) ); FanOutProperties fanOutProperties = new FanOutProperties(testConfig, streams); assertEquals(fanOutProperties.getStreamConsumerArns(), streams); }</code><technical_language>@xiaolong-sn 5 lines reduced List<String> streams = Arrays.asList("fakedstream1", "fakedstream2");</technical_language>

Target:
public void testNoneStrategyWithStreams() { List<String> streams = Arrays.asList("fakedstream1", "fakedstream2"); Properties testConfig = TestUtils.getStandardProperties(); testConfig.setProperty(ConsumerConfigConstants.RECORD_PUBLISHER_TYPE, RecordPublisherType.EFO.toString()); testConfig.setProperty(ConsumerConfigConstants.EFO_REGISTRATION_TYPE, EFORegistrationType.NONE.toString()); streams.forEach( stream -> testConfig.setProperty(ConsumerConfigConstants.EFO_CONSUMER_ARN_PREFIX + "." + stream, stream) ); FanOutProperties fanOutProperties = new FanOutProperties(testConfig, streams); Map<String, String> expectedStreamArns = new HashMap<>(); expectedStreamArns.put("fakedstream1", "fakedstream1"); expectedStreamArns.put("fakedstream2", "fakedstream2"); assertEquals(fanOutProperties.getStreamConsumerArn("fakedstream1"), Optional.of("fakedstream1")); assertEquals(fanOutProperties.getStreamConsumerArns(), Optional.of(expectedStreamArns)); }
Source:
<code>public void shouldGenerateEntityProxy() throws SecurityException, NoSuchMethodException { <START> Class<?> proxyClass = EmployeeProxy.class; <END> ProxyFor proxyAnnotation = EmployeeProxy.class.getAnnotation(ProxyFor.class); proxyClass.getMethod("getDisplayName"); proxyClass.getMethod("getSupervisorKey"); proxyClass.getMethod("getId"); proxyClass.getMethod("getSupervisor"); proxyClass.getMethod("setSupervisorKey", Long.class); proxyClass.getMethod("setSupervisor", EmployeeProxy.class); proxyClass.getMethod("setVersion", Integer.class); proxyClass.getMethod("stableId"); boolean filteredFields = false; try { proxyClass.getMethod("setId", Long.class); proxyClass.getMethod("getVersion", Long.class); } catch (NoSuchMethodException e) { filteredFields = true; } assertTrue(EntityProxy.class.isAssignableFrom(EmployeeProxy.class)); assertTrue(filteredFields); assertEquals(proxyAnnotation.value(), Employee.class); assertEquals(proxyAnnotation.locator(), EmployeeLocator.class); }</code><technical_language>label // a employee proxy a explanation, this for follow :)</technical_language>

Target:
public void shouldGenerateEntityProxy() throws SecurityException, NoSuchMethodException { Class<?> proxyClass = EmployeeProxy.class; ProxyFor proxyAnnotation = EmployeeProxy.class.getAnnotation(ProxyFor.class); proxyClass.getMethod("getDisplayName"); proxyClass.getMethod("getSupervisorKey"); proxyClass.getMethod("getId"); proxyClass.getMethod("getSupervisor"); proxyClass.getMethod("setSupervisorKey", Long.class); proxyClass.getMethod("setSupervisor", EmployeeProxy.class); proxyClass.getMethod("setVersion", Integer.class); proxyClass.getMethod("stableId"); boolean filteredFieldsWereNotGenerated = false; try { proxyClass.getMethod("setId", Long.class); proxyClass.getMethod("getVersion", Long.class); } catch (NoSuchMethodException e) { filteredFieldsWereNotGenerated = true; } assertTrue(filteredFieldsWereNotGenerated); assertTrue(EntityProxy.class.isAssignableFrom(EmployeeProxy.class)); assertEquals(proxyAnnotation.value(), Employee.class); assertEquals(proxyAnnotation.locator(), EmployeeLocator.class); }
Source:
<code>public Value get( String fieldName ) { if( current == null ) { throw new ClientException( "The current record is null. " + "It is either because the result contains no more record, or the record has not been set. " + "You could call `next()` to test if there are more records in this result " + <START> "and/or to iterate over result records." ); <END> } Value value = current.get( fieldName ); if( value == null ) { throw new ValueNotFoundException( fieldName ); } else { return value; } }</code><technical_language>I enough explain case of called next - this happen if user call next, check result of next true. If reached end of stream, result point record (assuming record). Eg. this altered highlight scenario more - In order access fields of a record in a result, call next() point result next record in result stream</technical_language>

Target:
public Value get( String fieldName ) { if( current == null ) { throw new ClientException( "In order to access fields of a record in a result, " + "you must first call next() to point the result to the next record in the result stream." ); } return current.get( fieldName ); }
Source:
<code>public QuotaManager(FrontendConfig frontendConfig) { try { quota = new JSONObject(frontendConfig.restRequestQuota); } catch (JSONException ex) { throw new IllegalStateException("Invalid config value: " + frontendConfig.restRequestQuota, ex); } for (RestMethod restMethod : RestMethod.values()) { <START> quotaMap.put(restMethod, new RejectThrottler(quota.getInt(restMethod.name()))); <END> } }</code><technical_language>TIL JSONException is a RuntimeException. Anyways, handle case config JSON entry for a HTTP verb quota.optString(method, -1)</technical_language>

Target:
public QuotaManager(FrontendConfig frontendConfig) { JSONObject quota = new JSONObject(frontendConfig.restRequestQuota); quotaMap = new HashMap<>(); for (RestMethod restMethod : RestMethod.values()) { quotaMap.put(restMethod, new RejectThrottler(quota.optInt(restMethod.name(), -1))); } }
Source:
<code>private void rematchAllNetworksAndRequests() { final long now = SystemClock.elapsedRealtime(); final NetworkReassignment changes = computeNetworkReassignment(); if (VDBG || DDBG) log(changes.toString()); applyNetworkReassignment(changes, <START> oldDefaultNetwork, now); <END> }</code><technical_language>hmm, I this get build pass</technical_language>

Target:
private void rematchAllNetworksAndRequests() { final long now = SystemClock.elapsedRealtime(); final NetworkReassignment changes = computeNetworkReassignment(); if (VDBG || DDBG) log(changes.toString()); applyNetworkReassignment(changes, now); }
Source:
<code>public ItemUseHandler getItemHandler() { <START> return e -> { Optional<Block> block = e.getClickedBlock(); if (block.isPresent()) { if (!block.get().getType().isInteractable()) { e.cancel(); } } else { e.cancel(); } <END> }; }</code><technical_language>Correct</technical_language>

Target:
public ItemUseHandler getItemHandler() { return e -> { Optional<Block> block = e.getClickedBlock(); if (!block.isPresent() || !block.get().getType().isInteractable()) { e.cancel(); } }; }
Source:
<code>public DynamicSourcesWorkingSetUpdater() { fUpdateJob= new Job(WorkingSetMessages.JavaSourcesWorkingSets_updating) { @Override protected IStatus run(IProgressMonitor monitor) { return updateElements(monitor); } }; fUpdateUIJob = new UpdateUIJob(); <START> fUpdateJob.setSystem(true); <END> fResourceChangeListener= new ResourceChangeListener(); ResourcesPlugin.getWorkspace().addResourceChangeListener(fResourceChangeListener, IResourceChangeEvent.POST_CHANGE); fJavaElementChangeListener= new JavaElementChangeListener(); JavaCore.addElementChangedListener(fJavaElementChangeListener, ElementChangedEvent.POST_CHANGE); }</code><technical_language>do: fUpdateUIJob.setSystem(true);</technical_language>

Target:
public DynamicSourcesWorkingSetUpdater() { fResourceChangeListener= new ResourceChangeListener(); ResourcesPlugin.getWorkspace().addResourceChangeListener(fResourceChangeListener, IResourceChangeEvent.POST_CHANGE); fJavaElementChangeListener= new JavaElementChangeListener(); JavaCore.addElementChangedListener(fJavaElementChangeListener, ElementChangedEvent.POST_CHANGE); }
Source:
<code>protected void run() throws Exception { if (createBranch) { final ObjectId head = db.resolve(Constants.HEAD); if (head == null) throw die(CLIText.get().onBranchToBeBorn); } CheckoutCommand command = new Git(db).checkout(); command.setCreateBranch(createBranch); command.setName(name); command.setForce(force); try { String oldBranch = db.getBranch(); Ref ref = command.call(); if (ref == null) return; if (Repository.shortenRefName(ref.getName()).equals(oldBranch)) { outw.println(MessageFormat.format( CLIText.get().alreadyOnBranch, name)); return; } if (createBranch) outw.println(MessageFormat.format( CLIText.get().switchedToNewBranch, Repository.shortenRefName(ref.getName()))); else outw.println(MessageFormat.format( CLIText.get().switchedToBranch, Repository.shortenRefName(ref.getName()))); } catch (RefNotFoundException e) { outw.println(MessageFormat.format( CLIText.get().pathspecDidNotMatch, name)); } catch (RefAlreadyExistsException e) { throw die(MessageFormat.format(CLIText.get().branchAlreadyExists, name)); } catch (CheckoutConflictException e) { outw.println(MessageFormat.format(CLIText.get().checkoutConflict, <START> name)); <END> } }</code><technical_language>name is branch name display path of conflicting file(s) match behavior of native git</technical_language>

Target:
protected void run() throws Exception { if (createBranch) { final ObjectId head = db.resolve(Constants.HEAD); if (head == null) throw die(CLIText.get().onBranchToBeBorn); } CheckoutCommand command = new Git(db).checkout(); command.setCreateBranch(createBranch); command.setName(name); command.setForce(force); try { String oldBranch = db.getBranch(); Ref ref = command.call(); if (ref == null) return; if (Repository.shortenRefName(ref.getName()).equals(oldBranch)) { outw.println(MessageFormat.format( CLIText.get().alreadyOnBranch, name)); return; } if (createBranch) outw.println(MessageFormat.format( CLIText.get().switchedToNewBranch, Repository.shortenRefName(ref.getName()))); else outw.println(MessageFormat.format( CLIText.get().switchedToBranch, Repository.shortenRefName(ref.getName()))); } catch (RefNotFoundException e) { outw.println(MessageFormat.format( CLIText.get().pathspecDidNotMatch, name)); } catch (RefAlreadyExistsException e) { throw die(MessageFormat.format(CLIText.get().branchAlreadyExists, name)); } catch (CheckoutConflictException e) { outw.println(CLIText.get().checkoutConflict); for (String path : e.getConflictingPaths()) outw.println(MessageFormat.format( CLIText.get().checkoutConflictPathLine, path)); } }
Source:
<code>public void testAssignExecutorInvalidExecutor() throws Exception { final ExecutableFlow flow = TestUtils.createExecutableFlow("exectest1", "exec1"); this.executionFlowDao.uploadExecutableFlow(flow); assertThatThrownBy( <START> () -> this.assignExecutor.assignExecutor(flow.getExecutionId(), 1)) <END> .isInstanceOf(ExecutorManagerException.class) .hasMessageContaining("non-existent executor"); }</code><technical_language>assignExecutor(1, flow.getExecutionId())? do 1 is a non-existent executor</technical_language>

Target:
public void testAssignExecutorInvalidExecutor() throws Exception { final ExecutableFlow flow = TestUtils.createExecutableFlow("exectest1", "exec1"); this.executionFlowDao.uploadExecutableFlow(flow); assertThatThrownBy( () -> this.assignExecutor.assignExecutor(1, flow.getExecutionId())) .isInstanceOf(ExecutorManagerException.class) .hasMessageContaining("non-existent executor"); }
Source:
<code>public void test() throws Exception { URI uri = new URI("<LINK_0>"); UserTuneElement.Builder builder = UserTuneElement.getBuilder(); UserTuneElement userTuneElement1 = builder.setArtist("Yes") .setLength(686) .setRating(8) .setSource("Yessongs") .setTitle("Heart of the Sunrise") .setTrack(3) .setUri(uri) .build(); IntegrationTestRosterUtil.ensureBothAccountsAreSubscribedToEachOther(conOne, conTwo, timeout); final SimpleResultSyncPoint userTuneReceived = new SimpleResultSyncPoint(); final UserTuneListener userTuneListener = new UserTuneListener() { @Override public void onUserTuneUpdated(BareJid jid, Message message, UserTuneElement userTuneElement) { <START> if (isEqual(userTuneElement,userTuneElement1)) { <END> userTuneReceived.signal(); } } }; utm2.addUserTuneListener(userTuneListener); try { utm1.publishUserTune(userTuneElement1); userTuneReceived.waitForResult(timeout); } finally { utm2.removeUserTuneListener(userTuneListener); } }</code><technical_language>add equals() method UserTuneElement</technical_language>

Target:
public void test() throws Exception { URI uri = new URI("<LINK_0>"); UserTuneElement.Builder builder = UserTuneElement.getBuilder(); UserTuneElement userTuneElement1 = builder.setArtist("Yes") .setLength(686) .setRating(8) .setSource("Yessongs") .setTitle("Heart of the Sunrise") .setTrack("3") .setUri(uri) .build(); IntegrationTestRosterUtil.ensureBothAccountsAreSubscribedToEachOther(conOne, conTwo, timeout); final SimpleResultSyncPoint userTuneReceived = new SimpleResultSyncPoint(); final UserTuneListener userTuneListener = new UserTuneListener() { @Override public void onUserTuneUpdated(BareJid jid, Message message, UserTuneElement userTuneElement) { if (userTuneElement.equals(userTuneElement1)) { userTuneReceived.signal(); } } }; utm2.addUserTuneListener(userTuneListener); try { utm1.publishUserTune(userTuneElement1); userTuneReceived.waitForResult(timeout); } finally { utm2.removeUserTuneListener(userTuneListener); } }
Source:
<code>public PDOMCPPMemberBlock(PDOMLinkage linkage) throws CoreException { Database db = linkage.getDB(); this.linkage = linkage; this.record = db.malloc(RECORD_SIZE); db.clearBytes(record, RECORD_SIZE); <START> updateNextPosition(); <END> }</code><technical_language>Redundant call</technical_language>

Target:
public PDOMCPPMemberBlock(PDOMLinkage linkage) throws CoreException { Database db = linkage.getDB(); this.linkage = linkage; this.record = db.malloc(RECORD_SIZE); db.clearBytes(record, RECORD_SIZE); }
Source:
<code>private Map<Account.Id, AccountGroupMember> getMembers( final AccountGroup.Id groupId) throws OrmException, NoSuchGroupException { final Map<Account.Id, AccountGroupMember> memberMap = Maps.newHashMap(); final List<AccountGroupMember> members = <START> db.accountGroupMembers().byGroup(groupId).toList(); <END> for (final AccountGroupMember m : members) { memberMap.put(m.getAccountId(), m); } return memberMap; }</code><technical_language>Inline this for loop avoid toList(). I improves readability runtime performance</technical_language>

Target:
private Map<Account.Id, AccountGroupMember> getMembers( final AccountGroup.Id groupId) throws OrmException, NoSuchGroupException { final Map<Account.Id, AccountGroupMember> members = Maps.newHashMap(); for (final AccountGroupMember m : db.accountGroupMembers().byGroup(groupId)) { members.put(m.getAccountId(), m); } return members; }
Source:
<code>public TimeLockStatus getStatus() { <START> if (healthCheckSupplier != null) { <END> return healthCheckSupplier.get().getStatus(); } else { return TimeLockStatus.NO_QUORUM; } }</code><technical_language>this Optional nullable</technical_language>

Target:
public TimeLockStatus getStatus() { return healthCheckSupplier.get().getStatus(); }
Source:
<code>public void shutdown() { long startTime = SystemTime.getInstance().milliseconds(); try { logger.info("shutdown started"); if (scheduler != null) { scheduler.shutdown(); <START> if (!scheduler.awaitTermination(1, TimeUnit.DAYS)) { <END> logger.error("Could not terminate all tasks after scheduler shutdown"); } } if (networkServer != null) { networkServer.shutdown(); } if (requestHandlerPool != null) { requestHandlerPool.shutdown(); } if (replicationManager != null) { replicationManager.shutdown(); } if (storeManager != null) { storeManager.shutdown(); } if (connectionPool != null) { connectionPool.shutdown(); } if (reporter != null) { reporter.stop(); } if (notificationSystem != null) { try { notificationSystem.close(); } catch (IOException e) { logger.error("Error while closing notification system.", e); } } logger.info("shutdown completed"); } catch (Exception e) { logger.error("Error while shutting down server", e); } finally { shutdownLatch.countDown(); long processingTime = SystemTime.getInstance().milliseconds() - startTime; metrics.serverShutdownTimeInMs.update(processingTime); logger.info("Server shutdown time in Ms " + processingTime); } }</code><technical_language>again, 1 day</technical_language>

Target:
public void shutdown() { long startTime = SystemTime.getInstance().milliseconds(); try { logger.info("shutdown started"); if (scheduler != null) { scheduler.shutdown(); if (!scheduler.awaitTermination(5, TimeUnit.MINUTES)) { logger.error("Could not terminate all tasks after scheduler shutdown"); } } if (networkServer != null) { networkServer.shutdown(); } if (requestHandlerPool != null) { requestHandlerPool.shutdown(); } if (replicationManager != null) { replicationManager.shutdown(); } if (storageManager != null) { storageManager.shutdown(); } if (connectionPool != null) { connectionPool.shutdown(); } if (reporter != null) { reporter.stop(); } if (notificationSystem != null) { try { notificationSystem.close(); } catch (IOException e) { logger.error("Error while closing notification system.", e); } } logger.info("shutdown completed"); } catch (Exception e) { logger.error("Error while shutting down server", e); } finally { shutdownLatch.countDown(); long processingTime = SystemTime.getInstance().milliseconds() - startTime; metrics.serverShutdownTimeInMs.update(processingTime); logger.info("Server shutdown time in Ms " + processingTime); } }
Source:
<code>public Orchestrator build() { String version = getSonarVersion(); Configuration.Builder configBuilder = Configuration.builder(); Configuration finalConfig = configBuilder .addConfiguration(config) .addMap(overriddenProperties) .setUpdateCenter(getUpdateCenter()) .build(); checkState(!isEmpty(version), "Missing Sonar version"); this.distribution.setVersion(Version.create(version)); <START> this.distribution.addPluginLocation(ResourceLocation.create("/com/sonar/orchestrator/sonar-reset-data-plugin-1.0-SNAPSHOT.jar")); <END> return new Orchestrator(finalConfig, distribution, startupLogWatcher); }</code><technical_language>so, removed dead code related support for SQ prior 5.6, right</technical_language>

Target:
public Orchestrator build() { getSonarVersion().ifPresent(s -> this.distribution.setVersion(Version.create(s))); checkState(distribution.getZipFile().isPresent() || distribution.version().isPresent(), "Version or path to ZIP of SonarQube is missing"); Configuration.Builder configBuilder = Configuration.builder(); Configuration finalConfig = configBuilder .addConfiguration(config) .addMap(overriddenProperties) .setUpdateCenter(getUpdateCenter()) .build(); this.distribution.addPluginLocation(ResourceLocation.create("/com/sonar/orchestrator/sonar-reset-data-plugin-1.0-SNAPSHOT.jar")); return new Orchestrator(finalConfig, distribution, startupLogWatcher); }
Source:
<code>public void testCanDoActionNonExistingStorageDomain() { <START> assertFalse("canDoActtion shouldn't be possible for a non-existant storage domain", command.canDoAction()); <END> command.setActionMessageParameters(); List<String> messages = command.getReturnValue().getCanDoActionMessages(); assertEquals("Wrong number of messages", 3, messages.size()); assertEquals("Wrong message", VdcBllMessages.ACTION_TYPE_FAILED_STORAGE_DOMAIN_NOT_EXIST.name(), messages.get(0)); assertEquals("Wrong message", VdcBllMessages.VAR__TYPE__STORAGE__DOMAIN.name(), messages.get(1)); assertEquals("Wrong message", VdcBllMessages.VAR__ACTION__REMOVE.name(), messages.get(2)); }</code><technical_language>called automatically</technical_language>

Target:
public void testCanDoActionNonExistingStorageDomain() { assertFalse("canDoActtion shouldn't be possible for a non-existant storage domain", command.canDoAction()); checkMessages(command, VdcBllMessages.ACTION_TYPE_FAILED_STORAGE_DOMAIN_NOT_EXIST); }
Source:
<code>public void shouldCallProcedureWithByteArrayWithParameterAndYield() throws Throwable { try ( Transaction ignore = db.beginTx() ) { <START> Result res = db.execute( "CALL org.neo4j.procedure.bytesWithDefault([7,8,9]) YIELD bytes RETURN bytes" ); <END> assertThat( res.columnAs( "bytes" ).next(), equalTo( new byte[]{7, 8, 9} ) ); assertFalse( res.hasNext() ); } }</code><technical_language>is onboard list syntax for byte arrays</technical_language>

Target:
public void shouldCallProcedureWithByteArrayWithParameterAndYield() throws Throwable { try ( Transaction ignore = db.beginTx() ) { Result res = db.execute( "WITH $param AS b CALL org.neo4j.procedure.incrBytes(b) YIELD bytes RETURN bytes", map( "param", new byte[]{7, 8, 9} ) ); assertThat( res.columnAs( "bytes" ).next(), equalTo( new byte[]{8, 9, 10} ) ); assertFalse( res.hasNext() ); } }
Source:
<code>public void testNoSessionKey() throws Exception { exception.expectMessage("attribute sessionKey must be specified"); exception.expect(ConfigurationException.class); pipe.setValue("val"); pipe.configure(); PipeRunResult res = pipe.doPipe(null, session); <START> assertFalse(res.getPipeForward().getName().isEmpty()); <END> }</code><technical_language>expecting doPipe throw exception? this assertion is required :)</technical_language>

Target:
public void testNoSessionKey() throws Exception { exception.expectMessage("attribute sessionKey must be specified"); exception.expect(ConfigurationException.class); pipe.setValue("val"); pipe.configure(); pipe.doPipe(null, session); fail("this is expected to fail"); }
Source:
<code>public void testUnregisterTwice() { Handler<Message> handler = msg -> {}; eb.registerHandler(ADDRESS1, handler, ar -> { assertTrue(ar.succeeded()); EventBusRegistration reg = ar.result(); reg.unregister(); reg.unregister(); }); <START> testComplete(); <END> }</code><technical_language>Need await in async tests. test complete handler is called</technical_language>

Target:
public void testUnregisterTwice() { Handler<Message> handler = msg -> {}; Registration reg = eb.registerHandler(ADDRESS1, handler); reg.unregister(); reg.unregister(); testComplete(); }
Source:
<code>void convertWhenSimpleWeeksShouldReturnPeriod(ConversionService conversionService) { <START> assertThat(convert(conversionService, "10W")).isEqualTo(Period.ofWeeks(10)); <END> assertThat(convert(conversionService, "+10w")).isEqualTo(Period.ofWeeks(10)); assertThat(convert(conversionService, "-10W")).isEqualTo(Period.ofWeeks(-10)); }</code><technical_language>I this testing lower-case w (the line is testing upper-case W)</technical_language>

Target:
void convertWhenSimpleWeeksShouldReturnPeriod(ConversionService conversionService) { assertThat(convert(conversionService, "10w")).isEqualTo(Period.ofWeeks(10)); assertThat(convert(conversionService, "10W")).isEqualTo(Period.ofWeeks(10)); assertThat(convert(conversionService, "+10w")).isEqualTo(Period.ofWeeks(10)); assertThat(convert(conversionService, "-10W")).isEqualTo(Period.ofWeeks(-10)); }
Source:
<code>public void testOutstandingSplitSize() throws Exception { DataSize maxOutstandingSplitsSize = new DataSize(1, MEGABYTE); HiveSplitSource hiveSplitSource = new HiveSplitSource( "client-id", SESSION, "database", "table", TupleDomain.all(), 10, 10000, maxOutstandingSplitsSize, new TestingHiveSplitLoader(), Executors.newFixedThreadPool(5), new CounterStat()); InternalHiveSplit testSplit = new InternalHiveSplit( "partition-name", "path", 0, 100, 100, new Properties(), ImmutableList.of(new HivePartitionKey("pk_col", "pk_value")), <START> ImmutableList.of(new InternalHiveSplit.InternalHiveBlock(0, 100, ImmutableList.of(HostAddress.fromString("localhost")))), <END> OptionalInt.empty(), true, false, ImmutableMap.of()); int testSplitSizeInBytes = testSplit.getEstimatedSizeInBytes(); int maxSplitCount = toIntExact(maxOutstandingSplitsSize.toBytes()) / testSplitSizeInBytes; for (int i = 0; i < maxSplitCount; i++) { hiveSplitSource.addToQueue(testSplit); assertEquals(hiveSplitSource.getBufferedInternalSplitCount(), i + 1); } assertEquals(getFutureValue(hiveSplitSource.getNextBatch(maxSplitCount)).size(), maxSplitCount); for (int i = 0; i < maxSplitCount; i++) { hiveSplitSource.addToQueue(testSplit); assertEquals(hiveSplitSource.getBufferedInternalSplitCount(), i + 1); } try { hiveSplitSource.addToQueue(testSplit); fail("expect failure"); } catch (PrestoException e) { assertContains(e.getMessage(), "Split buffering for database.table exceeded memory limit"); } }</code><technical_language>Qualifying InternalHiveBlock needed</technical_language>

Target:
public void testOutstandingSplitSize() throws Exception { DataSize maxOutstandingSplitsSize = new DataSize(1, MEGABYTE); HiveSplitSource hiveSplitSource = new HiveSplitSource( SESSION, "database", "table", TupleDomain.all(), 10, 10000, maxOutstandingSplitsSize, new TestingHiveSplitLoader(), Executors.newFixedThreadPool(5), new CounterStat()); InternalHiveSplit testSplit = new InternalHiveSplit( "partition-name", "path", 0, 100, 100, new Properties(), ImmutableList.of(new HivePartitionKey("pk_col", "pk_value")), ImmutableList.of(new InternalHiveBlock(0, 100, ImmutableList.of(HostAddress.fromString("localhost")))), OptionalInt.empty(), true, false, ImmutableMap.of()); int testSplitSizeInBytes = testSplit.getEstimatedSizeInBytes(); int maxSplitCount = toIntExact(maxOutstandingSplitsSize.toBytes()) / testSplitSizeInBytes; for (int i = 0; i < maxSplitCount; i++) { hiveSplitSource.addToQueue(testSplit); assertEquals(hiveSplitSource.getBufferedInternalSplitCount(), i + 1); } assertEquals(getFutureValue(hiveSplitSource.getNextBatch(maxSplitCount)).size(), maxSplitCount); for (int i = 0; i < maxSplitCount; i++) { hiveSplitSource.addToQueue(testSplit); assertEquals(hiveSplitSource.getBufferedInternalSplitCount(), i + 1); } try { hiveSplitSource.addToQueue(testSplit); fail("expect failure"); } catch (PrestoException e) { assertContains(e.getMessage(), "Split buffering for database.table exceeded memory limit"); } }
Source:
<code>public void testHashPipe() throws Exception { <START> session.put("key", "3:2342:2342" ); <END> pipe.configure(); PipeRunResult res = doPipe(pipe, "password", session); assertEquals("success", res.getPipeForward().getName()); }</code><technical_language>This session do I correct</technical_language>

Target:
public void testHashPipe() throws Exception { pipe.configure(); PipeRunResult res = doPipe(pipe, "password", session); assertTrue(PasswordHash.validatePassword("password", res.getResult().asString())); int hashLength = res.getResult().asString().length(); assertEquals("success", res.getPipeForward().getName()); assertTrue(hashLength == 135); }
Source:
<code><START> public boolean remove(R resource) { Thread myself = Thread.currentThread(); return resources.remove(myself, resource); } <END></code><technical_language>I this work in combination existing cleanupIdleConnections method. called idle connection sweeper's pool of threads threads rpc requests "myself" key wrong</technical_language>

Target:
public boolean remove(R resource) { return resources.values().remove(resource); }
Source:
<code>public void testGetNonePermittedStorageDomains() { List<StorageDomain> result = dao.getPermittedStorageDomainsByStoragePool(EXISTING_USER_ID, ActionGroup.CONSUME_QUOTA, <START> Guid.createGuidFromString("6d849ebf-755f-4552-ad09-9a090cda105d")); <END> assertNotNull(result); assertTrue(result.isEmpty()); }</code><technical_language>new Guid(String) - this literal null</technical_language>

Target:
public void testGetNonePermittedStorageDomains() { List<StorageDomain> result = dao.getPermittedStorageDomainsByStoragePool(EXISTING_USER_ID, ActionGroup.CONSUME_QUOTA, new Guid("6d849ebf-755f-4552-ad09-9a090cda105d")); assertNotNull(result); assertTrue(result.isEmpty()); }
Source:
<code>ColumnIndexWriterProjection(StreamInput in) throws IOException { super(in); if (in.readBoolean()) { targetColsSymbolsExclPartition = Symbols.listFromStream(in); } else { targetColsSymbolsExclPartition = Collections.emptyList(); } if (in.readBoolean()) { int length = in.readVInt(); targetColsExclPartitionCols = new ArrayList<>(length); for (int i = 0; i < length; i++) { targetColsExclPartitionCols.add(Reference.fromStream(in)); } } else { targetColsExclPartitionCols = Collections.emptyList(); } ignoreDuplicateKeys = in.readBoolean(); if (in.readBoolean()) { int mapSize = in.readVInt(); onDuplicateKeyAssignments = new HashMap<>(mapSize); for (int i = 0; i < mapSize; i++) { onDuplicateKeyAssignments.put(Reference.fromStream(in), Symbols.fromStream(in)); } } else { onDuplicateKeyAssignments = Collections.emptyMap(); } <START> if (in.getVersion().onOrAfter(Version.V_4_1_0)) { <END> int mapSize = in.readVInt(); allTargetColumns = new ArrayList<>(); for (int i = 0; i < mapSize; i++) { allTargetColumns.add(Reference.fromStream(in)); } } else { allTargetColumns = List.of(); } }</code><technical_language>This V_4_2_0</technical_language>

Target:
ColumnIndexWriterProjection(StreamInput in) throws IOException { super(in); if (in.readBoolean()) { targetColsSymbolsExclPartition = Symbols.listFromStream(in); } else { targetColsSymbolsExclPartition = Collections.emptyList(); } if (in.readBoolean()) { int length = in.readVInt(); targetColsExclPartitionCols = new ArrayList<>(length); for (int i = 0; i < length; i++) { targetColsExclPartitionCols.add(Reference.fromStream(in)); } } else { targetColsExclPartitionCols = Collections.emptyList(); } ignoreDuplicateKeys = in.readBoolean(); if (in.readBoolean()) { int mapSize = in.readVInt(); onDuplicateKeyAssignments = new HashMap<>(mapSize); for (int i = 0; i < mapSize; i++) { onDuplicateKeyAssignments.put(Reference.fromStream(in), Symbols.fromStream(in)); } } else { onDuplicateKeyAssignments = Collections.emptyMap(); } if (in.getVersion().onOrAfter(Version.V_4_2_0)) { int mapSize = in.readVInt(); allTargetColumns = new ArrayList<>(); for (int i = 0; i < mapSize; i++) { allTargetColumns.add(Reference.fromStream(in)); } } else { allTargetColumns = List.of(); } }
Source:
<code>private static IAtomContainer parseSmiles(String smiles) throws CDKException { parser = new SmilesParser(SilentChemObjectBuilder.getInstance()); IAtomContainer molecule = parser.parseSmiles(smiles); AtomContainerManipulator.percieveAtomTypesAndConfigureUnsetProperties(molecule); CDKHydrogenAdder hAdder = CDKHydrogenAdder.getInstance(molecule.getBuilder()); <START> hAdder.addImplicitHydrogens(molecule); <END> AtomContainerManipulator.convertImplicitToExplicitHydrogens(molecule); Aromaticity.cdkLegacy().apply(molecule); return molecule; }</code><technical_language>Mmm... this needed... I this is done SmilesParser already. @johnmay</technical_language>

Target:
private static IAtomContainer parseSmiles(String smiles) throws CDKException { parser = new SmilesParser(SilentChemObjectBuilder.getInstance()); IAtomContainer molecule = parser.parseSmiles(smiles); AtomContainerManipulator.percieveAtomTypesAndConfigureUnsetProperties(molecule); AtomContainerManipulator.convertImplicitToExplicitHydrogens(molecule); Aromaticity.cdkLegacy().apply(molecule); return molecule; }
Source:
<code>public void testGetMessage() throws Exception { Field[] fields = BKException.Code.class.getFields(); int count = 0; for (Field f : fields) { if (f.getType() == Integer.TYPE && !f.getName().equals("UNINITIALIZED")) { int code = f.getInt(null); String msg = BKException.getMessage(code); if (code == UnexpectedConditionException) { <START> assertThat(msg, equalTo("Unexpected condition")); <END> } else { assertThat("failure on code " + f.getName(), msg, not(equalTo("Unexpected condition"))); } count++; } } assertThat(count, greaterThan(2)); }</code><technical_language>reason "assertEquals"</technical_language>

Target:
public void testGetMessage() throws Exception { Field[] fields = BKException.Code.class.getFields(); int count = 0; for (Field f : fields) { if (f.getType() == Integer.TYPE && !f.getName().equals("UNINITIALIZED")) { int code = f.getInt(null); String msg = BKException.getMessage(code); if (code == UnexpectedConditionException) { assertEquals("Unexpected condition", msg); } else { assertNotEquals("failure on code " + f.getName(), "Unexpected condition", msg); } count++; } } assertTrue(count > 2); }
Source:
<code>private void handleTagDeleted(ContentTagDeletedEvent tagDeletedEvent) { if (!isNotableTagName(tagDeletedEvent.getDeletedTagInfo().getName())) { return; } try { long contentID = tagDeletedEvent.getDeletedTagInfo().getContentID(); Content content = Case.getCurrentCaseThrows().getSleuthkitCase().getContentById(contentID); TagsManager tagsManager = Case.getCurrentCaseThrows().getServices().getTagsManager(); List<ContentTag> tags = tagsManager.getContentTagsByContent(content); <START> AbstractFile af = getAbstractFile(content); <END> if (af == null) return; if (hasNotableTag(tags)) { String comment = getMostRecentNotableComment(tags); setContentKnownStatus(af, comment, TskData.FileKnown.BAD, false); } else { String comment = tagDeletedEvent.getDeletedTagInfo().getComment(); setContentKnownStatus(af, comment, TskData.FileKnown.UNKNOWN, true); } } catch (TskCoreException | NoCurrentCaseException ex) { LOGGER.log(Level.SEVERE, "Failed to find content", ex); return; } }</code><technical_language>getAbstractFile hides fact casting. clearer if knowledge AbstractFiles tagged, appearances contrary, calling getAbstractFileById instead of getContentById</technical_language>

Target:
private void handleTagDeleted(ContentTagDeletedEvent tagDeletedEvent) { if (!isNotableTagName(tagDeletedEvent.getDeletedTagInfo().getName())) { return; } Long contentID = null; try { contentID = tagDeletedEvent.getDeletedTagInfo().getContentID(); Content content = Case.getCurrentCaseThrows().getSleuthkitCase().getContentById(contentID); TagsManager tagsManager = Case.getCurrentCaseThrows().getServices().getTagsManager(); List<ContentTag> tags = tagsManager.getContentTagsByContent(content); AbstractFile af = Case.getCurrentCaseThrows().getSleuthkitCase().getAbstractFileById(contentID); if (af == null) { return; } CorrelationAttributeInstance eamArtifact = CorrelationAttributeUtil.makeCorrAttrFromFile(af); AttributeUpdate attrUpdate = assessTagDelete(dbManager, eamArtifact, tags); setContentKnownStatus(af, attrUpdate.getComment(), attrUpdate.getFileKnown()); } catch (TskCoreException | NoCurrentCaseException ex) { LOGGER.log(Level.SEVERE, "Failed to find content", ex); } catch (CentralRepoException | CorrelationAttributeNormalizationException ex) { LOGGER.log(Level.SEVERE, "Failed to query database for current correlation attribute instance for content with ID: " + contentID, ex); } }
Source:
<code>CheckoutConflictException(String message, List<String> conflictingPaths) { super(message); <START> this.conflictingPaths=(conflictingPaths); <END> }</code><technical_language>Style-nit: Unnecessary parens. If a place need parens, right in assignment of variable another. :-)</technical_language>

Target:
CheckoutConflictException(String message, List<String> conflictingPaths) { super(message); this.conflictingPaths = conflictingPaths; }
Source:
<code>public static boolean isJenkinsAvailable() { logger.trace("Checking Jenkins availability..."); try { HttpURLConnection conn = (HttpURLConnection) new URL(JENKINS_URL).openConnection(); try (AutoCloseable ac = conn::disconnect) { conn.setConnectTimeout(JENKINS_TIMEOUT); if (conn.getResponseCode() == HttpURLConnection.HTTP_OK) { logger.trace("Jenkins is available at {}", JENKINS_URL); return true; } else { <START> throw new ConnectException(); <END> } } } catch (IOException e) { e.printStackTrace(); logger.warn("Could not connect to Jenkins at {}", JENKINS_URL); } catch (Exception e) { logger.warn("Failed to disconnect an HttpURLConnection to Jenkins at {}", JENKINS_URL); } return false; }</code><technical_language>I get throw exception - coudn't return false;</technical_language>

Target:
public static boolean isJenkinsAvailable() { logger.trace("Checking Jenkins availability..."); try { HttpURLConnection conn = (HttpURLConnection) new URL(JENKINS_URL).openConnection(); try (AutoCloseable ac = conn::disconnect) { conn.setConnectTimeout(JENKINS_TIMEOUT); if (conn.getResponseCode() == HttpURLConnection.HTTP_OK) { logger.trace("Jenkins is available at {}", JENKINS_URL); return true; } } } catch (Exception e) { logger.warn("Could not connect to Jenkins at - {}", JENKINS_URL, e.getMessage()); } return false; }
Source:
<code><START> public void execute() throws InsufficientCapacityException, ConcurrentOperationException { <END> User callerUser = _accountService.getActiveUser(CallContext.current().getCallingUserId()); Account callerAccount = _accountService.getActiveAccountById(callerUser.getAccountId()); Network network = _networkService.getNetwork(id); if (network == null) { throw new InvalidParameterValueException("Couldn't find network by id"); } Network result = _networkService.migrateGuestNetwork(getId(), getNetworkOfferingId(), callerAccount, callerUser, getResume()); if (result != null) { NetworkResponse response = _responseGenerator.createNetworkResponse(ResponseView.Restricted, result); response.setResponseName(getCommandName()); setResponseObject(response); } else { throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, "Failed to update network"); } }</code><technical_language>if I missing here, I do exceptions launched; is necessary add throws InsufficientCapacityException, ConcurrentOperationException? compiler complain removing them</technical_language>

Target:
public void execute() { User callerUser = _accountService.getActiveUser(CallContext.current().getCallingUserId()); Account callerAccount = _accountService.getActiveAccountById(callerUser.getAccountId()); Network network = _networkService.getNetwork(id); if (network == null) { throw new InvalidParameterValueException("Couldn't find network by id"); } Network result = _networkService.migrateGuestNetwork(getId(), getNetworkOfferingId(), callerAccount, callerUser, getResume()); if (result != null) { NetworkResponse response = _responseGenerator.createNetworkResponse(ResponseView.Restricted, result); response.setResponseName(getCommandName()); setResponseObject(response); } else { throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, "Failed to update network"); } }
Source:
<code>public String getDocumentation() { String documentation = super.getDocumentation(); <START> if (documentation == null || documentation.isEmpty()) { <END> return ""; } else { return documentation; } }</code><technical_language>isEmpty</technical_language>

Target:
public String getDocumentation() { String documentation = super.getDocumentation(); if (StringUtils.isEmpty(documentation)) { String defaultDocumentation = workItemDefinition.getDocumentation(); return defaultDocumentation == null ? "" : defaultDocumentation; } else { return documentation; } }
Source:
<code>private void checkFestEqualityAsserts(MethodInvocationTree mit, boolean flipped) { if (mit.arguments().isEmpty()) { return; } ExpressionTree expected = mit.arguments().get(0); ExpressionTree actual = findActualValueForFest(mit); boolean actualIsLiteral = actual != null && isBoolOrNullLiteral(actual); if (isBoolOrNullLiteral(expected) && actualIsLiteral) { <START> reportIssue(mit, DEFAULT_MESSAGE); <END> } else if (isBoolOrNullLiteral(expected)) { checkEqualityAssertWithOneLiteral(mit, expected, flipped, IS); } else if (actualIsLiteral) { checkEqualityAssertWithOneLiteral(mit, actual, flipped, IS); } }</code><technical_language>a general observation, tree is reported, define is highlighted in products, if tree is big, result in ugly. This is especially true for method invocation: arguments unusually long, a huge lambda, a huge message for example! actual code is reporting literal, I this behavior in order avoid issues changing line. eventually add a secondary location (see #2915 for example of secondary) method identifier clearer! (you ExpressionUtils::methodName for this)</technical_language>

Target:
private void checkFestEqualityAsserts(MethodInvocationTree mit, boolean flipped) { if (mit.arguments().isEmpty()) { return; } Optional<LiteralTree> expectedLiteral = getBoolOrNullLiteral(mit.arguments().get(0)); Optional<LiteralTree> actualLiteral = findActualLiteralForFest(mit); IdentifierTree methodName = ExpressionUtils.methodName(mit); if (expectedLiteral.isPresent() && actualLiteral.isPresent()) { reportDefaultMessage(methodName, Arrays.asList(expectedLiteral.get(), actualLiteral.get())); } else { expectedLiteral.ifPresent(literal -> checkEqualityAssertWithOneLiteral(methodName, literal, flipped, IS)); actualLiteral.ifPresent(literal -> checkEqualityAssertWithOneLiteral(methodName, literal, flipped, IS)); } }
Source:
<code>public boolean include(final TreeWalk walker) { final int n = walker.getTreeCount(); if (n == 1) return true; final int m = walker.getRawMode(baseTree); for (int i = 1; i < n; i++) { DirCacheIterator baseDirCache = walker.getTree(baseTree, DirCacheIterator.class); DirCacheIterator newDirCache = walker.getTree(i, DirCacheIterator.class); if (baseDirCache != null && newDirCache != null) { DirCacheEntry baseDci = baseDirCache.getDirCacheEntry(); DirCacheEntry newDci = newDirCache.getDirCacheEntry(); if (baseDci != null && newDci != null) { if (baseDci.isAssumeValid() != newDci.isAssumeValid()) return true; <START> if (baseDci.isAssumeValid() && newDci.isAssumeValid()) <END> return false; } } if (walker.getRawMode(i) != m || !walker.idEqual(i, baseTree)) return true; } return false; }</code><technical_language>if (baseDci.isAssumeValid()) sufficient in previous line? baseDci.isAssumeValid() is equal newDci.isAssumeValid(). need check values</technical_language>

Target:
public boolean include(final TreeWalk walker) { final int n = walker.getTreeCount(); if (n == 1) return true; final int m = walker.getRawMode(baseTree); for (int i = 1; i < n; i++) { DirCacheIterator baseDirCache = walker.getTree(baseTree, DirCacheIterator.class); DirCacheIterator newDirCache = walker.getTree(i, DirCacheIterator.class); if (baseDirCache != null && newDirCache != null) { DirCacheEntry baseDci = baseDirCache.getDirCacheEntry(); DirCacheEntry newDci = newDirCache.getDirCacheEntry(); if (baseDci != null && newDci != null) { if (baseDci.isAssumeValid() != newDci.isAssumeValid()) return true; if (baseDci.isAssumeValid()) return false; } } if (walker.getRawMode(i) != m || !walker.idEqual(i, baseTree)) return true; } return false; }
Source:
<code>public void chain8() { final List<Comparable> result = $.chain($.chain($.class.getDeclaredMethods()) <START> .reduce(new FunctionAccum<List<String>, Method>() { <END> public List<String> apply(final List<String> accum, final Method method) { accum.add(method.getName()); return accum; } }, new ArrayList<String>()).item()) .reject(new Predicate<String>() { public Boolean apply(final String name) { return name.contains("$"); } }) .uniq(new Function1<String, Character>() { public Character apply(final String name) { return name.charAt(0); } }) .sort() .first(4) .value(); assertEquals(4, result.size()); }</code><technical_language>Hi, Thanks for improvements! map() method instead of reduce()</technical_language>

Target:
public void chain8() { final List<Comparable> result = $.chain($.class.getDeclaredMethods()) .map(new Function1<Method, String>() { public String apply(final Method method) { return method.getName(); } }) .reject(new Predicate<String>() { public Boolean apply(final String name) { return name.contains("$"); } }) .uniq(new Function1<String, Character>() { public Character apply(final String name) { return name.charAt(0); } }) .sort() .first(4) .value(); assertEquals(4, result.size()); }
Source:
<code>private void readMetaDataText(Reader metadataTextInput) throws IOException, RecognitionException, ParseException { CommonTree tree = createAST(metadataTextInput); fTreeParser = new IOStructGen(tree, fTrace); fTreeParser.generate(); ByteOrder localDetectedByteOrder = getDetectedByteOrder(); <START> if (fDetectedByteOrder != null && fTrace.getByteOrder() != localDetectedByteOrder) { <END> throw new ParseException("Metadata byte order and trace byte order inconsistent."); } }</code><technical_language>Nice catch</technical_language>

Target:
private void readMetaDataText(Reader metadataTextInput) throws IOException, RecognitionException, ParseException { CommonTree tree = createAST(metadataTextInput); fTreeParser = new IOStructGen(tree, fTrace); fTreeParser.generate(); ByteOrder detectedByteOrder = getDetectedByteOrder(); if (detectedByteOrder != null && fTrace.getByteOrder() != detectedByteOrder) { throw new ParseException("Metadata byte order and trace byte order inconsistent."); } }
Source:
<code>public Promise<Map<JSITImport, JSITDefinitions>> getImportDefinitionsAsync(final Metadata metadata, final List<JSITImport> imports) { if (!imports.isEmpty()) { return loadDMNDefinitions().then(otherDefinitions -> { final Map<JSITImport, JSITDefinitions> importDefinitions = new HashMap<>(); for (final Map.Entry<String, JSITDefinitions> entry : otherDefinitions.entrySet()) { final JSITDefinitions def = Js.uncheckedCast(entry.getValue()); findImportByDefinitions(def, imports).ifPresent(anImport -> { final JSITImport foundImported = Js.uncheckedCast(anImport); importDefinitions.put(foundImported, def); }); } return promises.resolve(importDefinitions); }); } <START> return promises.resolve(new HashMap<>()); <END> }</code><technical_language>Colelctions.emptyMap() resolved map is iterated</technical_language>

Target:
public Promise<Map<JSITImport, JSITDefinitions>> getImportDefinitionsAsync(final Metadata metadata, final List<JSITImport> imports) { if (!imports.isEmpty()) { return loadDMNDefinitions().then(otherDefinitions -> { final Map<JSITImport, JSITDefinitions> importDefinitions = new HashMap<>(); for (final Map.Entry<String, JSITDefinitions> entry : otherDefinitions.entrySet()) { final JSITDefinitions def = Js.uncheckedCast(entry.getValue()); findImportByDefinitions(def, imports).ifPresent(anImport -> { final JSITImport foundImported = Js.uncheckedCast(anImport); importDefinitions.put(foundImported, def); }); } return promises.resolve(importDefinitions); }); } return promises.resolve(Collections.emptyMap()); }
Source:
<code>public String toString() { <START> Request.Builder reqBuilder = Request.newBuilder().setHeader(request.getHeader()); <END> reqBuilder.setAddRequest( AddRequest.newBuilder().mergeFrom(request.getAddRequest()).clearBody().clearMasterKey().buildPartial()); return reqBuilder.buildPartial().toString(); }</code><technical_language>constructing another request masterkey. formatting string (which void building another object)? e.g. @Override public String toString() { return MoreObjects.toStringHelper() .add("ledgerId", request.getLedgerId()) ... .toString(); }</technical_language>

Target:
public String toString() { return RequestUtils.toSafeString(request); }
Source:
<code>public void decorate(JLabel label) { label.setText("" + "<html>" + <START> "URL: <a style='text-decoration: none' href=\"" + server.getBaseUrl().toString() + "\">" + getDescription() + "</a><br/>" + <END> "Project ID: " + server.getProjectId() + "" + "</html>"); makeClickable(label, () -> uncheckedBrowse(server.getBaseUrl())); }</code><technical_language>nitpicky, double quotes for HTML tag attribute values. I annoying escape :/</technical_language>

Target:
public void decorate(JLabel label) { label.setText("" + "<html>" + "URL: <a style=\"text-decoration: none\" href=\"" + server.getBaseUrl().toString() + "\">" + getDescription() + "</a><br/>" + "Project ID: " + server.getProjectId() + "" + "</html>"); makeClickable(label, () -> uncheckedBrowse(server.getBaseUrl())); }
Source:
<code><START> public FluoQuery createPcj( <END> final String pcjId, final String sparql, final Set<ExportStrategy> strategies, final FluoClient fluo) throws MalformedQueryException, UnsupportedQueryException { requireNonNull(pcjId); requireNonNull(sparql); requireNonNull(fluo); FluoQuery fluoQuery = makeFluoQuery(sparql, pcjId, strategies); writeFluoQuery(fluo, fluoQuery, pcjId); return fluoQuery; }</code><technical_language>@params match</technical_language>

Target:
public FluoQuery createPcj( final String pcjId, final String sparql, final Set<ExportStrategy> strategies, final FluoClient fluo) throws MalformedQueryException, UnsupportedQueryException { requireNonNull(pcjId); requireNonNull(sparql); requireNonNull(strategies); requireNonNull(fluo); FluoQuery fluoQuery = makeFluoQuery(sparql, pcjId, strategies); writeFluoQuery(fluo, fluoQuery, pcjId); return fluoQuery; }