auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void execute(CommandLineArgument[] args) { if (arguments[ARG_IMPORT_FILE_IDX].isEnabled()) { List<String> errors = new ArrayList<>(); for (String file : args[ARG_IMPORT_FILE_IDX].getArguments()) { File f = new File(file); if (f.canRead()) { if (arguments[ARG_TARGET_URL_IDX].isEnabled()) { for (String url : arguments[ARG_TARGET_URL_IDX].getArguments()) { try { errors.addAll(this.importOpenApiDefinition(f, new URL(url), false)); } catch (MalformedURLException e) { <START> CommandLine.error("malformed target url: " + url); <END> } } } else { errors.addAll(this.importOpenApiDefinition(f, null, false)); } if (!errors.isEmpty()) { for (String error : errors) { CommandLine.error("Error importing definition: " + error); } } } else { CommandLine.error("Cannot read Open API file: " + f.getAbsolutePath()); } } } if (arguments[ARG_IMPORT_URL_IDX].isEnabled()) { for (String urlstr : args[ARG_IMPORT_URL_IDX].getArguments()) { try { List<String> errors = new ArrayList<>(); URI url = new URI(urlstr, false); if (arguments[ARG_SITE_OVERRIDE_IDX].isEnabled()) { for (String host : arguments[ARG_SITE_OVERRIDE_IDX].getArguments()) { try { new URI("http://" + host, false); errors.addAll(this.importOpenApiDefinition(url, host, false)); } catch (Exception e) { CommandLine.error(e.getMessage(), e); } } } else { errors.addAll(this.importOpenApiDefinition(url, null, false)); } if (!errors.isEmpty()) { for (String error : errors) { CommandLine.error("Error importing definition: " + error); } } } catch (Exception e) { CommandLine.error(e.getMessage(), e); } } } }</code><technical_language>change this a leading capital? Malformed</technical_language>

Target:
public void execute(CommandLineArgument[] args) { if (arguments[ARG_IMPORT_FILE_IDX].isEnabled()) { for (String file : args[ARG_IMPORT_FILE_IDX].getArguments()) { File f = new File(file); if (f.canRead()) { List<String> errors = new ArrayList<>(); if (arguments[ARG_TARGET_URL_IDX].isEnabled()) { for (String target : arguments[ARG_TARGET_URL_IDX].getArguments()) { try { errors.addAll(this.importOpenApiDefinition(f, target, false)); } catch (InvalidUrlException e) { CommandLine.error(e.getMessage()); } } } else { errors.addAll(this.importOpenApiDefinition(f, null, false)); } if (!errors.isEmpty()) { for (String error : errors) { CommandLine.error("Error importing definition: " + error); } } } else { CommandLine.error("Cannot read Open API file: " + f.getAbsolutePath()); } } } if (arguments[ARG_IMPORT_URL_IDX].isEnabled()) { for (String urlstr : args[ARG_IMPORT_URL_IDX].getArguments()) { try { List<String> errors = new ArrayList<>(); URI url = new URI(urlstr, false); if (arguments[ARG_TARGET_URL_IDX].isEnabled()) { for (String target : arguments[ARG_TARGET_URL_IDX].getArguments()) { try { errors.addAll(this.importOpenApiDefinition(url, target, false)); } catch (InvalidUrlException e) { CommandLine.error(e.getMessage()); } } } else { errors.addAll(this.importOpenApiDefinition(url, null, false)); } if (!errors.isEmpty()) { for (String error : errors) { CommandLine.error("Error importing definition: " + error); } } } catch (Exception e) { CommandLine.error(e.getMessage(), e); } } } }
Source:
<code><START> public Set<Task> getScheduledTasks() { <END> synchronized (this.taskMap) { return Sets.<Task>newHashSet(this.taskMap.values()); } }</code><technical_language>return type more specific cast SchedulerTask</technical_language>

Target:
protected Set<Task> getScheduledTasks() { synchronized (this.taskMap) { return Sets.<Task>newHashSet(this.taskMap.values()); } }
Source:
<code>public int[] executeBatch() throws SQLException { checkExecStatus(); synchronized (getConnectionSynchronization()) { setupContextStack(); <START> int i = 0; <END> clearResultSets(); Vector stmts = batchStatements; batchStatements = null; int size; if (stmts == null) size = 0; else size = stmts.size(); int[] returnUpdateCountForBatch = new int[size]; SQLException sqle; Iterator it = stmts.iterator(); activation.setBatch(it, this); try { activation.nextBatchElement(); returnUpdateCountForBatch = new int[size]; int r = 0; do { if (super.executeStatement(activation, false, true)) throw newSQLException(SQLState.RESULTSET_RETURN_NOT_ALLOWED); for (int j = 0; j < updateCounts.length; ++j) { returnUpdateCountForBatch[r++] = (int) updateCounts[j]; } } while(activation.nextBatchElement()); InterruptStatus.restoreIntrFlagIfSeen(lcc); return returnUpdateCountForBatch; } catch (StandardException se) { sqle = handleException(se); } catch (SQLException sqle2) { sqle = sqle2; } finally { activation.setBatch(null, null); restoreContextStack(); } int successfulUpdateCount[] = new int[i]; System.arraycopy(returnUpdateCountForBatch, 0, successfulUpdateCount, 0, i); SQLException batch = new java.sql.BatchUpdateException(sqle.getMessage(), sqle.getSQLState(), sqle.getErrorCode(), successfulUpdateCount, sqle); batch.setNextException(sqle); throw batch; } }</code><technical_language>is "I" for? useful</technical_language>

