auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code><START> public List<String> getCaFilePaths() { <END> List<String> caFilePaths = new ArrayList<>(); if(hasMultipleCaCerts()) { caFilePaths = Arrays.asList(caFilePath.split(CA_CERT_FILE_DELIMITER)); } else { caFilePaths.add(caFilePath); } return caFilePaths; }</code><technical_language>getCaFilePath() behave multiple CAs defined in config file</technical_language>

Target:
public List<String> getCaFilePaths() { List<String> caFilePaths = new ArrayList<>(); if(StringUtils.isNotBlank(caFilePath)) { if(hasMultipleCaCerts()) { caFilePaths = Arrays.asList(caFilePath.split(CA_CERT_FILE_DELIMITER)); } else { caFilePaths.add(caFilePath); } } return caFilePaths; }
Source:
<code>public Object fromString(String param) { String[] elements = param.split(separator); Object array = Array.newInstance(rawType.getComponentType(), elements.length); try { for (int i = 0; i < elements.length; i++) { Array.set(array, i, stringParameterInjector.extractValue(elements[i])); } } catch (Exception e) { <START> throw new RuntimeException("Ooops"); <END> } return array; }</code><technical_language>Ooops forgot change this :D</technical_language>

Target:
public Object fromString(String param) { String[] elements = param.split(separator); Object array = Array.newInstance(rawType.getComponentType(), elements.length); try { for (int i = 0; i < elements.length; i++) { Array.set(array, i, stringParameterInjector.extractValue(elements[i])); } } catch (Exception e) { throw new RuntimeException(Messages.MESSAGES.errorCreatingArray(param)); } return array; }
Source:
<code>protected synchronized void doStart() { try { try (Admin admin = getPeerConnection().getAdmin()) { String version = admin.getClusterMetrics().getHBaseVersion(); LOG.info("Peer cluster version is for peer {}", version, ctx.getPeerId()); <START> if (Integer.parseInt(version.split("\\.")[0]) >= 3) { <END> setRegionServers(fetchSlavesAddresses()); fetchServersUseZk = false; fetchServersChore = new FetchServersChore(ctx.getServer(), this); ctx.getServer().getChoreService().scheduleChore(fetchServersChore); } else { fetchServersUseZk = true; } } catch (Throwable t) { fetchServersUseZk = true; LOG.warn("Peer try to fetch servers by admin failed. Using zk impl.", ctx.getPeerId(), t); } reloadZkWatcher(); notifyStarted(); } catch (IOException e) { notifyFailed(e); } }</code><technical_language>interesting add a config flag allowing 3+ versions optionally ZK</technical_language>

Target:
protected synchronized void doStart() { this.shortOperationTimeout = ctx.getLocalConfiguration().getInt( HBASE_RPC_SHORTOPERATION_TIMEOUT_KEY, DEFAULT_HBASE_RPC_SHORTOPERATION_TIMEOUT); try { if (ctx.getLocalConfiguration().getBoolean(FETCH_SERVERS_USE_ZK_CONF_KEY, false)) { fetchServersUseZk = true; } else { try { if (ReplicationUtils.isPeerClusterSupportReplicationOffload(getPeerConnection())) { fetchServersChore = new FetchServersChore(ctx.getServer(), this); ctx.getServer().getChoreService().scheduleChore(fetchServersChore); fetchServersUseZk = false; } else { fetchServersUseZk = true; } } catch (Throwable t) { fetchServersUseZk = true; LOG.warn("Peer try to fetch servers by admin failed. Using zk impl.", ctx.getPeerId(), t); } } reloadZkWatcher(); notifyStarted(); } catch (IOException e) { notifyFailed(e); } }
Source:
<code>public void deleteOldVersionPath(String path, List<String> zNodeIds, int numVersionsToLeave, Comparator<String> c) { if (StringUtils.isEmpty(path) || zNodeIds == null) { <START> LOG.warn("cannot cleanup empty path or empty list in ZK"); <END> } LOG.info("ids = " + zNodeIds); if (zNodeIds.size() > numVersionsToLeave) { Collections.sort(zNodeIds, c); LOG.info("sorted ids = " + zNodeIds); int i = 0; int size = zNodeIds.size(); LOG.info("starting cleaning of barrier versions. size=" + size + "; num to leave=" + numVersionsToLeave); for (String znodeId: zNodeIds) { i++; if (size - i < numVersionsToLeave) { break; } LOG.info(path + "/" + znodeId); zkClient.deleteRecursive(path + "/" + znodeId); } } }</code><technical_language>Right behavior return throw exception(may forgot)</technical_language>

Target:
void deleteOldVersionPath(String path, List<String> zNodeIds, int numVersionsToLeave, Comparator<String> c) { if (StringUtils.isEmpty(path) || zNodeIds == null) { LOG.warn("cannot cleanup empty path or empty list in ZK"); return; } if (zNodeIds.size() > numVersionsToLeave) { Collections.sort(zNodeIds, c); int size = zNodeIds.size(); List<String> zNodesToDelete = zNodeIds.subList(0, zNodeIds.size() - numVersionsToLeave); LOG.info("Starting cleanup of barrier version zkNodes. From size=" + size + " to size " + zNodesToDelete.size() + "; numberToLeave=" + numVersionsToLeave); for (String znodeId : zNodesToDelete) { String pathToDelete = path + "/" + znodeId; try { LOG.info("deleting " + pathToDelete); zkClient.deleteRecursive(pathToDelete); } catch (Exception e) { LOG.warn("delete of node " + pathToDelete + " failed.", e); } } } }
Source:
<code>public static HyperLogLog newInstance(int numberOfBuckets) { <START> Preconditions.checkArgument(numberOfBuckets <= 8192, "numberOfBuckets must be <= 8192, actual: %s", numberOfBuckets); <END> return new HyperLogLog(new SparseHll(indexBitLength(numberOfBuckets))); }</code><technical_language>Static import</technical_language>

Target:
public static HyperLogLog newInstance(int numberOfBuckets) { checkArgument(numberOfBuckets <= MAX_NUMBER_OF_BUCKETS, "numberOfBuckets must be <= %s, actual: %s", MAX_NUMBER_OF_BUCKETS, numberOfBuckets); return new HyperLogLog(new SparseHll(indexBitLength(numberOfBuckets))); }
Source:
<code>public void updateModel(final OperationContext context, ModelNode model, ListAttributeDefinition attributeDefinition, ModelNode listAttribute) throws OperationFailedException { <START> listAttribute.set(Collections.<ModelNode>emptyList()); <END> }</code><technical_language>listAttribute.setEmptyList() accomplish thing</technical_language>

