auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void cleanupOrphanedTransactions() { log.tracef("About to cleanup orphaned transactions older than %ld ms", configuration.transaction().completedTxTimeout()); long beginning = System.nanoTime(); Set<GlobalTransaction> toKill = new HashSet<GlobalTransaction>(); for(Map.Entry<GlobalTransaction, RemoteTransaction> e : remoteTransactions.entrySet()) { GlobalTransaction gt = e.getKey(); RemoteTransaction remoteTx = e.getValue(); log.tracef("Checking transaction %s", gt); long txTimestamp = beginning - remoteTx.getCreationTime(); if (txTimestamp >= TimeUnit.MILLISECONDS.toNanos(configuration.transaction().completedTxTimeout())) { toKill.add(gt); } } for(Map.Entry<GlobalTransaction, LocalTransaction> e : globalToLocalTransactions.entrySet()) { GlobalTransaction gt = e.getKey(); LocalTransaction localTx = e.getValue(); log.tracef("Checking transaction %s", gt); long txTimestamp = beginning - localTx.getCreationTime(); if (txTimestamp >= TimeUnit.MILLISECONDS.toNanos(configuration.transaction().completedTxTimeout())) { toKill.add(gt); } } <START> rollbackStaleTransactions(toKill); <END> }</code><technical_language>log stale transactions a warning</technical_language>

Target:
public void cleanupOrphanedTransactions() { log.tracef("About to cleanup orphaned transactions older than %ld ms", configuration.transaction().completedTxTimeout()); long beginning = timeService.time(); long orphanedTxTimestamp = beginning - TimeUnit.MILLISECONDS.toNanos(configuration.transaction().completedTxTimeout()); Set<GlobalTransaction> toKill = new HashSet<GlobalTransaction>(); for(Map.Entry<GlobalTransaction, RemoteTransaction> e : remoteTransactions.entrySet()) { GlobalTransaction gt = e.getKey(); RemoteTransaction remoteTx = e.getValue(); if(remoteTx != null) { log.tracef("Checking transaction %s", gt); if (remoteTx.getCreationTime() < orphanedTxTimestamp) { long duration = timeService.timeDuration(remoteTx.getCreationTime(), beginning, TimeUnit.MILLISECONDS); log.warnf("Found and rolling back orphaned transaction: %s. Age %d millis", gt, duration); toKill.add(gt); } } } rollbackStaleTransactions(toKill); }
Source:
<code>public void onBackPressed() { <START> if (!shouldShowDualPane) { <END> if (isDrawerOpen()) { toggleNavigationDrawer(false); } } if (navigationDrawer.fetchDefaultPosition() != navigationDrawer.getCheckedItemPosition()) { navigationDrawer.checkDefaultItemPosition(); return; } super.onBackPressed(); }</code><technical_language>single check out of this if (!shouldShowDualPane && isDrawerOpen()) interested in single outcome</technical_language>

Target:
public void onBackPressed() { if (!shouldShowDualPane && isDrawerOpen()) { toggleNavigationDrawer(false); } if (navigationDrawer.fetchDefaultPosition() != navigationDrawer.getCheckedItemPosition()) { navigationDrawer.checkDefaultItemPosition(); return; } super.onBackPressed(); }
Source:
<code>@Test public void testPushAggregateThroughtOuterJoin8() { final HepProgram preProgram = new HepProgramBuilder() .addRuleInstance(AggregateProjectMergeRule.INSTANCE) .build(); final HepProgram program = new HepProgramBuilder() .addRuleInstance(AggregateJoinTransposeRule.EXTENDED) .build(); final String sql = "select d.name\n" + "from (select * from sales.emp where empno = 10) as e\n" <START> + "join sales.dept as d on e.job = d.name\n" <END> + "and e.deptno + e.empno = d.deptno + 5\n" + "group by d.name"; checkPlanning(tester, preProgram, new HepPlanner(program), sql); }</code><technical_language>This outer</technical_language>

Target:
@Test public void testPushAggregateThroughtOuterJoin8() { final HepProgram preProgram = new HepProgramBuilder() .addRuleInstance(AggregateProjectMergeRule.INSTANCE) .build(); final HepProgram program = new HepProgramBuilder() .addRuleInstance(AggregateJoinTransposeRule.EXTENDED) .build(); final String sql = "select d.name\n" + "from (select * from sales.emp where empno = 10) as e\n" + "left outer join sales.dept as d on e.job = d.name\n" + "and e.deptno + e.empno = d.deptno + 5\n" + "group by d.name"; checkPlanning(tester, preProgram, new HepPlanner(program), sql); }
Source:
<code>protected Boolean authSAS(@org.jclouds.location.Provider Supplier<Credentials> creds) { String credential = creds.get().credential; <START> return credential.matches("sv.*se.*sig.*"); <END> }</code><technical_language>So, a valid SAS credential? svsesig I need some unit tests clear valid SAS formats, a stronger regular expression. Do SAS credential delimiters separating field? enforce start sv?, etc</technical_language>

