auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>void inPool(CassandraClientPool cassandraClientPool) { CassandraClientPoolingContainer container = mock(CassandraClientPoolingContainer.class); when(container.getHost()).thenReturn(address); try { OngoingStubbing<Object> stubbing = when(container.runWithPooledResource( Mockito.<FunctionCheckedException<CassandraClient, Object, Exception>>any())); for (Exception ex : exceptions) { stubbing = stubbing.thenThrow(ex); } if (returnsValue) { stubbing.thenReturn("Response"); } } catch (Exception ex) { Throwables.throwIfUnchecked(ex); <START> } <END> cassandraClientPool.getCurrentPools().put(address, container); }</code><technical_language>I this swallows a checked exception - I realise fine this is setting mocks, a bit awkward</technical_language>

Target:
void inPool(CassandraClientPool cassandraClientPool) { CassandraClientPoolingContainer container = mock(CassandraClientPoolingContainer.class); when(container.getHost()).thenReturn(address); try { OngoingStubbing<Object> stubbing = when(container.runWithPooledResource( Mockito.<FunctionCheckedException<CassandraClient, Object, Exception>>any())); for (Exception ex : exceptions) { stubbing = stubbing.thenThrow(ex); } if (returnsValue) { stubbing.thenReturn("Response"); } } catch (Exception ex) { throw new RuntimeException(ex); } cassandraClientPool.getCurrentPools().put(address, container); }
Source:
<code>public final void onSuccess(Object message) { Result result = Result.EMPTY_RESULT; try { <START> if (message instanceof Result) { <END> result = (Result) message; } } catch (Throwable throwable) { onFailure(throwable); return; } resultsArray[index] = result; resultFuture.set(result); if (callback != null) { try { callback.update(NO_REGION, row.getRow(), (T) result); } catch (Throwable t) { LOG.error("User callback threw an exception for " + Bytes.toString(result.getRow())); } } }</code><technical_language>else this be</technical_language>

Target:
public final void onSuccess(Object message) { final Result result; if (message instanceof Result) { result = (Result) message; } else { result = Result.EMPTY_RESULT; } resultsArray[index] = result; resultFuture.set(result); if (callback != null) { try { callback.update(NO_REGION, row.getRow(), (T) result); } catch (Throwable t) { LOG.error("User callback threw an exception for " + Bytes.toString(result.getRow())); } } }
Source:
<code>public static Test suite() throws Exception { <START> final SftpProviderTestSuite suite = new SftpProviderTestSuite(new SftpProviderClosedExecChannelTestCase()); <END> suite.addTests(PermissionsTests.class); return suite; }</code><technical_language>@apearc03 ASL header is missing causing build fail Travis</technical_language>

Target:
public static Test suite() throws Exception { final SftpProviderTestSuite suite = new SftpProviderTestSuite(new SftpProviderClosedExecChannelTestCase()){ @Override protected void addBaseTests() throws Exception { addTests(ProviderReadTests.class); addTests(ProviderWriteTests.class); addTests(ProviderDeleteTests.class); addTests(ProviderRenameTests.class); addTests(NamingTests.class); addTests(PermissionsTests.class); } }; return suite; }
Source:
<code>public View addViewModel(int layoutResID, ViewModel viewModel, String id) { if (getBoundActivity() == null) { throw new RuntimeException("Bound Activity is null!"); } if (viewModel == null) { throw new RuntimeException("viewModel is null!"); } if (mViewModels == null) { mViewModels = new HashMap<>(); } if (mViewModels.values().contains(viewModel)) { throw new RuntimeException("cannot add same instance of viewModel twice!"); } if (viewModel instanceof INeedsActivity) { ((INeedsActivity) viewModel).setParentActivity(new WeakReference<>(getBoundActivity())); } if (viewModel instanceof INeedsFragmentManager) { <START> ((INeedsFragmentManager) viewModel).setFragmentManager(getBoundActivity().getFragmentManager()); <END> } viewModel.setLogger(getLogger()); mViewModels.put(id, viewModel); View view = getViewBinder().inflate(getBoundActivity(), viewModel, layoutResID, null); if (isShouldCallCreate()) { onCreate(mCreateBundle); } return view; }</code><technical_language>I get utility of INeedsFragmentManager. avoid call getter getParentActivity</technical_language>

Target:
public View addViewModel(int layoutResID, ViewModel viewModel, String id) { if (getBoundActivity() == null) { throw new RuntimeException("Bound Activity is null!"); } if (viewModel == null) { throw new RuntimeException("viewModel is null!"); } if (mViewModels == null) { mViewModels = new HashMap<>(); } if (mViewModels.values().contains(viewModel)) { throw new RuntimeException("cannot add same instance of viewModel twice!"); } if (viewModel instanceof INeedsActivity) { ((INeedsActivity) viewModel).setParentActivity(getBoundActivity()); } if (viewModel instanceof INeedsFragmentManager) { ((INeedsFragmentManager) viewModel).setFragmentManager(getBoundActivity().getFragmentManager()); } viewModel.setLogger(getLogger()); mViewModels.put(id, viewModel); View view = getViewBinder().inflate(getBoundActivity(), viewModel, layoutResID, null); if (isShouldCallCreate()) { onCreate(mCreateBundle); } return view; }
Source:
<code>public AboutDialogE4(final Shell parentShell) { super(parentShell); <START> product = Platform.getProduct(); <END> if (product != null) { productName = product.getName(); } if (productName == null) { productName = E4DialogMessages.AboutDialog_defaultProductName; } }</code><technical_language>another type encapsulate alternatives anumber o for "null" product</technical_language>

Target:
public AboutDialogE4(final Shell parentShell) { super(parentShell); product = new ProductInformation(Platform.getProduct()); }
Source:
<code>public ModelServiceImpl(IEclipseContext appContext) { if (appContext == null) { throw new NullPointerException("No application context given!"); } this.appContext = appContext; IEventBroker eventBroker = appContext.get(IEventBroker.class); if (eventBroker == null) { StringBuilder msg = new StringBuilder(); msg.append("Could not retrieve event broker!"); <START> Bundle bundle = Platform.getBundle("org.eclipse.equinox.event"); END> if (bundle == null) { msg.append(" Bundle 'org.eclipse.equinox.event' is missing. Please check your configuration."); } else if (bundle.getState() != Bundle.ACTIVE) { msg.append(" Bundle 'org.eclipse.equinox.event' is not active. Please check your configuration."); } throw new IllegalStateException(msg.toString()); } eventBroker.subscribe(UIEvents.UIElement.TOPIC_WIDGET, hostedElementHandler); mApplicationElementFactory = new GenericMApplicationElementFactoryImpl( appContext.get(IExtensionRegistry.class)); }</code><technical_language>This hard-wire org.eclipse.equinox.event, implementation different. correct do this, add info capacity is required. (IIRC Dirk Fauth added require provide capacity OSGi metadata)</technical_language>

