auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>private int calculateAlignmentValuesForText(Paint paint, String text) { switch (alignment) { <START> case ShowTextColorAndSizeBrick.ALIGNEMNT_STYLE_LEFT: <END> paint.setTextAlign(Align.LEFT); return 0; case ShowTextColorAndSizeBrick.ALIGNEMNT_STYLE_RIGHT: paint.setTextAlign(Align.RIGHT); return (int) paint.measureText(text); default: paint.setTextAlign(Align.CENTER); return (int) (paint.measureText(text) / 2); } }</code><technical_language>suggestion case ShowTextColorSizeAlignBrick.ALIGNMENT_STYLE_LEFT:</technical_language>

Target:
private int calculateAlignmentValuesForText(Paint paint, String text) { switch (alignment) { case ShowTextColorSizeAlignmentBrick.ALIGNMENT_STYLE_LEFT: paint.setTextAlign(Align.LEFT); return 0; case ShowTextColorSizeAlignmentBrick.ALIGNMENT_STYLE_RIGHT: paint.setTextAlign(Align.RIGHT); return (int) paint.measureText(text); default: paint.setTextAlign(Align.CENTER); return (int) (paint.measureText(text) / 2); } }
Source:
<code>public void write(JobContext jobCtx, MarshallerWriteContext outputCtx) throws IOException { outputCtx.writeShort( PersisterEnums.EXPIRE_TIMER ); ExpireJobContext ejobCtx = (ExpireJobContext) jobCtx; WorkingMemoryReteExpireAction expireAction = ejobCtx.getExpireAction(); outputCtx.writeInt( expireAction.getFactHandle().getId() ); DefaultJobHandle jobHandle = (DefaultJobHandle) ejobCtx.getJobHandle(); PointInTimeTrigger trigger = (PointInTimeTrigger) jobHandle.getTimerJobInstance().getTrigger(); <START> if (trigger.hasNextFireTime() != null) { <END> outputCtx.writeLong( trigger.hasNextFireTime().getTime() ); } }</code><technical_language>I if change unmarshall I suggested this check avoided</technical_language>

Target:
public void write(JobContext jobCtx, MarshallerWriteContext outputCtx) throws IOException { ExpireJobContext ejobCtx = (ExpireJobContext) jobCtx; DefaultJobHandle jobHandle = (DefaultJobHandle) ejobCtx.getJobHandle(); PointInTimeTrigger trigger = (PointInTimeTrigger) jobHandle.getTimerJobInstance().getTrigger(); Date nextFireTime = trigger.hasNextFireTime(); if (nextFireTime != null) { outputCtx.writeShort(PersisterEnums.EXPIRE_TIMER); outputCtx.writeInt(ejobCtx.getExpireAction().getFactHandle().getId()); outputCtx.writeLong(nextFireTime.getTime()); } }
Source:
<code>private void drawHighlightLine(Position position) { if (position.isDeleted()) return; int widgetOffset= 0; if (fViewer instanceof ITextViewerExtension5) { ITextViewerExtension5 extension= (ITextViewerExtension5) fViewer; widgetOffset= extension.modelOffset2WidgetOffset(position.getOffset()); if (widgetOffset <START> < 0) <END> return; } else { IRegion visible= fViewer.getVisibleRegion(); widgetOffset= position.getOffset() - visible.getOffset(); if (widgetOffset < 0 || visible.getLength() < widgetOffset ) return; } StyledText textWidget= fViewer.getTextWidget(); int charCount= textWidget.getCharCount(); if (widgetOffset > charCount) { return; } Point upperLeft= textWidget.getLocationAtOffset(widgetOffset); int width= textWidget.getClientArea().width + textWidget.getHorizontalPixel(); int height; if (position.length == 0 || !textWidget.getWordWrap()) { height= textWidget.getLineHeight(widgetOffset); } else { int offsetEnd= widgetOffset + position.length - 1; if(offsetEnd >= charCount){ offsetEnd = charCount - 1; } height= textWidget.getTextBounds(widgetOffset, offsetEnd).height; } textWidget.redraw(0, upperLeft.y, width, height, false); }</code><technical_language>modelOffset2WidgetOffset(...) return values -1. This change necessary</technical_language>

Target:
private void drawHighlightLine(Position position) { if (position.isDeleted()) return; int widgetOffset= 0; if (fViewer instanceof ITextViewerExtension5) { ITextViewerExtension5 extension= (ITextViewerExtension5) fViewer; widgetOffset= extension.modelOffset2WidgetOffset(position.getOffset()); if (widgetOffset == -1) return; } else { IRegion visible= fViewer.getVisibleRegion(); widgetOffset= position.getOffset() - visible.getOffset(); if (widgetOffset < 0 || visible.getLength() < widgetOffset ) return; } StyledText textWidget= fViewer.getTextWidget(); int charCount= textWidget.getCharCount(); if (widgetOffset > charCount) { return; } Point upperLeft= textWidget.getLocationAtOffset(widgetOffset); int width= textWidget.getClientArea().width + textWidget.getHorizontalPixel(); int height; if (position.length == 0 || !textWidget.getWordWrap()) { height= textWidget.getLineHeight(widgetOffset); } else { int offsetEnd= widgetOffset + position.length - 1; if(offsetEnd >= charCount){ offsetEnd = charCount - 1; } height= textWidget.getTextBounds(widgetOffset, offsetEnd).height; } textWidget.redraw(0, upperLeft.y, width, height, false); }
Source:
<code>void destroy() { Log.d(TAG, "Vungle banner adapter destroy:" + this); mVisibility = false; <START> mVungleManager.removeActiveBannerAd(placementId); <END> vungleBannerAd.detach(); vungleBannerAd = null; mPendingRequestBanner = false; }</code><technical_language>destroy vungleBannerAd new adapter refresh</technical_language>