Target:
public int[] executeBatch() throws SQLException { checkExecStatus(); synchronized (getConnectionSynchronization()) { setupContextStack(); clearResultSets(); Vector stmts = batchStatements; batchStatements = null; int size; if (stmts == null) size = 0; else size = stmts.size(); int[] returnUpdateCountForBatch = new int[size]; SQLException sqle; Iterator it = stmts.iterator(); activation.setBatch(it, this); try { activation.nextBatchElement(); returnUpdateCountForBatch = new int[size]; int r = 0; do { if (super.executeStatement(activation, false, true)) throw newSQLException(SQLState.RESULTSET_RETURN_NOT_ALLOWED); for (int j = 0; j < updateCounts.length; ++j) { returnUpdateCountForBatch[r++] = (int) updateCounts[j]; } } while(activation.nextBatchElement()); InterruptStatus.restoreIntrFlagIfSeen(lcc); return returnUpdateCountForBatch; } catch (StandardException se) { sqle = handleException(se); } catch (SQLException sqle2) { sqle = sqle2; } finally { activation.setBatch(null, null); restoreContextStack(); } int successfulUpdateCount[] = new int[0]; SQLException batch = new java.sql.BatchUpdateException(sqle.getMessage(), sqle.getSQLState(), sqle.getErrorCode(), successfulUpdateCount, sqle); batch.setNextException(sqle); throw batch; } }
Source:
<code>public void testLogFindAuditEvents() { <START> System.out.println("testLogFindAuditEvents"); <END> AuditRepositoryLogger instance = new AuditRepositoryLogger(); FindAuditEventsMessageType message = new FindAuditEventsMessageType(); FindAuditEventsType auditEvent = new FindAuditEventsType(); auditEvent.setPatientId("D12345"); message.setFindAuditEvents(auditEvent); LogEventRequestType result = instance.logFindAuditEvents(message, NhincConstants.AUDIT_LOG_OUTBOUND_DIRECTION, NhincConstants.AUDIT_LOG_NHIN_INTERFACE); assertNotNull(result); assertEquals(NhincConstants.AUDIT_LOG_OUTBOUND_DIRECTION, result.getDirection()); assertNotNull(result.getAuditMessage()); }</code><technical_language>This log statement adding (plus a system out call....) -- remove this</technical_language>

Target:
public void testLogFindAuditEvents() { AuditRepositoryLogger instance = new AuditRepositoryLogger(); FindAuditEventsMessageType message = new FindAuditEventsMessageType(); FindAuditEventsType auditEvent = new FindAuditEventsType(); auditEvent.setPatientId("D12345"); message.setFindAuditEvents(auditEvent); LogEventRequestType result = instance.logFindAuditEvents(message, NhincConstants.AUDIT_LOG_OUTBOUND_DIRECTION, NhincConstants.AUDIT_LOG_NHIN_INTERFACE); assertNotNull(result); assertEquals(NhincConstants.AUDIT_LOG_OUTBOUND_DIRECTION, result.getDirection()); assertNotNull(result.getAuditMessage()); }
Source:
<code>public CompletableFuture<SlotProfile> getSlotProfileFuture(ExecutionSlotSharingGroup executionSlotSharingGroup) { ResourceProfile totalSlotResourceProfile = ResourceProfile.ZERO; Collection<AllocationID> priorAllocations = new HashSet<>(); Collection<CompletableFuture<Collection<TaskManagerLocation>>> preferredLocationsPerExecution = new ArrayList<>(); for (ExecutionVertexID execution : executionSlotSharingGroup.getExecutionVertexIds()) { totalSlotResourceProfile = totalSlotResourceProfile.merge(resourceProfileRetriever.apply(execution)); priorAllocations.add(prioAllocationIdRetriever.apply(execution)); preferredLocationsPerExecution.add(preferredLocationsRetriever .getPreferredLocations(execution, producersToIgnore)); } CompletableFuture<Collection<TaskManagerLocation>> preferredLocationsFuture = FutureUtils .combineAll(preferredLocationsPerExecution) .thenApply(executionPreferredLocations -> <START> executionPreferredLocations.stream().flatMap(Collection::stream).collect(Collectors.toSet())); <END> ResourceProfile physicalSlotResourceProfile = totalSlotResourceProfile; return preferredLocationsFuture.thenApply( preferredLocations -> SlotProfile.priorAllocation( physicalSlotResourceProfile, physicalSlotResourceProfile, preferredLocations, priorAllocations, allBulkPriorAllocationIds)); }</code><technical_language>I a list is needed instead of a set, frequency of a location accounted selecting location in LocationPreferenceSlotSelectionStrategy</technical_language>

Target:
public CompletableFuture<SlotProfile> getSlotProfileFuture(ExecutionSlotSharingGroup executionSlotSharingGroup) { ResourceProfile totalSlotResourceProfile = ResourceProfile.ZERO; Collection<AllocationID> priorAllocations = new HashSet<>(); Collection<CompletableFuture<Collection<TaskManagerLocation>>> preferredLocationsPerExecution = new ArrayList<>(); for (ExecutionVertexID execution : executionSlotSharingGroup.getExecutionVertexIds()) { totalSlotResourceProfile = totalSlotResourceProfile.merge(resourceProfileRetriever.apply(execution)); priorAllocations.add(priorAllocationIdRetriever.apply(execution)); preferredLocationsPerExecution.add(preferredLocationsRetriever .getPreferredLocations(execution, producersToIgnore)); } CompletableFuture<Collection<TaskManagerLocation>> preferredLocationsFuture = FutureUtils .combineAll(preferredLocationsPerExecution) .thenApply(executionPreferredLocations -> executionPreferredLocations.stream().flatMap(Collection::stream).collect(Collectors.toList())); ResourceProfile physicalSlotResourceProfile = totalSlotResourceProfile; return preferredLocationsFuture.thenApply( preferredLocations -> SlotProfile.priorAllocation( physicalSlotResourceProfile, physicalSlotResourceProfile, preferredLocations, priorAllocations, allBulkPriorAllocationIds)); }
Source:
<code>public Reference getReference() throws NamingException { Reference ref = createReference(); StringBuilder serverString = new StringBuilder(); for (String serverName : serverNames) { if (serverString.length() > 0) { serverString.append(","); } serverString.append(serverName); } ref.add(new StringRefAddr("serverName", serverString.toString())); <START> if (portNumbers != null) { <END> StringBuilder portString = new StringBuilder(); for (int i = 0; i < portNumbers.length; i++) { if (i > 0) { portString.append(","); } int p = portNumbers[i]; portString.append(Integer.toString(p)); } ref.add(new StringRefAddr("portNumber", portString.toString())); } ref.add(new StringRefAddr("databaseName", databaseName)); if (user != null) { ref.add(new StringRefAddr("user", user)); } if (password != null) { ref.add(new StringRefAddr("password", password)); } for (PGProperty property : PGProperty.values()) { if (property.isPresent(properties)) { ref.add(new StringRefAddr(property.getName(), property.get(properties))); } } return ref; }</code><technical_language>Assuming changes portNumbers null</technical_language>

