auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>private boolean doLoginCheck(UsernamePasswordCredentials credentials) { String username = credentials.getUsername(); String password = allowedUsersMap.get(username); boolean loginCheckOk = false; if (password != null){ if(password.equals(credentials.getPassword())){ String userGroup = userGroups.get(username); if (userGroup != null){ <START> userGroupCredentials = new UserGroupCredentials(credentials.getEndpoint(),userGroup); <END> sharedState.put(SecurityConstants.ATTRIBUTE_CREDENTIALS, credentials); loginCheckOk = true; } else { logger.log(Level.WARNING, "User Group not found for user " + username); loginCheckOk = false; } } } else { logger.log(Level.WARNING, "User details not found for " + username); loginCheckOk = false; } return loginCheckOk; }</code><technical_language>add authentication name part of information Subject. Nonetheless, a problem for this code sample</technical_language>

Target:
private void doLoginCheck(UsernamePasswordCredentials credentials) throws FailedLoginException { String username = credentials.getUsername(); String password = ALLOWED_USERS_MAP.get(username); if (password != null) { if (password.equals(credentials.getPassword())) { String userGroup = USER_GROUPS.get(username); if (userGroup != null) { userGroupCredentials = new UserGroupCredentials(credentials.getEndpoint(), userGroup); sharedState.put(SecurityConstants.ATTRIBUTE_CREDENTIALS, credentials); } else { logger.log(Level.WARNING, "User Group not found for user " + username); throw new FailedLoginException("User Group not found for user " + username); } } } else { logger.log(Level.WARNING, "User details not found for " + username); throw new FailedLoginException("User details not found for " + username); } }
Source:
<code>public void shoudNotSaveItem_withExisitingItem() { final InMemoryDatabaseTemplate databaseTemplate = new InMemoryDatabaseTemplate(databaseSchemaHolder); final StubItem stubItem = dataGenerator.randomStubItem(); databaseTemplate.create(stubItem); <START> ItemConstraintViolationException actaulException = null; <END> try { databaseTemplate.create(stubItem); } catch (final ItemConstraintViolationException e) { actaulException = e; } assertNotNull(actaulException); }</code><technical_language>Typo variable name</technical_language>

Target:
public void shoudNotSaveItem_withExisitingItem() { final InMemoryDatabaseTemplate databaseTemplate = new InMemoryDatabaseTemplate(databaseSchemaHolder); final StubItem stubItem = dataGenerator.randomStubItem(); databaseTemplate.create(stubItem); ItemConstraintViolationException actualException = null; try { databaseTemplate.create(stubItem); } catch (final ItemConstraintViolationException e) { actualException = e; } assertNotNull(actualException); }
Source:
<code>public PageWithPosition poll() { PageWithPosition result = null; for (MergeSource mergeSource : mergeSources) { PageWithPosition current = mergeSource.getPage(); if (current == null) { if (mergeSource.isFinished()) { continue; } else { return null; } } if (result == null) { result = current; <START> } <END> else { int compareResult = comparator.compareTo(current.getPage(), current.getPosition(), result.getPage(), result.getPosition()); if (compareResult < 0) { result = current; } } } return result; }</code><technical_language>is weird this method work caller call incrementPosition returned object. increment position a method return copied PageWithPosition</technical_language>

Target:
public PageWithPosition poll() { if (!isQueueInitialized()) { List<PageWithPosition> initialRows = new ArrayList<>(); for (MergeSource mergeSource : mergeSources) { PageWithPosition page = mergeSource.getPage(); if (page == null) { if (mergeSource.isFinished()) { continue; } else { return null; } } initialRows.add(page); } mergeQueue.addAll(initialRows); } else { MergeSource mergeSource = mergeSources.get(lastPooledIndex); PageWithPosition page = mergeSource.getPage(); if (page == null && !mergeSource.isFinished()) { return null; } if (page != null) { mergeQueue.add(page); } } PageWithPosition result = mergeQueue.poll(); if (result == null) { return null; } lastPooledIndex = result.getStreamIndex(); return result; }
Source:
<code>default boolean anyExists() { <START> return flatData(RelativePath.empty().get(), Pager.builder().withPageSize(1).orderBy(Order.unspecified()) <END> .build()).hasNext(); }</code><technical_language>Pager builder replaced Pager.single()</technical_language>

Target:
default boolean anyExists() { return flatData(RelativePath.empty().get(), Pager.single()).hasNext(); }
Source:
<code>public void testReentrantLock(boolean fair) throws Exception { for (int i = 0; i < NODES_CNT; i++) { <START> final IgniteLock lock = createReentrantLock(i, fair ? "lock1" : "lock2", fair); <END> GridTestUtils.waitForCondition(new GridAbsPredicate() { @Override public boolean apply() { return !lock.isLocked(); } }, 500L); lock.lock(); assertTrue(lock.isLocked()); assertTrue(lock.isHeldByCurrentThread()); lock.lock(); assertTrue(lock.isLocked()); assertTrue(lock.isHeldByCurrentThread()); lock.unlock(); assertTrue(lock.isLocked()); assertTrue(lock.isHeldByCurrentThread()); lock.unlock(); assertFalse(lock.isLocked()); assertFalse(lock.isHeldByCurrentThread()); } }</code><technical_language>mb, "lock1" -> "unfair lock" "lock2" -> "fair lock"</technical_language>

Target:
private void testReentrantLock(boolean fair) throws Exception { for (int i = 0; i < NODES_CNT; i++) { final IgniteLock lock = createReentrantLock(i, fair ? "fair lock" : "unfair lock", fair); assertEquals(0, lock.getHoldCount()); GridTestUtils.waitForCondition(() -> !lock.isLocked(), 500L); lock.lock(); assertEquals(1, lock.getHoldCount()); assertTrue(lock.isLocked()); assertTrue(lock.isHeldByCurrentThread()); lock.lock(); assertEquals(2, lock.getHoldCount()); assertTrue(lock.isLocked()); assertTrue(lock.isHeldByCurrentThread()); lock.unlock(); assertEquals(1, lock.getHoldCount()); assertTrue(lock.isLocked()); assertTrue(lock.isHeldByCurrentThread()); lock.unlock(); assertEquals(0, lock.getHoldCount()); assertFalse(lock.isLocked()); assertFalse(lock.isHeldByCurrentThread()); } }
Source:
<code>public LoggerContext configure(Props props) { LoggerContext ctx = helper.getRootContext(); ctx.reset(); configureRootLogger(props); helper.apply(logLevelConfig, props); <START> helper.enableJulChangePropagation(ctx); <END> configureDirectToConsoleLoggers(ctx, STARTUP_LOGGER_NAME); extendConfigure(); return ctx; }</code><technical_language>is reason moving call enableJulChangePropagation configuring loggers in SLF4J</technical_language>