Target:
void destroy() { Log.d(TAG, "Vungle banner adapter destroy:" + this); mVisibility = false; mVungleManager.removeActiveBannerAd(placementId, vungleBannerAd); if (vungleBannerAd != null) { vungleBannerAd.detach(); vungleBannerAd.destroyAd(); } vungleBannerAd = null; mPendingRequestBanner = false; }
Source:
<code><START> protected Coordinate pointToGrid(Coordinate coordinate) { <END> long gridMinXMultiplier = (long) (coordinate.x / (GRID_FACTOR * MAX_POSITIONAL_ERROR)); long gridMinYMultiplier = (long) (coordinate.y / (GRID_FACTOR * MAX_POSITIONAL_ERROR)); double gridMinXNearPoint = gridMinXMultiplier * (GRID_FACTOR * MAX_POSITIONAL_ERROR); double gridMinYNearPoint = gridMinYMultiplier * (GRID_FACTOR * MAX_POSITIONAL_ERROR); while ((gridMinXNearPoint + (GRID_FACTOR * MAX_POSITIONAL_ERROR)) < coordinate.x) { gridMinXNearPoint = gridMinXNearPoint + (GRID_FACTOR * MAX_POSITIONAL_ERROR); } while ((gridMinYNearPoint + (GRID_FACTOR * MAX_POSITIONAL_ERROR)) < coordinate.y) { gridMinYNearPoint = gridMinYNearPoint + (GRID_FACTOR * MAX_POSITIONAL_ERROR); } Coordinate minDistGridPoint = null; for (Coordinate cellCoordinate : getGridCellCoordinates(gridMinXNearPoint, gridMinYNearPoint)) { double distance = coordinate.distance(cellCoordinate); if (distance <= MAX_POSITIONAL_ERROR) { if (minDistGridPoint != null) { if (distance < coordinate.distance(minDistGridPoint)) minDistGridPoint = cellCoordinate; } else { minDistGridPoint = cellCoordinate; } } } return minDistGridPoint; }</code><technical_language>is hard create test cases test functionality of interpolation exactly, I good test parts of algorithm easily unit tested. This method is a candidate for that, is easy tests cover corner cases</technical_language>

Target:
protected Coordinate pointToGrid(Coordinate coordinate) { long gridMinXMultiplier = (long) (coordinate.x / (GRID_FACTOR * MAX_POSITIONAL_ERROR)); long gridMinYMultiplier = (long) (coordinate.y / (GRID_FACTOR * MAX_POSITIONAL_ERROR)); double gridMinXNearPoint = round(gridMinXMultiplier * (GRID_FACTOR * MAX_POSITIONAL_ERROR), ROUNDING_SCALE); double gridMinYNearPoint = round(gridMinYMultiplier * (GRID_FACTOR * MAX_POSITIONAL_ERROR), ROUNDING_SCALE); while ((gridMinXNearPoint + (GRID_FACTOR * MAX_POSITIONAL_ERROR)) < coordinate.x) { gridMinXNearPoint = gridMinXNearPoint + (GRID_FACTOR * MAX_POSITIONAL_ERROR); } while ((gridMinYNearPoint + (GRID_FACTOR * MAX_POSITIONAL_ERROR)) < coordinate.y) { gridMinYNearPoint = gridMinYNearPoint + (GRID_FACTOR * MAX_POSITIONAL_ERROR); } Coordinate minDistGridPoint = null; for (Coordinate cellCoordinate : getGridCellCoordinates(gridMinXNearPoint, gridMinYNearPoint)) { double distance = coordinate.distance(cellCoordinate); if (distance <= MAX_POSITIONAL_ERROR) { if (minDistGridPoint != null) { if (distance < coordinate.distance(minDistGridPoint)) minDistGridPoint = cellCoordinate; } else { minDistGridPoint = cellCoordinate; } } } return minDistGridPoint; }
Source:
<code>protected void saveStatsData(Bundle outState) { <START> outState.putSerializable(ARG_REST_RESPONSE, mVisitsData); <END> outState.putInt(ARG_SELECTED_GRAPH_BAR, mSelectedBarGraphBarIndex); outState.putInt(ARG_PREV_NUMBER_OF_BARS, mPrevNumberOfBarsGraph); outState.putInt(ARG_SELECTED_OVERVIEW_ITEM, mSelectedOverviewItemIndex); outState.putBoolean(ARG_CHECKBOX_SELECTED, mVisitorsCheckbox.isChecked()); }</code><technical_language>check if hasDataAvailable() adding this</technical_language>

Target:
protected void saveStatsData(Bundle outState) { if (hasDataAvailable()) { outState.putSerializable(ARG_REST_RESPONSE, mVisitsData); } outState.putInt(ARG_SELECTED_GRAPH_BAR, mSelectedBarGraphBarIndex); outState.putInt(ARG_PREV_NUMBER_OF_BARS, mPrevNumberOfBarsGraph); outState.putInt(ARG_SELECTED_OVERVIEW_ITEM, mSelectedOverviewItemIndex); outState.putBoolean(ARG_CHECKBOX_SELECTED, mVisitorsCheckbox.isChecked()); }
Source:
<code>public Single<TrackedEntityInstance> getTeiEnrollmentToSubmit(String enrollmentUid, String teiUid) { <START> return Single.fromCallable(() -> <END> trackedEntityModule.trackedEntityInstances.byUid().eq(teiUid).one().get() ).map(tei -> { Enrollment enrollment = enrollmentModule.enrollments.byUid().eq(enrollmentUid).one().get(); List<TrackedEntityAttributeValue> attributes = trackedEntityModule.trackedEntityAttributeValues.byTrackedEntityInstance().eq(teiUid).get(); return tei.toBuilder() .trackedEntityAttributeValues(attributes) .enrollments(Collections.singletonList(enrollment)) .build(); }); }</code><technical_language>replaced by: Single.fromCallable(() -> trackedEntityModule.trackedEntityInstances.uid(teiUid).withTrackedEntityAttributeValues().get().toBuilder().enrollments(Collections.singletonList(enrollment)).build());</technical_language>

Target:
public Single<TrackedEntityInstance> getTeiEnrollmentToSubmit(String enrollmentUid, String teiUid) { return Single.fromCallable(() -> { Enrollment enrollment = enrollmentModule.enrollments.byUid().eq(enrollmentUid).one().get(); return trackedEntityModule.trackedEntityInstances.withTrackedEntityAttributeValues() .byUid().eq(teiUid).one().get().toBuilder() .enrollments(Collections.singletonList(enrollment)) .build(); }); }
Source:
<code>private static File createResultsDirectory() { File resultsDirectory = CONFIG.getResultsDirectory(); if (resultsDirectory.exists() || resultsDirectory.mkdirs()) { return resultsDirectory; } else { LOGGER.error( <START> String.format("Results directory <%s> doesn't exists and can't be created", <END> resultsDirectory.getAbsolutePath()) ); return Files.createTempDir(); } }</code><technical_language>write create temp folder</technical_language>