Target:
public Reference getReference() throws NamingException { Reference ref = createReference(); StringBuilder serverString = new StringBuilder(); for (int i = 0; i < serverNames.length; i++) { if (i > 0) { serverString.append(","); } String serverName = serverNames[i]; serverString.append(serverName); } ref.add(new StringRefAddr("serverName", serverString.toString())); StringBuilder portString = new StringBuilder(); for (int i = 0; i < portNumbers.length; i++) { if (i > 0) { portString.append(","); } int p = portNumbers[i]; portString.append(Integer.toString(p)); } ref.add(new StringRefAddr("portNumber", portString.toString())); ref.add(new StringRefAddr("databaseName", databaseName)); if (user != null) { ref.add(new StringRefAddr("user", user)); } if (password != null) { ref.add(new StringRefAddr("password", password)); } for (PGProperty property : PGProperty.values()) { if (property.isPresent(properties)) { ref.add(new StringRefAddr(property.getName(), property.get(properties))); } } return ref; }
Source:
<code>public void setGuestCpuType(String value) { <START> if (!Objects.equals(createdByUser, value)) { <END> guestCpuType = value; onPropertyChanged(new PropertyChangedEventArgs("GuestCpuType")); } }</code><technical_language>this comparison !Objects.equals(guestCpuType, value)</technical_language>

Target:
public void setGuestCpuType(String value) { if (!Objects.equals(guestCpuType, value)) { guestCpuType = value; onPropertyChanged(new PropertyChangedEventArgs("GuestCpuType")); } }
Source:
<code>public CompletionStage<MetricsQueryResult> executeQuery(final BoundedMetricsQuery query) { <START> return CompletableFuture.completedFuture(query).thenCompose(this::executeQueryInner); <END> }</code><technical_language>thought CompletableFuture.completedFuture(x).thenCompose(y) semantically identical y(x) -- difference</technical_language>

Target:
public CompletionStage<MetricsQueryResult> executeQuery(final BoundedMetricsQuery query) { try { return executeQueryInner(query); } catch (final Exception e) { final CompletableFuture<MetricsQueryResult> future = new CompletableFuture<>(); future.completeExceptionally(e); return future; } }
Source:
<code>public static DataType<?> ofNameOrNull(String typeName) { <START> return TYPES_BY_NAME_OR_ALIAS.getOrDefault(typeName, null); <END> }</code><technical_language>get instead of .getOrDefault</technical_language>

Target:
public static DataType<?> ofNameOrNull(String typeName) { return TYPES_BY_NAME_OR_ALIAS.get(typeName); }
Source:
<code>public void setup() throws SourceUnavailableException, IngestException { when(mockMetacard1.getId()).thenReturn("1"); when(mockMetacard2.getId()).thenReturn("2"); when(mockMetacard1.getTitle()).thenReturn("title1"); when(mockMetacard2.getTitle()).thenReturn("title2"); when(deleteResponse.getDeletedMetacards()).thenReturn(ImmutableList.of(mockMetacard1)); when(catalogFramework.delete(any(DeleteRequest.class))).thenReturn(deleteResponse); <START> when(catalogFramework.delete(any(DeleteRequestImpl.class))).thenReturn(deleteResponse); <END> cmd = new SearchFormsManageCommand() { @Override protected List<Metacard> retrieveSystemMetacards() { return ImmutableList.of(mockMetacard1, mockMetacard2); } }; cmd.catalogFramework = catalogFramework; }</code><technical_language>need this stub for DeleteRequest</technical_language>

Target:
public void setup() throws SourceUnavailableException, IngestException { when(mockMetacard1.getId()).thenReturn("1"); when(mockMetacard2.getId()).thenReturn("2"); when(mockMetacard1.getTitle()).thenReturn("title1"); when(mockMetacard2.getTitle()).thenReturn("title2"); when(deleteResponse.getDeletedMetacards()).thenReturn(ImmutableList.of(mockMetacard1)); when(catalogFramework.delete(any(DeleteRequest.class))).thenReturn(deleteResponse); when(endpointUtil.getMetacardListByTag(SYSTEM_TEMPLATE)) .thenReturn(ImmutableList.of(mockMetacard1, mockMetacard2)); cmd = new SearchFormsManageCommand(); cmd.setEndpointUtil(endpointUtil); cmd.setCatalogFramework(catalogFramework); }
Source:
<code>private boolean verifyAllHostsMountedDatastore(List<Pair<ManagedObjectReference, String>> lstHosts, DatastoreMO dsMO) throws Exception { int numHostsChecked = 0; for (Pair<ManagedObjectReference, String> host : lstHosts) { ManagedObjectReference morHostToMatch = host.first(); HostMO hostToMatchMO = new HostMO(dsMO.getContext(), morHostToMatch); List<DatastoreHostMount> datastoreHostMounts = dsMO.getHostMounts(); for (DatastoreHostMount datastoreHostMount : datastoreHostMounts) { ManagedObjectReference morHost = datastoreHostMount.getKey(); HostMO hostMO = new HostMO(dsMO.getContext(), morHost); if (hostMO.getHostName().equals(hostToMatchMO.getHostName())) { if (datastoreHostMount.getMountInfo().isMounted() && datastoreHostMount.getMountInfo().isAccessible()) { numHostsChecked++; } else { return false; } } } } return lstHosts.size() == numHostsChecked; <START> } <END></code><technical_language>a log show admin a data storage is mounted accessible</technical_language>

Target:
private boolean verifyAllHostsMountedDatastore(List<Pair<ManagedObjectReference, String>> lstHosts, DatastoreMO dsMO) throws Exception { List<HostMO> hostMOs = new ArrayList<>(lstHosts.size()); for (Pair<ManagedObjectReference, String> host : lstHosts) { ManagedObjectReference morHostToMatch = host.first(); HostMO hostToMatchMO = new HostMO(dsMO.getContext(), morHostToMatch); hostMOs.add(hostToMatchMO); } return verifyAllHostsMountedDatastore2(hostMOs, dsMO); }
Source:
<code>private static Set<AccountGroup.Id> groupsFor( SchemaFactory<ReviewDb> dbfactory, String[] groupNames) { final Set<AccountGroup.Id> result = new HashSet<AccountGroup.Id>(); <START> ReviewDb db = null; <END> try { db = dbfactory.open(); for (String name : groupNames) { AccountGroup group = db.accountGroups().get(new AccountGroup.NameKey(name)); if (group == null) { log.warn("Group \"" + name + "\" not in database," + " removing from authGroup"); } else { result.add(group.getId()); } } } catch (OrmException e) { log.error("Database error: " + e); } finally { db.close(); } return result; }</code><technical_language>This is bad practice. assigned null this field missed potential NullPointerException in finally block. A idiom is: try { final ReviewDb db = dbfactory.open(); try { .... } finally { db.close(); } } catch (OrmException e) { ... } is for control reach db.close allocation succeed. Further, compiler validate this for you, if break assumption changing code, yell</technical_language>