Target:
public LoggerContext configure(Props props) { LoggerContext ctx = helper.getRootContext(); ctx.reset(); helper.enableJulChangePropagation(ctx); configureRootLogger(props); helper.apply(logLevelConfig, props); configureDirectToConsoleLoggers(ctx, STARTUP_LOGGER_NAME); extendConfigure(); return ctx; }
Source:
<code>public void insertPastState(long stateStartTime, long stateEndTime, int quark, ITmfStateValue value) throws TimeRangeException { <START> final int nbAttributes = ((ITmfStateSystem) fPartialSS).getNbAttributes(); <END> if (!fInitialized) { fCheckpointQuark = fPartialSS.createCheckpoint(CHECKPOINT_NAME); fQuarksAdded.clear(); rebuildQuarks(nbAttributes, Boolean.TRUE); fCheckpoints.add(fPartialInput.getStartTime()); fInitialized = true; } if (quark == fCheckpointQuark) { fQuarksAdded.clear(); rebuildQuarks(nbAttributes, Boolean.TRUE); fCheckpoints.add(stateEndTime); return; } if (stateEndTime > fLatestTime) { fLatestTime = stateEndTime; } boolean insert = false; if (quark >= fQuarksAdded.size()) { rebuildQuarks(nbAttributes, Boolean.FALSE); insert = true; } if (fQuarksAdded.get(quark)) { fQuarksAdded.set(quark, Boolean.FALSE); insert = true; } if (insert) { fInnerHistory.insertPastState(stateStartTime, stateEndTime, quark, value); } }</code><technical_language>get a number here, call createCheckpoint() (which create a new quark). pass this (non-updated) number parameter rebuildQuarks. This sounds flaky. Is this intended behavior</technical_language>

Target:
public void insertPastState(long stateStartTime, long stateEndTime, int quark, ITmfStateValue value) throws TimeRangeException { int nbAttributes = ((ITmfStateSystem) fPartialSS).getNbAttributes(); if (!fInitialized) { fCheckpointQuark = fPartialSS.createCheckpoint(CHECKPOINT_NAME); fAddedQuarks.clear(); rebuildQuarks(nbAttributes, Boolean.TRUE); fCheckpoints.add(fPartialInput.getStartTime()); fInitialized = true; } if (quark == fCheckpointQuark) { fAddedQuarks.clear(); rebuildQuarks(nbAttributes, Boolean.TRUE); fCheckpoints.add(stateEndTime); return; } if (stateEndTime > fLatestTime) { fLatestTime = stateEndTime; } boolean insert = false; if (quark >= fAddedQuarks.size()) { rebuildQuarks(nbAttributes, Boolean.FALSE); insert = true; } if (fAddedQuarks.get(quark)) { fAddedQuarks.set(quark, Boolean.FALSE); insert = true; } if (insert) { fInnerHistory.insertPastState(stateStartTime, stateEndTime, quark, value); } }
Source:
<code>public void testPushOptionsNotSupported() throws Exception { try (Git git = new Git(db); Git git2 = new Git(createBareRepository())) { final StoredConfig config = git.getRepository().getConfig(); RemoteConfig remoteConfig = new RemoteConfig(config, "test"); remoteConfig.addURI(new URIish( git2.getRepository().getDirectory().toURI().toURL())); remoteConfig.addFetchRefSpec( new RefSpec("+refs/heads/*:refs/remotes/origin/*")); remoteConfig.update(config); <START> config.save(); <END> writeTrashFile("f", "content of f"); git.add().addFilepattern("f").call(); RevCommit commit = git.commit().setMessage("adding f").call(); git.checkout().setName("not-pushed").setCreateBranch(true).call(); git.checkout().setName("branchtopush").setCreateBranch(true).call(); assertNull( git2.getRepository().resolve("refs/heads/branchtopush")); assertNull( git2.getRepository().resolve("refs/heads/not-pushed")); assertNull( git2.getRepository().resolve("refs/heads/master")); List<String> pushOptions = new ArrayList<>(); PushCommand pushCommand = git.push().setRemote("test") .setPushOptions(pushOptions); pushCommand.call(); assertEquals(commit.getId(), git2.getRepository().resolve("refs/heads/branchtopush")); assertNull(git2.getRepository().resolve("refs/heads/not-pushed")); assertNull(git2.getRepository().resolve("refs/heads/master")); } }</code><technical_language>Do want explicit configuration relying default turn push options</technical_language>

Target:
public void testPushOptionsNotSupported() throws Exception { try (Git git = new Git(db); Git git2 = new Git(createBareRepository())) { final StoredConfig config = git.getRepository().getConfig(); RemoteConfig remoteConfig = new RemoteConfig(config, "test"); remoteConfig.addURI(new URIish( git2.getRepository().getDirectory().toURI().toURL())); remoteConfig.addFetchRefSpec( new RefSpec("+refs/heads/*:refs/remotes/origin/*")); remoteConfig.update(config); config.save(); final StoredConfig config2 = git2.getRepository().getConfig(); config2.setBoolean("receive", null, "pushoptions", false); config2.save(); writeTrashFile("f", "content of f"); git.add().addFilepattern("f").call(); RevCommit commit = git.commit().setMessage("adding f").call(); git.checkout().setName("not-pushed").setCreateBranch(true).call(); git.checkout().setName("branchtopush").setCreateBranch(true).call(); assertNull(git2.getRepository().resolve("refs/heads/branchtopush")); assertNull(git2.getRepository().resolve("refs/heads/not-pushed")); assertNull(git2.getRepository().resolve("refs/heads/master")); List<String> pushOptions = new ArrayList<>(); PushCommand pushCommand = git.push().setRemote("test") .setPushOptions(pushOptions); pushCommand.call(); assertEquals(commit.getId(), git2.getRepository().resolve("refs/heads/branchtopush")); assertNull(git2.getRepository().resolve("refs/heads/not-pushed")); assertNull(git2.getRepository().resolve("refs/heads/master")); } }
Source:
<code>public BigtableOptions build() { if (bulkOptions == null) { <START> int maxInflightRpcs = <END> BulkOptions.BIGTABLE_MAX_INFLIGHT_RPCS_PER_CHANNEL_DEFAULT * dataChannelCount; bulkOptions = new BulkOptions.Builder().setMaxInflightRpcs(maxInflightRpcs).build(); } return new BigtableOptions( clusterAdminHost, tableAdminHost, dataHost, port, projectId, zoneId, clusterId, credentialOptions, userAgent, retryOptions, dataChannelCount, bulkOptions, usePlaintextNegotiation); }</code><technical_language>Is in a non-HBase environment a non-null bulkOptions set maxInflightRpcs? If a valid pass ResourceLimiter</technical_language>