Target:
protected boolean authSAS(@org.jclouds.location.Provider Supplier<Credentials> creds) { String credential = creds.get().credential; String formattedCredential = credential.startsWith("?") ? credential.substring(1) : credential; List<String> required = ImmutableList.of("sv", "se", "sig", "sp"); try { Map<String, String> tokens = Splitter.on('&').withKeyValueSeparator('=').split(formattedCredential); return all(required, in(tokens.keySet())); } catch (Exception ex) { return false; } }
Source:
<code>private String getName(URL url) { String name = url.getPath(); if (name != null) { <START> { <END> final int j = name.lastIndexOf(File.pathSeparatorChar); if (j >= 0) name = name.substring(j + 1); } if (File.pathSeparatorChar != '/') { final int j = name.lastIndexOf('/'); if (j >= 0) name = name.substring(j + 1); } } return name; }</code><technical_language>extra {</technical_language>

Target:
private String getName(URL url) { String name = url.getPath(); if (name != null) { final int i = name.lastIndexOf(File.pathSeparatorChar); if (i >= 0) name = name.substring(i + 1); if (File.pathSeparatorChar != '/') { final int j = name.lastIndexOf('/'); if (j >= 0) name = name.substring(j + 1); } } return name; }
Source:
<code>public void checkSourceDirectory(String path, String expected) throws IOException, XmlPullParserException { SpoonPom pomModel = new SpoonPom(path, null, MavenLauncher.SOURCE_TYPE.APP_SOURCE, new StandardEnvironment()); SpoonPom childModel = pomModel.getModules().get(0); <START> assertEquals(expected, childModel.getSourceDirectories().get(0).getAbsolutePath(), expected); <END> }</code><technical_language>expected argument, remove one, message, relevant</technical_language>

Target:
public void checkSourceDirectory(String path, String expected) throws IOException, XmlPullParserException { SpoonPom pomModel = new SpoonPom(path, null, MavenLauncher.SOURCE_TYPE.APP_SOURCE, new StandardEnvironment()); SpoonPom childModel = pomModel.getModules().get(0); assertEquals(expected, childModel.getSourceDirectories().get(0).getAbsolutePath()); }
Source:
<code>public void testOwnerProject() { <START> grant(true, OWN, admin, "refs/*", 1); <END> ProjectControl uBlah = user(devs); ProjectControl uAdmin = user(devs, admin); assertFalse("not owner", uBlah.isOwner()); assertTrue("is owner", uAdmin.isOwner()); }</code><technical_language>I say, true for local false for inherited is obtuse. obtuse for a unit test. rename "local" "inherited" localRights inheritedRights, rename project name keys "local" "parent". do grant(local, ..) grant(parent, ...) pass in name key, in grant name key decide collection into</technical_language>

Target:
public void testOwnerProject() { grant(local, OWN, admin, "refs/*", 1); ProjectControl uBlah = user(devs); ProjectControl uAdmin = user(devs, admin); assertFalse("not owner", uBlah.isOwner()); assertTrue("is owner", uAdmin.isOwner()); }
Source:
<code>private void mergeValue() <START> throws Exception <END> { String key = parser.getKey(); String value = parser.getValue(); String version = parser.getVersion(); if (version == null) { version = startVersionDialog(key); } ConfigKey configKey = fetchConfigKey(key, version); if (configKey != null && configKey.getKey() != null) { if (!configKey.isPasswordKey() && configKey.getDisplayValue().trim().length() > 0) { String val = configKey.getDisplayValue(); if (!val.endsWith(";")) { val += ";"; } val += value; value = val; } } boolean sucessUpdate = persist(key, value, version); if (!sucessUpdate) { log.debug("setValue: error concatenating " + key + "'s value. No such entry" + (version == null ? "" : " with version " + version) + "."); throw new IllegalArgumentException("Error setting " + key + "'s value. No such entry" + (version == null ? "" : " with version " + version) + "."); } }</code><technical_language>I 2 questions : 1) handle prevent case of merged handled code 2) allow merge key values a comma separator get this done merge option (for example adding custom fencing configuration)</technical_language>

Target:
private void mergeValue() throws Exception { String key = parser.getKey(); String value = parser.getValue(); if(!keysConfig.getBoolean(key + "/" + MERGABLE_TOKEN, false)) { console.writeFormat(MERGE_NOT_SUPPORTED_MSG, key); console.writeLine(); return; } String version = parser.getVersion(); if (version == null) { version = startVersionDialog(key); } ConfigKey configKey = fetchConfigKey(key, version); if (configKey != null && configKey.getKey() != null && configKey.getDisplayValue().trim().length() > 0) { String valueInDb = configKey.getDisplayValue().trim(); String delimiter = keysConfig.getString(key + "/" + DELIMITER_TOKEN, ";"); value = concatenatedValues(value, valueInDb, delimiter); if(valueInDb.equals(value)) { console.writeFormat(MERGE_SAME_VALUE_MSG); console.writeLine(); return; } } if (!persist(key, value, version)) { log.debug("setValue: error concatenating " + key + "'s value. No such entry" + (version == null ? "" : " with version " + version) + "."); throw new IllegalArgumentException("Error setting " + key + "'s value. No such entry" + (version == null ? "" : " with version " + version) + "."); } }
Source:
<code>public void testTokenValueIncorrect() { <START> final Integer id = 184; <END> try { TokenUtils.getTokenName(id); fail("IllegalArgumentException is expected"); } catch (IllegalArgumentException expected) { assertEquals("given id " + id, expected.getMessage()); } }</code><technical_language>please explain this change. 184 good extension of tokens next java syntax extension, is value</technical_language>

Target:
public void testTokenValueIncorrect() throws IllegalAccessException { int maxId = 0; final Field[] fields = TokenTypes.class.getDeclaredFields(); for (final Field field : fields) { if (field.getType() != Integer.TYPE) { continue; } final String name = field.getName(); final int id = field.getInt(name); if (id > maxId) { maxId = id; } } final int nextAfterMaxId = maxId + 1; try { TokenUtils.getTokenName(nextAfterMaxId); fail("IllegalArgumentException is expected"); } catch (IllegalArgumentException expected) { assertEquals("given id " + nextAfterMaxId, expected.getMessage()); } }
Source:
<code>private void configureCluster(Settings.Builder builder) { <START> int replicationFactor = props.valueAsInt(ProcessProperties.SEARCH_REPLICAS, 0); <END> int minimumMasterNodes = props.valueAsInt(ProcessProperties.SEARCH_MINIMUM_MASTER_NODES, 1); String initialStateTimeOut = readInitialStateTimeout(); if (clusterEnabled) { if (!props.contains(ProcessProperties.SEARCH_REPLICAS)) { replicationFactor = 1; } if (!props.contains(ProcessProperties.SEARCH_MINIMUM_MASTER_NODES)) { minimumMasterNodes = 2; } if (!props.contains(ProcessProperties.SEARCH_INITIAL_STATE_TIMEOUT)) { initialStateTimeOut = "120s"; } String hosts = props.value(ProcessProperties.CLUSTER_SEARCH_HOSTS, ""); LOGGER.info("Elasticsearch cluster enabled. Connect to hosts [{}]", hosts); builder.put("discovery.zen.ping.unicast.hosts", hosts); } builder.put("discovery.zen.minimum_master_nodes", minimumMasterNodes) .put("discovery.initial_state_timeout", initialStateTimeOut) .put(IndexMetaData.SETTING_NUMBER_OF_REPLICAS, replicationFactor) .put("cluster.name", getClusterName()) .put("cluster.routing.allocation.awareness.attributes", "rack_id") .put("node.rack_id", nodeName) .put("node.name", nodeName) .put("node.data", true) .put("node.master", true); }</code><technical_language>Replica do sense default, cluster is disabled. 0 default overridable cluster is enabled</technical_language>

