auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code><START> public static List<IAnalysisParameterProvider> getParameterProviders(IAnalysisModule module, ITmfTrace trace) { <END> List<IAnalysisParameterProvider> providerList = new ArrayList<IAnalysisParameterProvider>(); if (!fParameterProviders.containsKey(module.getId())) { return providerList; } for (Class<? extends IAnalysisParameterProvider> providerClass : fParameterProviders.get(module.getId())) { try { IAnalysisParameterProvider provider = providerClass.newInstance(); if (provider.appliesToTrace(trace)) { providerList.add(provider); } } catch (IllegalArgumentException e) { Activator.logError("Error instantiating parameter provider", e); } catch (SecurityException e) { Activator.logError("Error instantiating parameter provider", e); } catch (InstantiationException e) { Activator.logError("Error instantiating parameter provider", e); } catch (IllegalAccessException e) { Activator.logError("Error instantiating parameter provider", e); } } return providerList; }</code><technical_language>this called multiple threads? If is a concurrency issue. fParameterProviders needs handle thread safe in this class</technical_language>

Target:
public static List<IAnalysisParameterProvider> getParameterProviders(IAnalysisModule module, ITmfTrace trace) { List<IAnalysisParameterProvider> providerList = new ArrayList<IAnalysisParameterProvider>(); synchronized (fParameterProviders) { if (!fParameterProviders.containsKey(module.getId())) { return providerList; } for (Class<? extends IAnalysisParameterProvider> providerClass : fParameterProviders.get(module.getId())) { try { IAnalysisParameterProvider provider = providerClass.newInstance(); if (provider.appliesToTrace(trace)) { providerList.add(provider); } } catch (IllegalArgumentException e) { Activator.logError(Messages.TmfAnalysisManager_ErrorParameterProvider, e); } catch (SecurityException e) { Activator.logError(Messages.TmfAnalysisManager_ErrorParameterProvider, e); } catch (InstantiationException e) { Activator.logError(Messages.TmfAnalysisManager_ErrorParameterProvider, e); } catch (IllegalAccessException e) { Activator.logError(Messages.TmfAnalysisManager_ErrorParameterProvider, e); } } } return providerList; }
Source:
<code>private void addIssueIfNotCaught(Type thrown, Tree node) { if (isNotCaught(thrown)) { <START> String message = "Handle the " + "\"" + thrown + "\"" + " thrown here in a \"try/catch\" block."; <END> reportIssue(node, message); } }</code><technical_language>rely ToString() method of a type. if change in implementation class for debugging purpose? thrown.name() instead</technical_language>

Target:
private void addIssueIfNotCaught(Type thrown, Tree node) { if (isNotCaught(thrown)) { String message = "Handle the " + "\"" + thrown.name() + "\"" + " thrown here in a \"try/catch\" block."; reportIssue(node, message); } }
Source:
<code>private static TestResult findPreviousTestResult(Run<?, ?> originalBuild, TaskListener listener, String alternateJob) { Run<?,?> b = originalBuild.getPreviousBuild(); if (alternateJob != null && !alternateJob.equals("")) { TopLevelItem item = Jenkins.getInstance().getItem(alternateJob); if (item != null) { if (item instanceof Job) { <START> Job j = (Job) item; <END> if (!j.getBuilds().isEmpty()) { b = j.getLastBuild(); } else { listener.getLogger().printf("Job named %s does not have any existing builds, falling back to current job", alternateJob); } } else { listener.getLogger().printf("Item named %s is not a job, falling back to current job.", alternateJob); } } else { listener.getLogger().printf("Could not find job named %s, falling back to current job.", alternateJob); } } for (int i = 0; i < NUMBER_OF_BUILDS_TO_SEARCH - 1; i++) { if (b == null) break; if(!RESULTS_OF_BUILDS_TO_CONSIDER.contains(b.getResult())) continue; AbstractTestResultAction tra = b.getAction(AbstractTestResultAction.class); if (tra == null) continue; Object o = tra.getResult(); if (o instanceof TestResult) { listener.getLogger().printf("Using build #%d as reference\n", b.getNumber()); return (TestResult) o; } b = b.getPreviousBuild(); } return null; }</code><technical_language>bug: no. java Job j = Jenkins.getInstance().getItem(alternateJob, originalBuild.getParent(), Job.class);</technical_language>

Target:
private static TestResult findPreviousTestResult(Run<?, ?> originalBuild, TaskListener listener, String alternateJob) { Run<?,?> b = originalBuild.getPreviousBuild(); if (alternateJob != null && !alternateJob.equals("")) { Job j = Jenkins.getInstance().getItem(alternateJob, originalBuild.getParent(), Job.class); if (j != null) { if (!j.getBuilds().isEmpty()) { b = j.getLastBuild(); } else { listener.getLogger().printf("Job named %s does not have any existing builds, falling back to current job", alternateJob); } } else { listener.getLogger().printf("Could not find job named %s, falling back to current job.", alternateJob); } } for (int i = 0; i < NUMBER_OF_BUILDS_TO_SEARCH - 1; i++) { if (b == null) break; if(!RESULTS_OF_BUILDS_TO_CONSIDER.contains(b.getResult())) continue; AbstractTestResultAction tra = b.getAction(AbstractTestResultAction.class); if (tra == null) continue; Object o = tra.getResult(); if (o instanceof TestResult) { listener.getLogger().printf("Using build #%d as reference\n", b.getNumber()); return (TestResult) o; } b = b.getPreviousBuild(); } return null; }
Source:
<code>public <V> DataSet<V> readParquetFile(int[] baseColumnMap, int[] partitionColumnMap, String location, OperationContext context, Qualifier[][] qualifiers, DataValueDescriptor probeValue, ExecRow execRow, boolean useSample, double sampleFraction) throws StandardException { try { Dataset<Row> table = null; try { table = SpliceSpark.getSession().read().parquet(location); } catch (Exception e) { return handleExceptionInCaseOfEmptySet(e,location); } table = processExternalDataset(table,baseColumnMap,qualifiers,probeValue); if (useSample) { return new SparkDataSet(table .rdd().toJavaRDD() <START> .map(new RowToLocatedRowFunction(context, execRow)).sample(false, sampleFraction)); <END> } else { return new SparkDataSet(table .rdd().toJavaRDD() .map(new RowToLocatedRowFunction(context, execRow))); } } catch (Exception e) { throw StandardException.newException( SQLState.EXTERNAL_TABLES_READ_FAILURE,e.getMessage()); } }</code><technical_language>a bit faster sample applying map</technical_language>