Target:
public BigtableOptions build() { if (bulkOptions == null) { int maxInflightRpcs = BulkOptions.BIGTABLE_MAX_INFLIGHT_RPCS_PER_CHANNEL_DEFAULT * dataChannelCount; bulkOptions = new BulkOptions.Builder().setMaxInflightRpcs(maxInflightRpcs).build(); } else if (bulkOptions.getMaxInflightRpcs() <= 0) { int maxInflightRpcs = BulkOptions.BIGTABLE_MAX_INFLIGHT_RPCS_PER_CHANNEL_DEFAULT * dataChannelCount; bulkOptions = bulkOptions.toBuilder().setMaxInflightRpcs(maxInflightRpcs).build(); } return new BigtableOptions( clusterAdminHost, tableAdminHost, dataHost, port, projectId, zoneId, clusterId, credentialOptions, userAgent, retryOptions, dataChannelCount, bulkOptions, usePlaintextNegotiation); }
Source:
<code>public WorkProcessor.ProcessorState<Page> process(Optional<WindowPartition> partitionOptional) { boolean finishing = !partitionOptional.isPresent(); if (finishing) { if (pageBuilder.isEmpty()) { return finished(); } Page page = pageBuilder.build(); pageBuilder.reset(); <START> return ofResult(page, false); <END> } WindowPartition partition = partitionOptional.get(); while (!pageBuilder.isFull()) { if (!partition.hasNext()) { return needsMoreData(); } partition.processNextRow(pageBuilder); } Page page = pageBuilder.build(); pageBuilder.reset(); return ofResult(page, !partition.hasNext()); }</code><technical_language>curious, happen if returned ofResult(page, /*needsMoreData*/ true), told is more data</technical_language>

Target:
public ProcessorState<Page> process(Optional<WindowPartition> partitionOptional) { boolean finishing = !partitionOptional.isPresent(); if (finishing) { if (pageBuilder.isEmpty()) { return finished(); } Page page = pageBuilder.build(); pageBuilder.reset(); return ofResult(page, false); } WindowPartition partition = partitionOptional.get(); while (!pageBuilder.isFull()) { if (!partition.hasNext()) { return needsMoreData(); } partition.processNextRow(pageBuilder); } Page page = pageBuilder.build(); pageBuilder.reset(); return ofResult(page, !partition.hasNext()); }
Source:
<code>public Attribute getAttributeValue(PerunSessionImpl sess, User user, AttributeDefinition destinationAttributeDefinition) throws InternalErrorException { Attribute attribute = super.getAttributeValue(sess, user, destinationAttributeDefinition); Attribute destinationAttribute = new Attribute(destinationAttributeDefinition); Set<String> valuesWithoutDuplicities = new HashSet<>(); if (attribute != null) { Object value = attribute.getValue(); if (value != null && value instanceof Set) { Object[] valArray = ((Set) value).toArray(); if (valArray instanceof String[]) { for (String val: (String[]) valArray) { valuesWithoutDuplicities.add(val); } } } <START> } <END> Attribute attribute1 = null; try { attribute1 = sess.getPerunBl().getAttributesManagerBl().getAttribute(sess, user, SOURCE_ATTRIBUTE_FRIENDLY_NAME_EPSA_MA_URN); } catch (WrongAttributeAssignmentException | AttributeNotExistsException e) { log.error("cannot read " + SOURCE_ATTRIBUTE_FRIENDLY_NAME_EPSA_MA + " of user " + user.getId(), e); } if (attribute1 != null) { Object value = attribute1.getValue(); if (value != null && value instanceof Map) { Map<String, String> valueAsMap = (Map<String, String>) value; valuesWithoutDuplicities.addAll(valueAsMap.keySet()); } } destinationAttribute.setValue(new ArrayList<>(valuesWithoutDuplicities)); return destinationAttribute; }</code><technical_language>This is complicated code. I prefer more human-readable. work situation, super implementation of method getAttributeValue return null instead of Attribute (if is not-null) is instance of ArrayList (not Set)</technical_language>

Target:
public Attribute getAttributeValue(PerunSessionImpl sess, User user, AttributeDefinition destinationAttributeDefinition) throws InternalErrorException { Attribute attribute = super.getAttributeValue(sess, user, destinationAttributeDefinition); Attribute destinationAttribute = new Attribute(destinationAttributeDefinition); Set<String> valuesWithoutDuplicities = new HashSet<>(attribute.valueAsList()); Attribute manualEPSAAttr = null; try { manualEPSAAttr = sess.getPerunBl().getAttributesManagerBl() .getAttribute(sess, user, getSecondarySourceAttributeName()); } catch (WrongAttributeAssignmentException e) { throw new InternalErrorException("Wrong assignment of " + getSecondarySourceAttributeFriendlyName() + " for user " + user.getId(), e); } catch (AttributeNotExistsException e) { log.debug("Attribute " + getSecondarySourceAttributeFriendlyName() + " of user " + user.getId() + "does not exist, values will be skipped", e); } if (manualEPSAAttr != null) { Map<String, String> value = manualEPSAAttr.valueAsMap(); if (value != null) { valuesWithoutDuplicities.addAll(value.keySet()); } } destinationAttribute.setValue(new ArrayList<>(valuesWithoutDuplicities)); return destinationAttribute; }
Source:
<code>public static void actool(Config config, File inDir, File outDir) throws IOException { List<Object> opts = new ArrayList<>(); String appIconSetName = null; String launchImagesName = null; final String appiconset = "appiconset"; final String launchimage = "launchimage"; for (String fileName : inDir.list()) { String ext = FilenameUtils.getExtension(fileName); if (ext.equals(appiconset)) { appIconSetName = FilenameUtils.getBaseName(fileName); } else if (ext.equals(launchimage)) { launchImagesName = FilenameUtils.getBaseName(fileName); } } if (appIconSetName != null || launchImagesName != null) { if (appIconSetName != null) { opts.add("--app-icon"); opts.add(appIconSetName); } if (launchImagesName != null) { opts.add("--launch-image"); opts.add(launchImagesName); } File partialInfoPlist = new File(inDir, "assetcatalog-partial-info.plist"); opts.add("--output-partial-info-plist"); opts.add(partialInfoPlist); } opts.add("--platform"); if (IOSTarget.isDeviceArch(config.getArch())) { opts.add("iphoneos"); } else if (IOSTarget.isSimulatorArch(config.getArch())) { opts.add("iphonesimulator"); } new Executor(config.getLogger(), getACTool()) .args(inDir, opts, "--output-format", "human-readable-text", <START> "--minimum-deployment-target", "6.0", "--compile", outDir).exec(); <END> }</code><technical_language>Xcode hardcode 6.0 is this need able pass in based MinimumOSVersion in user's Info.plist.xml</technical_language>