Target:
public ModelServiceImpl(IEclipseContext appContext) { if (appContext == null) { throw new NullPointerException("No application context given!"); } this.appContext = appContext; IEventBroker eventBroker = appContext.get(IEventBroker.class); if (eventBroker == null) { throw new IllegalStateException( "Could not get an IEventBroker instance. Please check your configuration that a providing bundle is present and active."); } eventBroker.subscribe(UIEvents.UIElement.TOPIC_WIDGET, hostedElementHandler); mApplicationElementFactory = new GenericMApplicationElementFactoryImpl( appContext.get(IExtensionRegistry.class)); }
Source:
<code>private String createResults() { QueryResults queryResults = new QueryResults( "20160128_214710_00012_rk68b", <START> server.url("query.html?20160128_214710_00012_rk68b").uri(), <END> null, null, ImmutableList.of(new Column("_col0", BigintType.BIGINT)), ImmutableList.of(ImmutableList.of(123)), StatementStats.builder().setState("FINISHED").build(), null, null, null); return QUERY_RESULTS_CODEC.toJson(queryResults); }</code><technical_language>commit message is misleading: * fixing anything. tests pass, this is simply cosmetic. * real paths absolute, in a subdirectory. new values "wrong". remove this commit. update in commit match path a real server</technical_language>

Target:
private String createResults() { QueryResults queryResults = new QueryResults( "20160128_214710_00012_rk68b", server.url("/query.html?20160128_214710_00012_rk68b").uri(), null, null, ImmutableList.of(new Column("_col0", BigintType.BIGINT)), ImmutableList.of(ImmutableList.of(123)), StatementStats.builder().setState("FINISHED").build(), null, null, null); return QUERY_RESULTS_CODEC.toJson(queryResults); }
Source:
<code>public Future<VdcReturnValueBase> executeAsyncCommand(final VdcActionType actionType, final VdcActionParametersBase parameters, final CommandContext cmdContext) { final CommandBase<?> command = CommandsFactory.createCommand(actionType, parameters, cmdContext); command.persistCommand(command.getParameters().getParentCommand(), cmdContext, true); CommandCallBack callBack = command.getCallBack(); if (callBack != null) { cmdCallBackMap.put(command.getCommandId(), callBack); } Future<VdcReturnValueBase> future = null; try { future = executor.submit(new Callable<VdcReturnValueBase>() { @Override public VdcReturnValueBase call() throws Exception { return executeCommand(command, cmdContext); } }); } catch(RejectedExecutionException ex) { command.setCommandStatus(CommandStatus.FAILED); } <START> return future; <END> }</code><technical_language>impact of returning null here</technical_language>

Target:
public Guid executeAsyncCommand(final VdcActionType actionType, final VdcActionParametersBase parameters, final CommandContext cmdContext) { final CommandBase<?> command = CommandsFactory.createCommand(actionType, parameters, cmdContext); command.persistCommand(command.getParameters().getParentCommand(), cmdContext, true); CommandCallBack callBack = command.getCallBack(); if (callBack != null) { cmdCallBackMap.put(command.getCommandId(), callBack); } try { executor.submit(new Callable<VdcReturnValueBase>() { @Override public VdcReturnValueBase call() throws Exception { return executeCommand(command, cmdContext); } }); } catch(RejectedExecutionException ex) { command.setCommandStatus(CommandStatus.FAILED); } return command.getCommandId(); }
Source:
<code>public Object[] getElements(Object inputElement) { <START> Set<RepositoryTreeNode> output = new HashSet<RepositoryTreeNode>(); <END> for (Repository repo : ((List<Repository>) inputElement)) { output.add(new RepositoryTreeNode<Repository>(null, RepositoryTreeNodeType.REPO, repo, repo)); } return output.toArray(); }</code><technical_language>Please sort</technical_language>

Target:
public Object[] getElements(Object inputElement) { Comparator<RepositoryTreeNode<Repository>> sorter = new Comparator<RepositoryTreeNode<Repository>>() { public int compare(RepositoryTreeNode<Repository> o1, RepositoryTreeNode<Repository> o2) { return getRepositoryName(o1.getObject()).compareTo( getRepositoryName(o2.getObject())); } }; Set<RepositoryTreeNode<Repository>> output = new TreeSet<RepositoryTreeNode<Repository>>( sorter); for (Repository repo : ((List<Repository>) inputElement)) { output.add(new RepositoryTreeNode<Repository>(null, RepositoryTreeNodeType.REPO, repo, repo)); } return output.toArray(); }
Source:
<code>public AsyncContext startAsync() throws IllegalStateException { if (_asyncNotSupportedSource != null) throw new IllegalStateException("!asyncSupported: " + _asyncNotSupportedSource); HttpChannelState state = getHttpChannelState(); if (_async == null) _async = new AsyncContextState(state); <START> AsyncContextEvent event = new AsyncContextEvent(_context, _async, state, this, this, getResponse(), false); <END> state.startAsync(event); return _async; }</code><technical_language>I if constructor here, extra boolean argument</technical_language>

Target:
public AsyncContext startAsync() throws IllegalStateException { if (_asyncNotSupportedSource != null) throw new IllegalStateException("!asyncSupported: " + _asyncNotSupportedSource); HttpChannelState state = getHttpChannelState(); if (_async == null) _async = new AsyncContextState(state); AsyncContextEvent event = new AsyncContextEvent(_context, _async, state, this, this, getResponse()); state.startAsync(event); return _async; }
Source:
<code>public void kill(DataSegment segment) throws SegmentLoadingException { final File path = getDirectory(segment); <START> try { <END> if (path.getName().endsWith(".zip")) { File partitionNumDir = path.getParentFile(); FileUtils.deleteDirectory(partitionNumDir); File versionDir = partitionNumDir.getParentFile(); if (versionDir.delete()) { File intervalDir = versionDir.getParentFile(); if (intervalDir.delete()) { File dataSourceDir = intervalDir.getParentFile(); dataSourceDir.delete(); } } } else { throw new SegmentLoadingException("Unknown file type[%s]", path); } } catch (IOException e) { throw new SegmentLoadingException(e, "Unable to kill segment"); } }</code><technical_language>add a log line killing a segment located location <somethign></technical_language>