Target:
public <V> DataSet<V> readParquetFile(int[] baseColumnMap, int[] partitionColumnMap, String location, OperationContext context, Qualifier[][] qualifiers, DataValueDescriptor probeValue, ExecRow execRow, boolean useSample, double sampleFraction) throws StandardException { try { Dataset<Row> table = null; try { table = SpliceSpark.getSession().read().parquet(location); } catch (Exception e) { return handleExceptionInCaseOfEmptySet(e,location); } table = processExternalDataset(table,baseColumnMap,qualifiers,probeValue); if (useSample) { return new SparkDataSet(table .rdd().toJavaRDD() .sample(false, sampleFraction) .map(new RowToLocatedRowFunction(context, execRow))); } else { return new SparkDataSet(table .rdd().toJavaRDD() .map(new RowToLocatedRowFunction(context, execRow))); } } catch (Exception e) { throw StandardException.newException( SQLState.EXTERNAL_TABLES_READ_FAILURE,e.getMessage()); } }
Source:
<code>public ThingBuilder withChannels(Channel... channels) { <START> List<Channel> newChannels = Arrays.asList(channels); <END> validateChannelUIDs(newChannels); ThingHelper.ensureUniqueChannels(newChannels); this.thing.setChannels(new ArrayList<>(newChannels)); return this; }</code><technical_language>call return withChannels(Arrays.asList(channels)); instead of adding implementation twice? do I miss something</technical_language>

Target:
public ThingBuilder withChannels(Channel... channels) { return withChannels(Arrays.asList(channels)); }
Source:
<code>protected void processPacket(QuorumPacket qp) throws IOException{ switch (qp.getType()) { case Leader.PING: ping(qp); break; case Leader.PROPOSAL: LOG.warn("Ignoring proposal"); break; case Leader.COMMIT: LOG.warn("Ignoring commit"); break; case Leader.UPTODATE: LOG.error("Received an UPTODATE message after Observer started"); break; case Leader.REVALIDATE: revalidate(qp); break; case Leader.SYNC: ((ObserverZooKeeperServer)zk).sync(); break; case Leader.INFORM: TxnHeader hdr = new TxnHeader(); Record txn = SerializeUtils.deserializeTxn(qp.getData(), hdr); Request request = new Request (null, hdr.getClientId(), hdr.getCxid(), hdr.getType(), null, null); request.txn = txn; request.hdr = hdr; ObserverZooKeeperServer obs = (ObserverZooKeeperServer)zk; obs.commitRequest(request); break; default: <START> LOG.error("Invalid packet type received by Observer: " + qp.getType()); <END> } }</code><technical_language>change log message improve readability. LOG.error("Invalid packet type: received Observer", qp.getType());</technical_language>

Target:
protected void processPacket(QuorumPacket qp) throws IOException{ switch (qp.getType()) { case Leader.PING: ping(qp); break; case Leader.PROPOSAL: LOG.warn("Ignoring proposal"); break; case Leader.COMMIT: LOG.warn("Ignoring commit"); break; case Leader.UPTODATE: LOG.error("Received an UPTODATE message after Observer started"); break; case Leader.REVALIDATE: revalidate(qp); break; case Leader.SYNC: ((ObserverZooKeeperServer)zk).sync(); break; case Leader.INFORM: TxnHeader hdr = new TxnHeader(); Record txn = SerializeUtils.deserializeTxn(qp.getData(), hdr); Request request = new Request (null, hdr.getClientId(), hdr.getCxid(), hdr.getType(), null, null); request.txn = txn; request.hdr = hdr; ObserverZooKeeperServer obs = (ObserverZooKeeperServer)zk; obs.commitRequest(request); break; default: LOG.error("Invalid packet type: received by Observer", qp.getType()); } }
Source:
<code>private static Configuration createClusteredLockCacheConfiguration(ClusteredLockManagerConfiguration config) { ConfigurationBuilder builder = new ConfigurationBuilder(); builder.transaction().transactionMode(TransactionMode.NON_TRANSACTIONAL); if (config.consistency() == Consistency.LOW) { <START> builder.clustering() <END> .cacheMode(CacheMode.DIST_SYNC).hash().numOwners(config.numOwners()); } else { builder.clustering().cacheMode(CacheMode.REPL_SYNC); } if (config.reliability() == Reliability.CONSISTENT) { builder.clustering() .partitionHandling().whenSplit(PartitionHandling.DENY_READ_WRITES); } else { builder.clustering().partitionHandling().whenSplit(PartitionHandling.ALLOW_READ_WRITES); } return builder.build(); }</code><technical_language>suggestion builder.clustering().cacheMode(CacheMode.DIST_SYNC) .hash().numOwners(config.numOwners());</technical_language>

Target:
private static Configuration createClusteredLockCacheConfiguration(ClusteredLockManagerConfiguration config) { ConfigurationBuilder builder = new ConfigurationBuilder(); builder.transaction().transactionMode(TransactionMode.NON_TRANSACTIONAL); if (config.numOwners() > 0) { builder.clustering().cacheMode(CacheMode.DIST_SYNC) .hash().numOwners(config.numOwners()); } else { builder.clustering().cacheMode(CacheMode.REPL_SYNC); } if (config.reliability() == Reliability.CONSISTENT) { builder.clustering() .partitionHandling().whenSplit(PartitionHandling.DENY_READ_WRITES); } else { builder.clustering().partitionHandling().whenSplit(PartitionHandling.ALLOW_READ_WRITES); } return builder.build(); }
Source:
<code>public Collection getApplicableContributors(IStructuredSelection selection) { boolean isMultiSelection = selection.size() > 1; Iterator it = selection.iterator(); Collection result = null; while (it.hasNext()) { Object element = it.next(); Collection collection = getApplicableContributors(element); if (isMultiSelection) { <START> it = collection.iterator(); <END> while (it.hasNext()) { RegistryPageContributor contrib = (RegistryPageContributor) it.next(); if (!contrib.supportsMultipleSelection()) { it.remove(); } } } if (result == null) result = new LinkedHashSet(collection); else result.retainAll(collection); if (result.isEmpty()) { return Collections.emptyList(); } } return result; }</code><technical_language>"it" contained selection elements contains contributors? Is generify this code avoid confusion</technical_language>