Target:
public static void actool(Config config, File inDir, File outDir) throws IOException { List<Object> opts = new ArrayList<>(); String appIconSetName = null; String launchImagesName = null; final String appiconset = "appiconset"; final String launchimage = "launchimage"; for (String fileName : inDir.list()) { String ext = FilenameUtils.getExtension(fileName); if (ext.equals(appiconset)) { appIconSetName = FilenameUtils.getBaseName(fileName); } else if (ext.equals(launchimage)) { launchImagesName = FilenameUtils.getBaseName(fileName); } } if (appIconSetName != null || launchImagesName != null) { if (appIconSetName != null) { opts.add("--app-icon"); opts.add(appIconSetName); } if (launchImagesName != null) { opts.add("--launch-image"); opts.add(launchImagesName); } File partialInfoPlist = new File(inDir, "assetcatalog-partial-info.plist"); opts.add("--output-partial-info-plist"); opts.add(partialInfoPlist); } opts.add("--platform"); if (IOSTarget.isDeviceArch(config.getArch())) { opts.add("iphoneos"); } else if (IOSTarget.isSimulatorArch(config.getArch())) { opts.add("iphonesimulator"); } String minOSVersion = config.getOs().getMinVersion(); if (config.getIosInfoPList() != null) { String v = config.getIosInfoPList().getMinimumOSVersion(); if (v != null) { minOSVersion = v; } } new Executor(config.getLogger(), getACTool()) .args(inDir, opts, "--output-format", "human-readable-text", "--minimum-deployment-target", minOSVersion, "--compile", outDir).exec(); }
Source:
<code>public void deleteUser(String userName) throws UserStoreException { Connection dbConnection = null; PreparedStatement preparedStatement = null; userName = UserCoreUtil.addDomainToName(userName, getMyDomainName()); String domain = UserCoreUtil.extractDomainFromName(userName); if (domain!=null){ domain = domain.toUpperCase(); } try { dbConnection = getDBConnection(); preparedStatement = dbConnection.prepareStatement(HybridJDBCConstants.REMOVE_USER_SQL); preparedStatement.setString(1, UserCoreUtil.removeDomainFromName(userName)); preparedStatement.setInt(2, tenantId); preparedStatement.setInt(3, tenantId); preparedStatement.setString(4, domain); preparedStatement.execute(); dbConnection.commit(); } catch (SQLException e) { <START> String errorMessage = "Error occurred while deleting user :"+userName; <END> throw new UserStoreException(errorMessage, e); } finally { DatabaseUtil.closeAllConnections(dbConnection, preparedStatement); } }</code><technical_language>add debug log, correct spacing</technical_language>

Target:
public void deleteUser(String userName) throws UserStoreException { Connection dbConnection = null; PreparedStatement preparedStatement = null; userName = UserCoreUtil.addDomainToName(userName, getMyDomainName()); String domain = UserCoreUtil.extractDomainFromName(userName); if (domain!=null){ domain = domain.toUpperCase(); } try { dbConnection = getDBConnection(); preparedStatement = dbConnection.prepareStatement(HybridJDBCConstants.REMOVE_USER_SQL); preparedStatement.setString(1, UserCoreUtil.removeDomainFromName(userName)); preparedStatement.setInt(2, tenantId); preparedStatement.setInt(3, tenantId); preparedStatement.setString(4, domain); preparedStatement.execute(); dbConnection.commit(); } catch (SQLException e) { String errorMessage = "Error occurred while deleting user : " + userName; if (log.isDebugEnabled()) { log.debug(errorMessage, e); } throw new UserStoreException(errorMessage, e); } finally { DatabaseUtil.closeAllConnections(dbConnection, preparedStatement); } }
Source:
<code>public void truncate() throws IOException, InterruptedException, RuntimeException { String token = this.getToken(); openshiftClient = new OpenshiftClientWrapper(this.namespace, this.KUBERNETES_MASTER_URL, token); Pod[] cassandraPods = openshiftClient.getPodsWithPrefix(this.cassandraPodPrefix); ArrayList<String> cassandraHosts = new ArrayList<>(); Arrays.stream(cassandraPods).forEach((pod) -> { cassandraHosts.add(pod.getIP()); }); if (cassandraPods.length > 0 || this.forceTruncation) { if (openshiftClient.hasLargePartition(this.partitionThreshold)) { IndexTruncator truncator = new IndexTruncator(cassandraHosts, this.keyspace, this.useSSL); truncator.setCassandraConnectionMaxDelay(cassandraConnectionMaxDelay); truncator.setCassandraConnectionMaxRetries(cassandraConnectionMaxRetries); int metricsReplicas = openshiftClient.getReplicationControllerScale(this.metricsRC); int heapsterReplicas = openshiftClient.getReplicationControllerScale(this.heapsterRC); openshiftClient.scaleReplicaitonController(this.heapsterRC, 0); openshiftClient.scaleReplicaitonController(this.metricsRC, 0); truncator.execute(); <START> openshiftClient.scaleReplicaitonController(this.heapsterRC, heapsterReplicas); <END> openshiftClient.scaleReplicaitonController(this.metricsRC, metricsReplicas); } else { log.info("Truncation was not necessary, skipping."); } } else { throw new RuntimeException("No cassandra pods available, Is Hawkular Metrics running?"); } }</code><technical_language>scaling performed if truncator.execute() fails leave cluster hawkular heapster... In case truncation fails for time is retried, in retry get metricsReplicas=0 heapsterReplicas=0 scale appropriately</technical_language>