Target:
void updateModel(final OperationContext context, ModelNode model, ListAttributeDefinition attributeDefinition, ModelNode listAttribute) throws OperationFailedException { listAttribute.setEmptyList(); }
Source:
<code>protected GroupRest createAndReturn(Context context) throws AuthorizeException, RepositoryMethodNotImplementedException { HttpServletRequest req = getRequestService().getCurrentRequest().getHttpServletRequest(); ObjectMapper mapper = new ObjectMapper(); GroupRest groupRest = null; try { groupRest = mapper.readValue(req.getInputStream(), GroupRest.class); } catch (IOException excIO) { <START> throw new UnprocessableEntityException("error parsing the body... maybe this is not the right error code"); <END> } Group group = null; try { group = gs.create(context); gs.setName(group, groupRest.getName()); gs.update(context, group); if (groupRest.getMetadata() != null) { for (MetadataEntryRest mer: groupRest.getMetadata()) { String[] metadatakey = mer.getKey().split("\\."); gs.addMetadata(context, group, metadatakey[0], metadatakey[1], metadatakey.length == 3 ? metadatakey[2] : null, mer.getLanguage(), mer.getValue()); } } } catch (SQLException excSQL) { throw new RuntimeException(excSQL.getMessage(), excSQL); } return converter.convert(group); }</code><technical_language>This error message is a bit odd. "error parsing body: " + excIO.getMessage()</technical_language>

Target:
protected GroupRest createAndReturn(Context context) throws AuthorizeException, RepositoryMethodNotImplementedException { HttpServletRequest req = getRequestService().getCurrentRequest().getHttpServletRequest(); ObjectMapper mapper = new ObjectMapper(); GroupRest groupRest = null; try { groupRest = mapper.readValue(req.getInputStream(), GroupRest.class); } catch (IOException excIO) { throw new UnprocessableEntityException("error parsing the body ..." + excIO.getMessage()); } Group group = null; try { group = gs.create(context); gs.setName(group, groupRest.getName()); gs.update(context, group); if (groupRest.getMetadata() != null) { for (MetadataEntryRest mer: groupRest.getMetadata()) { String[] metadatakey = mer.getKey().split("\\."); gs.addMetadata(context, group, metadatakey[0], metadatakey[1], metadatakey.length == 3 ? metadatakey[2] : null, mer.getLanguage(), mer.getValue()); } } } catch (SQLException excSQL) { throw new RuntimeException(excSQL.getMessage(), excSQL); } return converter.convert(group); }
Source:
<code><START> public Value get( String fieldName ) <END> { Integer fieldIndex = fieldLookup.get( fieldName ); if ( fieldIndex == null ) { throw new ValueNotFoundException( fieldName ); } else { return fields[fieldIndex]; } }</code><technical_language>This fix apply get( int ) well, I think, consistent</technical_language>

Target:
public Value get( String fieldName ) { Integer fieldIndex = fieldLookup.get( fieldName ); if ( fieldIndex == null ) { return null; } else { return fields[fieldIndex]; } }
Source:
<code>protected void loadConfig() throws IOException { if (this.config_location != null && !this.config_location.isEmpty()) { this.loadConfig(this.config_location); return; } final ArrayList<String> file_locations = new ArrayList<String>(); file_locations.add("opentsdb.conf"); if (System.getProperty("os.name").toUpperCase().contains("WINDOWS")) { file_locations.add("C:\\Program Files\\opentsdb\\opentsdb.conf"); file_locations.add("C:\\Program Files (x86)\\opentsdb\\opentsdb.conf"); } else { file_locations.add("/etc/opentsdb.conf"); file_locations.add("/etc/opentsdb/opentsdb.conf"); file_locations.add("/opt/opentsdb/opentsdb.conf"); } for (String file : file_locations) { try { FileInputStream file_stream = new FileInputStream(file); this.properties.clear(); this.properties.load(file_stream); } catch (Exception e) { <START> LOG.debug("Unable to find or load " + file); <END> continue; } LOG.info("Successfully loaded configuration file: " + file); this.config_location = file; return; } LOG.info("No configuration found, will use defaults"); }</code><technical_language>pass e a argument debug</technical_language>

Target:
protected void loadConfig() throws IOException { if (this.config_location != null && !this.config_location.isEmpty()) { this.loadConfig(this.config_location); return; } final ArrayList<String> file_locations = new ArrayList<String>(); file_locations.add("opentsdb.conf"); if (System.getProperty("os.name").toUpperCase().contains("WINDOWS")) { file_locations.add("C:\\Program Files\\opentsdb\\opentsdb.conf"); file_locations.add("C:\\Program Files (x86)\\opentsdb\\opentsdb.conf"); } else { file_locations.add("/etc/opentsdb.conf"); file_locations.add("/etc/opentsdb/opentsdb.conf"); file_locations.add("/opt/opentsdb/opentsdb.conf"); } for (String file : file_locations) { try { FileInputStream file_stream = new FileInputStream(file); Properties props = new Properties(); props.load(file_stream); this.loadHashMap(props); } catch (Exception e) { LOG.debug("Unable to find or load " + file, e); continue; } LOG.info("Successfully loaded configuration file: " + file); this.config_location = file; return; } LOG.info("No configuration found, will use defaults"); }
Source:
<code>public static SimpleFeatureType getSchema(Map<String, Object> json, CoordinateReferenceSystem crs, boolean ignoreGeometryProperties) { replaceGeometry(json, GeoJSONReader2::toGeometry); if (ignoreGeometryProperties) { replaceMapProperties(json, it -> { try { <START> GeoJSONReader2.toGeometry(it); <END> return null; } catch (Exception e) { return it; } }); } else { replaceMapProperties(json, it -> { try { return GeoJSONReader2.toGeometry(it); } catch (Exception e) { return it; } }); } Map<String, Class<?>> bindings = new HashMap<>(); String type = GeoJSONUtil.getString(json, GeoJSON.TYPE); switch (type) { case GeoJSON.FEATURE_COLLECTION: List<Object> features = GeoJSONUtil.getList(json, GeoJSON.FEATURES); for (Object feature : features) { addAttributes((Map<String, Object>) feature, bindings); } break; case GeoJSON.FEATURE: addAttributes(json, bindings); break; default: throw new IllegalArgumentException("Invalid type"); } if (bindings.isEmpty()) { return null; } SimpleFeatureTypeBuilder sftb = new SimpleFeatureTypeBuilder(); sftb.setName("FeatureType"); sftb.setNamespaceURI("<LINK_0>"); sftb.setDefaultGeometry(GeoJSONUtil.DEFAULT_GEOMETRY_ATTRIBUTE_NAME); sftb.setCRS(crs); for (Map.Entry<String, Class<?>> attribute : bindings.entrySet()) { String name = attribute.getKey(); Class<?> cl = attribute.getValue(); if (cl.isAssignableFrom(Geometry.class)) { sftb.add(name, cl, crs); } else { sftb.add(name, cl); } } return sftb.buildFeatureType(); }</code><technical_language>this read if geom handling check is try catch? Object geom = GeoJSONReader2.toGeometry(it); if (ignoreGeometryProperties) { return null; } return geom;</technical_language>