Target:
private void configureCluster(Settings.Builder builder) { int replicationFactor = 0; int minimumMasterNodes = 1; String initialStateTimeOut = "30s"; if (clusterEnabled) { replicationFactor = props.valueAsInt(ProcessProperties.SEARCH_REPLICAS, 1); minimumMasterNodes = props.valueAsInt(ProcessProperties.SEARCH_MINIMUM_MASTER_NODES, 2); initialStateTimeOut = props.value(ProcessProperties.SEARCH_INITIAL_STATE_TIMEOUT, "120s"); String hosts = props.value(ProcessProperties.CLUSTER_SEARCH_HOSTS, ""); LOGGER.info("Elasticsearch cluster enabled. Connect to hosts [{}]", hosts); builder.put("discovery.zen.ping.unicast.hosts", hosts); } builder.put("discovery.zen.minimum_master_nodes", minimumMasterNodes) .put("discovery.initial_state_timeout", initialStateTimeOut) .put(IndexMetaData.SETTING_NUMBER_OF_REPLICAS, replicationFactor) .put("cluster.name", getClusterName()) .put("cluster.routing.allocation.awareness.attributes", "rack_id") .put("node.rack_id", nodeName) .put("node.name", nodeName) .put("node.data", true) .put("node.master", true); }
Source:
<code>T get() throws Exception; } private boolean isFormJSONValid(FormTemplate jsonInFormTemplate) { String jsonString = GSON.toJson(jsonInFormTemplate); if (isJSONValid(jsonString)) { return true; } <START> LOGGER.debug("Invalid JSON found: {}", jsonString); <END> return false; }</code><technical_language>log a message WARN (but JSON string) a system administrator about</technical_language>

Target:
T get() throws Exception; } private boolean isFormJSONValid(FormTemplate formTemplate) { String jsonString = GSON.toJson(formTemplate); if (isJSONValid(jsonString)) { return true; } LOGGER.warn("Filtered out invalid JSON"); LOGGER.debug("Invalid JSON: {}", jsonString); return false; }
Source:
<code>public BindFBO(String fboName) { this.fboName = fboName; <START> if (fboName.equals(DEFAULT_FRAME_BUFFER_NAME)) { <END> fboId = DEFAULT_FRAME_BUFFER_ID; } else { fboId = frameBuffersManager.getFBO(fboName).fboId; } }</code><technical_language>I case is fboName is default frame buffer. a minor issue, leave is</technical_language>

Target:
public BindFBO(String fboName) { this.fboName = fboName; }
Source:
<code>protected final void onInit() { if (this.getBeanFactory() != null) { this.messagingTemplate.setBeanFactory(getBeanFactory()); if (StringUtils.hasText(this.outputChannelName)) { Assert.isNull(this.outputChannel, "'outputChannelName' and 'outputChannel' are mutually exclusive."); this.outputChannel = this.getBeanFactory().getBean(this.outputChannelName, MessageChannel.class); } <START> } <END> if (!CollectionUtils.isEmpty(this.adviceChain)) { ProxyFactory proxyFactory = new ProxyFactory(new AdvisedRequestHandler()); for (Advice advice : this.adviceChain) { proxyFactory.addAdvice(advice); } this.advisedRequestHandler = (RequestHandler) proxyFactory.getProxy(this.beanClassLoader); } this.doInit(); }</code><technical_language>here; if oc null, ocn be</technical_language>

Target:
protected final void onInit() { if (this.getBeanFactory() != null) { this.messagingTemplate.setBeanFactory(getBeanFactory()); if (StringUtils.hasText(this.outputChannelName)) { Assert.isNull(this.outputChannel, "'outputChannelName' and 'outputChannel' are mutually exclusive."); try { this.outputChannel = this.getBeanFactory().getBean(this.outputChannelName, MessageChannel.class); } catch (BeansException e) { throw new DestinationResolutionException("Failed to look up MessageChannel with name '" + this.outputChannelName + "' in the BeanFactory."); } } } if (!CollectionUtils.isEmpty(this.adviceChain)) { ProxyFactory proxyFactory = new ProxyFactory(new AdvisedRequestHandler()); for (Advice advice : this.adviceChain) { proxyFactory.addAdvice(advice); } this.advisedRequestHandler = (RequestHandler) proxyFactory.getProxy(this.beanClassLoader); } this.doInit(); }
Source:
<code>public void testNoLeaksAfterShutdown() { withRemoteCacheManager(new RemoteCacheManagerCallable(getRemoteCacheManager()) { @Override public void call() { RemoteCache<Integer, String> remote = rcm.getCache(); EventLogListener<Integer> eventListener = new EventLogListener<>(); remote.addClientListener(eventListener); eventListener.expectNoEvents(); remote.put(1, "one"); eventListener.expectOnlyCreatedEvent(1, cache()); } }); <START> detectThreadLeaks(".*Client-Listener.*"); <END> }</code><technical_language>this fail if another test running in parallel client listeners</technical_language>

Target:
public void testNoLeaksAfterShutdown() { withRemoteCacheManager(new RemoteCacheManagerCallable(getRemoteCacheManager()) { @Override public void call() { RemoteCache<Integer, String> remote = rcm.getCache(cacheName); EventLogListener<Integer> eventListener = new EventLogListener<>(); remote.addClientListener(eventListener); eventListener.expectNoEvents(); remote.put(1, "one"); eventListener.expectOnlyCreatedEvent(1, cache(cacheName)); } }); detectThreadLeaks(".*Client-Listener-ClientListenerLeakTest-.*"); }
Source:
<code>public void testPermissionsException() { try { Map<String, EnumSet<Permissions>> userPerms = db.getPermissions(); <START> Assume.assumeTrue(false); <END> }catch(UnsupportedOperationException e){ } }</code><technical_language>I find logic for Assume confusing - fault. a message "skipping test DB is a Cloudant service"</technical_language>