Target:
public Collection getApplicableContributors(IStructuredSelection selection) { boolean isMultiSelection = selection.size() > 1; Iterator<Object> selIter = selection.iterator(); Collection<RegistryPageContributor> result = null; while (selIter.hasNext()) { Object selectionElement = selIter.next(); Collection<RegistryPageContributor> collection = getApplicableContributors(selectionElement); if (isMultiSelection) { Iterator<RegistryPageContributor> resIter = collection.iterator(); while (resIter.hasNext()) { RegistryPageContributor contrib = resIter.next(); if (!contrib.supportsMultipleSelection()) { resIter.remove(); } } } if (result == null) { result = new LinkedHashSet<>(collection); } else { result.retainAll(collection); } if (result.isEmpty()) { return Collections.emptyList(); } } return result; }
Source:
<code><START> public AdUser() { <END> mUserId = Guid.Empty; mGroups = new HashMap<String, AdGroups>(); }</code><technical_language>Please rename this entity - again, work AD ;)</technical_language>

Target:
public AdUser() { mUserId = Guid.Empty; mGroups = new HashMap<String, LdapGroup>(); }
Source:
<code>public static ConnectorPropertiesWrapperJPAEntity getFromObject( Object property) { Class<?> clazz = property.getClass(); ConnectorPropertiesWrapperJPAEntity wrapper = new ConnectorPropertiesWrapperJPAEntity(); List<ConnectorPropertyJPAEntity> propList = new ArrayList<ConnectorPropertyJPAEntity>(); wrapper.setProperties(propList); wrapper.setCollectionType(clazz.getName()); <START> Object[] arr; <END> if (clazz.isArray()) { Class<?> compClass = clazz.getComponentType(); if (compClass.isPrimitive()) { compClass = ClassUtils.primitiveToWrapper(compClass); int length = Array.getLength(property); Object wrapperArray = Array.newInstance(compClass, length); for (int i = 0; i < length; i++) { Array.set(wrapperArray, i, Array.get(property, i)); } arr = (Object[]) wrapperArray; } else { arr = (Object[]) property; } loopProperties(Arrays.asList(arr), propList); return wrapper; } if (Collection.class.isAssignableFrom(clazz)) { Collection<Object> coll = (Collection<Object>) property; loopProperties(coll, propList); return wrapper; } wrapper.setCollectionType(null); propList.add(ConnectorPropertyJPAEntity.getFromObject(property)); return wrapper; }</code><technical_language>move Object array if block</technical_language>

Target:
public static ConnectorPropertiesWrapperJPAEntity getFromObject( Object property) { Class<?> clazz = property.getClass(); ConnectorPropertiesWrapperJPAEntity wrapper = new ConnectorPropertiesWrapperJPAEntity(); List<ConnectorPropertyJPAEntity> propList = new ArrayList<ConnectorPropertyJPAEntity>(); wrapper.setProperties(propList); wrapper.setCollectionType(clazz.getName()); if (clazz.isArray()) { Object[] arr; Class<?> compClass = clazz.getComponentType(); if (compClass.isPrimitive()) { compClass = ClassUtils.primitiveToWrapper(compClass); int length = Array.getLength(property); Object wrapperArray = Array.newInstance(compClass, length); for (int i = 0; i < length; i++) { Array.set(wrapperArray, i, Array.get(property, i)); } arr = (Object[]) wrapperArray; } else { arr = (Object[]) property; } loopProperties(Arrays.asList(arr), propList); return wrapper; } else { if (Collection.class.isAssignableFrom(clazz)) { Collection<Object> coll = (Collection<Object>) property; loopProperties(coll, propList); return wrapper; } else { wrapper.setCollectionType(null); propList.add(ConnectorPropertyJPAEntity.getFromObject(property)); return wrapper; } } }
Source:
<code>public void setSerial(int port) { <START> this.localPort = port; <END> serial = EMULATOR_SERIAL_PREFIX + port; }</code><technical_language>I this line is incorrect removed</technical_language>

Target:
public void setSerial(int port) { serial = EMULATOR_SERIAL_PREFIX + port; }
Source:
<code>public void process(Element element, JCodeModel codeModel, EBeanHolder holder) throws JClassAlreadyExistsException { ExecutableElement executableElement = (ExecutableElement) element; UiThread annotation = element.getAnnotation(UiThread.class); Propagation propagation = annotation.propagation(); JMethod delegatingMethod = helper.overrideAnnotatedMethod(executableElement, holder); JBlock clonedBody = null; if (propagation == Propagation.REUSE) { clonedBody = cloneMethodBody(delegatingMethod); } JDefinedClass anonymousRunnableClass = helper.createDelegatingAnonymousRunnableClass(holder, delegatingMethod); { long delay = annotation.delay(); if (holder.handler == null) { JClass handlerClass = holder.classes().HANDLER; holder.handler = holder.generatedClass.field(JMod.PRIVATE, handlerClass, "handler_", JExpr._new(handlerClass)); } <START> if (propagation == Propagation.REUSE) { <END> addUIThreadCheck(delegatingMethod, clonedBody, codeModel); } if (delay == 0) { delegatingMethod.body().invoke(holder.handler, "post").arg(_new(anonymousRunnableClass)); } else { delegatingMethod.body().invoke(holder.handler, "postDelayed").arg(_new(anonymousRunnableClass)).arg(lit(delay)); } } }</code><technical_language>If delay is positive, thread reused code enqueued. Or, check in UiThreadValidator' ifPropagation.REUSEis adelay</technical_language>

Target:
public void process(Element element, JCodeModel codeModel, EBeanHolder holder) throws JClassAlreadyExistsException { ExecutableElement executableElement = (ExecutableElement) element; UiThread annotation = element.getAnnotation(UiThread.class); Propagation propagation = annotation.propagation(); JMethod delegatingMethod = helper.overrideAnnotatedMethod(executableElement, holder); JDefinedClass anonymousRunnableClass = helper.createDelegatingAnonymousRunnableClass(holder, delegatingMethod); { long delay = annotation.delay(); if (holder.handler == null) { JClass handlerClass = holder.classes().HANDLER; holder.handler = holder.generatedClass.field(JMod.PRIVATE, handlerClass, "handler_", JExpr._new(handlerClass)); } if (delay == 0) { if (propagation == Propagation.REUSE) { addUIThreadCheck(delegatingMethod, codeModel, holder); } delegatingMethod.body().invoke(holder.handler, "post").arg(_new(anonymousRunnableClass)); } else { delegatingMethod.body().invoke(holder.handler, "postDelayed").arg(_new(anonymousRunnableClass)).arg(lit(delay)); } } }
Source:
<code>private void reportSLOs(final long startingNanos, final int totalSizeBytes, final int eventCount, final EventPublishResult eventPublishResult, final String eventTypeName, final Client client) { if (eventPublishResult.getStatus() == EventPublishingStatus.SUBMITTED) { final long msSpent = TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - startingNanos); final String applicationName = client.getClientId(); LOG.info("[SLO] [publishing-latency] time={} size={} count={} eventTypeName={} app={}", msSpent, totalSizeBytes, eventCount, eventTypeName, applicationName); <START> nakadiKpiPublisher.publish(kpiBatchPublishedEventType, () -> new JSONObject() <END> .put("event_type", eventTypeName) .put("app", applicationName) .put("app_hashed", nakadiKpiPublisher.hash(applicationName)) .put("number_of_events", eventCount) .put("batch_size", totalSizeBytes)); } }</code><technical_language>I understand "msSpent" is needed for task, do need in future? event-type compatible add if needed</technical_language>