Target:
public void truncate() throws IOException, InterruptedException, RuntimeException { String token = this.getToken(); openshiftClient = new OpenshiftClientWrapper(this.namespace, this.KUBERNETES_MASTER_URL, token); Pod[] cassandraPods = openshiftClient.getPodsWithPrefix(this.cassandraPodPrefix); ArrayList<String> cassandraHosts = new ArrayList<>(); Arrays.stream(cassandraPods).forEach((pod) -> { cassandraHosts.add(pod.getIP()); }); int replicas; if (cassandraPods.length > 0) { if (openshiftClient.hasLargePartition(this.partitionThreshold, this.keyspace, this.cassandraPodPrefix) || this.forceTruncation) { IndexTruncator truncator = new IndexTruncator(cassandraHosts, this.keyspace, this.useSSL); truncator.setCassandraConnectionMaxDelay(cassandraConnectionMaxDelay); truncator.setCassandraConnectionMaxRetries(cassandraConnectionMaxRetries); replicas = openshiftClient.getReplicationControllerScale(this.metricsRC); if (replicas != 0) { metricsReplicas = replicas; } replicas = openshiftClient.getReplicationControllerScale(this.heapsterRC); if (replicas != 0) { heapsterReplicas = replicas; } openshiftClient.scaleReplicaitonController(this.heapsterRC, 0); openshiftClient.scaleReplicaitonController(this.metricsRC, 0); truncator.execute(); } else { log.info("Truncation was not necessary, skipping."); } } else { throw new RuntimeException("No cassandra pods available, Is Hawkular Metrics running?"); } }
Source:
<code>public ShiftableTextArea() { super(); <START> new KeyboardNavigationHandler(this) { <END> @Override public void onEnter(NativeEvent evt) { super.onEnter(evt); int cursorPos = getCursorPos(); setText(new StringBuilder(getText()).insert(cursorPos, '\n').toString()); setCursorPos(cursorPos + 1); } }; }</code><technical_language>move this code constructor? This weird complicated for reading</technical_language>

Target:
public ShiftableTextArea() { super(); initializeEnterKeyHandler(); }
Source:
<code><START> public void setBaseOptions(Options options) { <END> setPriorityOptions(options); setNameOptions(options); setIteratorTypeOptions(options); }</code><technical_language>I this refactoring, nice. this method protected</technical_language>

Target:
protected void setBaseOptions(Options options) { setPriorityOptions(options); setNameOptions(options); setIteratorTypeOptions(options); }
Source:
<code>public Object last(List<Object> list) { <START> if (list.size() > 0) { <END> return list.get(list.size() - 1); } return null; }</code><technical_language>a listiterator's hasprevious previous methods. This protect non-random access lists retrieving "last" element costly</technical_language>

Target:
public Object last(List<Object> list) { ListIterator<Object> it = list.listIterator(list.size()); if (it.hasPrevious()) { return it.previous(); } return null; }
Source:
<code>public <R extends Service> void startService(ServiceBinding<R> serviceBinding) { <START> if ( StartStoppable.class.isInstance( serviceBinding.getService() ) ) { <END> ( (StartStoppable) serviceBinding.getService() ).start( configuration, sessionFactory ); } else { super.startService( serviceBinding ); } }</code><technical_language>a interesting debate. StartStoppable mutually exclusive Startable? sense. lose call in super of Manageable. in end Startable StartStoppable sense, simply call super. super.startService</technical_language>

Target:
public <R extends Service> void startService(ServiceBinding<R> serviceBinding) { super.startService( serviceBinding ); if ( StartStoppable.class.isInstance( serviceBinding.getService() ) ) { ( (StartStoppable) serviceBinding.getService() ).start( configuration, sessionFactory ); } }
Source:
<code>private Set<QualifierKind> findAllTheSupers( @UnderInitialization DefaultQualifierKindHierarchy this, @KeyFor("#2") QualifierKind qualifierKind, Map<DefaultQualifierKind, Set<DefaultQualifierKind>> directSuperMap) { Queue<DefaultQualifierKind> queue = new ArrayDeque<>(directSuperMap.get(qualifierKind)); Set<QualifierKind> allSupers = new TreeSet<>(directSuperMap.get(qualifierKind)); Set<DefaultQualifierKind> visited = new HashSet<>(); while (!queue.isEmpty()) { DefaultQualifierKind superQualKind = queue.remove(); if (superQualKind == qualifierKind) { throw new TypeSystemError("Cycle in hierarchy: %s", qualifierKind); } if (visited.contains(superQualKind)) { continue; } <START> visited.add(superQualKind); <END> if (superQualKind.isPoly()) { continue; } if (directSuperMap.containsKey(superQualKind)) { Set<DefaultQualifierKind> superSuperQuals = directSuperMap.get(superQualKind); queue.addAll(superSuperQuals); allSupers.addAll(superSuperQuals); } else { throw new TypeSystemError( "Found a super qualifier kind not a key in the directSuperMap. Found: %s", superQualKind); } } return allSupers; }</code><technical_language>add returns a boolean, instead of contains, code slightly shorter slightly more efficient</technical_language>