Target:
public void testPermissionsException() { Map<String, EnumSet<Permissions>> userPerms = db.getPermissions(); }
Source:
<code>public void requestBannerAd(Context context, MediationBannerListener mediationBannerListener, Bundle serverParameters, AdSize adSize, MediationAdRequest mediationAdRequest, Bundle networkExtras) { mContext = context; mMediationBannerListener = mediationBannerListener; mChartboostParams = ChartboostAdapterUtils.createChartboostParams( serverParameters, networkExtras); if (!ChartboostAdapterUtils.isValidChartboostParams(mChartboostParams)) { if (mMediationBannerListener != null) { mMediationBannerListener.onAdFailedToLoad( ChartboostAdapter.this, AdRequest.ERROR_CODE_INVALID_REQUEST); } return; } BannerSize supportedAdSize = chartboostAdSizeFromLocalExtras(context, adSize); if (supportedAdSize == null) { mMediationBannerListener.onAdFailedToLoad(ChartboostAdapter.this, ERROR_BANNER_SIZE_MISMATCH); return; } <START> isAdLoaded.set(false); <END> mChartboostParams.setBannerSize(supportedAdSize); ChartboostSingleton.startChartboostBanner( context, mChartboostBannerDelegate, this); }</code><technical_language>I this is needed is initialized false</technical_language>

Target:
public void requestBannerAd(Context context, MediationBannerListener mediationBannerListener, Bundle serverParameters, AdSize adSize, MediationAdRequest mediationAdRequest, Bundle networkExtras) { mContext = context; mMediationBannerListener = mediationBannerListener; mChartboostParams = ChartboostAdapterUtils.createChartboostParams( serverParameters, networkExtras); if (!ChartboostAdapterUtils.isValidChartboostParams(mChartboostParams)) { if (mMediationBannerListener != null) { mMediationBannerListener.onAdFailedToLoad( ChartboostAdapter.this, AdRequest.ERROR_CODE_INVALID_REQUEST); } return; } BannerSize supportedAdSize = chartboostAdSizeFromLocalExtras(context, adSize); if (supportedAdSize == null) { mMediationBannerListener.onAdFailedToLoad(ChartboostAdapter.this, ERROR_BANNER_SIZE_MISMATCH); return; } mChartboostParams.setBannerSize(supportedAdSize); ChartboostSingleton.startChartboostBanner( context, mChartboostBannerDelegate, this); }
Source:
<code>public void getBeacon(final ScanEvent scanEvent, SortedMap<String, String> attributes, final BeaconResponseHandler beaconResponseHandler) { String networkInfo = NetworkInfoBroadcastReceiver.latestNetworkInfo != null ? NetworkInfoBroadcastReceiver.getNetworkInfoString() : ""; getApiService() .getBeacon(scanEvent.getBeaconId().getPid(), networkInfo, attributes) .enqueue(new Callback<ResolveResponse>() { @Override public void onResponse(Call<ResolveResponse> call, Response<ResolveResponse> response) { if (response.isSuccessful() && response.body() != null) { save(response.body()); onSuccess(response.body()); } else { onFail(new Throwable("No Content, Invalid Api Key")); } } @Override public void onFailure(Call<ResolveResponse> call, Throwable t) { onFail(t); } void onSuccess(ResolveResponse resolveResponse) { List<BeaconEvent> beaconEvents = checkSuccessfulBeaconResponse(scanEvent, resolveResponse); beaconResponseHandler.onSuccess(beaconEvents); checkShouldCallBeaconResponseHandlers(resolveResponse); } void onFail(Throwable t) { if (lastSuccess == null) { beaconResponseHandler.onFailure(t); } else { <START> onSuccess(lastSuccess); <END> } } }); }</code><technical_language>I good logError here. a fail resolve. do think</technical_language>