Target:
private void reportSLOs(final long startingNanos, final int totalSizeBytes, final int eventCount, final EventPublishResult eventPublishResult, final String eventTypeName, final Client client) { if (eventPublishResult.getStatus() == EventPublishingStatus.SUBMITTED) { final long msSpent = TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - startingNanos); final String applicationName = client.getClientId(); LOG.info("[SLO] [publishing-latency] time={} size={} count={} eventTypeName={} app={}", msSpent, totalSizeBytes, eventCount, eventTypeName, applicationName); nakadiKpiPublisher.publish(kpiBatchPublishedEventType, () -> new JSONObject() .put("event_type", eventTypeName) .put("app", applicationName) .put("app_hashed", nakadiKpiPublisher.hash(applicationName)) .put("number_of_events", eventCount) .put("ms_spent", msSpent) .put("batch_size", totalSizeBytes)); } }
Source:
<code>public void checkVertxHealthChecksBoosterProjectCommands() { consoles.executeCommandFromProjectExplorer( VERTX_HEALTH_CHECKS_BOOSTER, BUILD_GOAL, BUILD_COMMAND, BUILD_SUCCESS); consoles.executeCommandFromProjectExplorer( VERTX_HEALTH_CHECKS_BOOSTER, RUN_GOAL, RUN_COMMAND, "[INFO] INFO: Succeeded in deploying verticle"); <START> consoles.checkWebElementVisibilityAtPreviewPage(By.id("_vert_x_health_check_booster")); <END> consoles.closeProcessTabWithAskDialog(RUN_COMMAND); consoles.executeCommandFromProcessesArea( "dev-machine", DEBUG_GOAL, DEBUG_COMMAND, "[INFO] Listening for transport dt_socket at address: 5005"); consoles.closeProcessTabWithAskDialog(DEBUG_COMMAND); }</code><technical_language>clear "_vert_x_health_check_booster" is for. this point name id</technical_language>

Target:
public void checkVertxHealthChecksBoosterProjectCommands() { By textOnPreviewPage = By.id("_vert_x_health_check_booster"); consoles.executeCommandFromProjectExplorer( HEALTH_CHECKS_BOOSTER_PROJECT, BUILD_GOAL, BUILD_COMMAND, BUILD_SUCCESS); consoles.executeCommandFromProjectExplorer( HEALTH_CHECKS_BOOSTER_PROJECT, RUN_GOAL, RUN_COMMAND, "[INFO] INFO: Succeeded in deploying verticle"); consoles.checkWebElementVisibilityAtPreviewPage(textOnPreviewPage); consoles.closeProcessTabWithAskDialog(RUN_COMMAND); consoles.executeCommandFromProcessesArea( "dev-machine", DEBUG_GOAL, DEBUG_COMMAND, "[INFO] Listening for transport dt_socket at address: 5005"); consoles.closeProcessTabWithAskDialog(DEBUG_COMMAND); }
Source:
<code>public static Object[][] data() { return new Object[][] { <START> { "generic", new GenericContainer<>(new RemoteDockerImage(IMAGE_FUTURE)), true }, <END> { "anonymous generic", new GenericContainer(new RemoteDockerImage(IMAGE_FUTURE)) {}, true }, { "custom", new CustomContainer(), true }, { "anonymous custom", new CustomContainer() {}, true }, { "custom with containerIsCreated", new CustomContainerWithContainerIsCreated(), false }, }; }</code><technical_language>FTR if I remember correctly, I IMAGE_FUTURE avoid side effect in constructor, simple new GenericContainer<>(TINY_IMAGE)</technical_language>

Target:
public static Object[][] data() { return new Object[][] { { "generic", new GenericContainer<>(TINY_IMAGE), true }, { "anonymous generic", new GenericContainer(TINY_IMAGE) {}, true }, { "custom", new CustomContainer(), true }, { "anonymous custom", new CustomContainer() {}, true }, { "custom with containerIsCreated", new CustomContainerWithContainerIsCreated(), false }, }; }
Source:
<code><START> public int getHstsMaxAge() <END> { return hstsMaxAge; }</code><technical_language>Add min/max bean validation</technical_language>

Target:
public Duration getHstsMaxAge() { return hstsMaxAge; }
Source:
<code>protected boolean canDoAction() { if (getParameters().getParametersList().isEmpty()) { <START> return failCanDoAction(VdcBllMessages.ACTION_TYPE_FAILED_DISK_NOT_EXIST); <END> } return updateParameters(); }</code><technical_language>error message for this BllMessage is: "Cannot ${action} ${type}. disk exist". I - need a new message for "Cannot ${action} ${type}. disk moves requested."</technical_language>

Target:
protected boolean canDoAction() { if (getParameters().getParametersList().isEmpty()) { return failCanDoAction(VdcBllMessages.ACTION_TYPE_FAILED_NO_DISKS_SPECIFIED); } return updateParameters(); }
Source:
<code><START> public static Command<Set<Cookie>> getAllCookies() { <END> return new Command<>(domainName + ".getAllCookies", ImmutableMap.of(), map("cookies", new TypeToken<Set<Cookie>>() {}.getType())); }</code><technical_language>Selenium a org.openqa.selenium.Cookie. friction other parts of Selenium APIs if return extant Selenium types. I fine extend types more meaningful if necessary</technical_language>

Target:
public static Command<Cookies> getAllCookies() { return new Command<>(DOMAIN_NAME + ".getAllCookies", ImmutableMap.of(), map("cookies", Cookies.class)); }
Source:
<code>public static boolean initialize(Context context) { preferences = PreferenceManager.getDefaultSharedPreferences(context); <START> if (CrashReporter.isReportingEnabled()) { <END> reporter.initialize(context); Log.d(TAG, "INITIALIZED!"); return true; } Log.d(TAG, "INITIALIZATION FAILED! [ Report : " + isReportingEnabled() + "]"); return false; }</code><technical_language>~CrashReporter.~isReportingEnabled()</technical_language>