Target:
public static SimpleFeatureType getSchema(Map<String, Object> json, CoordinateReferenceSystem crs, boolean ignoreGeometryProperties) { replaceGeometry(json, GeoJSONReader2::toGeometry); replaceMapProperties(json, it -> { try { Geometry g = GeoJSONReader2.toGeometry(it); if (ignoreGeometryProperties) { return null; } return g; } catch (Exception e) { return it; } }); Map<String, Class<?>> bindings = new HashMap<>(); String type = GeoJSONUtil.getString(json, GeoJSON.TYPE); switch (type) { case GeoJSON.FEATURE_COLLECTION: List<Object> features = GeoJSONUtil.getList(json, GeoJSON.FEATURES); for (Object feature : features) { addAttributes((Map<String, Object>) feature, bindings); } break; case GeoJSON.FEATURE: addAttributes(json, bindings); break; default: throw new IllegalArgumentException("Invalid type"); } if (bindings.isEmpty()) { return null; } SimpleFeatureTypeBuilder sftb = new SimpleFeatureTypeBuilder(); sftb.setName("FeatureType"); sftb.setNamespaceURI("<LINK_0>"); sftb.setDefaultGeometry(GeoJSONUtil.DEFAULT_GEOMETRY_ATTRIBUTE_NAME); sftb.setCRS(crs); for (Map.Entry<String, Class<?>> attribute : bindings.entrySet()) { String name = attribute.getKey(); Class<?> cl = attribute.getValue(); if (cl.isAssignableFrom(Geometry.class)) { sftb.add(name, cl, crs); } else { sftb.add(name, cl); } } return sftb.buildFeatureType(); }
Source:
<code>public View onCreateView(LayoutInflater inflater, ViewGroup container, final Bundle savedInstanceState) { View rootView = inflater.inflate(R.layout.fragment_page, container, false); webView = (ObservableWebView) rootView.findViewById(R.id.page_web_view); initWebViewListeners(); <START> webView.getSettings().setBuiltInZoomControls(true); <END> webView.getSettings().setDisplayZoomControls(false); tocDrawer = (WikiDrawerLayout) rootView.findViewById(R.id.page_toc_drawer); tocDrawer.setDragEdgeWidth(getResources().getDimensionPixelSize(R.dimen.drawer_drag_margin)); tocButton = (FloatingActionButton) rootView.findViewById(R.id.floating_toc_button); tocButton.setOnClickListener(tocButtonOnClickListener); refreshView = (SwipeRefreshLayoutWithScroll) rootView .findViewById(R.id.page_refresh_container); int swipeOffset = getContentTopOffsetPx(getActivity()) + REFRESH_SPINNER_ADDITIONAL_OFFSET; refreshView.setProgressViewOffset(false, -swipeOffset, swipeOffset); refreshView.setScrollableChild(webView); refreshView.setOnRefreshListener(pageRefreshListener); errorView = (WikiErrorView)rootView.findViewById(R.id.page_error); return rootView; }</code><technical_language>I this leaked in Id7bfaef3f61a7cb4b578110110319eef514fe4ef</technical_language>

Target:
public View onCreateView(LayoutInflater inflater, ViewGroup container, final Bundle savedInstanceState) { View rootView = inflater.inflate(R.layout.fragment_page, container, false); webView = (ObservableWebView) rootView.findViewById(R.id.page_web_view); initWebViewListeners(); tocDrawer = (WikiDrawerLayout) rootView.findViewById(R.id.page_toc_drawer); tocDrawer.setDragEdgeWidth(getResources().getDimensionPixelSize(R.dimen.drawer_drag_margin)); tocButton = (FloatingActionButton) rootView.findViewById(R.id.floating_toc_button); tocButton.setOnClickListener(tocButtonOnClickListener); refreshView = (SwipeRefreshLayoutWithScroll) rootView .findViewById(R.id.page_refresh_container); int swipeOffset = getContentTopOffsetPx(getActivity()) + REFRESH_SPINNER_ADDITIONAL_OFFSET; refreshView.setProgressViewOffset(false, -swipeOffset, swipeOffset); refreshView.setScrollableChild(webView); refreshView.setOnRefreshListener(pageRefreshListener); errorView = (WikiErrorView)rootView.findViewById(R.id.page_error); return rootView; }
Source:
<code>public String apply(Location input) { <START> return new StringBuilder().append(input.getId()).append(':').append(input.getScope()).toString(); <END> }</code><technical_language>compiler do this for automatically</technical_language>

Target:
public String apply(Location input) { return input.getId() + ":" + input.getScope(); }
Source:
<code>public HTML(final Response res) { <START> super(RsWithType.make(res, RsWithType.TYPE_HTML)); <END> }</code><technical_language>@essobedo invoke this(res, StandardCharsets.UTF_8) here</technical_language>

Target:
public HTML(final Response res) { this(res, new Opt.Empty<Charset>()); }
Source:
<code>private void <START> onEditSnapshotSchedule() <END> { final GlusterVolumeSnapshotModel snapshotModel = (GlusterVolumeSnapshotModel) getWindow(); if (!snapshotModel.validate(false)) { return; } setConfirmWindow(null); scheduleSnapshot(snapshotModel, true); }</code><technical_language>onEditSnapshotScheduleInternal</technical_language>

Target:
public void onEditSnapshotSchedule() { final GlusterVolumeSnapshotModel snapshotModel = (GlusterVolumeSnapshotModel) getWindow(); if (snapshotModel.getRecurrence().getSelectedItem() == GlusterVolumeSnapshotScheduleRecurrence.UNKNOWN) { confirmDeleteVolumeSnapshotSchedule(); } else { onEditSnapshotScheduleInternal(); } }
Source:
<code>public final void clear() { ref.clear(); <START> } <END></code><technical_language>remove this call</technical_language>