Target:
public void getBeacon(final ScanEvent scanEvent, SortedMap<String, String> attributes, final BeaconResponseHandler beaconResponseHandler) { String networkInfo = NetworkInfoBroadcastReceiver.latestNetworkInfo != null ? NetworkInfoBroadcastReceiver.getNetworkInfoString() : ""; getApiService() .getBeacon(scanEvent.getBeaconId().getPid(), networkInfo, attributes) .enqueue(new Callback<ResolveResponse>() { @Override public void onResponse(Call<ResolveResponse> call, Response<ResolveResponse> response) { if (response.isSuccessful() && response.body() != null) { save(response.body()); onSuccess(response.body()); } else { onFail(new Throwable("No Content, Invalid Api Key")); } } @Override public void onFailure(Call<ResolveResponse> call, Throwable t) { onFail(t); } void onSuccess(ResolveResponse resolveResponse) { List<BeaconEvent> beaconEvents = checkSuccessfulBeaconResponse(scanEvent, resolveResponse); beaconResponseHandler.onSuccess(beaconEvents); checkShouldCallBeaconResponseHandlers(resolveResponse); } void onFail(Throwable t) { if (lastSuccess == null) { beaconResponseHandler.onFailure(t); } else { Logger.log.logError("resolution failed, but we have a backup:" + scanEvent.getBeaconId().toTraditionalString(), t); onSuccess(lastSuccess); } } }); }
Source:
<code>public ItemType getItemType() { <START> return ItemTypeRegistryModule.getInstance().getById( <END> this.statId.substring(this.statId.lastIndexOf('.') + 1)).get(); }</code><technical_language>put return .getById( next line reads: .getById(this.statId.substring(this.statId.lastIndexOf('.') + 1)).get();</technical_language>

Target:
public ItemType getItemType() { return ItemTypeRegistryModule.getInstance().get( CatalogKey.resolve(this.statId.substring(this.statId.lastIndexOf('.') + 1))).get(); }
Source:
<code>private void addTask(VDSReturnValue vdsReturnValue, boolean isRevertedTask) { AsyncTaskCreationInfo taskCreationInfo = vdsReturnValue.getCreationInfo(); getReturnValue().getInternalTaskIdList().add(cmd.createTask( taskCreationInfo, cmd.getActionType(), getTaskObjectType(), getTaskObjects()) ); Guid taskId = taskCreationInfo.getTaskID(); getReturnValue().getTaskIdList().add(taskId); <START> if (isRevertedTask) { <END> log.infoFormat("Reverted task ID is {0}.", taskId); } }</code><technical_language>Eli, I consider adding more useful information obtained taskCreationInfo</technical_language>

Target:
private void addTask(VDSReturnValue vdsReturnValue, boolean isRevertedTask) { AsyncTaskCreationInfo taskCreationInfo = vdsReturnValue.getCreationInfo(); getReturnValue().getInternalTaskIdList().add(cmd.createTask( taskCreationInfo, cmd.getActionType(), getTaskObjectType(), getTaskObjects()) ); Guid taskId = taskCreationInfo.getTaskID(); getReturnValue().getTaskIdList().add(taskId); if (isRevertedTask) { log.infoFormat("Reverting task {0} with ID {1} on DataCenter {2}.", taskCreationInfo.getTaskType().name(), taskId, taskCreationInfo.getStoragePoolID()); } }
Source:
<code>public Node visitInsert(SqlBaseParser.InsertContext context) { List<String> columns = identsToStrings(context.ident()); Table table; try { table = (Table) visit(context.table()); } catch (ClassCastException e) { TableFunction tf = (TableFunction) visit(context.table()); for (Expression ex : tf.functionCall().getArguments()) { if (!(ex instanceof QualifiedNameReference)) { throw new IllegalArgumentException(String.format(Locale.ENGLISH, "invalid table column reference %s", ex.toString())); } } throw e; } Node subquery = visit(context.insertSource().query()); return new Insert<>( table, <START> subquery == null ? null : (Query) subquery, <END> columns, createDuplicateKeyContext(context)); }</code><technical_language>null insert source is missing <LINK_0> . I a case. Also, do want calling subquery? insertSource().query() querySpec (subqquery) values, I is suitable name</technical_language>

Target:
public Node visitInsert(SqlBaseParser.InsertContext context) { List<String> columns = identsToStrings(context.ident()); Table table; try { table = (Table) visit(context.table()); } catch (ClassCastException e) { TableFunction tf = (TableFunction) visit(context.table()); for (Expression ex : tf.functionCall().getArguments()) { if (!(ex instanceof QualifiedNameReference)) { throw new IllegalArgumentException(String.format(Locale.ENGLISH, "invalid table column reference %s", ex.toString())); } } throw e; } return new Insert( table, (Query) visit(context.insertSource().query()), columns, createDuplicateKeyContext(context)); }
Source:
<code>public URN owner() { try { return URN.create( <START> new JdkRequest(UriBuilder.fromUri(this.home).build()) <END> .header(HttpHeaders.ACCEPT, MediaType.APPLICATION_XML) .header(HttpHeaders.AUTHORIZATION, this.token) .method(Request.GET) .fetch() .as(RestResponse.class) .assertStatus(HttpURLConnection.HTTP_OK) .as(XmlResponse.class) .xml() .xpath("/page/identity/urn/text()") .get(0) ); } catch (IOException ex) { throw new IllegalStateException(ex); } }</code><technical_language>need UriBuilder</technical_language>

Target:
public URN owner() { try { return URN.create( new JdkRequest(this.home) .header(HttpHeaders.ACCEPT, MediaType.APPLICATION_XML) .header(HttpHeaders.AUTHORIZATION, this.token) .fetch() .as(RestResponse.class) .assertStatus(HttpURLConnection.HTTP_OK) .as(XmlResponse.class) .xml() .xpath("/page/identity/urn/text()") .get(0) ); } catch (IOException ex) { throw new IllegalStateException(ex); } }
Source:
<code>protected synchronized void close(final Exception cause) { lock.lock(); queueMap.values().parallelStream() .filter(Objects::nonNull) .forEach(queue -> queue.offer(POISON_PILL)); queueMap.clear(); <START> lock.unlock(); <END> super.close(cause); }</code><technical_language>*three* types of locking here: synchronized, ConcurrentHashMap, ReentrantLock. fishy</technical_language>

Target:
protected void close(final Exception cause) { synchronized (this) { queueMap.values().parallelStream() .filter(Objects::nonNull) .forEach(queue -> queue.offer(POISON_PILL)); queueMap.clear(); } super.close(cause); }
Source:
<code>public static int getSizeInBytes() { <START> return 2 + 1 + 4 + 1 + 12 + 300; END> }</code><technical_language>call PairedReadSequence.getSizeInBytes() add that</technical_language>

Target:
public static int getSizeInBytes() { return PairedReadSequence.getSizeInBytes() + (3 * 4); }
Source:
<code>public void info(PrintStream out) { if (clusterId != null) { out.println("Minimesos cluster is running: " + clusterId); <START> out.println("Mesos version: " + clusterConfig.getMesosVersion()); <END> printServiceUrls(out); } }</code><technical_language>Mesos Version absent. printing then</technical_language>

Target:
public void info(PrintStream out) { if (clusterId != null) { out.println("Minimesos cluster is running: " + clusterId); if (getMesosVersion() != null) { out.println("Mesos version: " + clusterConfig.getMesosVersion()); } printServiceUrls(out); } }
Source:
<code>public void testSwitchToFilter() { final Rectangle cellBounds = SWTBotUtils.getCellBounds(fTableBot.widget, ROW, <START> 1); <END> ImageHelper before = ImageHelper.getScreenGrab(cellBounds); fTableBot.click(0, 1); fBot.text().typeText("00\n", 100); fTableBot.select(4); ImageHelper after = ImageHelper.getScreenGrab(cellBounds); fTableBot.click(0, 0); ImageHelper afterFilter = ImageHelper.getScreenGrab(cellBounds); fTableBot.click(0, 0); List<RGB> beforeLine = before.getPixelRow(2); List<RGB> afterLine = after.getPixelRow(2); List<RGB> afterFilterLine = afterFilter.getPixelRow(2); assertEquals(beforeLine.size(), afterLine.size()); assertEquals(beforeLine.size(), afterFilterLine.size()); for (int i = 0; i < beforeLine.size(); i++) { RGB afterFilterPixel = afterFilterLine.get(i); RGB beforePixel = beforeLine.get(i); RGB afterPixel = afterLine.get(i); assertEquals(beforePixel, afterFilterPixel); if (!afterPixel.equals(YELLOW)) { assertEquals(beforePixel, afterPixel); } else { assertNotEquals(YELLOW, beforePixel); } } assertEquals(beforeLine, afterFilterLine); assertNotEquals(afterLine, beforeLine); }</code><technical_language><field name>_COLUMN</technical_language>

Target:
public void testSwitchToFilter() { final Rectangle cellBounds = SWTBotUtils.getCellBounds(fTableBot.widget, ROW, TIMESTAMP_COLUMN); ImageHelper before = ImageHelper.getScreenGrab(cellBounds); fTableBot.click(0, TIMESTAMP_COLUMN); fBot.text().typeText("00\n", 100); fTableBot.select(ROW - 1); ImageHelper after = ImageHelper.getScreenGrab(cellBounds); fTableBot.click(0, 0); ImageHelper afterFilter = ImageHelper.getScreenGrab(cellBounds); fTableBot.click(0, 0); List<RGB> beforeLine = before.getPixelRow(2); List<RGB> afterLine = after.getPixelRow(2); List<RGB> afterFilterLine = afterFilter.getPixelRow(2); assertEquals(beforeLine.size(), afterLine.size()); assertEquals(beforeLine.size(), afterFilterLine.size()); for (int i = 0; i < beforeLine.size(); i++) { RGB afterFilterPixel = afterFilterLine.get(i); RGB beforePixel = beforeLine.get(i); RGB afterPixel = afterLine.get(i); assertEquals(beforePixel, afterFilterPixel); if (!afterPixel.equals(YELLOW)) { assertEquals(beforePixel, afterPixel); } else { assertNotEquals(YELLOW, beforePixel); } } assertEquals(beforeLine, afterFilterLine); assertNotEquals(afterLine, beforeLine); }
Source:
<code><START> public Collection<PotionEffect> getEffectList() { <END> return effectList; }</code><technical_language>this immutable</technical_language>

Target:
public Collection<PotionEffect> getEffectList() { return Collections.unmodifiableCollection(effectList); }
Source:
<code>public boolean equals(Object obj) { if (this == obj) { return true; } if (!super.equals(obj)) { return false; } if (!(obj instanceof CustomEvent)) { return false; } CustomEvent other = (CustomEvent) obj; if (fDefinition == null) { if (other.fDefinition != null) { return false; } } else if (!fDefinition.equals(other.fDefinition)) { <START> return false; <END> } return true; }</code><technical_language>Needs consider new custom fields</technical_language>

Target:
public boolean equals(Object obj) { if (this == obj) { return true; } if (!super.equals(obj)) { return false; } if (!(obj instanceof CustomEvent)) { return false; } CustomEvent other = (CustomEvent) obj; if (fDefinition == null) { if (other.fDefinition != null) { return false; } } else if (!fDefinition.equals(other.fDefinition)) { return false; } if (customEventTimestamp == null) { if (other.customEventTimestamp != null) { return false; } } else if (!customEventTimestamp.equals(other.customEventTimestamp)) { return false; } if (customEventContent == null) { if (other.customEventContent != null) { return false; } } else if (!customEventContent.equals(other.customEventContent)) { return false; } if (customEventType == null) { if (other.customEventType != null) { return false; } } else if (!customEventType.equals(other.customEventType)) { return false; } return true; }
Source:
<code><START> public static void process(String label, BufferedReader in) <END> throws IOException { processFile(label, in); }</code><technical_language>want deprecate this method well</technical_language>

Target:
public static void process(String label, BufferedReader in) throws IOException { validate(in, label); }
Source:
<code>public DatabaseJoinData() { super(); <START> notfound = null; <END> }</code><technical_language>keeping notfound = null? null this</technical_language>

Target:
public DatabaseJoinData() { super(); }
Source:
<code>public Aggregator factorize(ColumnSelectorFactory metricFactory) { return new LongLastAggregator( name, metricFactory.makeLongColumnSelector(fieldName), <START> metricFactory.makeLongColumnSelector(Column.TIME_COLUMN_NAME) <END> ); }</code><technical_language>minor nit: pass time column selector field selector consistent buffered aggregator</technical_language>

Target:
public Aggregator factorize(ColumnSelectorFactory metricFactory) { return new LongLastAggregator( name, metricFactory.makeLongColumnSelector(Column.TIME_COLUMN_NAME), metricFactory.makeLongColumnSelector(fieldName) ); }
Source:
<code>private Schema(@JsonProperty("id") final String id, @JsonProperty("url") final URI url, @JsonProperty("links") final Map<String, String> links, @JsonProperty("last_created") final String lastCreated, @JsonProperty("last_version") final Integer lastVersion) { this.id = id; this.url = url; this.links = links; <START> this.lastCreated = ZonedDateTime.parse(lastCreated); <END> this.lastVersion = lastVersion; }</code><technical_language>This in a separate PR</technical_language>

Target:
private Schema(@JsonProperty("id") final String id, @JsonProperty("url") final URI url, @JsonProperty("links") final Map<String, String> links, @JsonProperty("last_created") final String lastCreated, @JsonProperty("last_version") final Integer lastVersion) { this.id = id; this.url = url; this.links = links; this.lastCreated = DateConverter.iso8601DateTimeFromString(lastCreated); this.lastVersion = lastVersion; }
Source:
<code><START> private Map<Guid, Guid> loadImageGroupsIdsIfNeeded() { <END> List<Guid> requiredDisks = new ArrayList<>(); for (VmInternalData vmInternalData : _runningVms.values()) { for (DiskImageDynamic disk : vmInternalData.getVmDynamic().getDisks()) { requiredDisks.add(disk.getId()); } } if (_vdsManager.getRefreshStatistics()) return getDbFacade().getImageDao().getImageGroupIdMapForImageList(requiredDisks); return new HashMap<>(); }</code><technical_language>part lines 1722-1727 if clause in line 1729 well, used</technical_language>

Target:
private Map<Guid, Guid> loadImageGroupsIdsIfNeeded() { if (_vdsManager.getRefreshStatistics()) { Set<Guid> requiredDisks = new HashSet<>(); for (VmInternalData vmInternalData : _runningVms.values()) { for (DiskImageDynamic disk : vmInternalData.getVmDynamic().getDisks()) { requiredDisks.add(disk.getId()); } } return getDbFacade().getImageDao().getImageGroupIdMapForImageList(requiredDisks); } return Collections.emptyMap(); }
Source:
<code>public long getTimeOut(AbstractBuild<?, ?> build, BuildListener listener) throws InterruptedException, MacroEvaluationException, IOException { double elasticTimeout = getElasticTimeout(Integer.parseInt(expandAll(build,listener,getTimeoutPercentage())), build, listener); if (elasticTimeout == 0) { if (isFailSafeTimeoutDuration()) { <START> return Math.max(Integer.parseInt(getTimeoutMinutesElasticDefault()) * 60 * 1000, Integer.parseInt(expandAll(build, listener, getTimeoutMinutesElasticDefault())) * MINUTES); <END> } else { return Math.max(MINIMUM_TIMEOUT_MILLISECONDS, Integer.parseInt(expandAll(build, listener, getTimeoutMinutesElasticDefault())) * MINUTES); } } else { if (isFailSafeTimeoutDuration()) { return Math.max(Integer.parseInt(getTimeoutMinutesElasticDefault()) * 60 * 1000, (long) elasticTimeout); } else { return (long) Math.max(MINIMUM_TIMEOUT_MILLISECONDS, elasticTimeout); } } }</code><technical_language>a <LINK_0></technical_language>

Target:
public long getTimeOut(AbstractBuild<?, ?> build, BuildListener listener) throws InterruptedException, MacroEvaluationException, IOException { double elasticTimeout = getElasticTimeout(Integer.parseInt(expandAll(build,listener,getTimeoutPercentage())), build, listener); if (elasticTimeout == 0) { return Math.max(MINIMUM_TIMEOUT_MILLISECONDS, Integer.parseInt(expandAll(build, listener, getTimeoutMinutesElasticDefault())) * MINUTES); } else { if (isFailSafeTimeoutDuration()) { return Math.max(Integer.parseInt(expandAll(build, listener, getTimeoutMinutesElasticDefault())) * MINUTES, (long) elasticTimeout); } else { return (long) Math.max(MINIMUM_TIMEOUT_MILLISECONDS, elasticTimeout); } } }
Source:
<code>public List<FieldInfo> findIterableFields() { List<FieldInfo> iterableFields = new ArrayList<>(); try { for (FieldInfo fieldInfo : fieldsInfo().fields()) { Class type = getField(fieldInfo).getType(); if (type.isArray() || Iterable.class.isAssignableFrom(type)) { <START> iterableFields.add(fieldInfo); <END> } } return iterableFields; } catch (Exception e) { throw new RuntimeException(e); } }</code><technical_language>code style: is streaming api usage e.g. in change above</technical_language>

Target:
public List<FieldInfo> findIterableFields() { Predicate<FieldInfo> isIterable = f -> { Class type = getField(f).getType(); return type.isArray() || Iterable.class.isAssignableFrom(type); }; try { return fieldsInfo().fields().stream() .filter(isIterable).collect(toList()); } catch (Exception e) { throw new RuntimeException(e); } }
Source:
<code>public Context extractContext(String diagnosticId, Context context) { Objects.requireNonNull(context, "'context' cannot be null"); Objects.requireNonNull(diagnosticId, "'diagnosticId' cannot be null"); <START> Context local = context; <END> for (Tracer tracer : tracers) { local = tracer.extractContext(diagnosticId, local); } return local; }</code><technical_language>nit: optimise LOC, do: Context local = Objects.requireNonNull(context, "'context' null");. another instance above</technical_language>

Target:
public Context extractContext(String diagnosticId, Context context) { Context local = Objects.requireNonNull(context, "'context' cannot be null"); Objects.requireNonNull(diagnosticId, "'diagnosticId' cannot be null"); for (Tracer tracer : tracers) { local = tracer.extractContext(diagnosticId, local); } return local; }
Source:
<code>private static long timeout(Properties props) { String timeout = PGProperty.LOGIN_TIMEOUT.get(props); if (timeout != null) { try { return (long) (Float.parseFloat(timeout) * 1000); } catch (NumberFormatException e) { <START> LOGGER.log(Level.FINE, "Couldn't parse loginTimeout value: {0}", timeout); <END> } } return (long) DriverManager.getLoginTimeout() * 1000; }</code><technical_language>1 this in case user provides loginTimeout property ignored. happen often? I this log message in case put invalid value, warn is ignored</technical_language>

Target:
private static long timeout(Properties props) { String timeout = PGProperty.LOGIN_TIMEOUT.get(props); if (timeout != null) { try { return (long) (Float.parseFloat(timeout) * 1000); } catch (NumberFormatException e) { LOGGER.log(Level.WARNING, "Couldn't parse loginTimeout value: {0}", timeout); } } return (long) DriverManager.getLoginTimeout() * 1000; }
Source:
<code>public void transferMemoryToTaskContext(long taskBytes, LocalMemoryContext transferredBytesMemoryContext) { long bytes = operatorMemoryContext.getUserMemory(); <START> operatorMemoryContext.localUserMemoryContext().transferOwnership(transferredBytesMemoryContext); <END> try { transferredBytesMemoryContext.setBytes(transferredBytesMemoryContext.getBytes() + taskBytes - bytes); } catch (ExceededMemoryLimitException e) { transferredBytesMemoryContext.setBytes(0); throw e; } }</code><technical_language>supposed transfer taskBytes only, transferred</technical_language>

Target:
public void transferMemoryToTaskContext(long taskBytes, LocalMemoryContext transferredBytesMemoryContext) { checkArgument(taskBytes >= 0, "taskBytes is negative"); long bytes = operatorMemoryContext.getUserMemory(); long bytesBeforeTransfer = transferredBytesMemoryContext.getBytes(); operatorMemoryContext.localUserMemoryContext().transferMemory(transferredBytesMemoryContext); try { transferredBytesMemoryContext.setBytes(transferredBytesMemoryContext.getBytes() + taskBytes - bytes); } catch (ExceededMemoryLimitException e) { transferredBytesMemoryContext.setBytes(bytesBeforeTransfer); throw e; } }
Source:
<code>private void setDebugStatus( String message ) { <START> if ( debug ) { <END> log.setLogLevel(LogLevel.DEBUG); log.logBasic( message ); } }</code><technical_language>This is be. This be: if (log.isDebugEnabled()) { log.logDebug( message ); } Do "if (debug)" this is hardcoded enabled. Do "setLogLevel()" overriding system administrator's configuration</technical_language>

Target:
private void setDebugStatus( String message ) { if ( log.isDebug() ) { log.logBasic( message ); } }
Source:
<code>public Node visitWindowSpecification(SqlBaseParser.WindowSpecificationContext context) { return new WindowSpecification( getLocation(context), <START> Optional.ofNullable(context.identifier()).map(i -> (Identifier) visit(i)), <END> visit(context.partition, Expression.class), visit(context.sortItem(), SortItem.class), visitIfPresent(context.windowFrame(), WindowFrame.class)); }</code><technical_language>ditto, s/i/identifier/</technical_language>

Target:
public Node visitWindowSpecification(SqlBaseParser.WindowSpecificationContext context) { return new WindowSpecification( getLocation(context), Optional.ofNullable(context.identifier()).map(identifier -> (Identifier) visit(identifier)), visit(context.partition, Expression.class), visit(context.sortItem(), SortItem.class), visitIfPresent(context.windowFrame(), WindowFrame.class)); }
Source:
<code>public void objectOutputStreamBuilder_Test() throws IOException, URISyntaxException { final Ds3Object obj = new Ds3Object(OBJ_NAME, OBJ_BYTES.length); final Ds3ClientHelpers.Job job = HELPERS.startReadJob(BUCKET_NAME, Lists.newArrayList(obj)); final ByteArrayOutputStream stream = new ByteArrayOutputStream(); job.transfer(new ObjectOutputStreamBuilder() { @Override <START> public OutputStream buildOutputStream(String key) throws IOException { <END> return stream; } }); assertThat(stream.toByteArray(), is(OBJ_BYTES)); }</code><technical_language>Missing final</technical_language>

Target:
public void objectOutputStreamBuilder_Test() throws IOException, URISyntaxException { final Ds3Object obj = new Ds3Object(OBJ_NAME, OBJ_BYTES.length); final Ds3ClientHelpers.Job job = HELPERS.startReadJob(BUCKET_NAME, Lists.newArrayList(obj)); final ByteArrayOutputStream stream = new ByteArrayOutputStream(); job.transfer(new ObjectOutputStreamBuilder() { @Override public OutputStream buildOutputStream(final String key) throws IOException { return stream; } }); assertThat(stream.toByteArray(), is(OBJ_BYTES)); }
Source:
<code>public void onCreated(Iterable<CacheEntryEvent<? extends K, ? extends V>> events) throws CacheEntryListenerException { <START> for (Object event1 : events) { <END> CacheEntryEvent event = (CacheEntryEvent) event1; Assert.assertEquals(EventType.CREATED, event.getEventType()); this.created.incrementAndGet(); event.getKey(); event.getValue(); event.getSource(); } }</code><technical_language>This typed CacheEntryEvent<? extends K, ? extends V> cast needed next line</technical_language>

Target:
public void onCreated(Iterable<CacheEntryEvent<? extends K, ? extends V>> events) throws CacheEntryListenerException { for (CacheEntryEvent<? extends K, ? extends V> event : events) { Assert.assertEquals(EventType.CREATED, event.getEventType()); this.created.incrementAndGet(); } }
Source:
<code>public void testParameters() throws Exception { FreeStyleProject other = createArtifactProject(), p = createProject("${PROJSRC}", null, "${BASE}/*.txt", "${TARGET}/bar", false, false, false, true); ParameterDefinition paramDef = new StringParameterDefinition("PROJSRC",other.getName(), ""); ParameterDefinition paramDef2 = new StringParameterDefinition("BASE","",""); ParameterDefinition paramDef3 = new StringParameterDefinition("TARGET","foo",""); ParametersDefinitionProperty paramsDef = new ParametersDefinitionProperty(paramDef, paramDef2,paramDef3); p.addProperty(paramsDef); <START> CopyArtifactPermissionProperty cpp=new CopyArtifactPermissionProperty("/"+p.getName()); <END> other.addProperty(cpp); rule.assertBuildStatusSuccess(other.scheduleBuild2(0, new UserCause()).get()); FreeStyleBuild b = p.scheduleBuild2(0, new UserCause(), new ParametersAction(new StringParameterValue("PROJSRC", other.getName()), new StringParameterValue("BASE", "*r"), new StringParameterValue("TARGET", "foo"))).get(); rule.assertBuildStatusSuccess(b); assertFile(false, "foo/bar/foo.txt", b); assertFile(true, "foo/bar/subdir/subfoo.txt", b); }</code><technical_language>this need added now-why is in this PR? JENKINS-20398 related</technical_language>

Target:
public void testParameters() throws Exception { FreeStyleProject other = createArtifactProject(), p = createProject("$PROJSRC", null, "$BASE/*.txt", "$TARGET/bar", false, false, false, true); ParameterDefinition paramDef = new StringParameterDefinition("PROJSRC", other.getName(), ""); ParameterDefinition paramDef2 = new StringParameterDefinition("BASE", "", ""); ParameterDefinition paramDef3 = new StringParameterDefinition("TARGET", "foo", ""); ParametersDefinitionProperty paramsDef = new ParametersDefinitionProperty(paramDef, paramDef2, paramDef3); p.addProperty(paramsDef); rule.assertBuildStatusSuccess(other.scheduleBuild2(0, new UserCause()).get()); FreeStyleBuild b = p.scheduleBuild2(0, new UserCause(), new ParametersAction(new StringParameterValue("PROJSRC", other.getName()), new StringParameterValue("BASE", "*r"), new StringParameterValue("TARGET", "foo"))).get(); rule.assertBuildStatusSuccess(b); assertFile(false, "foo/bar/foo.txt", b); assertFile(true, "foo/bar/subdir/subfoo.txt", b); }
Source:
<code>public boolean contains(final String name) { try { <START> return !new JdkRequest(UriBuilder.fromUri(this.home).build()) <END> .header(HttpHeaders.ACCEPT, MediaType.APPLICATION_XML) .header(HttpHeaders.AUTHORIZATION, this.token) .method(Request.GET) .fetch() .as(RestResponse.class) .assertStatus(HttpURLConnection.HTTP_OK) .as(XmlResponse.class) .xml() .xpath(String.format("/page/rules/rule[name='%s']", name)) .isEmpty(); } catch (IOException ex) { throw new IllegalStateException(ex); } }</code><technical_language>need UriBuilder</technical_language>

Target:
public boolean contains(final String name) { try { return !new JdkRequest(this.home) .header(HttpHeaders.ACCEPT, MediaType.APPLICATION_XML) .header(HttpHeaders.AUTHORIZATION, this.token) .fetch() .as(RestResponse.class) .assertStatus(HttpURLConnection.HTTP_OK) .as(XmlResponse.class) .xml() .xpath(String.format("/page/rules/rule[name='%s']", name)) .isEmpty(); } catch (IOException ex) { throw new IllegalStateException(ex); } }
Source:
<code>public void validateSortedEncodedKeyComponents( Double lhs, Double rhs, Indexed<Double> lhsEncodings, Indexed<Double> rhsEncodings ) throws SegmentValidationException { if (!lhs.equals(rhs)) { <START> throw new SegmentValidationException("Dim [%s] value not equal. Expected [%s] found [%s]", lhs, rhs); <END> } }</code><technical_language>formatting params, args</technical_language>

Target:
public void validateSortedEncodedKeyComponents( Double lhs, Double rhs, Indexed<Double> lhsEncodings, Indexed<Double> rhsEncodings ) throws SegmentValidationException { if (!lhs.equals(rhs)) { throw new SegmentValidationException( "Dim [%s] value not equal. Expected [%s] found [%s]", dimensionName, lhs, rhs ); } }