Target:
private static Set<AccountGroup.Id> groupsFor( SchemaFactory<ReviewDb> dbfactory, String[] groupNames) { final Set<AccountGroup.Id> result = new HashSet<AccountGroup.Id>(); try { final ReviewDb db = dbfactory.open(); try { for (String name : groupNames) { AccountGroup group = db.accountGroups().get(new AccountGroup.NameKey(name)); if (group == null) { log.warn("Group \"" + name + "\" not in database," + " removing from authGroup"); } else { result.add(group.getId()); } } } finally { db.close(); } } catch (OrmException e) { log.error("Database error: " + e); } return result; }
Source:
<code><START> public static void propagateMetadataFromProps(Map<String, String> metaData, Props inputProps, String nodeType, <END> String nodeName, Logger logger) { if (null == metaData || null == inputProps || null == logger || Strings.isNullOrEmpty(nodeType) || Strings.isNullOrEmpty(nodeName)) { throw new IllegalArgumentException("Input params should not be null or empty."); } final String propsToPropagate = inputProps.getString(AZKABAN_EVENT_REPORTING_PROPERTIES_TO_PROPAGATE); if (Strings.isNullOrEmpty(propsToPropagate)) { logger.info(String.format("No properties to propagate to metadata for %s: %s", nodeType, nodeName)); return; } else { logger.info(String.format("Propagating: %s to metadata for %s: %s", propsToPropagate, nodeType, nodeName)); } final List<String> propsToPropagateList = SPLIT_ON_COMMA.splitToList(propsToPropagate); for (String propKey : propsToPropagateList) { if (!inputProps.containsKey(propKey)) { logger.warn(String.format("%s does not contains: %s property; " + "skipping propagation to metadata", nodeName, propKey)); continue; } metaData.put(propKey, inputProps.getString(propKey)); } }</code><technical_language>method package-private annotate @VisibleForTesting</technical_language>

Target:
static void propagateMetadataFromProps(Map<String, String> metaData, Props inputProps, String nodeType, String nodeName, Logger logger) { if (!inputProps.containsKey(AZKABAN_EVENT_REPORTING_PROPERTIES_TO_PROPAGATE)) { return; } if (null == metaData || null == inputProps || null == logger || Strings.isNullOrEmpty(nodeType) || Strings.isNullOrEmpty(nodeName)) { throw new IllegalArgumentException("Input params should not be null or empty."); } final String propsToPropagate = inputProps.getString(AZKABAN_EVENT_REPORTING_PROPERTIES_TO_PROPAGATE); if (Strings.isNullOrEmpty(propsToPropagate)) { logger.info(String.format("No properties to propagate to metadata for %s: %s", nodeType, nodeName)); return; } else { logger.info(String.format("Propagating: %s to metadata for %s: %s", propsToPropagate, nodeType, nodeName)); } final List<String> propsToPropagateList = SPLIT_ON_COMMA.splitToList(propsToPropagate); for (String propKey : propsToPropagateList) { if (!inputProps.containsKey(propKey)) { logger.warn(String.format("%s does not contains: %s property; " + "skipping propagation to metadata", nodeName, propKey)); continue; } metaData.put(propKey, inputProps.getString(propKey)); } }
Source:
<code>public String getHost() { String[] hosts = _host.split(","); if (_hostCounter >= hosts.length) { _hostCounter = 0; } <START> s_logger.info("Connecting to host: " + hosts[_hostCounter % hosts.length]); <END> return hosts[_hostCounter++ % hosts.length]; }</code><technical_language>this log message a bit deceiving. I ++ return statement moved above. text is deceiving; connection is in this method. other hand method name change getNextHost()..</technical_language>

Target:
public String getHost() { final String[] hosts = _host.split(","); if (_hostCounter >= hosts.length) { _hostCounter = 0; } final String host = hosts[_hostCounter % hosts.length]; _hostCounter++; return host; }
Source:
<code>public static CreateTableRequest adapt(HTableDescriptor desc, byte[][] splitKeys) { <START> CreateTableRequest createTableRequest = CreateTableRequest.of(desc.getTableName().getNameAsString()); <END> adapt(createTableRequest, desc); addSplitKeys(createTableRequest, splitKeys); return createTableRequest; }</code><technical_language>Please call variable request for short</technical_language>

Target:
public static CreateTableRequest adapt(HTableDescriptor desc, byte[][] splitKeys) { CreateTableRequest request = CreateTableRequest.of(desc.getTableName().getNameAsString()); adapt(desc, request); addSplitKeys(splitKeys, request); return request; }
Source:
<code>public void parseAccountId() { AccountId accountId = BridgeUtils.parseAccountId("test", "identifier"); assertEquals("test", accountId.getStudyId()); assertEquals("identifier", accountId.getId()); accountId = BridgeUtils.parseAccountId("test", "externalid:identifier"); assertEquals("test", accountId.getStudyId()); assertEquals("identifier", accountId.getExternalId()); accountId = BridgeUtils.parseAccountId("test", "externalId:identifier"); assertEquals("test", accountId.getStudyId()); <START> assertEquals("identifier", accountId.getExternalId()); <END> accountId = BridgeUtils.parseAccountId("test", "healthcode:identifier"); assertEquals("test", accountId.getStudyId()); assertEquals("identifier", accountId.getHealthCode()); accountId = BridgeUtils.parseAccountId("test", "healthCode:identifier"); assertEquals("test", accountId.getStudyId()); assertEquals("identifier", accountId.getHealthCode()); accountId = BridgeUtils.parseAccountId("test", "unk:identifier"); assertEquals("test", accountId.getStudyId()); assertEquals("unk:identifier", accountId.getId()); }</code><technical_language>Minor: identifier some upper case letters, verify accidentally case identifier. (Here for healthCode.)</technical_language>