Target:
public final void clear() { ref.clear(); manager.removeStallEntries(); }
Source:
<code>public Mono<HttpResponse> process(HttpPipelineCallContext context, HttpPipelineNextPolicy next) { final Flux<ByteBuffer> contents = context.getHttpRequest().body() == null ? Flux.just(getEmptyBuffer()) <START> : context.getHttpRequest().body(); <END> return credentials .getAuthorizationHeadersAsync( context.getHttpRequest().getUrl(), context.getHttpRequest().getHttpMethod().toString(), contents.defaultIfEmpty(getEmptyBuffer())) .flatMapMany(headers -> Flux.fromIterable(headers.entrySet())) .map(header -> context.getHttpRequest().setHeader(header.getKey(), header.getValue())) .last() .flatMap(request -> next.process()); }</code><technical_language>getBody()</technical_language>

Target:
public Mono<HttpResponse> process(HttpPipelineCallContext context, HttpPipelineNextPolicy next) { final Flux<ByteBuffer> contents = context.getHttpRequest().getBody() == null ? Flux.just(getEmptyBuffer()) : context.getHttpRequest().getBody(); return credentials .getAuthorizationHeadersAsync( context.getHttpRequest().getUrl(), context.getHttpRequest().getHttpMethod().toString(), contents.defaultIfEmpty(getEmptyBuffer())) .flatMapMany(headers -> Flux.fromIterable(headers.entrySet())) .map(header -> context.getHttpRequest().setHeader(header.getKey(), header.getValue())) .last() .flatMap(request -> next.process()); }
Source:
<code>public void setItemRegistry(ItemRegistry itemRegistry) { this.itemRegistry = itemRegistry; <START> LOGGER.debug("Item registry set for SiteWhere persistence."); <END> }</code><technical_language>decrease TRACE level</technical_language>

Target:
public void setItemRegistry(ItemRegistry itemRegistry) { this.itemRegistry = itemRegistry; LOGGER.trace("Item registry set for SiteWhere persistence."); }
Source:
<code>private void refreshHostStorage(Guid hostId) { getBackendCollection(VdcQueryType.GetDeviceList, new GetDeviceListQueryParameters(hostId, StorageType.ISCSI, Boolean.FALSE)); <START> } <END></code><technical_language>Worth checking if need this method all, in THIS patch. Regardless, Boolean.FALSE a primitive false</technical_language>

Target:
private void refreshHostStorage(Guid hostId) { getBackendCollection(VdcQueryType.GetDeviceList, new GetDeviceListQueryParameters(hostId, StorageType.ISCSI, false, null)); }
Source:
<code>public Optional<DecisionTable> getModelClass() { final DecisionTable dtable = new DecisionTable(); <START> return Optional.of(dtable); <END> }</code><technical_language>shorten return Optional.of(new DecisionTable());</technical_language>

Target:
public Optional<DecisionTable> getModelClass() { return Optional.of(new DecisionTable()); }
Source:
<code><START> private static boolean isInSetMethodsBlackList(final String methodName) { <END> return ("addAll".equals(methodName) || "containsAll".equals(methodName) || "iterator".equals(methodName) || "removeAll".equals(methodName) || "retainAll".equals(methodName) || "toArray".equals(methodName)); }</code><technical_language>isInCollectionMethodsBlackList isInSetMethodsBlackList I invoke isInCollectionMethodsBlackList isInSetMethodsBlackList</technical_language>

Target:
private static boolean isInSetMethodsBlackList(final String methodName) { return isInCollectionMethodsBlackList(methodName); }
Source:
<code>protected ResultQueueEntry<FlatRow> getNext() throws IOException { ResultQueueEntry<FlatRow> queueEntry; try { <START> queueEntry = resultQueue.take(); <END> } catch (InterruptedException e) { Thread.currentThread().interrupt(); throw new IOException("Interrupted while waiting for next result", e); } return queueEntry; }</code><technical_language>I a fan, recommend explicit timeout well</technical_language>

Target:
protected ResultQueueEntry<FlatRow> getNext() throws IOException { ResultQueueEntry<FlatRow> queueEntry; try { queueEntry = resultQueue.take(); if (queueEntry == null) { throw new IllegalStateException("Timed out awaiting next sync rows"); } } catch (InterruptedException e) { Thread.currentThread().interrupt(); throw new IOException("Interrupted while waiting for next result", e); } return queueEntry; }
Source:
<code>public void request(@NonNull Context context, @NonNull Site site, int age, <START> @NonNull final FeedClient.Callback cb) { <END> cb.success(Collections.singletonList(card)); }</code><technical_language>get access Site here, wasteful throw away, re-expose FeedCoordinatorBase. In other cards far, Site passed server request here. done for Random</technical_language>

Target:
public void request(@NonNull Context context, @NonNull Site site, int age, @NonNull final FeedClient.Callback cb) { cb.success(Collections.singletonList(getNewCard(site))); }
Source:
<code><START> public static void trace(String option, String message, Throwable throwable) { <END> String systemPrintableMessage = message; while (systemPrintableMessage.length() > 100) { String partial = systemPrintableMessage.substring(0, 100); systemPrintableMessage = systemPrintableMessage.substring(100); System.out.println(partial + "\\"); } System.out.println(systemPrintableMessage); if(fgDebugTrace != null) { fgDebugTrace.trace(option, message, throwable); } }</code><technical_language>This method behave DsfPlugin.debug() behave, else affect printouts. Therefore, I suggest moving line splitting stuff CommandCache, leaving this if (DEBUG) { sysout.println } tracing file</technical_language>

Target:
public static void trace(String option, String message, Throwable throwable) { if (DEBUG) { System.out.println(message); } if(fgDebugTrace != null) { fgDebugTrace.trace(option, message, throwable); } }
Source:
<code>private Content setCommonFields(Content content, LoveFilmDataRow source) { content.setActivelyPublished(true); content.setTitle(ITEM_NAME.valueFrom(source)); content.setImage(HEROSHOT_URL.valueFrom(source)); content.setYear(yearFrom(ORIGINAL_PUBLICATION_DATE.valueFrom(source))); content.setGenres(genresFrom(GENRE.valueFrom(source))); content.setPeople(peopleFrom(CONTRIBUTOR.valueFrom(source))); content.setLanguages(languagesFrom(LANGUAGE.valueFrom(source))); content.setCertificates(certificatesFrom(BBFC_RATING.valueFrom(source))); content.setMediaType(MediaType.VIDEO); content.setDescription(SYNOPSIS.valueFrom(source)); String asin = ASIN.valueFrom(source); <START> if (asin.length() > 0) { <END> content.addAliasUrl(AMAZON_ALIAS_URL_PREFIX + asin); content.addAlias(new Alias(ASIN_NAMESPACE, asin)); } String imdbId = IMDB_ID.valueFrom(source); if (imdbId.length() > 0) { content.addAliasUrl(IMDB_ALIAS_URL_PREFIX + imdbId); content.addAlias(new Alias(IMDB_NAMESPACE, imdbId)); } return content; }</code><technical_language>Strings.isNullOrEmpty() more robust. for imdbId</technical_language>