Target:
private static File createResultsDirectory() { File resultsDirectory = CONFIG.getResultsDirectory(); if (createDirectories(resultsDirectory)) { return resultsDirectory; } LOGGER.error(String.format( "Results directory <%s> doesn't exists and can't be created, using default directory", resultsDirectory.getAbsolutePath() )); resultsDirectory = getDefaultResultsDirectory(); if (createDirectories(resultsDirectory)) { return resultsDirectory; } LOGGER.error(String.format( "Default results directory <%s> doesn't exists and can't be created, using temp directory", resultsDirectory.getAbsolutePath() )); return Files.createTempDir(); }
Source:
<code>public Optional<MemberId> upstreamDatabase() { if ( counter.shouldReturnCoreMemberId() ) { return getCoreMemberId(); } else { List<MemberId> readReplicaMembers = new ArrayList<>( topologyService.localReadReplicas().members().keySet() ); Collections.shuffle( readReplicaMembers ); <START> <END> List<MemberId> coreMembers = new ArrayList<>( topologyService.localCoreServers().members().keySet() ); Collections.shuffle( coreMembers ); return Stream.concat( readReplicaMembers.stream(), coreMembers.stream() ).filter( not( myself::equals ) ).findFirst(); } }</code><technical_language>I this is call randomReadReplicaMemberId method removed</technical_language>

Target:
public Optional<MemberId> upstreamDatabase() { if ( counter.shouldReturnCoreMemberId() ) { return randomCoreMember(); } else { List<MemberId> readReplicaMembers = new ArrayList<>( topologyService.localReadReplicas().members().keySet() ); Collections.shuffle( readReplicaMembers ); List<MemberId> coreMembers = new ArrayList<>( topologyService.localCoreServers().members().keySet() ); Collections.shuffle( coreMembers ); return Stream.concat( readReplicaMembers.stream(), coreMembers.stream() ).filter( not( myself::equals ) ).findFirst(); } }
Source:
<code>public StashDropCommand(Repository repo) { super(repo); if (!(repo.getRefDatabase() instanceof RefDirectory)) { throw new <START> IllegalArgumentException(JGitText.get().stashDropNotSupported); <END> } }</code><technical_language>UnsupportedOperationException a match</technical_language>

Target:
public StashDropCommand(Repository repo) { super(repo); if (!(repo.getRefDatabase() instanceof RefDirectory)) { throw new UnsupportedOperationException( JGitText.get().stashDropNotSupported); } }
Source:
<code><START> public PagedMessage[] getMessages() { <END> throw new UnsupportedOperationException("get messages should not be called for index page cache"); }</code><technical_language>this dangerous? code relying this, need support it, remove interface/api people dont code it</technical_language>

Target:
public synchronized PagedMessage[] getMessages() { if (pagedMessages != null) { return pagedMessages; } else { try { openPage(); return page.read().toArray(new PagedMessage[numberOfMessages]); } catch (Exception e) { throw new RuntimeException(e.getMessage(), e); } finally { close(); } } }
Source:
<code>private boolean makeGitHubTeamMaintainer(Channel channel, User sender, String newTeamMaintainer, String team) { boolean result = false; if (!isSenderAuthorized(channel, sender)) { insufficientPermissionError(channel); return false; } OutputChannel out = channel.send(); try { GitHub github = GitHub.connect(); GHUser c = github.getUser(newTeamMaintainer); GHOrganization o = github.getOrganization(IrcBotConfig.GITHUB_ORGANIZATION); final GHTeam ghTeam = o.getTeamByName(team); if (ghTeam == null) { out.message("No team for " + team); return false; } ghTeam.add(c, GHTeam.Role.MAINTAINER); <START> String successMsg = "Added " + newTeamMaintainer + " as a GitHub maintainer for team " + team; <END> out.message(successMsg); result = true; } catch (IOException e) { out.message("Failed to make user maintainer of team: " + e.getMessage()); e.printStackTrace(); } return result; }</code><technical_language>above, this need a separate declaration instead of passing string out.message</technical_language>

Target:
private boolean makeGitHubTeamMaintainer(Channel channel, User sender, String newTeamMaintainer, String team) { boolean result = false; if (!isSenderAuthorized(channel, sender)) { insufficientPermissionError(channel); return false; } OutputChannel out = channel.send(); try { GitHub github = GitHub.connect(); GHUser c = github.getUser(newTeamMaintainer); GHOrganization o = github.getOrganization(IrcBotConfig.GITHUB_ORGANIZATION); final GHTeam ghTeam = o.getTeamByName(team); if (ghTeam == null) { out.message("No team for " + team); return false; } ghTeam.add(c, GHTeam.Role.MAINTAINER); out.message("Added " + newTeamMaintainer + " as a GitHub maintainer for team " + team); result = true; } catch (IOException e) { out.message("Failed to make user maintainer of team: " + e.getMessage()); e.printStackTrace(); } return result; }
Source:
<code>public synchronized void writeQueryResult(@Nonnull String name, @Nullable String type, @Nullable Object value) throws IOException { try { if (singleLine) { <START> getTemporaryFileWriter().write(name + "=" + value + ", "); <END> } else { getTemporaryFileWriter().write("["+dfISO8601.format(Calendar.getInstance().getTime()) +"] "+name + " " + value + "\n"); } } catch (IOException e) { releaseTemporaryWriter(); throw e; } }</code><technical_language>this a trailing comma line? avoid save metrics in writeQueryResult do actual write in postCollect</technical_language>

Target:
public synchronized void writeQueryResult(@Nonnull String name, @Nullable String type, @Nullable Object value) throws IOException { try { if (singleLine) { if (firstResult) { firstResult = false; } else { getTemporaryFileWriter().write(", "); } getTemporaryFileWriter().write(name + "=" + value); } else { getTemporaryFileWriter().write("["+dfISO8601.format(Calendar.getInstance().getTime()) +"] "+name + " " + value + "\n"); } } catch (IOException e) { releaseTemporaryWriter(); throw e; } }
Source:
<code>public void testUpdateUnknownTournament() { User admin = createAdmin(); Tournament updateTournament = updateTournament(); updateTournament.setId(123L); assertEquals(0, tournamentRepository.findAll().size()); given() .header(getXAuthTokenHeaderForUser(admin)) .body(updateTournament) .contentType(ContentType.JSON) .when() .post(TOURNAMENT_ENDPOINT + updateTournament.getId()) .then() <START> .statusCode(HttpStatus.SC_INTERNAL_SERVER_ERROR) <END> .body("message", equalTo("Could not find tournament")); assertEquals(0, tournamentRepository.findAll().size()); }</code><technical_language>This is a server error, handled a bad request</technical_language>