Target:
public void parseAccountId() { AccountId accountId = BridgeUtils.parseAccountId("test", "IdentifierA9"); assertEquals("test", accountId.getStudyId()); assertEquals("IdentifierA9", accountId.getId()); accountId = BridgeUtils.parseAccountId("test", "externalid:IdentifierA9"); assertEquals("test", accountId.getStudyId()); assertEquals("IdentifierA9", accountId.getExternalId()); accountId = BridgeUtils.parseAccountId("test", "externalId:IdentifierA9"); assertEquals("test", accountId.getStudyId()); assertEquals("IdentifierA9", accountId.getExternalId()); accountId = BridgeUtils.parseAccountId("test", "healthcode:IdentifierA9"); assertEquals("test", accountId.getStudyId()); assertEquals("IdentifierA9", accountId.getHealthCode()); accountId = BridgeUtils.parseAccountId("test", "healthCode:IdentifierA9"); assertEquals("test", accountId.getStudyId()); assertEquals("IdentifierA9", accountId.getHealthCode()); accountId = BridgeUtils.parseAccountId("test", "unk:IdentifierA9"); assertEquals("test", accountId.getStudyId()); assertEquals("unk:IdentifierA9", accountId.getId()); }
Source:
<code>public void start(C configuration, EmbeddedCacheManager cacheManager) { if (log.isDebugEnabled()) { log.debugf("Starting server with configuration: %s", configuration); } this.configuration = configuration; this.cacheManager = cacheManager; BasicComponentRegistry bcr = SecurityActions.getGlobalComponentRegistry(cacheManager).getComponent(BasicComponentRegistry.class.getName()); cacheIgnore = bcr.getComponent(CacheIgnoreManager.class).running(); if (cacheIgnore == null) { throw new IllegalStateException("CacheIgnoreManager is a required component"); } executor = bcr.getComponent(KnownComponentNames.BLOCKING_EXECUTOR, ExecutorService.class).running(); <START> if (executor instanceof BlockingTaskAwareExecutorServiceImpl) { <END> manageableThreadPoolExecutorService = new ManageableThreadPoolExecutorService( ((BlockingTaskAwareExecutorServiceImpl) executor).getExecutorService()); } else { manageableThreadPoolExecutorService = new ManageableThreadPoolExecutorService(executor); } try { startInternal(); } catch (RuntimeException t) { stop(); throw t; } }</code><technical_language>is this true? I create LazyInitializingBlockingTaskAwareExecutorService. is a ManageableExecutorService more wrapping</technical_language>

Target:
public void start(C configuration, EmbeddedCacheManager cacheManager) { if (log.isDebugEnabled()) { log.debugf("Starting server with configuration: %s", configuration); } this.configuration = configuration; this.cacheManager = cacheManager; BasicComponentRegistry bcr = SecurityActions.getGlobalComponentRegistry(cacheManager).getComponent(BasicComponentRegistry.class.getName()); cacheIgnore = bcr.getComponent(CacheIgnoreManager.class).running(); if (cacheIgnore == null) { throw new IllegalStateException("CacheIgnoreManager is a required component"); } executor = bcr.getComponent(KnownComponentNames.BLOCKING_EXECUTOR, ExecutorService.class).running(); manageableThreadPoolExecutorService = new ManageableThreadPoolExecutorService(executor); try { startInternal(); } catch (RuntimeException t) { stop(); throw t; } }
Source:
<code>public InvokeCancel(BigInteger jobId) { super(); this.jobId = jobId; <START> this.userName = ""; <END> }</code><technical_language>usual in Java null represent unset value, empty string</technical_language>

Target:
public InvokeCancel(BigInteger jobId) { super(); this.jobId = jobId; this.userName = null; }
Source:
<code>protected synchronized void processGetNodeData(GetNodeDataMessage msg) { if (logger.isTraceEnabled()) logger.trace( "Peer {}: processing GetNodeData, size [{}]", channel.getPeerIdShort(), msg.getStateRoots().size() ); List<Value> states = new ArrayList<>(); RepositoryRoot repositoryRoot = ((RepositoryRoot) repository); <START> for (byte[] stateRoot : msg.getStateRoots()) { <END> Value value = repositoryRoot.getState(stateRoot); if (value != null) { states.add(value); logger.trace("Eth63: " + Hex.toHexString(stateRoot).substring(0, 8) + " -> " + value); } else { } } sendMessage(new NodeDataMessage(states)); }</code><technical_language>please rename stateRoot(s) nodeKey nodeHash avoid confusion</technical_language>

Target:
protected synchronized void processGetNodeData(GetNodeDataMessage msg) { if (logger.isTraceEnabled()) logger.trace( "Peer {}: processing GetNodeData, size [{}]", channel.getPeerIdShort(), msg.getNodeKeys().size() ); List<Value> nodeValues = new ArrayList<>(); for (byte[] nodeKey : msg.getNodeKeys()) { byte[] rawNode = stateSource.get(nodeKey); if (rawNode != null) { Value value = new Value(rawNode); nodeValues.add(value); logger.trace("Eth63: " + Hex.toHexString(nodeKey).substring(0, 8) + " -> " + value); } } sendMessage(new NodeDataMessage(nodeValues)); }
Source:
<code>public static BluetoothGattDescriptor getClientCharacteristicConfigurationDescriptor() { BluetoothGattDescriptor descriptor = new BluetoothGattDescriptor( CLIENT_CHARACTERISTIC_CONFIGURATION_UUID, (BluetoothGattDescriptor.PERMISSION_READ | BluetoothGattDescriptor.PERMISSION_WRITE)); <START> try { <END> descriptor.setValue(new byte[]{0, 0}); } finally { return descriptor; } }</code><technical_language>do need set value</technical_language>

Target:
public static BluetoothGattDescriptor getClientCharacteristicConfigurationDescriptor() { BluetoothGattDescriptor descriptor = new BluetoothGattDescriptor( CLIENT_CHARACTERISTIC_CONFIGURATION_UUID, (BluetoothGattDescriptor.PERMISSION_READ | BluetoothGattDescriptor.PERMISSION_WRITE)); descriptor.setValue(new byte[]{0, 0}); return descriptor; }
Source:
<code>public void equal_shouldReturnFalseIfOnlyStartDateOfOneObjectIsNull() { CohortMembership cohortMembershipOne = new CohortMembership(13); <START> CohortMembership cohortMembershipTwo = new CohortMembership(12); <END> Date startDate1 = null; Date endDate1 = new Date(1577836801); Date startDate2 = new Date(1545140935); Date endDate2 = null; cohortMembershipOne.setEndDate(endDate1); cohortMembershipOne.setStartDate(startDate1); cohortMembershipTwo.setStartDate(startDate2); cohortMembershipTwo.setEndDate(endDate2); assertFalse(cohortMembershipOne.equals(cohortMembershipTwo)); }</code><technical_language>start date is null end date of object</technical_language>