Target:
private Content setCommonFields(Content content, LoveFilmDataRow source) { content.setActivelyPublished(true); content.setTitle(ITEM_NAME.valueFrom(source)); content.setImage(HEROSHOT_URL.valueFrom(source)); content.setYear(yearFrom(ORIGINAL_PUBLICATION_DATE.valueFrom(source))); content.setGenres(genresFrom(GENRE.valueFrom(source))); content.setPeople(peopleFrom(CONTRIBUTOR.valueFrom(source))); content.setLanguages(languagesFrom(LANGUAGE.valueFrom(source))); content.setCertificates(certificatesFrom(BBFC_RATING.valueFrom(source))); content.setMediaType(MediaType.VIDEO); content.setDescription(SYNOPSIS.valueFrom(source)); String asin = ASIN.valueFrom(source); if (!Strings.isNullOrEmpty(asin)) { content.addAliasUrl(AMAZON_ALIAS_URL_PREFIX + asin); content.addAlias(new Alias(ASIN_NAMESPACE, asin)); } String imdbId = IMDB_ID.valueFrom(source); if (!Strings.isNullOrEmpty(imdbId)) { content.addAliasUrl(IMDB_ALIAS_URL_PREFIX + imdbId); content.addAlias(new Alias(IMDB_NAMESPACE, imdbId)); } return content; }
Source:
<code>public boolean savePassword(final Network network, final NicProfile nic, final VirtualMachineProfile vm) throws ResourceUnavailableException { if (!canHandle(network, null)) { return false; } final List<DomainRouterVO> routers = _routerDao.listByNetworkAndRole(network.getId(), Role.VIRTUAL_ROUTER); if (routers == null || routers.isEmpty()) { s_logger.debug("Can't find virtual router element in network " + network.getId()); return true; } final VirtualMachineProfile uservm = vm; final DataCenterVO dcVO = _dcDao.findById(network.getDataCenterId()); final NetworkTopology networkTopology = networkTopologyContext.retrieveNetworkTopology(dcVO); if (!network.isRedundant()) { return networkTopology.savePasswordToRouter(network, nic, uservm, routers.get(0)); } Boolean applied; for (final VirtualRouter router : routers) { if (router.getState() == State.Running && router.getRedundantState() == VirtualRouter.RedundantState.MASTER) { applied = networkTopology.savePasswordToRouter(network, nic, uservm, router); <START> if (!applied) { <END> return false; } return true; } } final String password = (String) uservm.getParameter(VirtualMachineProfile.Param.VmPassword); final String password_encrypted = DBEncryptionUtil.encrypt(password); final UserVmVO userVmVO = _userVmDao.findById(vm.getId()); _userVmDao.loadDetails(userVmVO); userVmVO.setDetail("password", password_encrypted); _userVmDao.saveDetails(userVmVO); userVmVO.setUpdateParameters(true); _userVmDao.update(userVmVO.getId(), userVmVO); return true; }</code><technical_language>line 781 line 794 more simple for (final VirtualRouter router : routers) { if (router.getState() == State.Running && (!router.getIsRedundantRouter() || router.getRedundantState() == RedundantState.MASTER)) { return networkTopology.savePasswordToRouter(network, nic, uservm, router); } }</technical_language>

Target:
public boolean savePassword(final Network network, final NicProfile nic, final VirtualMachineProfile vm) throws ResourceUnavailableException { if (!canHandle(network, null)) { return false; } final List<DomainRouterVO> routers = _routerDao.listByNetworkAndRole(network.getId(), Role.VIRTUAL_ROUTER); if (routers == null || routers.isEmpty()) { s_logger.debug("Can't find virtual router element in network " + network.getId()); return true; } final VirtualMachineProfile uservm = vm; final DataCenterVO dcVO = _dcDao.findById(network.getDataCenterId()); final NetworkTopology networkTopology = networkTopologyContext.retrieveNetworkTopology(dcVO); if (!network.isRedundant()) { return networkTopology.savePasswordToRouter(network, nic, uservm, routers.get(0)); } for (final VirtualRouter router : routers) { if (router.getState() == State.Running && router.getRedundantState() == VirtualRouter.RedundantState.MASTER) { return networkTopology.savePasswordToRouter(network, nic, uservm, router); } } return savePassword(uservm); }
Source:
<code>public void putTicket(ImageTicket ticket) { BasicHttpEntityEnclosingRequest request = new BasicHttpEntityEnclosingRequest( "PUT", TICKETS_URI + ticket.getId(), HttpVersion.HTTP_1_1); StringEntity entity = new StringEntity(new Gson().toJson(ticket.toDict()), StandardCharsets.UTF_8); request.setEntity(entity); request.setHeader("content-length", String.valueOf(entity.getContentLength())); DefaultBHttpClientConnection conn = getConnection(); HttpEntity response = executeRequest(request, conn); <START> finalize(response, conn); <END> }</code><technical_language>above, try resources</technical_language>

Target:
public void putTicket(ImageTicket ticket) { BasicHttpEntityEnclosingRequest request = new BasicHttpEntityEnclosingRequest( "PUT", TICKETS_URI + ticket.getId(), HttpVersion.HTTP_1_1); try (DefaultBHttpClientConnection conn = getConnection()) { StringEntity entity = new StringEntity( new ObjectMapper().writeValueAsString(ticket.toDict()), StandardCharsets.UTF_8); request.setEntity(entity); request.setHeader("content-length", String.valueOf(entity.getContentLength())); consumeResponse(executeRequest(request, conn)); } catch (IOException e) { throw new RuntimeException(e.getMessage(), e); } }
Source:
<code>public final static String formatNumber(long value, int base, boolean signed) { String s; switch (base) { case 2: s = "0b" + Long.toBinaryString(value); break; case 8: s = "0" + Long.toOctalString(value); break; case 16: s = "0x" + Long.toHexString(value); break; case 10: default: if (!signed && value < 0) { BigInteger bigInteger = BigInteger.valueOf(value); BigInteger powerOfTwo = (BigInteger.valueOf(Long.MAX_VALUE)).add(BigInteger.valueOf(1)); powerOfTwo = powerOfTwo.multiply(BigInteger.valueOf(2)); <START> bigInteger = bigInteger.add(powerOfTwo); <END> s = bigInteger.toString(); } else { s = Long.toString(value); } break; } return s; }</code><technical_language>simplified bigInteger.add(BigInteger.valueOf(1).shiftLeft(64))</technical_language>