Target:
public static boolean initialize(Context context) { preferences = PreferenceManager.getDefaultSharedPreferences(context); if (isReportingEnabled()) { reporter.initialize(context); Log.d(TAG, "INITIALIZED!"); return true; } Log.d(TAG, "INITIALIZATION FAILED! [ Report : " + isReportingEnabled() + "]"); return false; }
Source:
<code>public boolean isDecorated() { return decorated; <START> } <END></code><technical_language>return !decorators.isEmpty()</technical_language>

Target:
public boolean isDecorated() { return !decorators.isEmpty(); }
Source:
<code>private static ITmfTimeAligned getReferenceView(TmfTimeViewAlignmentInfo alignmentInfo, TmfView blackListedView) { IWorkbenchWindow workbenchWindow = getWorkbenchWindow(alignmentInfo.getShell()); if (workbenchWindow == null || workbenchWindow.getActivePage() == null) { return null; } IWorkbenchPage page = workbenchWindow.getActivePage(); int lowestTimeAxisOffset = Integer.MAX_VALUE; ITmfTimeAligned referenceView = null; for (IViewReference ref : page.getViewReferences()) { IViewPart view = ref.getView(false); if (view != blackListedView && isTimeAlignedView(view)) { if (isCandidateForReferenceView((TmfView) view, alignmentInfo, lowestTimeAxisOffset)) { referenceView = (ITmfTimeAligned) view; <START> getClampedTimeAxisOffset(referenceView.getTimeViewAlignmentInfo()); <END> break; } } } return referenceView; }</code><technical_language>lowestTimeAxisOffset is above. a bug in findbugs</technical_language>

Target:
private static ITmfTimeAligned getReferenceView(TmfTimeViewAlignmentInfo alignmentInfo, TmfView blackListedView) { IWorkbenchWindow workbenchWindow = getWorkbenchWindow(alignmentInfo.getShell()); if (workbenchWindow == null || workbenchWindow.getActivePage() == null) { return null; } IWorkbenchPage page = workbenchWindow.getActivePage(); int lowestTimeAxisOffset = Integer.MAX_VALUE; ITmfTimeAligned referenceView = null; for (IViewReference ref : page.getViewReferences()) { IViewPart view = ref.getView(false); if (view != blackListedView && isTimeAlignedView(view)) { if (isCandidateForReferenceView((TmfView) view, alignmentInfo, lowestTimeAxisOffset)) { referenceView = (ITmfTimeAligned) view; lowestTimeAxisOffset = getClampedTimeAxisOffset(referenceView.getTimeViewAlignmentInfo()); } } } return referenceView; }
Source:
<code>public void makeSortable() { makeSortable(new Comparator<T>() { <START> SimpleStatusImageComparator imageComparator = new SimpleStatusImageComparator(); <END> @Override public int compare(T o1, T o2) { return imageComparator.compare(getImage(o1), getImage(o2)); } }); }</code><technical_language>Small thing, this be: private static final SimpleStatusImageComparator imageComparator = new SimpleStatusImageComparator(); in scope of NicActivateStatusColumn (outside this anonymous class declaration), comparators stateless a single instance. for existing TextColumnWithTooltip / LexoNumericComparator. do think</technical_language>

Target:
public void makeSortable() { makeSortable(new Comparator<T>() { private final SimpleStatusImageComparator imageComparator = new SimpleStatusImageComparator(); @Override public int compare(T o1, T o2) { return imageComparator.compare(getImage(o1), getImage(o2)); } }); }
Source:
<code>public ApplicationInfo createApplication(final String appId, final List<String> paths) throws InjectionException { final JavaConfigurationBuilder jcb = Tang.Factory.getTang().newConfigurationBuilder(); jcb.bindNamedParameter(ApplicationIdentifier.class, appId); jcb.bindNamedParameter(JarFilePath.class, paths.get(0)); final String groupId = ""; jcb.bindNamedParameter(GroupId.class, groupId); jcb.bindNamedParameter(PeriodicCheckpointPeriod.class, String.valueOf(checkpointPeriod)); final Injector injector = Tang.Factory.getTang().newInjector(jcb.build()); injector.bindVolatileInstance(MQTTResource.class, mqttSharedResource); injector.bindVolatileInstance(KafkaSharedResource.class, kafkaSharedResource); injector.bindVolatileInstance(NettySharedResource.class, nettySharedResource); injector.bindVolatileInstance(QueryInfoStore.class, planStore); final ApplicationInfo applicationInfo = injector.getInstance(ApplicationInfo.class); applicationMap.putIfAbsent(appId, applicationInfo); final Group group = injector.getInstance(Group.class); groupAllocationTableModifier.addEvent( new WritingEvent(WritingEvent.EventType.GROUP_ADD, new Tuple<>(applicationInfo, group))); <START> groupMap.putIfAbsent(groupId, group); <END> return applicationInfo; }</code><technical_language>I this processed in SingleWriterThread</technical_language>

Target:
public ApplicationInfo createApplication(final String appId, final List<String> paths) throws InjectionException { final JavaConfigurationBuilder jcb = Tang.Factory.getTang().newConfigurationBuilder(); jcb.bindNamedParameter(ApplicationIdentifier.class, appId); jcb.bindNamedParameter(JarFilePath.class, paths.get(0)); final String groupId = ""; jcb.bindNamedParameter(GroupId.class, groupId); jcb.bindNamedParameter(PeriodicCheckpointPeriod.class, String.valueOf(checkpointPeriod)); final Injector injector = Tang.Factory.getTang().newInjector(jcb.build()); injector.bindVolatileInstance(MQTTResource.class, mqttSharedResource); injector.bindVolatileInstance(KafkaSharedResource.class, kafkaSharedResource); injector.bindVolatileInstance(NettySharedResource.class, nettySharedResource); injector.bindVolatileInstance(QueryInfoStore.class, planStore); final ApplicationInfo applicationInfo = injector.getInstance(ApplicationInfo.class); applicationMap.putIfAbsent(appId, applicationInfo); final Group group = injector.getInstance(Group.class); groupAllocationTableModifier.addEvent( new WritingEvent(WritingEvent.EventType.GROUP_ADD, new Tuple<>(applicationInfo, group))); return applicationInfo; }
Source:
<code>private void checkoutCommit(RevCommit commit) throws IllegalStateException, IOException { try (RevWalk walk = new RevWalk(db)) { RevCommit head = walk.parseCommit(db.resolve(Constants.HEAD)); DirCacheCheckout dco = new DirCacheCheckout(db, head.getTree(), db.lockDirCache(), commit.getTree()); dco.setFailOnConflict(true); <START> dco.checkout(); <END> RefUpdate refUpdate = db.updateRef(Constants.HEAD, true); refUpdate.setNewObjectId(commit); refUpdate.setRefLogMessage("checkout: moving to " + head.getName(), false); refUpdate.forceUpdate(); } }</code><technical_language>(optional) In preimage walk is released --- a '}' de-indent do same. I guess I prefer wrote, though</technical_language>