Target:
public void testUpdateUnknownTournament() { User admin = createAdmin(); Tournament updateTournament = updateTournament(); updateTournament.setId(123L); assertEquals(0, tournamentRepository.findAll().size()); given() .header(getXAuthTokenHeaderForUser(admin)) .body(updateTournament) .contentType(ContentType.JSON) .when() .put(TOURNAMENT_ENDPOINT + updateTournament.getId()) .then() .statusCode(HttpStatus.SC_NOT_FOUND) .body("message", equalTo("Could not find tournament")); assertEquals(0, tournamentRepository.findAll().size()); }
Source:
<code>private T submitTask(SubTaskSpec<T> spec, int numAttempts) { T task = spec.newSubTask(numAttempts); try { <START> indexingServiceClient.runTask(task); <END> } catch (IllegalStateException e) { if (e.getMessage().contains("Could not resolve type id")) { log.warn(e, "Got an unknown type id error. Retrying with a backward compatible type."); task = spec.newSubTaskWithBackwardCompatibleType(numAttempts); indexingServiceClient.runTask(task); } } return task; }</code><technical_language>this approach able retry immediately, exhaust retries in indexingServiceClient first? (The loop in DruidLeaderClient) Ideally, this detects a problem submission, need exhaust retries moves backwards-compatible type. check it, if already</technical_language>

Target:
private T submitTask(SubTaskSpec<T> spec, int numAttempts) { T task = spec.newSubTask(numAttempts); try { indexingServiceClient.runTask(task); } catch (Exception e) { if (isUnknownTypeIdException(e)) { log.warn(e, "Got an unknown type id error. Retrying with a backward compatible type."); task = spec.newSubTaskWithBackwardCompatibleType(numAttempts); indexingServiceClient.runTask(task); } else { throw e; } } return task; }
Source:
<code>public void testStrictSettings() throws Exception { final DefaultConfiguration checkConfig = createCheckConfig(AvoidNestedBlocksCheck.class); final String[] expected = { "22:9: " + getCheckMessage(MSG_KEY_BLOCK_NESTED), "44:17: " + getCheckMessage(MSG_KEY_BLOCK_NESTED), "50:17: " + getCheckMessage(MSG_KEY_BLOCK_NESTED), "58:17: " + getCheckMessage(MSG_KEY_BLOCK_NESTED), }; <START> verify(checkConfig, getPath("InputAvoidNestedBlocksStrictSettings.java"), expected); <END> }</code><technical_language>splitting files individual tests yet. Please remove duplicated file point one</technical_language>

Target:
public void testStrictSettings() throws Exception { final DefaultConfiguration checkConfig = createCheckConfig(AvoidNestedBlocksCheck.class); final String[] expected = { "22:9: " + getCheckMessage(MSG_KEY_BLOCK_NESTED), "44:17: " + getCheckMessage(MSG_KEY_BLOCK_NESTED), "50:17: " + getCheckMessage(MSG_KEY_BLOCK_NESTED), "58:17: " + getCheckMessage(MSG_KEY_BLOCK_NESTED), }; verify(checkConfig, getPath("InputAvoidNestedBlocksDefault.java"), expected); }
Source:
<code>private boolean waitForCommands(){ log.debug("Waiting for host to send back all the player commands"); Map<Player, PlayerCommand> commands = new HashMap<Player, PlayerCommand>(); try { JsonObject obj = parser.parse(in).getAsJsonObject(); log.debug("Got player commands; applying commands: "+obj.toString()); commands = jsonToPlayerCommands(obj); } catch(IllegalArgumentException e) { log.error("Illegal argument while waiting for command"); cleanup(); } catch(JsonIOException e) { <START> log.error("JSON I/O error while waiting for command"); <END> cleanup(); } return getLocalGame().applyCommands(commands); }</code><technical_language>Do want log exceptions (just passing arg error)</technical_language>

Target:
private boolean waitForCommands(){ log.debug("Waiting for host to send back all the player commands"); Map<Player, PlayerCommand> commands = new HashMap<Player, PlayerCommand>(); try { JsonObject obj = parser.parse(in).getAsJsonObject(); log.debug("Got player commands; applying commands: "+obj.toString()); commands = jsonToPlayerCommands(obj); } catch(IllegalArgumentException e) { log.error("Illegal argument while waiting for command",e); cleanup(); } catch(JsonIOException e) { log.error("JSON I/O error while waiting for command",e); cleanup(); } return getLocalGame().applyCommands(commands); }
Source:
<code>private void updateGlusterHosts() { <START> for (VDS vds : allForCluster) { <END> glusterCommandHelper.initGlusterHost(vds); } }</code><technical_language>this a single line allForCluster.stream().forEach(glusterCommandHelper::initGlusterHost)</technical_language>

Target:
private void updateGlusterHosts() { allForCluster.stream().forEach(glusterCommandHelper::initGlusterHost); }
Source:
<code>List<ICommitMessageProvider> getCommitMessageProviders() { List<ICommitMessageProvider> providers = new ArrayList<>(); IExtensionRegistry registry = Platform.getExtensionRegistry(); IConfigurationElement[] configs = registry .getConfigurationElementsFor(COMMIT_MESSAGE_PROVIDER_ID); for (IConfigurationElement config : configs) { Object provider; String contributorName = "<unknown>"; try { contributorName = config.getDeclaringExtension() .getContributor().getName(); provider = config.createExecutableExtension("class"); if (provider instanceof ICommitMessageProvider) { providers.add((ICommitMessageProvider) provider); } else { Activator.logError( UIText.CommitDialog_WrongTypeOfCommitMessageProvider, null); } <START> } catch (CoreException | InvalidRegistryObjectException e) { <END> Activator.logError( MessageFormat.format( UIText.CommitDialog_ErrorCreatingCommitMessageProvider, contributorName), e); } } return providers; }</code><technical_language>catch RuntimeException here</technical_language>

Target:
List<ICommitMessageProvider> getCommitMessageProviders() { List<ICommitMessageProvider> providers = new ArrayList<>(); IExtensionRegistry registry = Platform.getExtensionRegistry(); IConfigurationElement[] configs = registry .getConfigurationElementsFor(COMMIT_MESSAGE_PROVIDER_ID); for (IConfigurationElement config : configs) { Object provider; String contributorName = "<unknown>"; String extensionId = "<unknown>"; try { extensionId = config.getDeclaringExtension() .getUniqueIdentifier(); contributorName = config.getContributor().getName(); provider = config.createExecutableExtension("class"); if (provider instanceof ICommitMessageProvider) { providers.add((ICommitMessageProvider) provider); } else { Activator.logError(MessageFormat.format( UIText.CommitDialog_WrongTypeOfCommitMessageProvider, extensionId, contributorName), null); } } catch (CoreException | RuntimeException e) { Activator.logError( MessageFormat.format( UIText.CommitDialog_ErrorCreatingCommitMessageProvider, extensionId, contributorName), e); } } return providers; }
Source:
<code><START> default byte[] getSubjectKeyIdentifier(KeyPair clientKeyPair, Set<X509ExtensionWrapper> extensions) <END> throws IOException, NoSuchAlgorithmException { return new JcaX509ExtensionUtils().createSubjectKeyIdentifier(clientKeyPair.getPublic()).getEncoded(); }</code><technical_language>I prefer this method definition DefaultSubjectKeyIdentifierWriter. reason stands, DefaultSubjectKeyIndentifierWriter is empty class</technical_language>