Target:
private Set<QualifierKind> findAllTheSupers( @UnderInitialization DefaultQualifierKindHierarchy this, @KeyFor("#2") QualifierKind qualifierKind, Map<DefaultQualifierKind, Set<DefaultQualifierKind>> directSuperMap) { Set<QualifierKind> allSupers = new TreeSet<>(directSuperMap.get(qualifierKind)); Queue<DefaultQualifierKind> toVisit = new ArrayDeque<>(directSuperMap.get(qualifierKind)); Set<DefaultQualifierKind> visited = new HashSet<>(); while (!toVisit.isEmpty()) { DefaultQualifierKind superQualKind = toVisit.remove(); if (superQualKind == qualifierKind) { throw new TypeSystemError("Cycle in hierarchy: %s", qualifierKind); } if (!visited.add(superQualKind) || superQualKind.isPoly()) { continue; } Set<DefaultQualifierKind> superSuperQuals = directSuperMap.get(superQualKind); if (superSuperQuals == null) { throw new TypeSystemError(superQualKind + " is not a key in the directSuperMap"); } toVisit.addAll(superSuperQuals); allSupers.addAll(superSuperQuals); } return allSupers; }
Source:
<code>private String exportHTML(XWikiContext context) throws XWikiException, IOException { XWikiRequest request = context.getRequest(); String description = request.get("description"); String name = request.get("name"); if (StringUtils.isBlank(name)) { name = context.getDoc().getFullName(); } String[] checkedPages = request.getParameterValues("pages"); String[] uncheckedPages = null; String checkedPagesRequestValue = request.get("checked-pages"); String uncheckedPagesRequestValue = request.get("unchecked-pages"); String otherPagesRequestValue = request.get("other-pages"); boolean otherPages = false; <START> if (!StringUtils.isEmpty(otherPagesRequestValue)) { <END> otherPages = Boolean.valueOf(otherPagesRequestValue); } if (!StringUtils.isEmpty(uncheckedPagesRequestValue)) { uncheckedPages = uncheckedPagesRequestValue.split("&"); } if (!StringUtils.isEmpty(checkedPagesRequestValue)) { checkedPages = checkedPagesRequestValue.split("&"); } Collection<DocumentReference> pageList = resolvePagesToExport(otherPages, checkedPages, uncheckedPages, context); if (pageList.isEmpty()) { return null; } HtmlPackager packager = new HtmlPackager(); if (name != null && name.trim().length() > 0) { packager.setName(name); } if (description != null) { packager.setDescription(description); } packager.addPageReferences(pageList); packager.export(context); return null; }</code><technical_language>Yes, assign Boolean.valueOf()</technical_language>

Target:
private String exportHTML(XWikiContext context) throws XWikiException, IOException { ExportArguments exportArguments = new ExportArguments(context); Collection<DocumentReference> pageList = resolvePages(exportArguments, context); if (pageList.isEmpty()) { return null; } HtmlPackager packager = new HtmlPackager(); if (exportArguments.name != null && exportArguments.name.trim().length() > 0) { packager.setName(exportArguments.name); } if (exportArguments.description != null) { packager.setDescription(exportArguments.description); } packager.addPageReferences(pageList); packager.export(context); return null; }
Source:
<code>public EmailProducerBuilder setToRecipient(String recipient) { List<String> recipients = new ArrayList<String>(); recipients.add(recipient); <START> return this.setToRecipients(recipients); <END> }</code><technical_language>shortcut return this.setToRecipients(Arrays.asList(recipient))</technical_language>