Target:
public final static String formatNumber(long value, int base, boolean signed) { String s; switch (base) { case 2: s = "0b" + Long.toBinaryString(value); break; case 8: s = "0" + Long.toOctalString(value); break; case 16: s = "0x" + Long.toHexString(value); break; case 10: default: if (!signed && value < 0) { BigInteger bigInteger = BigInteger.valueOf(value); bigInteger = bigInteger.add(BigInteger.valueOf(1).shiftLeft(64)); s = bigInteger.toString(); } else { s = Long.toString(value); } break; } return s; }
Source:
<code>protected void executeCommand() { String connectionId = getConnection().getid(); getStorageServerConnectionDao().remove(connectionId); <START> if (!Guid.isNullOrEmpty(getParameters().getVdsId())) { <END> disconnectStorage(); } setSucceeded(true); }</code><technical_language>Please consider add a log (maybe warning) someting : "connection " + connectionId + " removed DB, hosts disconnected" worth some scrubbing time</technical_language>

Target:
protected void executeCommand() { String connectionId = getConnection().getid(); getStorageServerConnectionDao().remove(connectionId); log.debugFormat("Removing connection %s from database ", connectionId); if (Guid.isNullOrEmpty(getParameters().getVdsId())) { log.debug("No vdsId passed - hosts will not be disconnected."); } else { disconnectStorage(); } setSucceeded(true); }
Source:
<code>public void onException(final Exception ex) { LOG.warn("Exception occurred while streaming", ex); if (!headersSent) { headersSent = true; try { <START> if (ex instanceof NakadiException) { <END> writeProblemResponse(Response.Status.BAD_REQUEST, ((NakadiException) ex).getProblemMessage()); } else { writeProblemResponse(Response.Status.SERVICE_UNAVAILABLE, "Failed to continue streaming"); } } catch (final IOException e) { LOG.error("Failed to write exception to response", e); } } else { LOG.warn("Exception found while streaming, but no data could be provided to client", ex); } }</code><technical_language>correct return 400 for Nakadi Exception 503 for else. I in reality more options</technical_language>

Target:
public void onException(final Exception ex) { LOG.warn("Exception occurred while streaming", ex); if (!headersSent) { headersSent = true; try { if (ex instanceof NakadiException) { writeProblemResponse(((NakadiException) ex).asProblem()); } else { writeProblemResponse(Problem.valueOf(Response.Status.SERVICE_UNAVAILABLE, "Failed to continue streaming")); } } catch (final IOException e) { LOG.error("Failed to write exception to response", e); } } else { LOG.warn("Exception found while streaming, but no data could be provided to client", ex); } }
Source:
<code>public CSCMatrix horzcatSparse(final List<Vector> vectors) { <START> List<breeze.linalg.SparseVector<Double>> breezeVecList = Lists.transform(vectors, <END> new Function<Vector, breeze.linalg.SparseVector<Double>>() { public breeze.linalg.SparseVector<Double> apply(final Vector vector) { return ((SparseVector) vector).getBreezeVector(); } }); return new CSCMatrix( breeze.linalg.SparseVector.horzcat(JavaConversions.asScalaBuffer(breezeVecList), ZERO, TAG)); }</code><technical_language>Please add final</technical_language>

Target:
public CSCMatrix horzcatSparse(final List<Vector> vectors) { final List<breeze.linalg.SparseVector<Double>> breezeVecList = Lists.transform(vectors, new Function<Vector, breeze.linalg.SparseVector<Double>>() { public breeze.linalg.SparseVector<Double> apply(final Vector vector) { return ((SparseVector) vector).getBreezeVector(); } }); return new CSCMatrix( breeze.linalg.SparseVector.horzcat(JavaConversions.asScalaBuffer(breezeVecList), ZERO, TAG)); }
Source:
<code>private void readRenameModeFromPreferences() { IPreferenceStore store = getPreferenceStore(); String renameMode = <START> store.getString(IWorkbenchPreferenceConstants.RESOURCE_RENAME_MODE_DIALOG); <END> if (renameMode == null || renameMode.isEmpty()) { renameMode = getRenameModePreference(); } renameModeInline = !IWorkbenchPreferenceConstants.RESOURCE_RENAME_MODE_DIALOG.equals(renameMode); }</code><technical_language>this is a value, a key :-)</technical_language>

Target:
private void readRenameModeFromPreferences() { IPreferenceStore store = getPreferenceStore(); String renameMode = store.getString(IWorkbenchPreferenceConstants.RESOURCE_RENAME_MODE); renameModeInline = !IWorkbenchPreferenceConstants.RESOURCE_RENAME_MODE_DIALOG.equals(renameMode); }
Source:
<code>private void validateSetOptimizeQueriesOption(boolean optimizeQueries) { if (!setCacheDeserializedValuesExplicitlyInvoked) { return; } <START> if (cacheDeserializedValues == CacheDeserializedValues.INDEX_ONLY) { <END> throw new ConfigurationException("Deprecated option 'optimize-queries' is being set" + ", but 'cacheDeserializedValues' was set to INDEX_ONLY. " + "These are conflicting options. Please remove the `optimize-queries'"); } else if (cacheDeserializedValues == CacheDeserializedValues.NEVER && optimizeQueries) { throw new ConfigurationException("Deprecated option 'optimize-queries' is being set to true, " + "but 'cacheDeserializedValues' is set to NEVER. " + "These are conflicting options. Please remove the `optimize-queries'"); } else if (cacheDeserializedValues == CacheDeserializedValues.ALWAYS && !optimizeQueries) { throw new ConfigurationException("Deprecated option 'optimize-queries' is being set to false, " + "but 'cacheDeserializedValues' is set to ALWAYS. " + "These are conflicting options. Please remove the `optimize-queries'"); } }</code><technical_language>Is this want? INDEX_ONLY is default, calling setCacheDeserializedValues(INDEX_ONLY) significant suddenly trigger error</technical_language>