Target:
byte[] getSubjectKeyIdentifier(KeyPair clientKeyPair, Set<X509ExtensionWrapper> extensions) throws IOException, NoSuchAlgorithmException; default byte[] toOctetString(byte[] bytesToEncode) throws IOException { DEROctetString octetStr = new DEROctetString(bytesToEncode); return octetStr.getEncoded(); }
Source:
<code>private synchronized void onSubscription() { if (service.isRegistered(this)) { logger.debug("Checking WeMo GENA subscription for '{}'", this); ThingTypeUID thingTypeUID = thing.getThingTypeUID(); String subscription = null; if (thingTypeUID.equals(THING_TYPE_INSIGHT)) { subscription = "insight1"; } else { subscription = "basicevent1"; <START> service.addSubscription(this, "basicevent1", SUBSCRIPTION_DURATION); <END> } if ((subscriptionState.get(subscription) == null) || !subscriptionState.get(subscription).booleanValue()) { logger.debug("Setting up GENA subscription {}: Subscribing to service {}...", getUDN(), subscription); service.addSubscription(this, subscription, SUBSCRIPTION_DURATION); subscriptionState.put(subscription, true); } } else { logger.warn("Setting up WeMo GENA subscription for '{}' FAILED - service.isRegistered(this) is FALSE", this); } }</code><technical_language>subscribe (L369) please remove L364</technical_language>

Target:
private synchronized void onSubscription() { if (service.isRegistered(this)) { logger.debug("Checking WeMo GENA subscription for '{}'", this); ThingTypeUID thingTypeUID = thing.getThingTypeUID(); String subscription = null; if (thingTypeUID.equals(THING_TYPE_INSIGHT)) { subscription = "insight1"; } else { subscription = "basicevent1"; } if ((subscriptionState.get(subscription) == null) || !subscriptionState.get(subscription).booleanValue()) { logger.debug("Setting up GENA subscription {}: Subscribing to service {}...", getUDN(), subscription); service.addSubscription(this, subscription, SUBSCRIPTION_DURATION); subscriptionState.put(subscription, true); } } else { logger.debug("Setting up WeMo GENA subscription for '{}' FAILED - service.isRegistered(this) is FALSE", this); } }
Source:
<code>TcpIgniteClient( Function<ClientChannelConfiguration, ClientChannel> chFactory, ClientConfiguration cfg ) throws ClientException { final ClientBinaryMetadataHandler metadataHandler = new ClientBinaryMetadataHandler(); marsh = new ClientBinaryMarshaller(metadataHandler, new ClientMarshallerContext()); marsh.setBinaryConfiguration(cfg.getBinaryConfiguration()); serDes = new ClientUtils(marsh); binary = new ClientBinary(marsh); ch = new ReliableChannel(chFactory, cfg, binary); <START> ch.initConnection(); <END> ch.addChannelFailListener(() -> metadataHandler.onReconnect()); transactions = new TcpClientTransactions(ch, marsh, new ClientTransactionConfiguration(cfg.getTransactionConfiguration())); cluster = new ClientClusterImpl(ch, marsh); compute = new ClientComputeImpl(ch, marsh, cluster.defaultClusterGroup()); services = new ClientServicesImpl(ch, marsh, cluster.defaultClusterGroup()); }</code><technical_language>Is a reason bring this of constructor</technical_language>

Target:
TcpIgniteClient( Function<ClientChannelConfiguration, ClientChannel> chFactory, ClientConfiguration cfg ) throws ClientException { final ClientBinaryMetadataHandler metadataHandler = new ClientBinaryMetadataHandler(); marsh = new ClientBinaryMarshaller(metadataHandler, new ClientMarshallerContext()); marsh.setBinaryConfiguration(cfg.getBinaryConfiguration()); serDes = new ClientUtils(marsh); binary = new ClientBinary(marsh); ch = new ReliableChannel(chFactory, cfg, binary); ch.channelsInit(); ch.addChannelFailListener(() -> metadataHandler.onReconnect()); transactions = new TcpClientTransactions(ch, marsh, new ClientTransactionConfiguration(cfg.getTransactionConfiguration())); cluster = new ClientClusterImpl(ch, marsh); compute = new ClientComputeImpl(ch, marsh, cluster.defaultClusterGroup()); services = new ClientServicesImpl(ch, marsh, cluster.defaultClusterGroup()); }
Source:
<code>private void resetData(AccountObject account) { mId = account.getId(); mUsername = account.getUsername(); mPassword = account.getPassword(); mEmail = account.getEmail(); mNickName = account.getNickName(); mEnable = account.getEnable(); <START> mRoles = null; <END> }</code><technical_language>remove mRoles</technical_language>

Target:
private void resetData(AccountObject account) { mId = account.getId(); mUsername = account.getUsername(); mPassword = account.getPassword(); mEmail = account.getEmail(); mNickName = account.getNickName(); mEnable = account.getEnable(); }
Source:
<code>public void nonEmptyArrayToUnmodifiableNonNullListOfStrings() { Object[] array = {"1", "2", "3"}; List<String> list = InternalQueries.unmodifiableNonNullListOfStrings(array); <START> assertThat(list.size()).isEqualTo(array.length); <END> for (int i = 0; i < array.length; i++) { assertThat(list.get(i)).isEqualTo(array[i]); } }</code><technical_language>hasSize(array.length)</technical_language>