Target:
public void equal_shouldReturnFalseIfOnlyStartDateOfOneObjectIsNull() { final int PATIENT_ID = 12; CohortMembership cohortMembershipOne = new CohortMembership(PATIENT_ID); CohortMembership cohortMembershipTwo = new CohortMembership(PATIENT_ID); final int TIMESTAMP_END_DATE = 1577836801; Date startDate1 = null; Date endDate1 = new Date(TIMESTAMP_END_DATE); Date startDate2 = new Date(1545140935); Date endDate2 = new Date(TIMESTAMP_END_DATE); cohortMembershipOne.setEndDate(endDate1); cohortMembershipOne.setStartDate(startDate1); cohortMembershipTwo.setStartDate(startDate2); cohortMembershipTwo.setEndDate(endDate2); assertFalse(cohortMembershipOne.equals(cohortMembershipTwo)); }
Source:
<code>private synchronized Text[] getCutPoints() throws IOException { if (cutPointArray == null) { String cutFileName = conf.get(CUTFILE_KEY); <START> Path[] cf = DistributedCacheHelper.getLocalCacheFiles(conf); <END> if (cf != null) { for (Path path : cf) { if (path.toUri().getPath() .endsWith(cutFileName.substring(cutFileName.lastIndexOf('/')))) { TreeSet<Text> cutPoints = new TreeSet<>(); try (Scanner in = new Scanner(new BufferedReader( new InputStreamReader(new FileInputStream(path.toString()), UTF_8)))) { while (in.hasNextLine()) cutPoints.add(new Text(Base64.getDecoder().decode(in.nextLine()))); } cutPointArray = cutPoints.toArray(new Text[cutPoints.size()]); break; } } } if (cutPointArray == null) throw new FileNotFoundException(cutFileName + " not found in distributed cache"); } return cutPointArray; }</code><technical_language>I guess safe call Job.getInstance() here? does. pass job context this private method</technical_language>

Target:
private synchronized Text[] getCutPoints() throws IOException { if (cutPointArray == null) { String cutFileName = conf.get(CUTFILE_KEY); Path[] cf = Job.getInstance().getLocalCacheFiles(); if (cf != null) { for (Path path : cf) { if (path.toUri().getPath() .endsWith(cutFileName.substring(cutFileName.lastIndexOf('/')))) { TreeSet<Text> cutPoints = new TreeSet<>(); try (Scanner in = new Scanner(new BufferedReader( new InputStreamReader(new FileInputStream(path.toString()), UTF_8)))) { while (in.hasNextLine()) cutPoints.add(new Text(Base64.getDecoder().decode(in.nextLine()))); } cutPointArray = cutPoints.toArray(new Text[cutPoints.size()]); break; } } } if (cutPointArray == null) throw new FileNotFoundException(cutFileName + " not found in distributed cache"); } return cutPointArray; }
Source:
<code>public boolean touch(ResourceLoader resourceLoader, ContainerContext context) throws Exception { ServletContext sc = context.getServletContext(); <START> return "DecoratingListener".equals(sc.getAttribute("org.eclipse.jetty.cdi")); <END> }</code><technical_language>string constants</technical_language>

Target:
public boolean touch(ResourceLoader resourceLoader, ContainerContext context) throws Exception { ServletContext sc = context.getServletContext(); return sc.getAttribute(JETTY_DECORATING_ATTRIBUTE) instanceof String || JETTY_CDI_VALUE.equals(sc.getAttribute(JETTY_CDI_ATTRIBUTE)); }
Source:
<code>private Image getReviewStateImage(int reviewScore) { <START> switch (reviewScore) { <END> case -2: return CommonImages.getImage(ReviewsImages.RED_NOT); case -1: return CommonImages.getImage(ReviewsImages.MINUS_ONE); case 1: return CommonImages.getImage(ReviewsImages.PLUS_ONE); case 2: return CommonImages.getImage(ReviewsImages.GREEN_CHECK); } return null; }</code><technical_language>remove this switch enum part of enum a (-2, -1, etc.) returns proper ImageDescriptor. This simplify this function to: return CommonImages.getImage(ReviewsImages.valueOf(reviewsScore.toString()))</technical_language>