Target:
private void validateSetOptimizeQueriesOption(boolean optimizeQueries) { if (!setCacheDeserializedValuesExplicitlyInvoked) { return; } if (cacheDeserializedValues != CacheDeserializedValues.ALWAYS && optimizeQueries) { throw new ConfigurationException("Deprecated option 'optimize-queries' is being set to true, " + "but 'cacheDeserializedValues' was set to NEVER or INDEX_ONLY. " + "These are conflicting options. Please remove the `optimize-queries'"); } else if (cacheDeserializedValues == CacheDeserializedValues.ALWAYS && !optimizeQueries) { throw new ConfigurationException("Deprecated option 'optimize-queries' is being set to false, " + "but 'cacheDeserializedValues' was set to ALWAYS. " + "These are conflicting options. Please remove the `optimize-queries'"); } }
Source:
<code>public void playerInteract(PlayerInteractEvent event) { if (event.action == PlayerInteractEvent.Action.RIGHT_CLICK_BLOCK && event.entityPlayer.getHeldItem().itemID < Block.blocksList.length && Block.blocksList[event.entityPlayer.getHeldItem().itemID] != null) { int x = event.x + (new int[] {0, 0, 0, 0, -1, 1})[event.face]; int y = event.y - 1 + (new int[] {-1, 1, 0, 0, 0, 0})[event.face]; <START> int z = event.z + (new int[] {0, 0, -1, 1, 0, 0})[event.face];getClass(); <END> if (event.entityPlayer.worldObj.getBlockId(x, y, z) == ModBlocks.aludel.blockID) { event.setCanceled(true); } } }</code><technical_language>wondering random "getClass();"</technical_language>

Target:
public void playerInteract(PlayerInteractEvent event) { if (event.action == PlayerInteractEvent.Action.RIGHT_CLICK_BLOCK && event.entityPlayer.getHeldItem().itemID < Block.blocksList.length && Block.blocksList[event.entityPlayer.getHeldItem().itemID] != null) { int x = event.x + (new int[] {0, 0, 0, 0, -1, 1})[event.face]; int y = event.y - 1 + (new int[] {-1, 1, 0, 0, 0, 0})[event.face]; int z = event.z + (new int[] {0, 0, -1, 1, 0, 0})[event.face]; if (event.entityPlayer.worldObj.getBlockId(x, y, z) == ModBlocks.aludel.blockID) { event.setCanceled(true); } } }
Source:
<code>private boolean checkForChangedDerivedProducts(Pool pool, Pool existingPool, Map<String, Product> changedProducts) { boolean productsChanged = false; if (pool.getDerivedProduct() != null) { productsChanged = !pool.getDerivedProduct().getId() .equals(existingPool.getDerivedProduct().getId()); productsChanged |= (changedProducts != null && changedProducts.containsKey(pool.getDerivedProduct().getId())); } if (productsChanged) { if (pool.getDerivedProduct() != null) { existingPool.setDerivedProduct(pool.getDerivedProduct()); } } else { <START> if (pool.getDerivedProduct() == null && existingPool.getDerivedProduct() != null) { <END> existingPool.setDerivedProduct(null); productsChanged = true; } } return productsChanged; }</code><technical_language>This block belongs block starts setting productsChanged. if block changes based state longer flipping anymore</technical_language>

Target:
private boolean checkForChangedDerivedProducts(Pool pool, Pool existingPool, Map<String, Product> changedProducts) { boolean dpChanged = false; Product existingDerived = existingPool.getDerivedProduct(); Product incomingDerived = pool.getDerivedProduct(); if (incomingDerived != null) { dpChanged = (existingDerived == null) || (existingDerived.getId() != null && !existingDerived.getId().equals(incomingDerived.getId())) || (changedProducts != null && changedProducts.containsKey(existingDerived.getId())); } else { dpChanged = existingDerived != null; } if (dpChanged) { existingPool.setDerivedProduct(incomingDerived); } return dpChanged; }
Source:
<code>public void testGetStudies() { String url = "/v2/studies?species=mmusculus&assembly=grcm38&pageNumber=0&pageSize=2"; ResponseEntity<String> response = restTemplate.getForEntity(url, String.class); assertEquals(HttpStatus.OK, response.getStatusCode()); Configuration configuration = Configuration.defaultConfiguration() .jsonProvider(new JacksonJsonProvider()) .mappingProvider(new JacksonMappingProvider(objectMapper)) .addOptions(Option.SUPPRESS_EXCEPTIONS); List<VariantStudySummary> variantList = JsonPath.using(configuration).parse(response.getBody()) .read("$['_embedded']['variantStudySummaryList']", new TypeRef<List<VariantStudySummary>>() { }); <START> System.out.println(response.getBody()); <END> assertEquals("studyId1", variantList.get(0).getStudyId()); assertEquals("studyName1", variantList.get(0).getStudyName()); assertEquals("studyId2", variantList.get(1).getStudyId()); assertEquals("studyName2", variantList.get(1).getStudyName()); }</code><technical_language>suggestion</technical_language>

Target:
public void testGetStudies() { String url = "/v2/studies?species=mmusculus&assembly=grcm38&pageNumber=0&pageSize=2"; ResponseEntity<String> response = restTemplate.getForEntity(url, String.class); assertEquals(HttpStatus.OK, response.getStatusCode()); Configuration configuration = Configuration.defaultConfiguration() .jsonProvider(new JacksonJsonProvider()) .mappingProvider(new JacksonMappingProvider(objectMapper)) .addOptions(Option.SUPPRESS_EXCEPTIONS); List<VariantStudySummary> variantList = JsonPath.using(configuration).parse(response.getBody()) .read("$['_embedded']['variantStudySummaryList']", new TypeRef<List<VariantStudySummary>>() { }); assertEquals("studyId1", variantList.get(0).getStudyId()); assertEquals("studyName1", variantList.get(0).getStudyName()); assertEquals("studyId2", variantList.get(1).getStudyId()); assertEquals("studyName2", variantList.get(1).getStudyName()); }
Source:
<code>public EventTypeControllerTest() throws Exception { final EventTypeController controller = new EventTypeController(eventTypeRepository, topicRepository, partitionResolver, enrichment, featureToggleService); final MappingJackson2HttpMessageConverter jackson2HttpMessageConverter = new MappingJackson2HttpMessageConverter(objectMapper); mockMvc = standaloneSetup(controller).setMessageConverters(new StringHttpMessageConverter(), jackson2HttpMessageConverter).build(); <START> doReturn(false).when(featureToggleService).isFeatureEnabled(any(FeatureToggleService.Feature.class)); <END> }</code><technical_language>In general write any()</technical_language>