Target:
public void nonEmptyArrayToUnmodifiableNonNullListOfStrings() { Object[] array = {"1", "2", "3"}; List<String> list = InternalQueries.unmodifiableNonNullListOfStrings(array); assertThat(list).hasSize(array.length); for (int i = 0; i < array.length; i++) { assertThat(list.get(i)).isEqualTo(array[i]); } }
Source:
<code>private void connect() { LOG.trace("Connecting to {}", remoteId.address); this.channel = new Bootstrap().group(rpcClient.group).channel(rpcClient.channelClass) .option(ChannelOption.TCP_NODELAY, rpcClient.isTcpNoDelay()) .option(ChannelOption.SO_KEEPALIVE, rpcClient.tcpKeepAlive) .option(ChannelOption.CONNECT_TIMEOUT_MILLIS, rpcClient.connectTO) .handler(new BufferCallBeforeInitHandler()).localAddress(rpcClient.localAddr) .option(ChannelOption.WRITE_BUFFER_WATER_MARK, <START> ((NettyRpcClient) rpcClient).writeBufferWaterMark) <END> .remoteAddress(remoteId.address).connect().addListener(new ChannelFutureListener() { @Override public void operationComplete(ChannelFuture future) throws Exception { Channel ch = future.channel(); if (!future.isSuccess()) { failInit(ch, toIOE(future.cause())); rpcClient.failedServers.addToFailedServers(remoteId.address, future.cause()); return; } ch.writeAndFlush(connectionHeaderPreamble.retainedDuplicate()); if (useSasl) { saslNegotiate(ch); } else { ch.write(connectionHeaderWithLength.retainedDuplicate()); established(ch); } } }).channel(); }</code><technical_language>This cast is redundant</technical_language>

Target:
private void connect() { LOG.trace("Connecting to {}", remoteId.address); this.channel = new Bootstrap().group(rpcClient.group).channel(rpcClient.channelClass) .option(ChannelOption.TCP_NODELAY, rpcClient.isTcpNoDelay()) .option(ChannelOption.SO_KEEPALIVE, rpcClient.tcpKeepAlive) .option(ChannelOption.CONNECT_TIMEOUT_MILLIS, rpcClient.connectTO) .option(ChannelOption.SO_REUSEADDR, rpcClient.tcpReuseAddr) .handler(new BufferCallBeforeInitHandler()).localAddress(rpcClient.localAddr) .option(ChannelOption.WRITE_BUFFER_WATER_MARK, rpcClient.writeBufferWaterMark) .remoteAddress(remoteId.address).connect().addListener(new ChannelFutureListener() { @Override public void operationComplete(ChannelFuture future) throws Exception { Channel ch = future.channel(); if (!future.isSuccess()) { failInit(ch, toIOE(future.cause())); rpcClient.failedServers.addToFailedServers(remoteId.address, future.cause()); return; } ch.writeAndFlush(connectionHeaderPreamble.retainedDuplicate()); if (useSasl) { saslNegotiate(ch); } else { ch.write(connectionHeaderWithLength.retainedDuplicate()); established(ch); } } }).channel(); }
Source:
<code>public byte[] convertToByteArray() { <START> byte[] secreteKeyBytes = secretKey.getEncoded(); <END> byte[] retVal = new byte[SecretKeyUtil.INITIALIZATION_VECTOR_LENGTH + secreteKeyBytes.length]; System.arraycopy(initializationVector, 0, retVal, 0, SecretKeyUtil.INITIALIZATION_VECTOR_LENGTH); System.arraycopy(secreteKeyBytes, 0, retVal, SecretKeyUtil.INITIALIZATION_VECTOR_LENGTH, secreteKeyBytes.length); return retVal; }</code><technical_language>secretKeyBytes</technical_language>

Target:
public byte[] convertToByteArray() { byte[] secretKeyBytes = secretKey.getEncoded(); byte[] retVal = new byte[SecretKeyUtil.INITIALIZATION_VECTOR_LENGTH + secretKeyBytes.length]; System.arraycopy(initializationVector, 0, retVal, 0, SecretKeyUtil.INITIALIZATION_VECTOR_LENGTH); System.arraycopy(secretKeyBytes, 0, retVal, SecretKeyUtil.INITIALIZATION_VECTOR_LENGTH, secretKeyBytes.length); return retVal; }
Source:
<code>public ServiceCache<S> build() { <START> if (_cachingPolicy.useSingletonPolicy()) { <END> return new SingletonServiceCache<>(_serviceFactory); } else { return new ServiceCache<>(_cachingPolicy, _serviceFactory, _metricRegistry); } }</code><technical_language>null check _cachingPolicy</technical_language>

Target:
public ServiceCache<S> build() { checkNotNull(_cachingPolicy); if (_cachingPolicy.useMultiThreadedClientPolicy()) { checkArgument(_cachingPolicy.evictionTTLForMultiThreadedClientPolicy() >= 0); checkNotNull(_serviceFactory); if(!(_serviceFactory instanceof ThreadSafeServiceFactory)) { throw new IllegalArgumentException("Please implement ThreadSafeServiceFactory to construct MultiThreadedClientServiceCache"); } return new MultiThreadedClientServiceCache<>((ThreadSafeServiceFactory<S>) _serviceFactory, _cachingPolicy.evictionTTLForMultiThreadedClientPolicy(), _metricRegistry); } else { checkNotNull(_serviceFactory); checkNotNull(_metricRegistry); return new SingleThreadedClientServiceCache<>(_cachingPolicy, _serviceFactory, _metricRegistry); } }
Source:
<code>protected IPersonAttributes getSinglePerson(final Set<IPersonAttributes> people) { IPersonAttributes person; try { person = DataAccessUtils.singleResult(people); } catch (final IncorrectResultSizeDataAccessException e) { logger.debug("Unexpected multiple people returned from person attribute DAO: : ", e.getClass().getName(), e.getMessage()); <START> people.stream().forEach(p -> { <END> logger.debug("Person: {}", p); }); throw e; } return person; }</code><technical_language>stream() unnecessary. Call forEach a lambda</technical_language>

Target:
protected IPersonAttributes getSinglePerson(final Set<IPersonAttributes> people) { IPersonAttributes person; try { person = DataAccessUtils.singleResult(people); } catch (final IncorrectResultSizeDataAccessException e) { logger.warn("Unexpected multiple people returned from person attribute DAO: : ", e.getClass().getName(), e.getMessage()); people.forEach(p -> logger.debug("Person: {}", p)); throw e; } return person; }
Source:
<code>public void testValue() throws Exception { ModelNode address = getAddress(); <START> ModelNode operation = getOperation(address); <END> ModelNode res = executeOperation(operation); ModelNode node = res.get("java: contexts", "java:jboss/exported", "test", "children", "Bean!org.jboss.as.test.integration.naming.remote.ejb.Remote", "class-name"); Assert.assertEquals("org.jboss.as.test.integration.naming.remote.ejb.Remote", node.asString()); }</code><technical_language>Please a org.jboss.as.controller.operations.common.Util.createEmptyOperation</technical_language>