Target:
public void kill(DataSegment segment) throws SegmentLoadingException { final File path = getPath(segment); log.info("killing segment[%s] mapped to path[%s]", segment.getIdentifier(), path); try { if (path.getName().endsWith(".zip")) { File partitionNumDir = path.getParentFile(); FileUtils.deleteDirectory(partitionNumDir); File versionDir = partitionNumDir.getParentFile(); if (versionDir.delete()) { File intervalDir = versionDir.getParentFile(); if (intervalDir.delete()) { File dataSourceDir = intervalDir.getParentFile(); dataSourceDir.delete(); } } } else { throw new SegmentLoadingException("Unknown file type[%s]", path); } } catch (IOException e) { throw new SegmentLoadingException(e, "Unable to kill segment"); } }
Source:
<code>public static boolean waitForJobComplete( final Ds3Client client, final UUID jobId) throws InterruptedException, IOException { int retries = 0; final int max_retries = 12; while (JobStatus.IN_PROGRESS == client.getJobSpectraS3(new GetJobSpectraS3Request(jobId)).getMasterObjectListResult().getStatus()) { <START> LOG.info("Sleeping 5 minutes while waiting for job to finish..."); <END> sleep(300 * 1000); if (++retries > max_retries) { return false; } } return true; }</code><technical_language>This is repeated above, consolidate a single method</technical_language>

Target:
public static boolean waitForJobComplete( final Ds3Client client, final UUID jobId) throws Exception { final String getJobCmd = "--http --output-format json -c get_job -i " + jobId.toString(); final int maxRetries = 512; final int pollingInterval = 5 * 60 * 1000; final Predicate<String> predicate = new Predicate<String>() { @Override public boolean apply(@Nullable final String input) { return input.contains("\"status\" : \"COMPLETED\""); } }; return waitForMessage(client, getJobCmd, predicate, maxRetries, pollingInterval); }
Source:
<code>public void onSceneRemoved(InternalScene scene) { scene = null; <START> if (this.getThing().getStatus().equals(ThingStatus.ONLINE)) { <END> updateStatus(ThingStatus.OFFLINE); } logger.debug("Set status on {}", getThing().getStatus()); }</code><technical_language>need check, simply set OFFLINE</technical_language>

Target:
public void onSceneRemoved(InternalScene scene) { scene = null; updateStatus(ThingStatus.OFFLINE); logger.debug("Set status on {}", getThing().getStatus()); }
Source:
<code>public void isAliveShouldReturnTrueWhenRemoteRepositoryCachedValueIsTrue() { RemoteRepository remoteRepository = createMockedRemoteRepository(); <START> <END> initializeCache(true); boolean alive = remoteRepositoryAlivenessCacheManager.isAlive(remoteRepository); assertTrue(alive); }</code><technical_language>this check is cached in Cache</technical_language>

Target:
public void isAliveShouldReturnTrueWhenRemoteRepositoryCachedValueIsTrue() { initializeCache(true); boolean alive = remoteRepositoryAlivenessCacheManager.isAlive(remoteRepository); assertThat(alive).isTrue(); }
Source:
<code>public void testReestablishConnectionWithNewAM() throws InterruptedException { this.containerHeartbeatMonitor = spy(new ContainerHeartbeatMonitor(this.onExpired, this.containerHeartbeatClient, this.scheduler, COORDINATOR_URL, "0", coordinatorStreamStore, true, 10)); CoordinatorStreamValueSerde serde = new CoordinatorStreamValueSerde(SetConfig.TYPE); byte[] newCoordinatorUrl = serde.toBytes("<LINK_0>"); ContainerHeartbeatResponse response1 = new ContainerHeartbeatResponse(false); ContainerHeartbeatResponse response2 = new ContainerHeartbeatResponse(true); when(this.containerHeartbeatClient.requestHeartbeat()).thenReturn(response1).thenReturn(response2); <START> when(this.containerHeartbeatMonitor.getContainerHeartbeatClient()).thenReturn(this.containerHeartbeatClient); <END> when(this.coordinatorStreamStore.get(CoordinationConstants.YARN_COORDINATOR_URL)).thenReturn(newCoordinatorUrl); this.containerHeartbeatMonitor.start(); boolean fixedRateTaskCompleted = this.schedulerFixedRateExecutionLatch.await(2, TimeUnit.SECONDS); assertTrue("Did not complete heartbeat check", fixedRateTaskCompleted); verify(this.scheduler, never()).schedule(any(Runnable.class), anyLong(), any()); verify(this.onExpired, never()).run(); this.containerHeartbeatMonitor.stop(); verify(this.scheduler).shutdown(); }</code><technical_language>This a bit strange recreating heartbeat client for sake of tests. Ideally, want verify if a new client is created a new URL used</technical_language>

Target:
public void testReestablishConnectionWithNewAM() throws InterruptedException { String containerExecutionId = "0"; String newCoordinatorUrl = "<LINK_0>"; this.containerHeartbeatMonitor = spy(new ContainerHeartbeatMonitor(this.onExpired, this.containerHeartbeatClient, this.scheduler, COORDINATOR_URL, containerExecutionId, coordinatorStreamStore, true, 5, 10)); CoordinatorStreamValueSerde serde = new CoordinatorStreamValueSerde(SetConfig.TYPE); when(this.containerHeartbeatClient.requestHeartbeat()).thenReturn(FAILURE_RESPONSE).thenReturn(SUCCESS_RESPONSE); when(this.containerHeartbeatMonitor.createContainerHeartbeatClient(newCoordinatorUrl, containerExecutionId)).thenReturn(this.containerHeartbeatClient); when(this.coordinatorStreamStore.get(CoordinationConstants.YARN_COORDINATOR_URL)).thenReturn(serde.toBytes(newCoordinatorUrl)); this.containerHeartbeatMonitor.start(); boolean fixedRateTaskCompleted = this.schedulerFixedRateExecutionLatch.await(2, TimeUnit.SECONDS); assertTrue("Did not complete heartbeat check", fixedRateTaskCompleted); verify(this.scheduler, never()).schedule(any(Runnable.class), anyLong(), any()); verify(this.onExpired, never()).run(); this.containerHeartbeatMonitor.stop(); verify(this.scheduler).shutdown(); }
Source:
<code>void resetCounter() { <START> versionCounter.compareAndSet(versionCounter.get(), 0); <END> }</code><technical_language>This versionCounter.set(0);. This a race condition if generateNew method invoked time CAS operation work properly get returned a incremented CAS occur, causing a missed reset</technical_language>