Target:
public EventTypeControllerTest() throws Exception { final EventTypeController controller = new EventTypeController(eventTypeRepository, topicRepository, partitionResolver, enrichment, featureToggleService, uuid); Mockito.doReturn(randomUUID).when(uuid).randomUUID(); final MappingJackson2HttpMessageConverter jackson2HttpMessageConverter = new MappingJackson2HttpMessageConverter(objectMapper); mockMvc = standaloneSetup(controller).setMessageConverters(new StringHttpMessageConverter(), jackson2HttpMessageConverter).build(); doReturn(false).when(featureToggleService).isFeatureEnabled(any()); }
Source:
<code>private void replicateLedgerFragment(LedgerHandle lh, final LedgerFragment ledgerFragment, <START> final Map<Integer, BookieSocketAddress> targetBookieAddresses) <END> throws InterruptedException, BKException { CompletableFuture<Void> result = new CompletableFuture<>(); ResultCallBack resultCallBack = new ResultCallBack(result); SingleFragmentCallback cb = new SingleFragmentCallback( resultCallBack, lh, ledgerFragment.getFirstEntryId(), getReplacedBookiesMap(ledgerFragment, targetBookieAddresses)); Set<BookieSocketAddress> targetBookieSet = Sets.newHashSet(); targetBookieSet.addAll(targetBookieAddresses.values()); asyncRecoverLedgerFragment(lh, ledgerFragment, cb, targetBookieSet); try { SyncCallbackUtils.waitForResult(result); } catch (BKException err) { throw BKException.create(bkc.getReturnRc(err.getCode())); } }</code><technical_language>multiple bookies passed for a single fragment? A fragment is subdivision of a segment is stored a single bookie</technical_language>

Target:
private void replicateLedgerFragment(LedgerHandle lh, final LedgerFragment ledgerFragment, final Map<Integer, BookieSocketAddress> targetBookieAddresses) throws InterruptedException, BKException { CompletableFuture<Void> result = new CompletableFuture<>(); ResultCallBack resultCallBack = new ResultCallBack(result); SingleFragmentCallback cb = new SingleFragmentCallback( resultCallBack, lh, ledgerFragment.getFirstEntryId(), getReplacementBookiesMap(ledgerFragment, targetBookieAddresses)); Set<BookieSocketAddress> targetBookieSet = Sets.newHashSet(); targetBookieSet.addAll(targetBookieAddresses.values()); asyncRecoverLedgerFragment(lh, ledgerFragment, cb, targetBookieSet); try { SyncCallbackUtils.waitForResult(result); } catch (BKException err) { throw BKException.create(bkc.getReturnRc(err.getCode())); } }
Source:
<code>public void filter(ContainerRequestContext requestContext) throws IOException { String operation = requestContext.getMethod() + " /" + requestContext.getUriInfo().getPath(); String traceId = requestContext.getHeaderString(TraceHttpHeaders.TRACE_ID); String spanId = requestContext.getHeaderString(TraceHttpHeaders.SPAN_ID); if (Strings.isNullOrEmpty(traceId)) { Tracer.initTrace(Optional.<Boolean>absent(), Tracers.randomId()); Tracer.startSpan(operation); } else { Tracer.initTrace(hasSampledHeader(requestContext), traceId); if (spanId == null) { Tracer.startSpan(operation); } else { Tracer.startSpan(operation, spanId); } } <START> Tracer.addEvent(Events.serverReceive(System.currentTimeMillis(), TimeUnit.MILLISECONDS)); <END> MDC.put(MDC_KEY, Tracer.getTraceId()); requestContext.setProperty("com.palantir.remoting1.traceId", Tracer.getTraceId()); }</code><technical_language>tempted a span start records enum general event annotation thing. Then, span closes fire server call span Zipkin converter generate event details span. a minimum, this fetch time again</technical_language>

Target:
public void filter(ContainerRequestContext requestContext) throws IOException { String operation = requestContext.getMethod() + " /" + requestContext.getUriInfo().getPath(); String traceId = requestContext.getHeaderString(TraceHttpHeaders.TRACE_ID); String spanId = requestContext.getHeaderString(TraceHttpHeaders.SPAN_ID); if (Strings.isNullOrEmpty(traceId)) { Tracer.initTrace(Optional.<Boolean>absent(), Tracers.randomId()); Tracer.startSpan(operation, SpanType.SERVER_INCOMING); } else { Tracer.initTrace(hasSampledHeader(requestContext), traceId); if (spanId == null) { Tracer.startSpan(operation, SpanType.SERVER_INCOMING); } else { Tracer.startSpan(operation, spanId, SpanType.SERVER_INCOMING); } } MDC.put(MDC_KEY, Tracer.getTraceId()); requestContext.setProperty("com.palantir.remoting1.traceId", Tracer.getTraceId()); }
Source:
<code>public void shouldCompactIfVacuumWasNeverPerformedAndTheDbTimeIsLessThanCompactInterval() throws Exception { SqlConnection sqlConnection = setUpSqlConnection(null, COMPACT_INTERVAL_MILLIS / SMALL_POSITIVE_FACTOR); <START> assertThatVacuumWasPerformed(sqlConnection, true); <END> }</code><technical_language>(nit) strange assertThatVacuumWasPerformed takes a boolean assertThatVacuumWasNotPerformed not. rename in cases, overload need pass in a boolean</technical_language>

Target:
public void shouldCompactIfVacuumWasNeverPerformedAndTheDbTimeIsLessThanCompactInterval() throws Exception { SqlConnection sqlConnection = setUpSqlConnection(null, COMPACT_INTERVAL_MILLIS / SMALL_POSITIVE_FACTOR); assertThatVacuumWasPerformed(sqlConnection); }
Source:
<code><START> public <TInput> void accept(GenericFeedRangeVisitor<TInput> visitor, TInput input) { <END> if (visitor == null) { throw new NullPointerException("visitor"); } visitor.visit(this, input); }</code><technical_language>do need version async sync accept</technical_language>

Target:
public <TInput> void accept(GenericFeedRangeVisitor<TInput> visitor, TInput input) { checkNotNull(visitor, "Argument 'visitor' must not be null"); visitor.visit(this, input); }