Target:
private void checkoutCommit(RevCommit commit) throws IllegalStateException, IOException { RevCommit head; try (RevWalk walk = new RevWalk(db)) { head = walk.parseCommit(db.resolve(Constants.HEAD)); DirCacheCheckout dco = new DirCacheCheckout(db, head.getTree(), db.lockDirCache(), commit.getTree()); dco.setFailOnConflict(true); dco.checkout(); } RefUpdate refUpdate = db.updateRef(Constants.HEAD, true); refUpdate.setNewObjectId(commit); refUpdate.setRefLogMessage("checkout: moving to " + head.getName(), false); refUpdate.forceUpdate(); }
Source:
<code>public String getUserHomeLink() { <START> switch(activityLog.getRoleWithoutMasquerade()) { <END> case Const.ActivityLog.ROLE_STUDENT: return Url.addParamToUrl(Const.ActionURIs.STUDENT_HOME_PAGE, Const.ParamsNames.USER_ID, activityLog.getGoogleId()); case Const.ActivityLog.ROLE_INSTRUCTOR: return Url.addParamToUrl(Const.ActionURIs.INSTRUCTOR_HOME_PAGE, Const.ParamsNames.USER_ID, activityLog.getGoogleId()); default: return activityLog.getGoogleId(); } }</code><technical_language>Surprised CheckStyle catch this missing space switch (</technical_language>

Target:
public String getUserHomeLink() { switch (activityLog.getRoleWithoutMasquerade()) { case Const.ActivityLog.ROLE_STUDENT: return Url.addParamToUrl(Const.ActionURIs.STUDENT_HOME_PAGE, Const.ParamsNames.USER_ID, activityLog.getGoogleId()); case Const.ActivityLog.ROLE_INSTRUCTOR: return Url.addParamToUrl(Const.ActionURIs.INSTRUCTOR_HOME_PAGE, Const.ParamsNames.USER_ID, activityLog.getGoogleId()); default: return null; } }
Source:
<code>public static Request GroupCreateRequest(Id sender, String name, List<Id> nodes) { Objects.requireNonNull(sender); Objects.requireNonNull(name); Objects.requireNonNull(nodes); if(nodes.size() < 1) { <START> throw new IllegalArgumentException(); <END> } RequestBuilder builder = new RequestBuilder( MessageKind.GROUPCREATE, sender); builder.addField(Fields.NAME, name); builder.addField(Fields.NODES, nodes); return builder.request(); }</code><technical_language>Error message? checked else too</technical_language>

Target:
public static Request GroupCreateRequest(Id sender, String name, List<Id> nodes) { Objects.requireNonNull(sender); Objects.requireNonNull(name); Objects.requireNonNull(nodes); if(nodes.size() < 1) { throw new IllegalArgumentException("No nodes to add..."); } RequestBuilder builder = new RequestBuilder( MessageKind.GROUPCREATE, sender); builder.addField(Fields.NAME, name); builder.addField(Fields.NODES, nodes); return builder.request(); }
Source:
<code>public ActionBuilderModule<DocumentSimilarity> instantiate(Configuration config, Agent agent, String actionSetId) { String thresholdStr = config.get(EXPORT_DOCUMENTSSIMILARITY_THRESHOLD); Float similarityThreshold = null; if (thresholdStr != null && !WorkflowRuntimeParameters.UNDEFINED_NONEMPTY_VALUE.equals(thresholdStr)) { similarityThreshold = Float.valueOf(thresholdStr); log.warn("setting documents similarity exporter threshold to: " + similarityThreshold); <START> } <END> return new DocumentSimilarityActionBuilderModule(provideTrustLevelThreshold(config), similarityThreshold, agent, actionSetId); }</code><technical_language>WorkflowRuntimeParameters.getParamValue</technical_language>

Target:
public ActionBuilderModule<DocumentSimilarity> instantiate(Configuration config, Agent agent, String actionSetId) { Float similarityThreshold = null; String thresholdStr = WorkflowRuntimeParameters.getParamValue(EXPORT_DOCUMENTSSIMILARITY_THRESHOLD, config); if (thresholdStr != null) { similarityThreshold = Float.valueOf(thresholdStr); log.info("setting documents similarity exporter threshold to: " + similarityThreshold); } return new DocumentSimilarityActionBuilderModule(provideTrustLevelThreshold(config), similarityThreshold, agent, actionSetId); }
Source:
<code>RestRequestMetricsGroup(String requestType, String resource) { blobMetrics = new RestRequestMetrics(AmbryBlobStorageService.class, requestType + resource, metricRegistry); blobSSLMetrics = new RestRequestMetrics(AmbryBlobStorageService.class, requestType + resource + SSL_SUFFIX, metricRegistry); encryptedBlobMetrics = <START> new RestRequestMetrics(AmbryBlobStorageService.class, requestType + ENCRYPTED + resource, metricRegistry); <END> encryptedBlobSSLMetrics = new RestRequestMetrics(AmbryBlobStorageService.class, requestType + ENCRYPTED + resource + SSL_SUFFIX, metricRegistry); }</code><technical_language>Encrypted in middle of name, end? Depends tend search for metrics</technical_language>

Target:
RestRequestMetricsGroup(String requestType, String resource) { blobMetrics = new RestRequestMetrics(AmbryBlobStorageService.class, requestType + resource, metricRegistry); blobSslMetrics = new RestRequestMetrics(AmbryBlobStorageService.class, requestType + resource + SSL_SUFFIX, metricRegistry); blobEncryptedMetrics = new RestRequestMetrics(AmbryBlobStorageService.class, requestType + resource + ENCRYPTED, metricRegistry); blobSslEncryptedMetrics = new RestRequestMetrics(AmbryBlobStorageService.class, requestType + resource + SSL_SUFFIX + ENCRYPTED, metricRegistry); }
Source:
<code>public Object invoke(String mthdName, boolean srvKeepBinary, Object[] args) throws IgniteCheckedException, NoSuchMethodException { if (proxy instanceof PlatformService) return ((PlatformService)proxy).invokeMethod(mthdName, srvKeepBinary, args); else { assert proxy instanceof GridServiceProxy; if (!srvKeepBinary) args = PlatformUtils.unwrapBinariesInArray(args); Method mtd = getMethod(serviceClass, mthdName, args); try { return ((GridServiceProxy)proxy).invokeMethod(mtd, args); } catch (Throwable t) { <START> throw new IgniteCheckedException(t); <END> } } }</code><technical_language>IgniteUtils.cast(t)</technical_language>

Target:
public Object invoke(String mthdName, boolean srvKeepBinary, Object[] args) throws IgniteCheckedException, NoSuchMethodException { if (proxy instanceof PlatformService) return ((PlatformService)proxy).invokeMethod(mthdName, srvKeepBinary, args); else { assert proxy instanceof GridServiceProxy; if (!srvKeepBinary) args = PlatformUtils.unwrapBinariesInArray(args); Method mtd = getMethod(serviceClass, mthdName, args); try { return ((GridServiceProxy)proxy).invokeMethod(mtd, args); } catch (Throwable t) { throw IgniteUtils.cast(t); } } }
Source:
<code>public void handleMessage(Message msg) { switch (msg.what) { case MSG_MEASURE: { if (mCached != null) { mMainHandler.obtainMessage(0, mCached).sendToTarget(); break; } synchronized (mLock) { if (mBound) { removeMessages(MSG_DISCONNECT); sendMessage(obtainMessage(MSG_CONNECTED, mDefaultContainer)); } else { Intent service = new Intent().setComponent(DEFAULT_CONTAINER_COMPONENT); mContext.bindServiceAsUser(service, mDefContainerConn, Context.BIND_AUTO_CREATE, UserHandle.OWNER); } } break; } case MSG_CONNECTED: { final IMediaContainerService imcs = (IMediaContainerService) msg.obj; measureExactStorage(imcs); break; } case MSG_DISCONNECT: { synchronized (mLock) { if (mBound) { mBound = false; mContext.unbindService(mDefContainerConn); } } if <START> (null != getLooper()) { <END> getLooper().quit(); } break; } case MSG_COMPLETED: { mCached = (MeasurementDetails) msg.obj; mMainHandler.obtainMessage(0, mCached).sendToTarget(); break; } case MSG_INVALIDATE: { mCached = null; break; } } }</code><technical_language>looper null? handler is running this thread necessary check looper running exists first. "getLooper() != null" is more normal</technical_language>

Target:
public void handleMessage(Message msg) { switch (msg.what) { case MSG_MEASURE: { if (mCached != null) { mMainHandler.obtainMessage(0, mCached).sendToTarget(); break; } synchronized (mLock) { if (mBound) { removeMessages(MSG_DISCONNECT); sendMessage(obtainMessage(MSG_CONNECTED, mDefaultContainer)); } else { Intent service = new Intent().setComponent(DEFAULT_CONTAINER_COMPONENT); mContext.bindServiceAsUser(service, mDefContainerConn, Context.BIND_AUTO_CREATE, UserHandle.OWNER); } } break; } case MSG_CONNECTED: { final IMediaContainerService imcs = (IMediaContainerService) msg.obj; measureExactStorage(imcs); break; } case MSG_DISCONNECT: { synchronized (mLock) { if (mBound) { mBound = false; mContext.unbindService(mDefContainerConn); } } getLooper().quit(); break; } case MSG_COMPLETED: { mCached = (MeasurementDetails) msg.obj; mMainHandler.obtainMessage(0, mCached).sendToTarget(); break; } case MSG_INVALIDATE: { mCached = null; break; } } }
Source:
<code>public List<ConfigOpacDoctype> getAllDoctypes() { <START> return ConfigOpac.getAllDoctypes(); <END> }</code><technical_language>doctypes a selection of divisions rule set. get this: public List<SelectItem> getAllDoctypes() { return createProcessForm.getRuleset() .getStructuralElements(createProcessForm.getPriorityList()).entrySet() .stream().map(entry -> new SelectItem(entry.getKey(), entry.getValue())) .collect(Collectors.toList()); }</technical_language>

Target:
public List<SelectItem> getAllDoctypes() { return createProcessForm.getRuleset() .getStructuralElements(createProcessForm.getPriorityList()).entrySet() .stream().map(entry -> new SelectItem(entry.getKey(), entry.getValue())) .collect(Collectors.toList()); }
Source:
<code>protected void executeCommand() { <START> boolean isDomainUpdatePossible = !Guid.isNullOrEmpty(getVdsId()); <END> boolean isDomainUpdateRequired = isDomainUpdatePossible && doDomainsUseConnection(getConnection()); List<StorageDomain> updatedDomains = new ArrayList<>(); boolean hasConnectStorageSucceeded = false; if (isDomainUpdateRequired) { hasConnectStorageSucceeded = connectToStorage(); VDSReturnValue returnValueUpdatedStorageDomain = null; if (hasConnectStorageSucceeded) { changeStorageDomainStatusInTransaction(StorageDomainStatus.Locked); for (StorageDomain domain : domains) { returnValueUpdatedStorageDomain = getStatsForDomain(domain); if (returnValueUpdatedStorageDomain.getSucceeded()) { StorageDomain updatedStorageDomain = (StorageDomain) returnValueUpdatedStorageDomain.getReturnValue(); updatedDomains.add(updatedStorageDomain); } } if (!updatedDomains.isEmpty()) { updateStorageDomain(updatedDomains); } } } getStorageConnDao().update(getParameters().getStorageServerConnection()); if (isDomainUpdateRequired) { for (StorageDomain domain : domains) { for (StoragePoolIsoMap map : getStoragePoolIsoMap(domain)) { restoreStateAfterUpdate(map); } } if (hasConnectStorageSucceeded) { disconnectFromStorage(); } } setSucceeded(true); }</code><technical_language>remove this boolean, clear enough this inline</technical_language>

Target:
protected void executeCommand() { boolean isDomainUpdateRequired = !Guid.isNullOrEmpty(getVdsId()) && doDomainsUseConnection(getConnection()); List<StorageDomain> updatedDomains = new ArrayList<>(); boolean hasConnectStorageSucceeded = false; if (isDomainUpdateRequired) { hasConnectStorageSucceeded = connectToStorage(); VDSReturnValue returnValueUpdatedStorageDomain = null; if (hasConnectStorageSucceeded) { changeStorageDomainStatusInTransaction(StorageDomainStatus.Locked); for (StorageDomain domain : domains) { returnValueUpdatedStorageDomain = getStatsForDomain(domain); if (returnValueUpdatedStorageDomain.getSucceeded()) { StorageDomain updatedStorageDomain = (StorageDomain) returnValueUpdatedStorageDomain.getReturnValue(); updatedDomains.add(updatedStorageDomain); } } if (!updatedDomains.isEmpty()) { updateStorageDomain(updatedDomains); } } } getStorageConnDao().update(getParameters().getStorageServerConnection()); if (isDomainUpdateRequired) { for (StorageDomain domain : domains) { for (StoragePoolIsoMap map : getStoragePoolIsoMap(domain)) { restoreStateAfterUpdate(map); } } if (hasConnectStorageSucceeded) { disconnectFromStorage(); } } setSucceeded(true); }
Source:
<code>public void updateDiseaseOccurrenceIsValidatedValues(int diseaseGroupId, DateTime modelRunPrepDate) { List<DiseaseOccurrence> occurrences = diseaseService.getDiseaseOccurrencesInValidation(diseaseGroupId); for (DiseaseOccurrence occurrence : occurrences) { boolean validated = occurrenceHasBeenInReviewForMoreThanAWeek(occurrence, modelRunPrepDate); <START> if (occurrence.isValidated() != validated) { <END> occurrence.setValidated(validated); diseaseService.saveDiseaseOccurrence(occurrence); } } }</code><technical_language>a nullable field, null protection good in case: if (occurrence.isValidated() != null && occurrence.isValidated() != validated)</technical_language>

Target:
public void updateDiseaseOccurrenceIsValidatedValues(int diseaseGroupId, DateTime modelRunPrepDate) { List<DiseaseOccurrence> occurrences = diseaseService.getDiseaseOccurrencesInValidation(diseaseGroupId); for (DiseaseOccurrence occurrence : occurrences) { boolean validated = occurrenceHasBeenInReviewForMoreThanAWeek(occurrence, modelRunPrepDate); if (occurrence.isValidated() != null && occurrence.isValidated() != validated) { occurrence.setValidated(validated); diseaseService.saveDiseaseOccurrence(occurrence); } } }
Source:
<code>public DoubleSelectiveStreamReader( StreamDescriptor streamDescriptor, Optional<TupleDomainFilter> filter, boolean outputRequired, LocalMemoryContext systemMemoryContext) { requireNonNull(filter, "filter is null"); checkArgument(filter.isPresent() || outputRequired, "filter must be present if outputRequired is false"); this.streamDescriptor = requireNonNull(streamDescriptor, "streamDescriptor is null"); this.filter = filter.orElse(null); this.outputRequired = outputRequired; this.systemMemoryContext = requireNonNull(systemMemoryContext, "systemMemoryContext is null"); <START> nullsAllowed = this.filter == null || this.filter.testNull(); <END> }</code><technical_language>This logic needs updating handle non-deterministic filters. latest BooleanSelectiveStreamReader</technical_language>

Target:
public DoubleSelectiveStreamReader( StreamDescriptor streamDescriptor, Optional<TupleDomainFilter> filter, boolean outputRequired, LocalMemoryContext systemMemoryContext) { requireNonNull(filter, "filter is null"); checkArgument(filter.isPresent() || outputRequired, "filter must be present if outputRequired is false"); this.streamDescriptor = requireNonNull(streamDescriptor, "streamDescriptor is null"); this.filter = filter.orElse(null); this.outputRequired = outputRequired; this.systemMemoryContext = requireNonNull(systemMemoryContext, "systemMemoryContext is null"); nonDeterministicFilter = this.filter != null && !this.filter.isDeterministic(); nullsAllowed = this.filter == null || nonDeterministicFilter || this.filter.testNull(); }
Source:
<code><START> public boolean close() { <END> try { Files.copy(fxmlFile.toPath(), foriginalXmlFile.toPath(), REPLACE_EXISTING); } catch (IOException e) { Activator.logError(e.getMessage(), e); } fxmlFile.delete(); return super.close(); }</code><technical_language>This method called if press ESC key click [X] widget. closes dialog if pending changes model</technical_language>

Target:
public boolean close() { if(modifsInStandby()) { MessageDialog dialog = new MessageDialog(Display.getDefault().getActiveShell(), Messages.XmlFilePropertiesDialog_closingWindowDialogText, null, Messages.XmlFilePropertiesDialog_closingWindowDialogQuestion2, MessageDialog.QUESTION, new String[] { Messages.XmlFilePropertiesDialog_closingWindowDialogYes, Messages.XmlFilePropertiesDialog_closingWindowDialogNo }, 0); if(dialog.open() == 1) { return false; } } try { Files.copy(fXmlFile.toPath(), fOriginalXmlFile.toPath(), REPLACE_EXISTING); } catch (IOException e) { Activator.logError(e.getMessage(), e); } fXmlFile.delete(); return super.close(); }
Source:
<code><START> public void initializeTests() <END> throws Exception { coordinatorProps = new Properties(); notificationSystem = new MockNotificationSystem(9); plaintextCluster = new MockCluster(notificationSystem, SystemTime.getInstance()); plaintextCluster.startServers(); }</code><technical_language>constructor. Test classes get instantiated for individual test</technical_language>

Target:
public void initializeTests() throws Exception { coordinatorProps = new Properties(); notificationSystem = new MockNotificationSystem(9); plaintextCluster = new MockCluster(notificationSystem, false, SystemTime.getInstance()); plaintextCluster.startServers(); }
Source:
<code>public void vuapDefaultStateAutoplayIsMutredAfrerReplayIsNotMuted(Page page, String slot, String videoIframeSelector) { openPageWithVideoInLocalStorage(page, VuapVideos.PORVATA_VAST, VuapVideos.VAST_VIDEO); final AutoplayVuap vuap = new AutoplayVuap(driver, slot, videoIframeSelector); VuapAssertions.verifyVideoUnmuteAndMute(vuap); <START> VuapAssertions.verifyVideoEndedAndReplyButtonDisplayed(vuap, MAX_AUTOPLAY_MOVIE_DURATION); <END> vuap.replay(); vuap.clickOnSoundButton(); VuapAssertions.verifyVideoUnmuteAndMute(vuap); }</code><technical_language>split this assertion int two</technical_language>

Target:
public void vuapDefaultStateAutoplayIsMutredAfrerReplayIsNotMuted(Page page, String slot, String videoIframeSelector) { openPageWithVideoInLocalStorage(page); final AutoplayVuap vuap = new AutoplayVuap(driver, slot, videoIframeSelector); VuapAssertions.verifyVideoUnmuteAndMute(vuap); VuapAssertions.verifyReplyButtonDisplayedAfterVideoEnds(vuap, MAX_AUTOPLAY_MOVIE_DURATION); vuap.replay(); vuap.mute(); VuapAssertions.verifyVideoUnmuteAndMute(vuap); }
Source:
<code>public String getIssuerName() { <START> issuerName = StringUtils.isEmpty(x509Cert.getIssuerDN().getName()) ? EMPTY_FIELD <END> : x509Cert.getIssuerDN().getName(); return issuerName; }</code><technical_language>Do in CertificateManagerServiceImpl . certificate class for sending to/from information View</technical_language>

Target:
public String getIssuerName() { return issuerName; }