Target:
void resetCounter() { versionCounter.set(0); }
Source:
<code>private void notifyTermChange(PartitionId partitionId, PrimaryTerm term) { <START> getLogger().debug("Changed " + term + ", listeners=" + listeners.size()); <END> listeners.values().forEach(session -> session.publish(CHANGE, new PrimaryElectionEvent(PrimaryElectionEvent.Type.CHANGED, partitionId, term))); }</code><technical_language>string replacement {}</technical_language>

Target:
private void notifyTermChange(PartitionId partitionId, PrimaryTerm term) { getLogger().debug("Changed {}, num listeners={}", term, listeners.size()); listeners.values().forEach(session -> session.publish(CHANGE, new PrimaryElectionEvent(PrimaryElectionEvent.Type.CHANGED, partitionId, term))); }
Source:
<code>public void setPorts(List<PortMapping> ports) { if (ports != null) { this.ports.clear(); <START> this.ports.addAll(ports); <END> } }</code><technical_language>jenkins (de)serializes is a bit confusing. In upgrades this.ports null field is in xml config. want is guard that, remove final (I reason it) do this.ports=ports</technical_language>

Target:
public void setPorts(List<PortMapping> ports) { this.ports = ports; }
Source:
<code>private void createDirectoryStructureIfNotExists(final File dir) { if (!dir.exists()) { try { Files.createParentDirs(new File(dir, DUMMY_FILE_NAME)); } catch (final IOException e) { <START> throw new RuntimeException("Failed to create directory: " + dir.getAbsolutePath(), e); <END> } } else if (!dir.isDirectory()) { throw new RuntimeException("This location is not a directory: " + dir.getAbsolutePath()); } }</code><technical_language>Is a conscious decision throw unchecked here</technical_language>

Target:
private void createDirectoryStructureIfNotExists(final File dir) throws IOException { if (!dir.exists()) { try { Files.createParentDirs(new File(dir, DUMMY_FILE_NAME)); } catch (final IOException e) { throw new IOException("Failed to create directory: " + dir.getAbsolutePath(), e); } } else if (!dir.isDirectory()) { throw new IOException("This location is not a directory: " + dir.getAbsolutePath()); } }
Source:
<code>public TriggerResource toResource(CronTriggerDto trigger) { TriggerResource triggerResource = instantiateResource(trigger); addSelfLink(triggerResource); <START> addEditLink(triggerResource); <END> return triggerResource; }</code><technical_language>true</technical_language>

Target:
public TriggerResource toResource(CronTriggerDto trigger) { TriggerResource triggerResource = instantiateResource(trigger); addSelfLink(triggerResource); return triggerResource; }
Source:
<code>public void <START> stop(BundleContext <END> context) throws Exception { SiriusEditPlugin.getPlugin().removeSessionFileLoadingListener(modelingProjectExpansionListener); modelingProjectExpansionListener = null; super.stop(context); }</code><technical_language>dispose/clear PageRegistry set field null</technical_language>

Target:
public void stop(BundleContext context) throws Exception { SiriusEditPlugin.getPlugin().removeSessionFileLoadingListener(modelingProjectExpansionListener); modelingProjectExpansionListener = null; pageRegistry = null; super.stop(context); }
Source:
<code>public static void readMockFile() throws IOException { JsonToObjectMapper objectMapper = new JsonToObjectMapper(); <START> HOTSPOTS = objectMapper.readHotspots("hotspots_db_dump.json"); <END> }</code><technical_language>Yeah dump in sync. comprehensible I import relevant data for testing, easy kind of data is expected. e.g. jq get relevant data. if necessary full dump, do a complete integration actual data import instead of sync this file</technical_language>