Target:
public void testValue() throws Exception { ModelNode operation = Util.createEmptyOperation("jndi-view", PathAddress.pathAddress("subsystem", "naming")); ModelNode res = ManagementOperations.executeOperation(managementClient.getControllerClient(), operation); ModelNode node = res.get("java: contexts", "java:jboss/exported", "test", "children", "Bean!org.jboss.as.test.integration.naming.remote.ejb.Remote", "class-name"); Assert.assertEquals("org.jboss.as.test.integration.naming.remote.ejb.Remote", node.asString()); }
Source:
<code>protected void executeVdsBrokerCommand() { String volumeName = getParameters().getVolumeName(); result = getBroker().glusterVolumeRebalanceStop(volumeName); GlusterVolumeTaskStatusEntity entity = result.getStatusDetails(); entity.setStatusTime(new Date()); entity.setStopTime(new Date()); <START> proceedProxyReturnValue(); <END> setReturnValue(entity); }</code><technical_language>move this line? basically proceedProxyReturnValue called result, in case of error null expected, NPE hide real error message vdsm</technical_language>

Target:
protected void executeVdsBrokerCommand() { String volumeName = getParameters().getVolumeName(); result = getBroker().glusterVolumeRebalanceStop(volumeName); proceedProxyReturnValue(); GlusterVolumeTaskStatusEntity entity = result.getStatusDetails(); entity.setStatusTime(new Date()); entity.setStopTime(new Date()); setReturnValue(entity); }
Source:
<code><START> private void onSessionStateChange(final Session session, SessionState state, Exception exception) { <END> if (state.isOpened()) { Request request = Request.newMeRequest(session, new Request.GraphUserCallback() { @Override public void onCompleted(GraphUser user, Response response) { if (session.equals(Session.getActiveSession()) && user != null) { String name = user.getName(); String tokenString = String.format(getString(R.string.facebook_token_format), session.getAccessToken()); loginSuccessCallback.onLoginSuccess(name, tokenString); } } }); Request.executeBatchAsync(request); } }</code><technical_language>exception in this line used. lose i guess haha</technical_language>

Target:
private void onSessionStateChange(final Session session, SessionState state, Exception exception) { if (exception != null) { Timber.e("Facebook error: %s", exception.getMessage()); } if (state.isOpened()) { Request request = Request.newMeRequest(session, new Request.GraphUserCallback() { @Override public void onCompleted(GraphUser user, Response response) { if (session.equals(Session.getActiveSession()) && user != null) { String name = user.getName(); String tokenString = String.format(getString(R.string.facebook_token_format), session.getAccessToken()); loginSuccessCallback.onLoginSuccess(name, tokenString); } } }); Request.executeBatchAsync(request); } }
Source:
<code>private void walkAST(LanguageConnectionContext lcc, Visitable queryTree, CompilationPhase phase) throws StandardException { ASTVisitor visitor = lcc.getASTVisitor(); try { if (visitor != null) { visitor.begin(statementText, phase); queryTree.accept(visitor); } <START> } catch (StandardException e) { <END> throw e; } finally { visitor.end(phase); } }</code><technical_language>need catch clause, want explicit</technical_language>

Target:
private void walkAST(LanguageConnectionContext lcc, Visitable queryTree, CompilationPhase phase) throws StandardException { ASTVisitor visitor = lcc.getASTVisitor(); if (visitor != null) { try { visitor.begin(statementText, phase); queryTree.accept(visitor); } finally { visitor.end(phase); } } }
Source:
<code>private Container createContainer() throws DotSecurityException, DotDataException { Container container = new Container(); container.setFriendlyName("test container"); container.setTitle("this is the title"); container.setMaxContentlets(5); container.setPreLoop("preloop code"); container.setPostLoop("postloop code"); <START> final Structure st=CacheLocator.getContentTypeCache().getStructureByVelocityVarName("host"); <END> final List<ContainerStructure> csList = new ArrayList<>(); ContainerStructure cs = new ContainerStructure(); cs.setStructureId(st.getInode()); cs.setCode("this is the code"); csList.add(cs); return containerAPI.save(container, csList, defaultHost, user, false); }</code><technical_language>this call replaced contentTypeAPI.find(inodeOrVar)</technical_language>

Target:
private Container createContainer() throws DotSecurityException, DotDataException { Container container = new Container(); container.setFriendlyName("test container"); container.setTitle("this is the title"); container.setMaxContentlets(5); container.setPreLoop("preloop code"); container.setPostLoop("postloop code"); final List<ContainerStructure> csList = new ArrayList<>(); ContainerStructure cs = new ContainerStructure(); cs.setStructureId(contentTypeAPI.find("host").inode()); cs.setCode("this is the code"); csList.add(cs); return containerAPI.save(container, csList, defaultHost, user, false); }
Source:
<code>public Poller(Duration pollInterval, Function<PollResponse<T>, Mono<PollResponse<T>>> pollOperation) { <START> <END> Objects.requireNonNull(pollInterval, "The poll interval input parameter cannot be null."); if (pollInterval.toNanos() <= 0) { throw new IllegalArgumentException("Negative or zero value for poll interval not allowed."); } Objects.requireNonNull(pollOperation, "The poll operation input parameter cannot be null."); this.pollInterval = pollInterval; this.pollOperation = pollOperation; this.pollResponse = new PollResponse<>(PollResponse.OperationStatus.NOT_STARTED, null); this.fluxHandle = asyncPollRequestWithDelay() .flux() .repeat() .takeUntil(pollResponse -> pollResponse != null && pollResponse.isDone()) .share(); this.fluxDisposable = fluxHandle.subscribe(); this.autoPollingEnabled = true; }</code><technical_language>Remove newline</technical_language>

Target:
public Poller(Duration pollInterval, Function<PollResponse<T>, Mono<PollResponse<T>>> pollOperation) { Objects.requireNonNull(pollInterval, "The poll interval input parameter cannot be null."); if (pollInterval.toNanos() <= 0) { throw new IllegalArgumentException("Negative or zero value for poll interval not allowed."); } Objects.requireNonNull(pollOperation, "The poll operation input parameter cannot be null."); this.pollInterval = pollInterval; this.pollOperation = pollOperation; this.pollResponse = new PollResponse<>(PollResponse.OperationStatus.NOT_STARTED, null); this.fluxHandle = asyncPollRequestWithDelay() .flux() .repeat() .takeUntil(pollResponse -> hasCompleted()) .share(); this.fluxDisposable = fluxHandle.subscribe(); this.autoPollingEnabled = true; }
Source:
<code><START> public Response getInsertionDetails(@PathParam("programUuid") long programUuid, @PathParam("uuid") long uuid) throws Exception { <END> ArtifactReadable configArt = atsServer.getQuery().andUuid(uuid).getResults().getExactlyOne(); return getResponse(atsServer.getConfigItemFactory().getInsertion(configArt)); }</code><technical_language>need handle case configArt == null</technical_language>