Target:
public EmailProducerBuilder setToRecipient(String recipient) { return this.setToRecipients(Arrays.asList(recipient)); }
Source:
<code><START> public void testSqlTimeType(){ <END> DateConverter converter = new DateConverter(); Map<String, Object> context = new HashMap<>(); context.put(ActionContext.LOCALE, new Locale("es_MX", "MX")); Object value = converter.convertValue(context, null, null, null, TIME_00_59_10, Time.class); assertEquals(Time.valueOf(TIME_00_59_10), value); }</code><technical_language>suggestion public void testSqlTimeType() {</technical_language>

Target:
public void testSqlTimeType() { DateConverter converter = new DateConverter(); Map<String, Object> context = new HashMap<>(); context.put(ActionContext.LOCALE, new Locale("es_MX", "MX")); Object value = converter.convertValue(context, null, null, null, TIME_01_59_10, Time.class); assertEquals("01:59:10", value.toString()); }
Source:
<code>protected void copyFile() throws CVSException { IWorkspaceRoot root =ResourcesPlugin.getWorkspace().getRoot(); String filePath = this.file.getPath(); IFile finalFile = root.getFileForLocation(new Path(filePath)); if(finalFile == null) { throw new CVSException("File '" + filePath + "' <START> can not be found."); <END> } InputStream fileInputStream = null; try { fileInputStream = new BufferedInputStream(new FileInputStream(tempFile)); if(!finalFile.exists()) { finalFile.create(fileInputStream, IResource.FORCE , null); } else { finalFile.setContents(fileInputStream, IResource.FORCE, null); } } catch (FileNotFoundException e) { throw CVSException.wrapException(e); } catch (CoreException e) { throw CVSException.wrapException(e); } finally{ if (fileInputStream != null) { try { fileInputStream.close(); } catch (IOException e) { } finally { if (tempFile != null) tempFile.delete(); } } } }</code><technical_language>. *in workspace*</technical_language>

Target:
protected void copyFile() throws CVSException { IWorkspaceRoot root =ResourcesPlugin.getWorkspace().getRoot(); String filePath = this.file.getPath(); IFile finalFile = root.getFileForLocation(new Path(filePath)); if(finalFile == null) { throw new CVSException("File '" + filePath + "' can not be found in workspace."); } InputStream fileInputStream = null; try { fileInputStream = new BufferedInputStream(new FileInputStream(tempFile)); if(!finalFile.exists()) { finalFile.create(fileInputStream, IResource.FORCE , null); } else { finalFile.setContents(fileInputStream, IResource.FORCE, null); } } catch (FileNotFoundException e) { throw CVSException.wrapException(e); } catch (CoreException e) { throw CVSException.wrapException(e); } finally{ if (fileInputStream != null) { try { fileInputStream.close(); } catch (IOException e) { } finally { if (tempFile != null) tempFile.delete(); } } } }
Source:
<code>SqliteConnectionManager(long poolSize, long poolReaperIntervalMs) { Utils.info(LOGGER, "Initiating connection poll: [poolSize='%d', poolReaperIntervalMs='%d'].", poolSize, poolReaperIntervalMs); try { Class.forName("org.sqlite.JDBC"); } catch (Exception exception) { throw Utils.exception(exception, "Error initiating sqlite driver."); } double poolSizeThreshold = poolSize * 0.9; double connectionsToRemove = poolSize * 0.1; <START> new Thread(() -> { <END> Utils.updateLogId("connection-reaper"); while (!stopPoolReaper) { Utils.debug(LOGGER, "Current pool size is '%d' and threshold is '%f'.", pool.size(), poolSizeThreshold); if (pool.size() > poolSizeThreshold) { Utils.info(LOGGER, "Reaping connections, current pool size %d.", pool.size()); List<PooledConnection> pooledConnections = new ArrayList<>(pool.values()); Collections.sort(pooledConnections); for (int i = 0; i < connectionsToRemove && i < pooledConnections.size(); i++) { pooledConnections.get(i).reapConnection(); } } try { Thread.sleep(poolReaperIntervalMs); } catch (Exception exception) { Thread.currentThread().interrupt(); Utils.warn(LOGGER, "Pool reaper interrupted."); } } Utils.warn(LOGGER, "Pool reaper stop."); }).start(); }</code><technical_language>This thread closed application shutdown</technical_language>

Target:
SqliteConnectionManager(long poolSize, long poolReaperIntervalMs) { if (LOGGER.isInfoEnabled()) { LOGGER.info(String.format("Initiating connection poll: [poolSize='%d', poolReaperIntervalMs='%d'].", poolSize, poolReaperIntervalMs)); } try { Class.forName("org.sqlite.JDBC"); } catch (Exception exception) { throw Utils.exception(exception, "Error initiating sqlite driver."); } double poolSizeThreshold = poolSize * 0.9; double connectionsToRemove = poolSize * 0.1; new Thread(() -> { while (!stopPoolReaper) { if (LOGGER.isDebugEnabled()) { LOGGER.debug(String.format("Current pool size is '%d' and threshold is '%f'.", pool.size(), poolSizeThreshold)); } if (pool.size() > poolSizeThreshold) { if (LOGGER.isInfoEnabled()) { LOGGER.info(String.format("Reaping connections, current pool size %d.", pool.size())); } List<PooledConnection> pooledConnections = new ArrayList<>(pool.values()); Collections.sort(pooledConnections); for (int i = 0; i < connectionsToRemove && i < pooledConnections.size(); i++) { pooledConnections.get(i).reapConnection(); } } try { Thread.sleep(poolReaperIntervalMs); } catch (Exception exception) { Thread.currentThread().interrupt(); if (LOGGER.isWarnEnabled()) { LOGGER.warn("Pool reaper interrupted."); } } } if (LOGGER.isWarnEnabled()) { LOGGER.warn("Pool reaper stop."); } }).start(); }
Source:
<code>private void assertAPIVersionIsAbove(String minimumVersion) throws DockerException, InterruptedException { final String apiVersion = version().apiVersion(); final int versionComparison = compareVersion(apiVersion, minimumVersion); <START> if (versionComparison > 0) { <END> throw new UnsupportedApiVersionException(apiVersion); } }</code><technical_language>This <. passes Travis, this test fails if I run a local docker 1.11.0 API version 1.23</technical_language>

Target:
private void assertAPIVersionIsAbove(String minimumVersion) throws DockerException, InterruptedException { final String apiVersion = version().apiVersion(); final int versionComparison = compareVersion(apiVersion, minimumVersion); if (versionComparison < 0) { throw new UnsupportedApiVersionException(apiVersion); } }
Source:
<code>protected Map<String, Object> getIcon(DocumentReference userReference) { Map<String, Object> icon = new HashMap<>(); XWikiContext xcontext = this.xcontextProvider.get(); try { XWikiDocument userProfileDocument = xcontext.getWiki().getDocument(userReference, xcontext); String avatar = userProfileDocument.getStringValue("avatar"); XWikiAttachment avatarAttachment = userProfileDocument.getAttachment(avatar); if (avatarAttachment != null && avatarAttachment.isImage(xcontext)) { icon.put(META_DATA_ICON, xcontext.getWiki().getURL(avatarAttachment.getReference(), "download", "width=30&height=30&keepAspectRatio=true", null, xcontext)); } } catch (XWikiException e) { this.logger.warn( "Failed to read the avatar of user [{}]. Root cause is [{}]. Using the default avatar instead.", userReference.getName(), ExceptionUtils.getRootCauseMessage(e)); } if (!icon.containsKey(META_DATA_ICON)) { <START> icon.put(META_DATA_ICON, xcontext.getWiki().getSkinFile("icons/xwiki/noavatar.png", true, xcontext)); <END> } return icon; }</code><technical_language>try icon theme here, for instance user icon</technical_language>

Target:
protected Map<String, Object> getIcon(DocumentReference userReference) { Map<String, Object> icon = new HashMap<>(); XWikiContext xcontext = this.xcontextProvider.get(); try { XWikiDocument userProfileDocument = xcontext.getWiki().getDocument(userReference, xcontext); String avatar = userProfileDocument.getStringValue("avatar"); XWikiAttachment avatarAttachment = userProfileDocument.getAttachment(avatar); if (avatarAttachment != null && avatarAttachment.isImage(xcontext)) { icon.put(IconManager.META_DATA_URL, xcontext.getWiki().getURL(avatarAttachment.getReference(), "download", "width=30&height=30&keepAspectRatio=true", null, xcontext)); icon.put(IconManager.META_DATA_ICON_SET_TYPE, IconType.IMAGE.name()); } } catch (XWikiException e) { this.logger.warn( "Failed to read the avatar of user [{}]. Root cause is [{}]. Using the default avatar instead.", userReference.getName(), ExceptionUtils.getRootCauseMessage(e)); } if (!icon.containsKey(IconManager.META_DATA_URL)) { try { icon = this.iconManager.getMetaData(DEFAULT_ICON_NAME); } catch (IconException e) { this.logger.warn("Error getting the icon [{}]. Root cause is [{}].", DEFAULT_ICON_NAME, ExceptionUtils.getRootCause(e)); } } return icon; }
Source:
<code>public List<LockWithClient> getLocks() { <START> return ImmutableList.copyOf(Iterables.transform(lockHolders.entrySet(), TO_LOCK_WITH_CLIENT_FUNCTION)); <END> }</code><technical_language>nit: function inline</technical_language>

Target:
public List<LockWithClient> getLocks() { return lockHolders.entrySet() .stream() .map(input -> ImmutableLockWithClient.builder() .lockDescriptor(input.getKey()) .lockClient(input.getValue()) .build()) .collect(Collectors.toList()); }
Source:
<code><START> private void testNoEntryMergePolicyConfigured() { <END> Exceptions.expectException(CacheException.class, () -> ConflictManagerFactory.get(getCache(0)).resolveConflicts()); }</code><technical_language>if runs private methods test</technical_language>

Target:
public void testNoEntryMergePolicyConfigured() { Exceptions.expectException(CacheException.class, () -> ConflictManagerFactory.get(getCache(0)).resolveConflicts()); }
Source:
<code>public static void main(String[] args) { System.out.println("Running the interpreter."); if(args.length != 1) { System.err.println("usage: wyvern <filename>"); System.exit(1); } String filename = args[0]; Path filepath = Paths.get(filename); if(!Files.isReadable(filepath)) { System.err.println("Cannot read file " + filename); System.exit(-1); } String source = TestUtil.readFile(filepath.toAbsolutePath().toString()); try { ExpressionAST ast = (ExpressionAST) TestUtil.getNewAST(source); Expression program = ast.generateIL(GenContext.empty()); TypeContext ctx = TypeContext.empty(); program.typeCheck(ctx); Value v = program.interpret(EvalContext.empty()); System.out.println(v.toString()); } catch(ParseException e) { System.err.println("Failed to parse file."); e.printStackTrace(); <START> } <END> }</code><technical_language>This is fine, catch ToolError (e.g. a typechecking error) print error message standard error</technical_language>

Target:
public static void main(String[] args) { if(args.length != 1) { System.err.println("usage: wyvern <filename>"); System.exit(1); } String filename = args[0]; Path filepath = Paths.get(filename); if(!Files.isReadable(filepath)) { System.err.println("Cannot read file " + filename); System.exit(1); } String source = TestUtil.readFile(filepath.toAbsolutePath().toString()); try { ExpressionAST ast = (ExpressionAST) TestUtil.getNewAST(source); Expression program = ast.generateIL(GenContext.empty()); TypeContext ctx = TypeContext.empty(); program.typeCheck(ctx); Value v = program.interpret(EvalContext.empty()); System.out.println(v.toString()); } catch(ParseException e) { System.err.println(e.toString()); } catch(ToolError e) { System.err.println(e.getTypecheckingErrorMessage().toString()); } }
Source:
<code>public static String[] getBootClassPathJars() { String classpath = null; String javaVersion = System.getProperty("java.specification.version"); if (javaVersion.equals("9") || javaVersion.equals("10") || javaVersion.equals("11")) { try { classpath = String.join( File.pathSeparator, Files.list(Paths.get(System.getenv("JAVA_HOME"), "jmods")) <START> .map(x -> x.toString()) <END> .collect(Collectors.toList())); } catch (IOException e) { } } else { classpath = System.getProperty("sun.boot.class.path"); } if (classpath == null) { throw new IllegalStateException("could not find boot classpath"); } String[] jars = classpath.split(File.pathSeparator); ArrayList<String> result = new ArrayList<>(); for (String jar : jars) { if ((jar.endsWith(".jar") || jar.endsWith(".jmod")) && (new File(jar)).exists()) { result.add(jar); } } return result.toArray(new String[0]); }</code><technical_language>Prefer method reference instead of lambda: .map(Path::toString)</technical_language>

Target:
public static String[] getBootClassPathJars() { String classpath = null; String javaVersion = System.getProperty("java.specification.version"); if (javaVersion.equals("9") || javaVersion.equals("10") || javaVersion.equals("11")) { try { classpath = String.join( File.pathSeparator, Files.list(Paths.get(System.getenv("JAVA_HOME"), "jmods")) .map(Path::toString) .collect(Collectors.toList())); } catch (IOException e) { throw new IllegalStateException(e); } } else { classpath = System.getProperty("sun.boot.class.path"); } if (classpath == null) { throw new IllegalStateException("could not find boot classpath"); } String[] jars = classpath.split(File.pathSeparator); ArrayList<String> result = new ArrayList<>(); for (String jar : jars) { if ((jar.endsWith(".jar") || jar.endsWith(".jmod")) && (new File(jar)).exists()) { result.add(jar); } } return result.toArray(new String[0]); }
Source:
<code>private boolean checkVdsGroupMatchQuota(QuotaConsumptionParametersWrapper parameters, QuotaConsumptionParameter param) { Quota quota = param.getQuota(); if (param instanceof QuotaVdsConsumptionParameter) { QuotaVdsConsumptionParameter paramVds = (QuotaVdsConsumptionParameter) param; <START> <END> if (paramVds.getVdsGroupId() == null) { parameters.getCanDoActionMessages().add(VdcBllMessages.ACTION_TYPE_FAILED_QUOTA_IS_NOT_VALID.toString()); log.errorFormat("Quota Vds parameters from command: {0} are missing vds group id", parameters.getAuditLogable().getClass().getName()); return false; } boolean vdsGroupInQuota = false; if(quota.getGlobalQuotaVdsGroup() != null) { vdsGroupInQuota = quota.getGlobalQuotaVdsGroup().getQuotaVdsGroupId().equals(paramVds.getVdsGroupId()); } else { for (QuotaVdsGroup vdsGroup : quota.getQuotaVdsGroups()) { if (vdsGroup.getVdsGroupId().equals(paramVds.getVdsGroupId())) { vdsGroupInQuota = true; break; } } } if (!vdsGroupInQuota) { parameters.getCanDoActionMessages().add(VdcBllMessages.ACTION_TYPE_FAILED_QUOTA_IS_NOT_VALID.toString()); log.errorFormat("Quota Vds parameters from command: {0}. Vds group does not match quota", parameters.getAuditLogable().getClass().getName()); return false; } } return true; }</code><technical_language>Again, pass validateInputs()</technical_language>

Target:
private boolean checkVdsGroupMatchQuota(QuotaConsumptionParametersWrapper parameters, QuotaConsumptionParameter param) { Quota quota = param.getQuota(); QuotaVdsConsumptionParameter paramVds = (QuotaVdsConsumptionParameter) param; if (paramVds.getVdsGroupId() == null) { parameters.getCanDoActionMessages().add(VdcBllMessages.ACTION_TYPE_FAILED_QUOTA_IS_NOT_VALID.toString()); log.errorFormat("Quota Vds parameters from command: {0} are missing vds group id", parameters.getAuditLogable().getClass().getName()); return false; } boolean vdsGroupInQuota = false; if(quota.getGlobalQuotaVdsGroup() != null) { vdsGroupInQuota = quota.getGlobalQuotaVdsGroup().getVdsGroupId().equals(paramVds.getVdsGroupId()); } else { for (QuotaVdsGroup vdsGroup : quota.getQuotaVdsGroups()) { if (vdsGroup.getVdsGroupId().equals(paramVds.getVdsGroupId())) { vdsGroupInQuota = true; break; } } } if (!vdsGroupInQuota) { parameters.getCanDoActionMessages().add(VdcBllMessages.ACTION_TYPE_FAILED_QUOTA_IS_NOT_VALID.toString()); log.errorFormat("Quota Vds parameters from command: {0}. Vds group does not match quota", parameters.getAuditLogable().getClass().getName()); return false; } return true; }