Target:
public static void readMockFile() throws IOException { JsonToObjectMapper objectMapper = new JsonToObjectMapper(); HOTSPOTS = objectMapper.readHotspots("hotspots_integration_test.json"); }
Source:
<code>private static Object handleUnknownError(ResponseMsg msg, ChannelHandlerContext ctx, IClientRouter r) throws Throwable { ByteString bs = msg.getPayload().getServerError().getUnknownError().getThrowable(); byte[] bytes = new byte[bs.size()]; bs.copyTo(bytes, 0); <START> ByteArrayInputStream bis = new ByteArrayInputStream(bytes); ObjectInputStream ois = new ObjectInputStream(bis); <END> throw (Throwable) ois.readObject(); }</code><technical_language>wrap this in a try-with-resources block? need consider case receive exception do recognize (can't deserialize)... context of rolling upgrades</technical_language>

Target:
private static Object handleUnknownError(ResponseMsg msg, ChannelHandlerContext ctx, IClientRouter r) throws Throwable { ByteString bs = msg.getPayload().getServerError().getUnknownError().getThrowable(); byte[] bytes = new byte[bs.size()]; bs.copyTo(bytes, 0); try (ByteArrayInputStream bis = new ByteArrayInputStream(bytes)) { try (ObjectInputStream ois = new ObjectInputStream(bis)) { throw (Throwable) ois.readObject(); } } }
Source:
<code>public List<Node> getChildren() { List<Node> children = new ArrayList<Node>(); children.add(variable); children.add(expression); <START> return children; <END> }</code><technical_language>try return Arrays.asList(variable, expression)</technical_language>

Target:
public List<Node> getChildren() { return Arrays.asList(variable, expression); }
Source:
<code>private void addNewFilter(ITmfFilter filter) { CLabel label = new CLabel(this, SWT.SHADOW_OUT); label.setBackground(fLabelBackground); String text; <START> if (filter instanceof TmfFilterNode) { text = ((TmfFilterNode) filter).getFilterName(); } else { text = filter.toString(); } <END> if (fCollapsed) { label.setToolTipText(text); label.setTopMargin(0); label.setBottomMargin(0); label.setRightMargin(COLLAPSED_RIGHT_MARGIN); } else { label.setImage(DELETE); label.setText(text); } label.setData(filter); label.addMouseListener(new MouseAdapter() { @Override public void mouseDown(MouseEvent e) { Rectangle bounds; if (fCollapsed) { bounds = new Rectangle(0, 0, 2 * COLLAPSED_IMAGE_MARGIN + DELETE_SMALL.getBounds().width, label.getBounds().height); } else { bounds = DELETE.getBounds(); bounds.x += label.getLeftMargin(); bounds.y = (label.getSize().y - bounds.height) / 2; } if (bounds.contains(e.x, e.y)) { fListener.filterRemoved((ITmfFilter) label.getData()); } else { fListener.filterSelected((ITmfFilter) label.getData()); getParent().layout(true, true); } } }); label.addPaintListener(e -> { if (fCollapsed) { e.gc.drawImage(DELETE_SMALL, COLLAPSED_IMAGE_MARGIN, COLLAPSED_IMAGE_MARGIN); } }); }</code><technical_language>I assume do this show long text in case of a filter node? text is useful too, toString a tooltip</technical_language>

Target:
private void addNewFilter(ITmfFilter filter) { CLabel label = new CLabel(this, SWT.SHADOW_OUT); label.setBackground(fLabelBackground); String text; if (filter instanceof TmfFilterNode) { text = ((TmfFilterNode) filter).getFilterName(); label.setData(TOOLTIP_KEY, filter.toString()); } else { text = filter.toString(); } if (fCollapsed) { label.setToolTipText(text); label.setTopMargin(0); label.setBottomMargin(0); label.setRightMargin(COLLAPSED_RIGHT_MARGIN); } else { label.setImage(DELETE); label.setText(text); label.setToolTipText((String) label.getData(TOOLTIP_KEY)); } label.setData(filter); label.addMouseListener(new MouseAdapter() { @Override public void mouseDown(MouseEvent e) { Rectangle bounds; if (fCollapsed) { bounds = new Rectangle(0, 0, 2 * COLLAPSED_IMAGE_MARGIN + DELETE_SMALL.getBounds().width, label.getBounds().height); } else { bounds = DELETE.getBounds(); bounds.x += label.getLeftMargin(); bounds.y = (label.getSize().y - bounds.height) / 2; } if (bounds.contains(e.x, e.y)) { fListener.filterRemoved((ITmfFilter) label.getData()); } else { fListener.filterSelected((ITmfFilter) label.getData()); getParent().layout(true, true); } } }); label.addPaintListener(e -> { if (fCollapsed) { e.gc.drawImage(DELETE_SMALL, COLLAPSED_IMAGE_MARGIN, COLLAPSED_IMAGE_MARGIN); } }); }
Source:
<code>public String decryptFile(String filePathOrEncrypted) { if (!EncryptedSecret.isEncryptedSecret(filePathOrEncrypted)) { return filePathOrEncrypted; } EncryptedSecret encryptedSecret = EncryptedSecret.parse(filePathOrEncrypted); SecretEngine secretEngine = secretEngineRegistry.getEngine(encryptedSecret.getEngineIdentifier()); if (secretEngine == null) { throw new InvalidSecretFormatException("Secret Engine does not exist: " + encryptedSecret.getEngineIdentifier()); <START> <END> } else { secretEngine.validate(encryptedSecret); return decryptedFilePath(secretEngine, encryptedSecret); } }</code><technical_language>Extra newline</technical_language>

Target:
public Path decryptFile(String filePathOrEncrypted) { if (!EncryptedSecret.isEncryptedSecret(filePathOrEncrypted)) { return null; } EncryptedSecret encryptedSecret = EncryptedSecret.parse(filePathOrEncrypted); SecretEngine secretEngine = secretEngineRegistry.getEngine(encryptedSecret.getEngineIdentifier()); if (secretEngine == null) { throw new InvalidSecretFormatException("Secret Engine does not exist: " + encryptedSecret.getEngineIdentifier()); } else { secretEngine.validate(encryptedSecret); return decryptedFilePath(secretEngine, encryptedSecret); } }
Source:
<code>public static void preInit() { zonePlanner = BlockBCBase_Neptune.register(new BlockZonePlanner(Material.ROCK, "block.zone_planner")); <START> TileBC_Neptune.registerTile(TileZonePlanner.class, "tile.zone_planner"); <END> }</code><technical_language>zone planner is useless robots, remove If (DEVELOPER) bit</technical_language>

Target:
public static void preInit() { if (BCLib.DEV) { zonePlanner = BlockBCBase_Neptune.register(new BlockZonePlanner(Material.ROCK, "block.zone_planner")); TileBC_Neptune.registerTile(TileZonePlanner.class, "tile.zone_planner"); } }
Source:
<code>public boolean validateConnection() { boolean status = false; final String constructedURL = String.format(URL_GET_PROJECTS, getUrl()); <START> final HttpResponse response = get(constructedURL); <END> int statusCode = response.getStatusLine().getStatusCode(); if (statusCode >= 200 && statusCode < 300) { final HttpEntity entity = response.getEntity(); String string = null; try { string = EntityUtils.toString(entity); if (string.startsWith("[") && string.endsWith("]")) return true; } catch (IOException e) { ZephyrPlugin.log(e); } } else if (statusCode == 401) { return true; } else if (statusCode == 404) { return false; } else { ZephyrPlugin.log("Unexpected response status: " + statusCode); } return status; }</code><technical_language>Response is closed</technical_language>

Target:
public boolean validateConnection() { boolean status = false; final String constructedURL = String.format(URL_GET_PROJECTS, getUrl()); try (final CloseableHttpResponse response = get(constructedURL)) { int statusCode = response.getStatusLine().getStatusCode(); if (statusCode >= 200 && statusCode < 300) { final HttpEntity entity = response.getEntity(); String string = null; try { string = EntityUtils.toString(entity); if (string.startsWith("[") && string.endsWith("]")) return true; } catch (IOException e) { ZephyrPlugin.log(e); } } else if (statusCode == 401) { return true; } else if (statusCode == 404) { return false; } else { ZephyrPlugin.log(String.format("Unexpected response status <%s> upon validate connection <%s>", statusCode, constructedURL)); } } catch (IOException e) { ZephyrPlugin.log(e); } return status; }
Source:
<code>public void testSendWithTransaction() { <START> Message message = Proton.message(); <END> message.setMessageId("id"); message.setBody(new AmqpValue("hello")); AmqpTransaction transaction = new AmqpTransaction(ByteBuffer.wrap("1".getBytes())); ReactorSender reactorSender = new ReactorSender(entityPath, sender, handler, reactorProvider, tokenManager, messageSerializer, Duration.ofSeconds(1), new ExponentialAmqpRetryPolicy(new AmqpRetryOptions())); ReactorSender spyReactorSender = spy(reactorSender); doReturn(Mono.empty()).when(spyReactorSender).send(any(byte[].class), anyInt(), anyInt(), any(AmqpTransaction.class)); StepVerifier.create(spyReactorSender.send(message, transaction)) .verifyComplete(); StepVerifier.create(spyReactorSender.send(message, transaction)) .verifyComplete(); verify(sender, times(1)).getRemoteMaxMessageSize(); verify(spyReactorSender, times(2)).send(any(byte[].class), anyInt(), anyInt(), ArgumentMatchers.same(transaction)); }</code><technical_language>// Arrange</technical_language>

Target:
public void testSendWithTransaction() { Message message = Proton.message(); message.setMessageId("id"); message.setBody(new AmqpValue("hello")); ReactorSender reactorSender = new ReactorSender(entityPath, sender, handler, reactorProvider, tokenManager, messageSerializer, Duration.ofSeconds(1), new ExponentialAmqpRetryPolicy(new AmqpRetryOptions())); ReactorSender spyReactorSender = spy(reactorSender); doReturn(Mono.empty()).when(spyReactorSender).send(any(byte[].class), anyInt(), anyInt(), eq(transactionalState)); StepVerifier.create(spyReactorSender.send(message, transactionalState)) .verifyComplete(); StepVerifier.create(spyReactorSender.send(message, transactionalState)) .verifyComplete(); verify(sender, times(1)).getRemoteMaxMessageSize(); verify(spyReactorSender, times(2)).send(any(byte[].class), anyInt(), eq(DeliveryImpl.DEFAULT_MESSAGE_FORMAT), eq(transactionalState)); }
Source:
<code>public void showAggregatedProcesses() { <START> if (migrateWorkflow) { <END> List<Process> processList = new ArrayList<>(); aggregatedProcesses.clear(); for (Project project : selectedProjects) { processList.addAll(project.getProcesses()); } for (Process process : processList) { if (Objects.isNull(process.getTemplate())) { addToAggregatedProcesses(aggregatedProcesses, process); } } processListShown = true; } else { for (Project project : selectedProjects) { TaskManager.addTask(new MigrationTask(project)); } projectListShown = false; } }</code><technical_language>please a separate button separate method, "showAggregtedProcesses()"</technical_language>

Target:
public void showAggregatedProcesses() { List<Process> processList = new ArrayList<>(); aggregatedProcesses.clear(); for (Project project : selectedProjects) { processList.addAll(project.getProcesses()); } for (Process process : processList) { if (Objects.isNull(process.getTemplate())) { addToAggregatedProcesses(aggregatedProcesses, process); } } processListShown = true; }
Source:
<code>public void testShouldSetAvailableForAnotherInput() { InputSelection secondAndThird = new InputSelection.Builder().select(2).select(3).build(); MultipleInputSelectionHandler selectionHandler = new MultipleInputSelectionHandler(() -> secondAndThird, 3); selectionHandler.nextSelection(); assertFalse(selectionHandler.shouldSetAvailableForAnotherInput()); selectionHandler.setUnavailableInput(0); assertFalse(selectionHandler.shouldSetAvailableForAnotherInput()); selectionHandler.setUnavailableInput(2); assertTrue(selectionHandler.shouldSetAvailableForAnotherInput()); <START> <END> selectionHandler.setAvailableInput(0); assertTrue(selectionHandler.shouldSetAvailableForAnotherInput()); }</code><technical_language>Add a check selectionHandler.setAvailableInput(2)</technical_language>

Target:
public void testShouldSetAvailableForAnotherInput() { InputSelection secondAndThird = new InputSelection.Builder().select(2).select(3).build(); MultipleInputSelectionHandler selectionHandler = new MultipleInputSelectionHandler(() -> secondAndThird, 3); selectionHandler.nextSelection(); assertFalse(selectionHandler.shouldSetAvailableForAnotherInput()); selectionHandler.setUnavailableInput(0); assertFalse(selectionHandler.shouldSetAvailableForAnotherInput()); selectionHandler.setUnavailableInput(2); assertTrue(selectionHandler.shouldSetAvailableForAnotherInput()); selectionHandler.setAvailableInput(0); assertTrue(selectionHandler.shouldSetAvailableForAnotherInput()); selectionHandler.setAvailableInput(2); assertFalse(selectionHandler.shouldSetAvailableForAnotherInput()); }
Source:
<code>public void webpackDevServerPortShouldBeReusedOnReload() { open(); String initialUUID = findElement( By.id(WebpackDevServerPortView.UUID_ID)).getText(); Assert.assertEquals(36, initialUUID.length()); String initialPort = findElement( By.id(WebpackDevServerPortView.WEBPACK_PORT_ID)).getText(); Assert.assertTrue(NumberUtils.isDigits(initialPort)); findElement(By.id(WebpackDevServerPortView.TRIGGER_RELOAD_ID)).click(); boolean reconnected = false; <START> for (int i = 0; i < 10; i++) { <END> getDriver().navigate().refresh(); try { waitUntil(driver -> !initialUUID.equals( findElement(By.id(WebpackDevServerPortView.UUID_ID)) .getText()), 2); } catch (TimeoutException e) { continue; } reconnected = true; break; } Assert.assertTrue(reconnected); String port = findElement( By.id(WebpackDevServerPortView.WEBPACK_PORT_ID)).getText(); Assert.assertEquals(initialPort, port); }</code><technical_language>10 is a magic constant. I waitUntil some hardcoded big timeout. waitUntil is effectively same: run condition true again. put getDriver().navigate().refresh(); return !initialUUID.equals( findElement(By.id(WebpackDevServerPortView.UUID_ID)) .getText() expected condition impl. add a parameter a timeout. allow avoid: magic 10 constant, cycle, reconnected flag, catch exception</technical_language>

Target:
public void webpackDevServerPortShouldBeReusedOnReload() throws InterruptedException { open(); String initialUUID = findElement( By.id(WebpackDevServerPortView.UUID_ID)).getText(); Assert.assertEquals(36, initialUUID.length()); String initialPort = findElement( By.id(WebpackDevServerPortView.WEBPACK_PORT_ID)).getText(); Assert.assertTrue(NumberUtils.isDigits(initialPort)); findElement(By.id(WebpackDevServerPortView.TRIGGER_RELOAD_ID)).click(); int totalWaitTime = 20000; boolean reconnected = false; long startTime = System.currentTimeMillis(); while (System.currentTimeMillis() < startTime + totalWaitTime && !reconnected) { Thread.sleep(2000); getDriver().navigate().refresh(); List<WebElement> elements = findElements( By.id(WebpackDevServerPortView.UUID_ID)); if (!elements.isEmpty() && !elements.get(0).getText().equals(initialUUID)) { reconnected = true; } } Assert.assertTrue("unable to verify connection to new Jetty instance", reconnected); String port = findElement( By.id(WebpackDevServerPortView.WEBPACK_PORT_ID)).getText(); Assert.assertEquals(initialPort, port); }
Source:
<code>void waitUntilConnected() { boolean isConnected; do { isConnected = _zkclient.waitUntilConnected(HelixZkClient.DEFAULT_CONNECTION_TIMEOUT, TimeUnit.MILLISECONDS); if (!isConnected) { LOG.error("fail to connect zkserver: " + _zkAddress + " in " + HelixZkClient.DEFAULT_CONNECTION_TIMEOUT + "ms. expiredSessionId: " + _sessionId + ", clusterName: " + _clusterName); continue; } <START> _sessionId = _zkclient.getHexSessionId(_zkclient.getSessionId()); <END> } while (!isConnected || ZERO_HEX_SESSION_ID.equals(_sessionId)); LOG.info("Handling new session, session id: " + _sessionId + ", instance: " + _instanceName + ", instanceTye: " + _instanceType + ", cluster: " + _clusterName); }</code><technical_language>do need this? if need a method for Hex Id, _zkClient.getHexSessionId return Hex(_sessionId)? This embeded calls a bit weird</technical_language>

Target:
void waitUntilConnected() { boolean isConnected; do { isConnected = _zkclient.waitUntilConnected(HelixZkClient.DEFAULT_CONNECTION_TIMEOUT, TimeUnit.MILLISECONDS); if (!isConnected) { LOG.error("fail to connect zkserver: " + _zkAddress + " in " + HelixZkClient.DEFAULT_CONNECTION_TIMEOUT + "ms. expiredSessionId: " + _sessionId + ", clusterName: " + _clusterName); continue; } _sessionId = ZKUtil.toHexSessionId(_zkclient.getSessionId()); } while (!isConnected || "0".equals(_sessionId)); LOG.info("Handling new session, session id: " + _sessionId + ", instance: " + _instanceName + ", instanceTye: " + _instanceType + ", cluster: " + _clusterName); }
Source:
<code>protected void onSetUpAfterOpeningDesignerPerspective() throws Exception { ReadOnlyPermissionAuthority readOnlyPermissionAuthority = new ReadOnlyPermissionAuthority(); IPermissionProvider permissionProvider = new DefaultPermissionProvider(readOnlyPermissionAuthority); permissionProviderDescriptor = new StandalonePermissionProviderDescriptor("org.eclipse.sirius.tests.swtbot.lockModelExplorerPermissionAuthorityProvider", ExtenderConstants.HIGHEST_PRIORITY, permissionProvider); PermissionService.addExtension(permissionProviderDescriptor); sessionAirdResource = new UIResource(designerProject, SESSION_RESOURCE_NAME); localSession = designerPerspective.openSessionFromFile(sessionAirdResource, true); editor = (SWTBotSiriusDiagramEditor) openRepresentation(localSession.getOpenedSession(), "Entities", REPRESENTATION_DESCRIPTION_NAME, DDiagram.class); modelExplorerView = bot.viewById(IModelExplorerView.ID); modelExplorerView.setFocus(); jobChangeListener = new IJobChangeListener() { @Override public void sleeping(IJobChangeEvent event) { } @Override public void scheduled(IJobChangeEvent event) { refreshJobScheduled = event.getJob().belongsTo(RefreshLabelImageJob.FAMILY); } @Override public void running(IJobChangeEvent event) { } @Override public void done(IJobChangeEvent event) { } @Override public void awake(IJobChangeEvent event) { } @Override public void aboutToRun(IJobChangeEvent event) { } <START> }; <END> Job.getJobManager().addJobChangeListener(jobChangeListener); }</code><technical_language>Instead of empty/useless methods, JobChangeAdapter</technical_language>

Target:
protected void onSetUpAfterOpeningDesignerPerspective() throws Exception { ReadOnlyPermissionAuthority readOnlyPermissionAuthority = new ReadOnlyPermissionAuthority(); IPermissionProvider permissionProvider = new DefaultPermissionProvider(readOnlyPermissionAuthority); permissionProviderDescriptor = new StandalonePermissionProviderDescriptor("org.eclipse.sirius.tests.swtbot.lockModelExplorerPermissionAuthorityProvider", ExtenderConstants.HIGHEST_PRIORITY, permissionProvider); PermissionService.addExtension(permissionProviderDescriptor); sessionAirdResource = new UIResource(designerProject, SESSION_RESOURCE_NAME); localSession = designerPerspective.openSessionFromFile(sessionAirdResource, true); editor = (SWTBotSiriusDiagramEditor) openRepresentation(localSession.getOpenedSession(), "Entities", REPRESENTATION_DESCRIPTION_NAME, DDiagram.class); modelExplorerView = bot.viewById(IModelExplorerView.ID); modelExplorerView.setFocus(); jobChangeListener = new JobChangeAdapter() { @Override public void scheduled(IJobChangeEvent event) { refreshJobScheduled = event.getJob().belongsTo(RefreshLabelImageJob.FAMILY); } }; Job.getJobManager().addJobChangeListener(jobChangeListener); }
Source:
<code>private static String getDataTypeDescription(MeasurementVariable trait) { <START> if (trait.getDataTypeId() == null || !dataTypeList.contains(trait.getDataTypeId())) { <END> return KsuFieldbookUtil.UNRECOGNIZED_FORMAT; } else if (trait.getDataTypeId() == TermId.NUMERIC_VARIABLE.getId()) { return KsuFieldbookUtil.NUMERIC_FORMAT; } else if (trait.getDataTypeId() == TermId.CATEGORICAL_VARIABLE.getId()) { return KsuFieldbookUtil.CATEGORICAL_FORMAT; } else if (trait.getDataTypeId() == TermId.DATE_VARIABLE.getId()) { return KsuFieldbookUtil.DATE_FORMAT; } return KsuFieldbookUtil.CHARACTER_FORMAT; }</code><technical_language>@cuenyad a switch work here??</technical_language>

Target:
private static String getDataTypeDescription(MeasurementVariable trait) { Integer dataType; if (trait.getDataTypeId() == null || !dataTypeList.contains(trait.getDataTypeId())) { dataType = 0; } else { dataType = trait.getDataTypeId(); } return dataTypeFormats.get(dataType); }
Source:
<code>static void loadTerms(String fileName) throws IOException { try ( InputStream resourceStream = SearchBoostTermsResolver.class.getResourceAsStream(fileName); BufferedReader br = new BufferedReader(new InputStreamReader(resourceStream)) <START> ) { <END> for (String line; (line = br.readLine()) != null; ) { processMapping(line); } } }</code><technical_language>Consider declaring InputStreamReader in try resources is closed if BufferedReader constructor throws exception</technical_language>

Target:
static void loadTerms(String fileName) throws IOException { try ( InputStream resourceStream = SearchBoostTermsResolver.class.getResourceAsStream(fileName); InputStreamReader inputStreamReader = new InputStreamReader(resourceStream); BufferedReader br = new BufferedReader(inputStreamReader) ) { for (String line; (line = br.readLine()) != null; ) { processMapping(line); } } }
Source:
<code>public static void setFinal(File file, boolean isFinal) { try { UserDefinedFileAttributeView view = getFileAttributeView(file); view.write(IS_FINAL_ATTRIBUTE, Charset.defaultCharset().encode(new Boolean(isFinal).toString())); <START> } catch (Exception e) { <END> } }</code><technical_language>I prefer if exceptions swallowed instead IOExceptions nice log debug in case wonders this working</technical_language>

Target:
public static void setFinal(File file, boolean isFinal) { try { UserDefinedFileAttributeView view = getFileAttributeView(file); if (view != null) { view.write(IS_FINAL_ATTRIBUTE, Charset.defaultCharset().encode(new Boolean(isFinal).toString())); } } catch (IOException e) { } }
Source:
<code>public void customClassPrefixNoJavaType() throws ClassNotFoundException { ClassLoader resultsClassLoader = generateAndCompile("/schema/properties/primitivePropertiesNoJavaType.json", "com.example", config("classNamePrefix","Prefix")); <START> resultsClassLoader.loadClass("com.example.PrefixprimitivePropertiesNoJavaType"); <END> }</code><technical_language>com.example.PrefixPrimitivePropertiesNoJavaType</technical_language>

Target:
public void customClassPrefixNoJavaType() throws ClassNotFoundException { ClassLoader resultsClassLoader = generateAndCompile("/schema/properties/primitivePropertiesNoJavaType.json", "com.example", config("classNamePrefix","Prefix")); resultsClassLoader.loadClass("com.example.PrefixPrimitivePropertiesNoJavaType"); }
Source:
<code>public void stop() { <START> if (!isRunning.get()) { <END> log.info("Garbage informer had been stopped."); return; } gcScheduler.shutdownNow(); log.info("Garbage informer is stopped."); isRunning.set(false); }</code><technical_language>I compareAndSet in start()</technical_language>

Target:
public synchronized void stop() { if (!isRunning) { log.info("Garbage informer had been stopped."); return; } gcScheduler.shutdownNow(); log.info("Garbage informer is stopped."); isRunning = false; }
Source:
<code>private Message makeCopy(final MessageReference ref, final boolean expiry, final boolean copyOriginalHeaders) throws Exception { if (ref == null) { <START> return null; <END> } Message message = ref.getMessage(); long newID = storageManager.generateID(); Message copy = message.copy(newID); if (copyOriginalHeaders) { copy.referenceOriginalMessage(message, ref.getQueue().getName().toString()); } copy.setExpiration(0); if (expiry) { copy.setAnnotation(Message.HDR_ACTUAL_EXPIRY_TIME, System.currentTimeMillis()); } copy.reencode(); return copy; }</code><technical_language>I throw Exception.. supposed a copy of a ref == null. a valid assertion, I throw NullPointerException here. (Or a chosen exception)</technical_language>

Target:
private Message makeCopy(final MessageReference ref, final boolean expiry, final boolean copyOriginalHeaders) throws Exception { if (ref == null) { ActiveMQServerLogger.LOGGER.nullRefMessage(); throw new ActiveMQNullRefException("Reference to message is null"); } Message message = ref.getMessage(); long newID = storageManager.generateID(); Message copy = message.copy(newID); if (copyOriginalHeaders) { copy.referenceOriginalMessage(message, ref.getQueue().getName().toString()); } copy.setExpiration(0); if (expiry) { copy.setAnnotation(Message.HDR_ACTUAL_EXPIRY_TIME, System.currentTimeMillis()); } copy.reencode(); return copy; }
Source:
<code>public HBaseDatasetRepository build() { HTablePool pool = new HTablePool(configuration, 10); HBaseAdmin admin; try { admin = new HBaseAdmin(configuration); } catch (MasterNotRunningException e) { throw new DatasetRepositoryException( "Problem creating HBaseDatasetRepository.", e); } catch (ZooKeeperConnectionException e) { throw new DatasetRepositoryException( "Problem creating HBaseDatasetRepository.", e); } catch (IOException e) { <START> throw new DatasetRepositoryException( <END> "Problem creating HBaseDatasetRepository.", e); } return new HBaseDatasetRepository(admin, pool, getRepositoryUri(configuration)); }</code><technical_language>DatasetIOException</technical_language>

Target:
public HBaseDatasetRepository build() { HTablePool pool = new HTablePool(configuration, 10); HBaseAdmin admin; try { admin = new HBaseAdmin(configuration); } catch (MasterNotRunningException e) { throw new DatasetRepositoryException( "Problem creating HBaseDatasetRepository.", e); } catch (ZooKeeperConnectionException e) { throw new DatasetRepositoryException( "Problem creating HBaseDatasetRepository.", e); } catch (IOException e) { throw new DatasetIOException( "Problem creating HBaseDatasetRepository.", e); } return new HBaseDatasetRepository(admin, pool, getRepositoryUri(configuration)); }