Target:
public Response getInsertionDetails(@PathParam("programUuid") long programUuid, @PathParam("uuid") long uuid) throws Exception { ArtifactReadable configArt = atsServer.getArtifactByUuid(uuid); if (configArt == null) { throw new OseeCoreException("Given uuid not found"); } return Response.ok().entity(atsServer.getConfigItemFactory().getInsertion(configArt)).build(); }
Source:
<code>public boolean onPrepareOptionsMenu(Menu menu) { super.onPrepareOptionsMenu(menu); if (isSearching()) { return false; } MenuItem otherLangItem = menu.findItem(R.id.menu_page_other_languages); MenuItem shareItem = menu.findItem(R.id.menu_page_share); MenuItem addToListItem = menu.findItem(R.id.menu_page_add_to_list); MenuItem findInPageItem = menu.findItem(R.id.menu_page_find_in_page); MenuItem contentIssues = menu.findItem(R.id.menu_page_content_issues); MenuItem similarTitles = menu.findItem(R.id.menu_page_similar_titles); MenuItem themeChooserItem = menu.findItem(R.id.menu_page_font_and_theme); MenuItem tabsItem = menu.findItem(R.id.menu_page_show_tabs); tabsItem.setIcon(ResourceUtil.getTabListIcon(pageFragment.getTabCount())); if (pageFragment.isLoading() || pageFragment.getErrorState()) { otherLangItem.setEnabled(false); shareItem.setEnabled(false); addToListItem.setEnabled(false); findInPageItem.setEnabled(false); contentIssues.setEnabled(false); similarTitles.setEnabled(false); themeChooserItem.setEnabled(false); } else { otherLangItem.setVisible(pageFragment.getPage() != null && pageFragment.getPage().getPageProperties().getLanguageCount() != 0); otherLangItem.setEnabled(true); shareItem.setEnabled(pageFragment.getPage() != null && pageFragment.getPage().isArticle()); addToListItem.setEnabled(pageFragment.getPage() != null && pageFragment.getPage().isArticle()); findInPageItem.setEnabled(true); <START> updateMenuPageInfo(menu); <END> themeChooserItem.setEnabled(true); } return true; }</code><technical_language>this themeChooserItem.setEnable(true)</technical_language>

Target:
public boolean onPrepareOptionsMenu(Menu menu) { super.onPrepareOptionsMenu(menu); if (isSearching()) { return false; } MenuItem otherLangItem = menu.findItem(R.id.menu_page_other_languages); MenuItem shareItem = menu.findItem(R.id.menu_page_share); MenuItem addToListItem = menu.findItem(R.id.menu_page_add_to_list); MenuItem findInPageItem = menu.findItem(R.id.menu_page_find_in_page); MenuItem contentIssues = menu.findItem(R.id.menu_page_content_issues); MenuItem similarTitles = menu.findItem(R.id.menu_page_similar_titles); MenuItem themeChooserItem = menu.findItem(R.id.menu_page_font_and_theme); MenuItem tabsItem = menu.findItem(R.id.menu_page_show_tabs); tabsItem.setIcon(ResourceUtil.getTabListIcon(pageFragment.getTabCount())); if (pageFragment.isLoading() || pageFragment.getErrorState()) { otherLangItem.setEnabled(false); shareItem.setEnabled(false); addToListItem.setEnabled(false); findInPageItem.setEnabled(false); contentIssues.setEnabled(false); similarTitles.setEnabled(false); themeChooserItem.setEnabled(false); } else { otherLangItem.setVisible(pageFragment.getPage() != null && pageFragment.getPage().getPageProperties().getLanguageCount() != 0); otherLangItem.setEnabled(true); shareItem.setEnabled(pageFragment.getPage() != null && pageFragment.getPage().isArticle()); addToListItem.setEnabled(pageFragment.getPage() != null && pageFragment.getPage().isArticle()); findInPageItem.setEnabled(true); themeChooserItem.setEnabled(true); updateMenuPageInfo(menu); } return true; }
Source:
<code>protected GlusterVolumeTaskStatusEntity fetchTaskStatusDetails() { VDSReturnValue returnValue = runVdsCommand(VDSCommandType.GetGlusterVolumeRemoveBricksStatus, new GlusterVolumeRemoveBricksVDSParameters(getUpServerId(clusterId), volume.getName(), getParameters().getBricks(), volume.getReplicaCount())); <START> GlusterVolumeTaskStatusEntity entity = setStartTime((GlusterVolumeTaskStatusEntity) returnValue.getReturnValue()); <END> return updateHostDetails(entity); }</code><technical_language>setStartTime moved super</technical_language>

Target:
protected GlusterVolumeTaskStatusEntity fetchTaskStatusDetails() { VDSReturnValue returnValue = runVdsCommand(VDSCommandType.GetGlusterVolumeRemoveBricksStatus, new GlusterVolumeRemoveBricksVDSParameters(getUpServerId(clusterId), volume.getName(), getParameters().getBricks(), volume.getReplicaCount())); return updateStatusEntity((GlusterVolumeTaskStatusEntity) returnValue.getReturnValue()); }
Source:
<code><START> void subscribe(@Nonnull StreamListener streamListener, @Nonnull String namespace, <END> @Nonnull String streamTag, @Nonnull List<String> tablesOfInterest, @Nullable Timestamp timestamp) { runtime.getTableRegistry().getStreamManager() .subscribe(streamListener, namespace, streamTag, tablesOfInterest, (timestamp == null) ? getTimestamp().getSequence() : timestamp.getSequence()); }</code><technical_language>Codacy issue: [Use explicit scoping instead of default package private level](<LINK_0></technical_language>

Target:
public void subscribe(@Nonnull StreamListener streamListener, @Nonnull String namespace, @Nonnull String streamTag, @Nonnull List<String> tablesOfInterest, @Nullable Timestamp timestamp) { runtime.getTableRegistry().getStreamManager() .subscribe(streamListener, namespace, streamTag, tablesOfInterest, (timestamp == null) ? getTimestamp().getSequence() : timestamp.getSequence()); }