Target:
private Image getReviewStateImage(int reviewScore) { switch (reviewScore) { case MINUS_TWO: return CommonImages.getImage(ReviewsImages.RED_NOT); case MINUS_ONE: return CommonImages.getImage(ReviewsImages.MINUS_ONE); case PLUS_ONE: return CommonImages.getImage(ReviewsImages.PLUS_ONE); case PLUS_TWO: return CommonImages.getImage(ReviewsImages.GREEN_CHECK); } return null; }
Source:
<code><START> static MapSettings.Builder builder() { <END> return Sponge.getGame().getRegistry().createBuilder(Builder.class); }</code><technical_language>java static Builder builder() {</technical_language>

Target:
static Builder builder() { return Sponge.getGame().getRegistry().createBuilder(Builder.class); }
Source:
<code>public void makeOnCreationCallbackListBoxMultipleSelect() { setupMultipleSelectTest(CallbackFactory.MAX_VISIBLE_ROWS, true); final Callback callback = CallbackFactory.makeOnCreationCallback(factory, cell, enumLookups); callback.callback(multiValueDOMElement); <START> <END> verifyMultipleSelectTest(CallbackFactory.MAX_VISIBLE_ROWS); }</code><technical_language>chance add verify(multiValueWidget).setVisibleItemCount(eq(CallbackFactory.MAX_VISIBLE_ROWS));</technical_language>

Target:
public void makeOnCreationCallbackListBoxMultipleSelect() { setupMultipleSelectTest(CallbackFactory.MAX_VISIBLE_ROWS, true); final Callback callback = CallbackFactory.makeOnCreationCallback(factory, cell, enumLookups); callback.callback(multiValueDOMElement); verifyMultipleSelectTest(CallbackFactory.MAX_VISIBLE_ROWS); verify(multiValueWidget).setVisibleItemCount(eq(CallbackFactory.MAX_VISIBLE_ROWS)); }
Source:
<code>public CompletableFuture<Void> visitRemoveCommand(InvocationContext ctx, RemoveCommand command) throws Throwable { ctx.onReturn((ctx1, command1, rv, throwable) -> { if (throwable != null) throw throwable; if (!isStoreEnabled(command) || ctx.isInTxScope() || !command.isSuccessful()) return null; if (!isProperWriter(ctx, command, command.getKey())) return null; Object key = command.getKey(); boolean resp = persistenceManager.deleteFromAllStores(key, BOTH); if (trace) getLog().tracef("Removed entry under key %s and got response %s from CacheStore", key, resp); return null; }); <START> return ctx.continueInvocation(); <END> }</code><technical_language>needed onReturn() returns CONTINUE_INVOCATION</technical_language>

Target:
public CompletableFuture<Void> visitRemoveCommand(InvocationContext ctx, RemoveCommand command) throws Throwable { return ctx.onReturn((rCtx, rCommand, rv, throwable) -> { if (throwable != null) throw throwable; RemoveCommand removeCommand = (RemoveCommand) rCommand; if (!isStoreEnabled(removeCommand) || rCtx.isInTxScope() || !removeCommand.isSuccessful()) return null; if (!isProperWriter(rCtx, removeCommand, removeCommand.getKey())) return null; Object key = removeCommand.getKey(); boolean resp = persistenceManager.deleteFromAllStores(key, BOTH); if (trace) getLog().tracef("Removed entry under key %s and got response %s from CacheStore", key, resp); return null; }); }
Source:
<code>public void onResume() { super.onResume(); authWebView.onResume(); <START> swipeContainer.setEnabled(false); <END> if (hasComponentCallback != null) { final CourseComponent component = hasComponentCallback.getComponent(); if (component != null && component.equals(unit)) { authWebView.loadUrl(false, unit.getBlockUrl()); } } }</code><technical_language>disabling swipe container in onResume callback in onCreate callback</technical_language>

Target:
public void onResume() { super.onResume(); authWebView.onResume(); if (hasComponentCallback != null) { final CourseComponent component = hasComponentCallback.getComponent(); if (component != null && component.equals(unit)) { authWebView.loadUrl(false, unit.getBlockUrl()); } } }
Source:
<code>public void process() { PerformanceMonitor.startActivity("rendering/finalPostProcessing"); ocUndistorted = dynamicFBM.getFBO(OC_UNDISTORTED_URN); sceneFinal = dynamicFBM.getFBO(DefaultDynamicFBOs.Final.getResourceUrn()); <START> sceneOpaque = dynamicFBM.getFBO(DefaultDynamicFBOs.ReadOnlyGBuffer.getResourceUrn()); <END> fullScale = sceneOpaque.dimensions(); if (!renderingDebugConfig.isEnabled()) { finalPost.enable(); } else { debug.enable(); } if (!renderingConfig.isOculusVrSupport()) { renderFinalMonoImage(); } else { renderFinalStereoImage(worldRenderer.getCurrentRenderStage()); } PerformanceMonitor.endActivity(); }</code><technical_language>in other nodes DynamicFBM.getFBO obtain default FBOs easily referenced DefaultDynamicFBOs.Final? Of is right require initialization update notified of changes, if necessary</technical_language>

Target:
public void process() { PerformanceMonitor.startActivity("rendering/finalPostProcessing"); ocUndistorted = displayResolutionDependentFBOs.get(OC_UNDISTORTED); fullScale = READ_ONLY_GBUFFER.dimensions(); if (!renderingDebugConfig.isEnabled()) { finalPost.enable(); } else { debug.enable(); } if (!renderingConfig.isOculusVrSupport()) { renderFinalMonoImage(); } else { renderFinalStereoImage(worldRenderer.getCurrentRenderStage()); } PerformanceMonitor.endActivity(); }
Source:
<code>private void processTCD(Attribution attribution, WebResourceImpl wRes) { Map<String, List<String>> creatorMap = new HashMap<>(); <START> if (creatorMap.size() == 0 && isNotBlank(wRes.getDcCreator())) { <END> creatorMap.putAll(createCreatorMap(wRes.getDcCreator())); } checkProxy(((AggregationImpl) wRes.getParentAggregation()).getParentBean().getProxies(), attribution, creatorMap); if (attribution.getTitle().size() == 0 && ArrayUtils.isNotEmpty(((AggregationImpl) wRes.getParentAggregation()).getParentBean().getTitle())) { attribution.getTitle().put("", collectListInLines(Arrays.asList(stripEmptyStrings(((AggregationImpl) wRes.getParentAggregation()).getParentBean().getTitle())))); } checkEuropeanaAggregration(attribution, ((AggregationImpl) wRes.getParentAggregation()).getParentBean().getEuropeanaAggregation(), creatorMap); checkCreatorLabel(attribution, (List<Agent>) ((AggregationImpl) wRes.getParentAggregation()).getParentBean().getAgents(), creatorMap); }</code><technical_language>this is NULL here, need check :)</technical_language>

Target:
private void processTCD(Attribution attribution, WebResourceImpl wRes) { Map<String, List<String>> creatorMap = new HashMap<>(); if (isNotBlank(wRes.getDcCreator())) { creatorMap.putAll(createCreatorMap(wRes.getDcCreator())); } checkProxy(((AggregationImpl) wRes.getParentAggregation()).getParentBean().getProxies(), attribution, creatorMap); if (attribution.getTitle().isEmpty() && ArrayUtils.isNotEmpty( ((AggregationImpl) wRes.getParentAggregation()).getParentBean().getTitle())) { attribution.getTitle().put("", collectListInLines(Arrays.asList( stripEmptyStrings(((AggregationImpl) wRes.getParentAggregation()).getParentBean().getTitle())))); } checkEuropeanaAggregration(attribution, ((AggregationImpl) wRes.getParentAggregation()).getParentBean() .getEuropeanaAggregation(), creatorMap); checkCreatorLabel(attribution, (List<Agent>) ((AggregationImpl) wRes.getParentAggregation()).getParentBean().getAgents(), creatorMap); }
Source:
<code>private DateTime calc(DateTime current, long periodMs) { <START> return new DateTime(getNextTimeAdjustedByDay( <END> current.getMillis(), periodMs, current.getZone()) ); }</code><technical_language>getNextTimeAdjustedByDay -> DateTimeUtils.getNextTimeAdjustedByDay</technical_language>

Target:
private DateTime calc(DateTime current, long periodMs) { return new DateTime(DateTimeUtils.getNextTimeAdjustedByDay( current.getMillis(), periodMs, current.getZone()) ); }
Source:
<code><START> public void testNotLogUserExceptionsAsMissingProperties() { <END> OgnlValueStack vs = createValueStack(); vs.setLogMissingProperties("true"); Dog dog = new Dog(); vs.push(dog); TestAppender testAppender = new TestAppender(); Logger logger = (Logger) LogManager.getLogger(OgnlValueStack.class); logger.addAppender(testAppender); testAppender.start(); try { vs.setValue("exception", "exceptionValue", false); vs.findValue("exception", false); vs.findValue("exception", String.class, false); vs.findValue("getException()", false); vs.findValue("getException()", String.class, false); vs.findValue("bite", false); vs.findValue("bite", void.class, false); vs.findValue("getBite()", false); vs.findValue("getBite()", void.class, false); assertEquals(0, testAppender.logEvents.size()); } finally { testAppender.stop(); logger.removeAppender(testAppender); } }</code><technical_language>@since 2.5.21</technical_language>

Target:
public void testNotLogUserExceptionsAsMissingProperties() { OgnlValueStack vs = createValueStack(); vs.setLogMissingProperties("true"); Dog dog = new Dog(); vs.push(dog); TestAppender testAppender = new TestAppender(); Logger logger = (Logger) LogManager.getLogger(OgnlValueStack.class); logger.addAppender(testAppender); testAppender.start(); try { vs.setValue("exception", "exceptionValue", false); vs.findValue("exception", false); vs.findValue("exception", String.class, false); vs.findValue("getException()", false); vs.findValue("getException()", String.class, false); vs.findValue("bite", false); vs.findValue("bite", void.class, false); vs.findValue("getBite()", false); vs.findValue("getBite()", void.class, false); vs.setLogMissingProperties("false"); vs.setValue("exception", "exceptionValue", false); vs.findValue("exception", false); vs.findValue("exception", String.class, false); vs.findValue("getException()", false); vs.findValue("getException()", String.class, false); vs.findValue("bite", false); vs.findValue("bite", void.class, false); vs.findValue("getBite()", false); vs.findValue("getBite()", void.class, false); assertEquals(0, testAppender.logEvents.size()); } finally { testAppender.stop(); logger.removeAppender(testAppender); } }
Source:
<code><START> public LinkedList<NavigatorLink> getNavigatorLinks(@RequestParam("entity") String entityTypeId) <END> { LinkedList<NavigatorLink> result = new LinkedList<>(); EntityType entityType = dataService.getEntityType(entityTypeId); if (entityType != null) { Package pack = entityType.getPackage(); getNavigatorLinks(result, pack); result.add(NavigatorLink.create(menuReaderService.getMenu().findMenuItemPath(NAVIGATOR) + "/", "glyphicon-home")); Collections.reverse(result); } return result; }</code><technical_language>Intelij Sonar warn impl of List 'LinkedList' suggest 'List'</technical_language>

Target:
public List<NavigatorLink> getNavigatorLinks(@RequestParam("entity") String entityTypeId) { List<NavigatorLink> result = new LinkedList<>(); EntityType entityType = dataService.getEntityType(entityTypeId); String navigatorPath = menuReaderService.getMenu().findMenuItemPath(NAVIGATOR); if (entityType != null) { Package pack = entityType.getPackage(); getNavigatorLinks(result, pack, navigatorPath); result.add(NavigatorLink.create(navigatorPath + "/", "glyphicon-home")); Collections.reverse(result); } return result; }
Source:
<code>public boolean updateChannelInfoFromDatabase() { ChannelDao cd = DaoFactory.getInstance().getChannelDao(context, DaoFactory.DaoType.SQLITE_DAO, null); List<Channel> channelList = cd.findAll(); <START> if (null == channelList) return false; <END> for (Channel aChannel : channelList) { channelMap.put(aChannel.getChannelIdentifier(), aChannel); } return true; }</code><technical_language>i api calls for return empty list if found. this check unnecessary. if want check for empty list do if findall fails</technical_language>

Target:
public boolean updateChannelInfoFromDatabase() { ChannelDao cd = DaoFactory.getInstance().getChannelDao(context, DaoFactory.DaoType.SQLITE_DAO, null); if (null == cd) return false; List<Channel> channelList = cd.findAll(); for (Channel aChannel : channelList) { channelMap.put(aChannel.getChannelIdentifier(), aChannel); } return true; }
Source:
<code>public void process(final KeyFilter filter, final CacheLoaderTask task, Executor executor, boolean fetchValue, boolean fetchMetadata) { scanForUnknownDirectories(); ExecutorAllCompletionService eacs = new ExecutorAllCompletionService(executor); final TaskContextImpl taskContext = new TaskContextImpl(); for (final DirectoryLoaderAdaptor dir : openDirectories.values()) { eacs.submit(new Callable<Void>() { @Override public Void call() throws Exception { try { final HashSet<MarshalledEntry> allInternalEntries = new HashSet<MarshalledEntry>(); dir.loadAllEntries(allInternalEntries, Integer.MAX_VALUE, ctx.getMarshaller()); for (MarshalledEntry me : allInternalEntries) { if (taskContext.isStopped()) break; if (filter == null || filter.shouldLoadKey(me.getKey())) { task.processEntry(me, taskContext); } } return null; } catch (Exception e) { log.errorExecutingParallelStoreTask(e); throw e; } } }); } eacs.waitUntilAllCompleted(); if (eacs.isExceptionThrown()) { <START> throw new CacheLoaderException("Execution exception!"); <END> } }</code><technical_language>above: "couldn't save occurred exception put a exception?"</technical_language>

Target:
public void process(final KeyFilter filter, final CacheLoaderTask task, Executor executor, boolean fetchValue, boolean fetchMetadata) { scanForUnknownDirectories(); ExecutorAllCompletionService eacs = new ExecutorAllCompletionService(executor); final TaskContextImpl taskContext = new TaskContextImpl(); for (final DirectoryLoaderAdaptor dir : openDirectories.values()) { eacs.submit(new Callable<Void>() { @Override public Void call() throws Exception { try { final HashSet<MarshalledEntry> allInternalEntries = new HashSet<MarshalledEntry>(); dir.loadAllEntries(allInternalEntries, Integer.MAX_VALUE, ctx.getMarshaller()); for (MarshalledEntry me : allInternalEntries) { if (taskContext.isStopped()) break; if (filter == null || filter.shouldLoadKey(me.getKey())) { task.processEntry(me, taskContext); } } return null; } catch (Exception e) { log.errorExecutingParallelStoreTask(e); throw e; } } }); } eacs.waitUntilAllCompleted(); if (eacs.isExceptionThrown()) { throw new CacheLoaderException("Execution exception!", eacs.getFirstException()); } }