auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>private int[] recordTravelTimesForTarget (int target, int[] percentileTravelTimesMinutes) { if (percentileTravelTimesMinutes.length != nPercentiles) { throw new ParameterException(percentileTravelTimesMinutes.length + " percentile values supplied; expected" + " " + nPercentiles); } <START> if (travelTimes != null) { <END> travelTimes.setTarget(target, percentileTravelTimesMinutes); } if (calculateAccessibility) { Grid grid = (Grid) accessibilityAccumulator.destinationPointSets[0]; int x = target % grid.width; int y = target / grid.width; double amount = grid.grid[x][y]; for (int p = 0; p < nPercentiles; p++) { if (percentileTravelTimesMinutes[p] < maxTripDurationMinutes) { accessibilityAccumulator.incrementAccessibility(0, 0, p, amount); } } } return percentileTravelTimesMinutes; }</code><technical_language>clearer conditional branches based boolean fields. If switch storing information a non-null place store result, do consistently, if (accessibilityAccumulator != null) instead of if (calculateAccessibility). I form booleans though</technical_language>

Target:
private int[] recordTravelTimesForTarget (int target, int[] percentileTravelTimesMinutes) { if (percentileTravelTimesMinutes.length != nPercentiles) { throw new ParameterException(percentileTravelTimesMinutes.length + " percentile values supplied; expected" + " " + nPercentiles); } if (calculateTravelTimes) { travelTimeResult.setTarget(target, percentileTravelTimesMinutes); } if (calculateAccessibility) { PointSet pointSet = accessibilityResult.destinationPointSets[0]; double amount = pointSet.getOpportunityCount(target); for (int p = 0; p < nPercentiles; p++) { if (percentileTravelTimesMinutes[p] < maxTripDurationMinutes) { accessibilityResult.incrementAccessibility(0, 0, p, amount); } } } return percentileTravelTimesMinutes; }
Source:
<code>Schema(String name, Pattern pathRegex, String path, String documentation, SchemaValidator schemaValidator, Allowance ephemeral, Allowance sequential, Allowance watched, boolean canBeDeleted, Map<String, String> metadata) { Preconditions.checkNotNull(pathRegex, "pathRegex cannot both be null"); <START> Preconditions.checkNotNull(path, "path cannot both be null"); <END> this.pathRegex = pathRegex; this.fixedPath = fixPath(path); this.metadata = ImmutableMap.copyOf(Preconditions.checkNotNull(metadata, "metadata cannot be null")); this.name = Preconditions.checkNotNull(name, "name cannot be null"); this.documentation = Preconditions.checkNotNull(documentation, "documentation cannot be null"); this.schemaValidator = Preconditions.checkNotNull(schemaValidator, "dataValidator cannot be null"); this.ephemeral = Preconditions.checkNotNull(ephemeral, "ephemeral cannot be null"); this.sequential = Preconditions.checkNotNull(sequential, "sequential cannot be null"); this.watched = Preconditions.checkNotNull(watched, "watched cannot be null"); this.canBeDeleted = canBeDeleted; }</code><technical_language>existing check incorrect, returned a boolean null. This change fix it, error messages incorrect. "pathRegex null" "path null" respectively</technical_language>

Target:
Schema(String name, Pattern pathRegex, String path, String documentation, SchemaValidator schemaValidator, Allowance ephemeral, Allowance sequential, Allowance watched, boolean canBeDeleted, Map<String, String> metadata) { Preconditions.checkArgument((pathRegex != null) || (path != null), "pathRegex and path cannot both be null"); this.pathRegex = pathRegex; this.fixedPath = fixPath(path); this.metadata = ImmutableMap.copyOf(Preconditions.checkNotNull(metadata, "metadata cannot be null")); this.name = Preconditions.checkNotNull(name, "name cannot be null"); this.documentation = Preconditions.checkNotNull(documentation, "documentation cannot be null"); this.schemaValidator = Preconditions.checkNotNull(schemaValidator, "dataValidator cannot be null"); this.ephemeral = Preconditions.checkNotNull(ephemeral, "ephemeral cannot be null"); this.sequential = Preconditions.checkNotNull(sequential, "sequential cannot be null"); this.watched = Preconditions.checkNotNull(watched, "watched cannot be null"); this.canBeDeleted = canBeDeleted; }
Source:
<code><START> public boolean contains(ItemGroup ctx, AbstractProject<?,?> job, EnvVars environment) { <END> String expandedJobs = getExpandedJobs(jobs, environment); if(!expandedJobs.contains(job.getName())) return false; String name = job.getFullName(); for (AbstractProject<?, ?> project : getJobList(ctx, environment)) { if (project.getFullName().equals(name)) return true; } return false; }</code><technical_language>If environment accepts null, mark <code>@CheckForNull</code></technical_language>

Target:
public boolean contains(ItemGroup ctx, AbstractProject<?,?> job, EnvVars environment) { String expandedJobs = getExpandedJobs(jobs, environment); if (expandedJobs == null) return false; if(!expandedJobs.contains(job.getName())) return false; String name = job.getFullName(); for (AbstractProject<?, ?> project : getJobList(ctx)) { if (project.getFullName().equals(name)) return true; } return false; }
Source:
<code>private byte[] httpGet(String requestUrl) throws IOException { GetMethod getMethod = new GetMethod(requestUrl); try { int responseCode = httpClient.executeMethod(getMethod); LOG.debug("Received response code for the get request on the url : {}", responseCode, requestUrl); byte[] response = getMethod.getResponseBody(); <START> if (responseCode != 200) { <END> throw new SamzaException(String.format("Received response code: %s for get request on: %s, with message: %s.", responseCode, requestUrl, StringUtils.newStringUtf8(response))); } return response; } finally { getMethod.releaseConnection(); } }</code><technical_language>constant for HTTP OK</technical_language>

Target:
private byte[] httpGet(String requestUrl) throws IOException { GetMethod getMethod = new GetMethod(requestUrl); try { int responseCode = httpClient.executeMethod(getMethod); LOG.debug("Received response code: for the get request on the url: {}", responseCode, requestUrl); byte[] response = getMethod.getResponseBody(); if (responseCode != HttpStatus.SC_OK) { throw new SamzaException(String.format("Received response code: %s for get request on: %s, with message: %s.", responseCode, requestUrl, StringUtils.newStringUtf8(response))); } return response; } finally { getMethod.releaseConnection(); } }
Source:
<code>public void addMapItem(MapItem mItem) { if (mMap != null) { if (MapItem.class.isAssignableFrom(mItem.getClass())) { int type = mItem.getMapItemType(); switch (type) { case 0: break; case MapItem.MARKERTYPE: if (mItem.getMarkerText() != null) { IconGenerator iconGenerator = new IconGenerator(mContext); iconGenerator.setBackground(mContext.getResources().getDrawable(R.drawable.usermarker)); iconGenerator.setTextAppearance(10); Bitmap bitmap = iconGenerator.makeIcon(mItem.getMarkerText()); Marker marker = mMap.addMarker(mItem.getMarkerOptions().icon(BitmapDescriptorFactory.fromBitmap(bitmap))); if (mItem.isDynamic()) { dynamicMarkers.add(marker); } } else { Marker marker; if (mItem.isVehicle()) { marker = mMap.addMarker(mItem.getMarkerOptions().icon(BitmapDescriptorFactory.fromResource(R.drawable.maps_shuttle_indicator))); } else { <START> if (isMapExpanded) { <END> marker = mMap.addMarker(mItem.getMarkerOptions().icon(BitmapDescriptorFactory.fromResource(R.drawable.map_stops))); } else { marker = mMap.addMarker(mItem.getMarkerOptions().icon(BitmapDescriptorFactory.fromResource(R.drawable.map_pin))); } } if (mItem.isDynamic()) { dynamicMarkers.add(marker); } else { staticMarkers.add(marker); } } break; case MapItem.POLYLINETYPE: Polyline polyline = mMap.addPolyline(mItem.getPolylineOptions()); if (mItem.isDynamic()) { dynamicLines.add(polyline); } break; case MapItem.POLYGONTYPE: Polygon polygon = mMap.addPolygon(mItem.getPolygonOptions()); if (mItem.isDynamic()) { dynamicPolygons.add(polygon); } break; } } } }</code><technical_language>this !isMapExpanded</technical_language>

Target:
public void addMapItem(MapItem mItem) { if (mMap != null) { if (MapItem.class.isAssignableFrom(mItem.getClass())) { int type = mItem.getMapItemType(); switch (type) { case 0: break; case MapItem.MARKERTYPE: if (mItem.getMarkerText() != null) { IconGenerator iconGenerator = new IconGenerator(mContext); iconGenerator.setBackground(mContext.getResources().getDrawable(R.drawable.usermarker)); iconGenerator.setTextAppearance(10); Bitmap bitmap = iconGenerator.makeIcon(mItem.getMarkerText()); Marker marker = mMap.addMarker(mItem.getMarkerOptions().icon(BitmapDescriptorFactory.fromBitmap(bitmap))); if (mItem.isDynamic()) { dynamicMarkers.add(marker); } } else { Marker marker; if (mItem.isVehicle()) { marker = mMap.addMarker(mItem.getMarkerOptions().icon(BitmapDescriptorFactory.fromResource(R.drawable.maps_shuttle_indicator))); } else { if (!isMapExpanded) { marker = mMap.addMarker(mItem.getMarkerOptions().icon(BitmapDescriptorFactory.fromResource(R.drawable.map_stops))); } else { marker = mMap.addMarker(mItem.getMarkerOptions().icon(BitmapDescriptorFactory.fromResource(R.drawable.map_pin))); } } if (mItem.isDynamic()) { dynamicMarkers.add(marker); } else { staticMarkers.add(marker); } } break; case MapItem.POLYLINETYPE: Polyline polyline = mMap.addPolyline(mItem.getPolylineOptions()); if (mItem.isDynamic()) { dynamicLines.add(polyline); } break; case MapItem.POLYGONTYPE: Polygon polygon = mMap.addPolygon(mItem.getPolygonOptions()); if (mItem.isDynamic()) { dynamicPolygons.add(polygon); } break; } } } }
Source:
<code>public Credential changePassword(Credential password, ChangePassword changePassword, Policy policy) { if (policy.isOption(Policy.AUTHORIZED_FOR_ALL_ACCOUNTS) || transformationService.compare(changePassword.getOldSecret(), password.getSecretValue())){ password.setSecretValue(transformationService.transform(changePassword.getNewSecret(), EncryptionConstants.HASH)); password = objectManager.persist(password); <START> validatePassword(changePassword.getNewSecret()); <END> return password; } throw new ClientVisibleException(ResponseCodes.BAD_REQUEST, "InvalidOldPassword"); }</code><technical_language>validatePassword() done persisting password DB, do check in caller instead of Dao</technical_language>

Target:
public Credential changePassword(Credential password, ChangePassword changePassword, Policy policy) { if (policy.isOption(Policy.AUTHORIZED_FOR_ALL_ACCOUNTS) || transformationService.compare(changePassword.getOldSecret(), password.getSecretValue())){ password.setSecretValue(transformationService.transform(changePassword.getNewSecret(), EncryptionConstants.HASH)); password = objectManager.persist(password); return password; } throw new ClientVisibleException(ResponseCodes.BAD_REQUEST, "InvalidOldPassword"); }
Source:
<code>public void testLibraryAliquotTableWarnings() { testLibraryAliquotTableWarningOnPoolWithError(801L, "(MISSING INDEX)"); <START> testLibraryAliquotTableWarningOnPoolWithError(802L, "(NEAR-DUPLICATE INDICES)"); <END> testLibraryAliquotTableWarningOnPoolWithError(803L, "DUPLICATE INDICES"); }</code><technical_language>This changed lower-case in some places, others</technical_language>

Target:
public void testLibraryAliquotTableWarnings() { testLibraryAliquotTableWarningOnPoolWithError(801L, "MISSING INDEX"); testLibraryAliquotTableWarningOnPoolWithError(802L, "Near-Duplicate Indices"); testLibraryAliquotTableWarningOnPoolWithError(803L, "DUPLICATE INDICES"); }
Source:
<code>private String processColorDefinition(State state, List<ColorArray> colorList) { if(colorList == null) return null; if(colorList.size() == 0) return null; String colorString = null; if(colorList.size() == 1 && colorList.get(0).getState() == null) <START> colorString = colorList.get(0).getArg(); <END> else { for (ColorArray color : colorList) { State cmpState = state; if(color.getState() == null) { logger.error("Error parsing color"); continue; } if(color.getItem() != null) { Item item; try { item = itemRegistry.getItem(color.getItem()); cmpState = item.getState(); } catch (ItemNotFoundException e) { logger.error("Cannot retrieve color item for widget", color.getItem()); } } String value; if(color.getSign() != null) value = color.getSign() + color.getState(); else value = color.getState(); if (matchStateToValue(cmpState, value, color.getCondition()) == true) { colorString = color.getArg(); break; } } } if(colorString == null) return null; if(colorString.startsWith("\"") && colorString.endsWith("\"")) colorString = colorString.substring(1, colorString.length()-1); return colorString; }</code><technical_language>add curly braces please</technical_language>

Target:
private String processColorDefinition(State state, List<ColorArray> colorList) { if(colorList == null) { return null; } if(colorList.size() == 0) { return null; } String colorString = null; if(colorList.size() == 1 && colorList.get(0).getState() == null) { colorString = colorList.get(0).getArg(); } else { for (ColorArray color : colorList) { State cmpState = state; if(color.getState() == null) { logger.error("Error parsing color"); continue; } if(color.getItem() != null) { Item item; try { item = itemRegistry.getItem(color.getItem()); cmpState = item.getState(); } catch (ItemNotFoundException e) { logger.warn("Cannot retrieve color item for widget", color.getItem()); } } String value; if(color.getSign() != null) value = color.getSign() + color.getState(); else value = color.getState(); if (matchStateToValue(cmpState, value, color.getCondition()) == true) { colorString = color.getArg(); break; } } } if(colorString == null) return null; if(colorString.startsWith("\"") && colorString.endsWith("\"")) colorString = colorString.substring(1, colorString.length()-1); return colorString; }
Source:
<code><START> public Task<T> batchable(final String desc, final K key) { <END> return Task.async(desc, ctx -> { final SettablePromise<T> result = Promises.settable(); Long planId = ctx.getPlanId(); BatchBuilder<K, T> builder = _batches.get(planId); if (builder == null) { builder = Batch.builder(); BatchBuilder<K, T> existingBuilder = _batches.putIfAbsent(planId, builder); if (existingBuilder != null) { builder = existingBuilder; } } builder.add(key, ctx.getShallowTraceBuilder(), result); return result; }); }</code><technical_language>a batchable(final key) method generates a simple desc for you? "Task for id $key"</technical_language>

Target:
public Task<T> batchable(final String desc, final K key) { return Task.async("batch: " + desc, ctx -> { final SettablePromise<T> result = Promises.settable(); Long planId = ctx.getPlanId(); BatchBuilder<K, T> builder = _batches.get(planId); if (builder == null) { builder = Batch.builder(); BatchBuilder<K, T> existingBuilder = _batches.putIfAbsent(planId, builder); if (existingBuilder != null) { builder = existingBuilder; } } builder.add(key, ctx.getShallowTraceBuilder(), result); return result; }); }
Source:
<code>public void testMergeSegments_BothClaimedByProcessor() throws Exception { initProcessor(TrackingEventProcessorConfiguration.forParallelProcessing(2)); tokenStore.initializeTokenSegments(testSubject.getName(), 2); <START> List<EventMessage<?>> handledEvents = new CopyOnWriteArrayList<>(); <END> when(mockHandler.handle(any())).thenAnswer(i -> handledEvents.add(i.getArgument(0))); publishEvents(10); testSubject.start(); waitForActiveThreads(2); assertTrue("Expected merge to succeed", testSubject.mergeSegment(0).join()); assertArrayEquals(new int[]{0}, tokenStore.fetchSegments(testSubject.getName())); waitForSegmentStart(0); while (!testSubject.processingStatus().get(0).isCaughtUp()) { Thread.sleep(10); } assertWithin(5, TimeUnit.SECONDS, () -> assertEquals(10, handledEvents.size())); Thread.sleep(100); assertEquals(10, handledEvents.size()); }</code><technical_language>suggest Set event handled twice</technical_language>

Target:
public void testMergeSegments_BothClaimedByProcessor() throws Exception { initProcessor(TrackingEventProcessorConfiguration.forParallelProcessing(2)); tokenStore.initializeTokenSegments(testSubject.getName(), 2); List<EventMessage<?>> handledEvents = new CopyOnWriteArrayList<>(); when(mockHandler.handle(any())).thenAnswer(i -> handledEvents.add(i.getArgument(0))); publishEvents(10); testSubject.start(); waitForActiveThreads(2); assertTrue("Expected merge to succeed", testSubject.mergeSegment(0).join()); assertArrayEquals(new int[]{0}, tokenStore.fetchSegments(testSubject.getName())); waitForSegmentStart(0); while (!testSubject.processingStatus().get(0).isCaughtUp()) { Thread.sleep(10); } assertWithin(5, TimeUnit.SECONDS, () -> assertEquals("Expected all 10 messages to be handled", 10, handledEvents.stream().map(EventMessage::getIdentifier).distinct().count())); Thread.sleep(100); assertEquals("Number of handler invocations doesn't match number of unique events", 10, handledEvents.size()); }
Source:
<code>private synchronized void reloadData() { _allDatastreamTasks = new HashMap<>(); List<String> connectorTypes = _zkclient.getChildren(_path); connectorTypes.removeAll(Arrays.asList("datastream", "instances", "liveinstances")); connectorTypes.forEach(connectorType -> { if (!_allDatastreamTasks.containsKey(connectorType)) { _allDatastreamTasks.put(connectorType, new ArrayList<>()); } String path = KeyBuilder.connector(_cluster, connectorType); List<String> tasksForConnector = _zkclient.getChildren(path); tasksForConnector.forEach(taskName -> { String p = KeyBuilder.connectorTask(_cluster, connectorType, taskName); String content = _zkclient.ensureReadData(p); <START> LOG.info("@@@ content: " + content); <END> DatastreamTask t = DatastreamTask.fromJson(content); _allDatastreamTasks.get(connectorType).add(t); }); }); }</code><technical_language>this debug level</technical_language>

Target:
private synchronized void reloadData() { _allDatastreamTasks = new HashMap<>(); List<String> connectorTypes = _zkclient.getChildren(_path); connectorTypes.forEach(connectorType -> { if (!_allDatastreamTasks.containsKey(connectorType)) { _allDatastreamTasks.put(connectorType, new ArrayList<>()); } String path = KeyBuilder.connector(_cluster, connectorType); List<String> tasksForConnector = _zkclient.getChildren(path); tasksForConnector.forEach(taskName -> { String p = KeyBuilder.connectorTask(_cluster, connectorType, taskName); String content = _zkclient.ensureReadData(p); DatastreamTask t = DatastreamTask.fromJson(content); _allDatastreamTasks.get(connectorType).add(t); }); }); }
Source:
<code>public void testSatisfiedThenLostNetworkRequestDoesNotTriggerOnUnavailable() { NetworkRequest nr = new NetworkRequest.Builder().addTransportType( NetworkCapabilities.TRANSPORT_WIFI).build(); final TestNetworkCallback networkCallback = new TestNetworkCallback(); final int requestTimeoutMs = 50; mCm.requestNetwork(nr, networkCallback, requestTimeoutMs); mWiFiNetworkAgent = new MockNetworkAgent(TRANSPORT_WIFI); mWiFiNetworkAgent.connect(false); final int assertTimeoutMs = requestTimeoutMs + TEST_CALLBACK_TIMEOUT_MS; networkCallback.expectAvailableCallbacks(mWiFiNetworkAgent, false, false, <START> assertTimeoutMs); <END> mWiFiNetworkAgent.disconnect(); networkCallback.expectCallback(CallbackState.LOST, mWiFiNetworkAgent); networkCallback.assertNoCallback(); }</code><technical_language>question, TEST_CALLBACK_TIMEOUT_MS</technical_language>

Target:
public void testSatisfiedThenLostNetworkRequestDoesNotTriggerOnUnavailable() { NetworkRequest nr = new NetworkRequest.Builder().addTransportType( NetworkCapabilities.TRANSPORT_WIFI).build(); final TestNetworkCallback networkCallback = new TestNetworkCallback(); final int requestTimeoutMs = 50; mCm.requestNetwork(nr, networkCallback, requestTimeoutMs); mWiFiNetworkAgent = new MockNetworkAgent(TRANSPORT_WIFI); mWiFiNetworkAgent.connect(false); networkCallback.expectAvailableCallbacks(mWiFiNetworkAgent, false, false, TEST_CALLBACK_TIMEOUT_MS); mWiFiNetworkAgent.disconnect(); networkCallback.expectCallback(CallbackState.LOST, mWiFiNetworkAgent); networkCallback.assertNoCallback(); }
Source:
<code>public static void main(String[] args) throws IOException, NoSuchAlgorithmException, InvalidKeyException, IllegalArgumentException { try { MinioClient minioClient = new MinioClient( "<LINK_0>", "Q3AM3UQ867SPQQA43P2F", "zuf+tfteSlswRu7BJ86wekitnifILbZam1KYY3TG"); minioClient.disableObjectLegalHold( DisableObjectLegalHoldArgs.builder() .bucket("my-bucketname") .object("my-objectname") .build()); System.out.println( " Is object legal hold is enabled " <START> + minioClient.isObjectLegalHoldEnabled( <END> IsObjectLegalHoldEnabledArgs.builder() .bucket("my-bucketname") .object("my-objectname") .versionId("object-versionId") .build())); } catch (MinioException e) { System.out.println("Error occurred: " + e); } }</code><technical_language>supposes error out if disabled</technical_language>

Target:
public static void main(String[] args) throws IOException, NoSuchAlgorithmException, InvalidKeyException, IllegalArgumentException { try { MinioClient minioClient = new MinioClient( "<LINK_0>", "Q3AM3UQ867SPQQA43P2F", "zuf+tfteSlswRu7BJ86wekitnifILbZam1KYY3TG"); minioClient.disableObjectLegalHold( DisableObjectLegalHoldArgs.builder() .bucket("my-bucketname") .object("my-objectname") .build()); System.out.println("Legal hold disabled on object successfully "); } catch (MinioException e) { System.out.println("Error occurred: " + e); } }
Source:
<code><START> protected void setSource(ClosableObservable<T1> source) { <END> cancelSubscription(); closeSource(); this.source = source; sourceObserver.onConnectionStatus(false); sourceObserver.onConnectionStatus(source.isConnected()); T1 value = source.getValue(); if (value != null) sourceObserver.onValue(value); Exception error = source.currentError(); if (error != null) sourceObserver.onError(error); sourceSubscription = source.addObserver(sourceObserver); }</code><technical_language>this synchronized? previous implementation in ForwardingObservable was</technical_language>

Target:
protected synchronized void setSource(ClosableObservable<T1> source) { cancelSubscription(); closeSource(); this.source = source; sourceObserver.onConnectionStatus(false); sourceObserver.onConnectionStatus(source.isConnected()); T1 value = source.getValue(); if (value != null) sourceObserver.onValue(value); Exception error = source.currentError(); if (error != null) sourceObserver.onError(error); sourceSubscription = source.addObserver(sourceObserver); }
Source:
<code>IterableResponse<EventData> receive(int maximumMessageCount) { <START> return new IterableResponse<>(Flux.empty()); <END> }</code><technical_language>ignorance of language specifics, stubbed out implemented. Is intentional</technical_language>

Target:
public IterableResponse<EventData> receive(int maximumMessageCount) { return new IterableResponse<>(Flux.empty()); }
Source:
<code>public void testIsSolidTumor() throws Exception { TumorType tumorType = new TumorType(); MainType mainType = new MainType(); tumorType.setMainType(mainType); <START> assertFalse("Empty tumor type is not solid tumor", TumorTypeUtils.isSolidTumor(tumorType)); <END> mainType.setName("Melanoma"); assertTrue("Melanoma is solid tumor", TumorTypeUtils.isSolidTumor(tumorType)); mainType.setName("Leukemia"); assertFalse("Leukemia is not solid tumor", TumorTypeUtils.isSolidTumor(tumorType)); mainType.setName("Leukemia"); tumorType.setTissue("EYE"); assertTrue("Tissue has higher priority than main type", TumorTypeUtils.isSolidTumor(tumorType)); }</code><technical_language>message printed out is of printed out</technical_language>

Target:
public void testIsSolidTumor() throws Exception { TumorType tumorType = new TumorType(); MainType mainType = new MainType(); tumorType.setMainType(mainType); assertFalse("Empty tumor type is solid tumor, but it should not be.", TumorTypeUtils.isSolidTumor(tumorType)); mainType.setName("Melanoma"); assertTrue("Melanoma is not solid tumor, but it should be.", TumorTypeUtils.isSolidTumor(tumorType)); mainType.setName("Leukemia"); assertFalse("Leukemia is solid tumor, but it should not be.", TumorTypeUtils.isSolidTumor(tumorType)); mainType.setName("Leukemia"); tumorType.setTissue("EYE"); assertTrue("Tissue does not have higher priority than main type, but it should have.", TumorTypeUtils.isSolidTumor(tumorType)); }
Source:
<code>public void subtract(final AbstractHistogram otherHistogram) throws ArrayIndexOutOfBoundsException, IllegalArgumentException { if (highestEquivalentValue(otherHistogram.getMaxValue()) > <START> highestEquivalentValue(valueFromIndex(this.countsArrayLength - 1))) { <END> throw new ArrayIndexOutOfBoundsException( "The other histogram includes values that do not fit in this histogram's range."); } for (int i = 0; i < otherHistogram.countsArrayLength; i++) { long otherCount = otherHistogram.getCountAtIndex(i); if (otherCount > 0) { long otherValue = otherHistogram.valueFromIndex(i); if (getCountAtValue(otherValue) < otherCount) { throw new IllegalArgumentException("otherHistogram count (" + otherCount + ") at value " + otherValue + " is larger than this one's (" + getCountAtValue(otherValue) + ")"); } recordValueWithCount(otherValue, -otherCount); } } if ((getCountAtValue(getMaxValue()) <= 0) || getCountAtValue(getMinNonZeroValue()) <= 0) { establishInternalTackingValues(); } }</code><technical_language>tests ok. If otherHistogram values this.getMaxValue() highestEquivalentValue(valueFromIndex(this.countsArrayLength - 1), values throw IllegalArgumentException in this code anyway. switch AIOOB IllegalArgumentException here? exception is thrown consistently..</technical_language>

Target:
public void subtract(final AbstractHistogram otherHistogram) throws ArrayIndexOutOfBoundsException, IllegalArgumentException { if (highestEquivalentValue(otherHistogram.getMaxValue()) > highestEquivalentValue(valueFromIndex(this.countsArrayLength - 1))) { throw new IllegalArgumentException( "The other histogram includes values that do not fit in this histogram's range."); } for (int i = 0; i < otherHistogram.countsArrayLength; i++) { long otherCount = otherHistogram.getCountAtIndex(i); if (otherCount > 0) { long otherValue = otherHistogram.valueFromIndex(i); if (getCountAtValue(otherValue) < otherCount) { throw new IllegalArgumentException("otherHistogram count (" + otherCount + ") at value " + otherValue + " is larger than this one's (" + getCountAtValue(otherValue) + ")"); } recordValueWithCount(otherValue, -otherCount); } } if ((getCountAtValue(getMaxValue()) <= 0) || getCountAtValue(getMinNonZeroValue()) <= 0) { establishInternalTackingValues(); } }
Source:
<code><START> public GHDiscussion wrapUp(GHTeam team) throws IOException { this.root = team.root; this.organization = team.getOrganization(); <END> this.team = team; return this; }</code><technical_language>suggestion public GHDiscussion wrapUp(GHTeam team) throws IOException { this.organization = team.getOrganization();</technical_language>

Target:
GHDiscussion wrapUp(GHTeam team) { this.team = team; return this; }
Source:
<code>public void testCountsBySpan(String acc, List<Chunk> chunks, int numberAlignments, int numberUnalignments) { if (!SRAAccession.isSupported()) return; SamReader reader = SamReaderFactory.make().validationStringency(ValidationStringency.SILENT).open( SamInputResource.of(new SRAAccession(acc)) ); final SAMRecordIterator samRecordIterator = ((SamReader.Indexing) reader).iterator(new BAMFileSpan(chunks)); int countAlignments = 0, countUnalignments = 0; while (true) { boolean hasRecord = samRecordIterator.hasNext(); SAMRecord record = null; try { record = samRecordIterator.next(); Assert.assertTrue(hasRecord); } catch (NoSuchElementException e) { Assert.assertFalse(hasRecord); } Assert.assertEquals(hasRecord, record != null); if (record == null) { break; } if (record.getReadUnmappedFlag()) { countUnalignments++; } else { countAlignments++; } <START> } <END> Assert.assertEquals(numberAlignments, countAlignments); Assert.assertEquals(numberUnalignments, countUnalignments); }</code><technical_language>This loop is repeated times in this test class -- extract a shared helper method takes expected counts arguments</technical_language>

Target:
public void testCountsBySpan(String acc, List<Chunk> chunks, int numberAlignments, int numberUnalignments) { if (!SRAAccession.isSupported()) return; SamReader reader = SamReaderFactory.make().validationStringency(ValidationStringency.SILENT).open( SamInputResource.of(new SRAAccession(acc)) ); final SAMRecordIterator samRecordIterator = ((SamReader.Indexing) reader).iterator(new BAMFileSpan(chunks)); checkAlignedUnalignedCountsByIterator(samRecordIterator, numberAlignments, numberUnalignments); }
Source:
<code>public void should_reload_profile_config_when_reloading_config() throws Exception { String query = "mockquery"; AtomicReference<String> configSource = new AtomicReference<>(""); DefaultDriverConfigLoader loader = new DefaultDriverConfigLoader( () -> <START> ConfigFactory.parseString( <END> "profiles.slow.request.consistency = ONE\nconfig-reload-interval = 2s\n" + configSource.get()) .withFallback(DEFAULT_CONFIG_SUPPLIER.get()), CoreDriverOption.values()); try (Cluster configCluster = Cluster.builder() .withConfigLoader(loader) .addContactPoints(simulacron.getContactPoints()) .build()) { simulacron.cluster().prime(when(query).then(noRows()).delay(1, TimeUnit.SECONDS)); Session session = configCluster.connect(); try { session.execute(SimpleStatement.builder(query).withConfigProfileName("slow").build()); fail("Expected DriverTimeoutException"); } catch (DriverTimeoutException e) { } configSource.set("profiles.slow.request.timeout = 10s"); TimeUnit.SECONDS.sleep(3); session.execute(SimpleStatement.builder(query).withConfigProfileName("slow").build()); } }</code><technical_language>I need check I allow this, in theory possible</technical_language>

Target:
public void should_reload_profile_config_when_reloading_config() throws Exception { String query = "mockquery"; AtomicReference<String> configSource = new AtomicReference<>(""); DefaultDriverConfigLoader loader = new DefaultDriverConfigLoader( () -> ConfigFactory.parseString( "profiles.slow.request.consistency = ONE\nconfig-reload-interval = 2s\n" + configSource.get()) .withFallback(DEFAULT_CONFIG_SUPPLIER.get()), CoreDriverOption.values()); try (Cluster configCluster = Cluster.builder() .withConfigLoader(loader) .addContactPoints(simulacron.getContactPoints()) .build()) { simulacron.cluster().prime(when(query).then(noRows()).delay(1, TimeUnit.SECONDS)); Session session = configCluster.connect(); try { session.execute(SimpleStatement.builder(query).withConfigProfileName("slow").build()); fail("Expected DriverTimeoutException"); } catch (DriverTimeoutException e) { } configSource.set("profiles.slow.request.timeout = 10s"); waitForConfigChange(configCluster, 3, TimeUnit.SECONDS); session.execute(SimpleStatement.builder(query).withConfigProfileName("slow").build()); } }
Source:
<code>public static String debugHttpResponse(HttpResponse response) throws ClientException { if (isHttpDebug) { String protocol = response.getSysUrl().split("://")[0].toUpperCase() + "/1.1"; StringBuilder debugString = new StringBuilder("< " + protocol + " " + response.getStatus() + "\n< "); <START> Map<String, String> reasponseHeaders = response.getSysHeaders(); <END> for (Entry<String, String> entry : reasponseHeaders.entrySet()) { debugString.append(entry.getKey() + " : " + entry.getValue() + "\n< "); } if (isHttpContentDebug) { debugString.append("\n" + response.getHttpContentString()); } log.info("\n" + debugString); return debugString.toString(); } else { return null; } }</code><technical_language>reasponseHeaders -> responseHeaders</technical_language>

Target:
public static String debugHttpResponse(HttpResponse response) throws ClientException { if (isHttpDebug) { String protocol = response.getSysUrl().split("://")[0].toUpperCase() + "/1.1"; StringBuilder debugString = new StringBuilder("< " + protocol + " " + response.getStatus() + "\n< "); Map<String, String> responseHeaders = response.getSysHeaders(); for (Entry<String, String> entry : responseHeaders.entrySet()) { debugString.append(entry.getKey() + " : " + entry.getValue() + "\n< "); } if (isHttpContentDebug) { debugString.append("\n" + response.getHttpContentString()); } log.info("\n" + debugString); return debugString.toString(); } else { return null; } }
Source:
<code>protected void configureServlets() { filter("/*").through(Key.get(CacheControlFilter.class)); bind(Key.get(CacheControlFilter.class)).in(SINGLETON); serve("/").with(HostPageServlet.class); serve("/Gerrit").with(LegacyGerritServlet.class); serve("/Gerrit/*").with(legacyGerritScreen()); serve("/cat/*").with(CatServlet.class); serve("/logout").with(HttpLogoutServlet.class); serve("/query").with(ChangeQueryServlet.class); serve("/signout").with(HttpLogoutServlet.class); serve("/ssh_info").with(SshInfoServlet.class); serve("/static/*").with(StaticServlet.class); serve("/tools/*").with(ToolServlet.class); filter("/p/*").through(ProjectAccessPathFilter.class); <START> if (authConfig.isTrustContainerAuth()) <END> filter("/p/*").through(ContainerAuthFilter.class); else filter("/p/*").through(ProjectDigestFilter.class); serve("/p/*").with(ProjectServlet.class); serve("/Main.class").with(notFound()); serve("/com/google/gerrit/launcher/*").with(notFound()); serve("/servlet/*").with(notFound()); serve("/all").with(query("status:merged")); serve("/mine").with(screen(PageLinks.MINE)); serve("/open").with(query("status:open")); serve("/watched").with(query("is:watched status:open")); serve("/starred").with(query("is:starred")); serveRegex("^/settings/?$").with(screen(PageLinks.SETTINGS)); serveRegex("^/register/?$").with(screen(PageLinks.REGISTER + "/")); serveRegex("^/([1-9][0-9]*)/?$").with(directChangeById()); serveRegex("^/r/(.+)/?$").with(DirectChangeByCommit.class); }</code><technical_language>Style Nit: braces for if/else</technical_language>

Target:
protected void configureServlets() { filter("/*").through(Key.get(CacheControlFilter.class)); bind(Key.get(CacheControlFilter.class)).in(SINGLETON); serve("/").with(HostPageServlet.class); serve("/Gerrit").with(LegacyGerritServlet.class); serve("/Gerrit/*").with(legacyGerritScreen()); serve("/cat/*").with(CatServlet.class); serve("/logout").with(HttpLogoutServlet.class); serve("/query").with(ChangeQueryServlet.class); serve("/signout").with(HttpLogoutServlet.class); serve("/ssh_info").with(SshInfoServlet.class); serve("/static/*").with(StaticServlet.class); serve("/tools/*").with(ToolServlet.class); filter("/p/*").through(ProjectAccessPathFilter.class); if (authConfig.isTrustContainerAuth()) { filter("/p/*").through(ContainerAuthFilter.class); } else { filter("/p/*").through(ProjectDigestFilter.class); } serve("/p/*").with(ProjectServlet.class); serve("/Main.class").with(notFound()); serve("/com/google/gerrit/launcher/*").with(notFound()); serve("/servlet/*").with(notFound()); serve("/all").with(query("status:merged")); serve("/mine").with(screen(PageLinks.MINE)); serve("/open").with(query("status:open")); serve("/watched").with(query("is:watched status:open")); serve("/starred").with(query("is:starred")); serveRegex("^/settings/?$").with(screen(PageLinks.SETTINGS)); serveRegex("^/register/?$").with(screen(PageLinks.REGISTER + "/")); serveRegex("^/([1-9][0-9]*)/?$").with(directChangeById()); serveRegex("^/r/(.+)/?$").with(DirectChangeByCommit.class); }
Source:
<code><START> boolean verify(char[] guess) throws InvalidKeyException { <END> byte[] guessedHashA1 = new byte[guess.length]; for (int i = 0; i < guess.length; i++) { guessedHashA1[i] = (byte)(0xff & guess[i]); } return verify(guessedHashA1); }</code><technical_language>Do want assume Latin-1</technical_language>

Target:
boolean verify(char[] guess) throws InvalidKeyException { if (guess == null) { throw new InvalidKeyException("Guess cannot be null"); } byte[] guessedHashA1 = (utf8Encoded ? new String(guess).getBytes(Charsets.UTF_8) : new String(guess).getBytes(Charsets.LATIN_1)); return verify(guessedHashA1); }
Source:
<code><START> @Override @Before public void setUp() throws Exception { <END> init(); }</code><technical_language>need put before/after annotation class is extended TestCase, simply remove inheritance annotation, I favor annotations inheritance in test. remember extended place</technical_language>

Target:
@Before public void setUp() { init(); }
Source:
<code>public static void loadValue(ITimeGraphPresentationProvider presentationProvider, StateItem stateItem) { IPreferenceStore store = getStore(); String rgbKey = getRgbKey(presentationProvider, stateItem); String widthKey = getWidthKey(presentationProvider, stateItem); Map<String, Object> styleMap = stateItem.getStyleMap(); RGB defaultRgb = stateItem.getStateColor(); <START> store.setDefault(rgbKey, RGBAUtil.fromRGBA(new RGBA(defaultRgb.red, defaultRgb.green, defaultRgb.blue, 255))); <END> int rgbColor = store.getInt(rgbKey); float defaultHeight = isLink(stateItem) ? 0.1f : 1.0f; float defaultWidth = (float) stateItem.getStyleMap().getOrDefault(ITimeEventStyleStrings.heightFactor(), defaultHeight); store.setDefault(widthKey, defaultWidth); float width = store.getFloat(widthKey); styleMap.put(ITimeEventStyleStrings.fillColor(), rgbColor); styleMap.put(ITimeEventStyleStrings.heightFactor(), width); }</code><technical_language>This sets default current opening legend. prevents store deleting preference is reset original default, is a change (wrong) default. I understand, if set a in store is equal default, preference is simply erased</technical_language>

Target:
public static void loadValue(ITimeGraphPresentationProvider presentationProvider, StateItem stateItem) { IPreferenceStore store = getStore(); String rgbKey = getRgbKey(presentationProvider, stateItem); String widthKey = getWidthKey(presentationProvider, stateItem); Map<String, Object> styleMap = stateItem.getStyleMap(); RGB defaultRgb = stateItem.getStateColor(); String defaultString = store.getDefaultString(rgbKey); if(defaultString.isEmpty()) { store.setDefault(rgbKey, new RGBAColor(defaultRgb.red, defaultRgb.green, defaultRgb.blue, 255).toString()); } int rgbColor = store.getInt(rgbKey); float defaultHeight = isLink(stateItem) ? 0.1f : 1.0f; float defaultWidth = (float) stateItem.getStyleMap().getOrDefault(ITimeEventStyleStrings.heightFactor(), defaultHeight); store.setDefault(widthKey, defaultWidth); float width = store.getFloat(widthKey); styleMap.put(ITimeEventStyleStrings.fillColor(), rgbColor); styleMap.put(ITimeEventStyleStrings.heightFactor(), width); }
Source:
<code>public void shouldMapIntegerSetToStringSet() { Source source = new Source(); <START> source.setAnotherIntegerSet( new HashSet<Integer>( Arrays.asList( 1, 2 ) ).stream() ); <END> Target target = SourceTargetMapper.INSTANCE.sourceToTarget( source ); assertThat( target ).isNotNull(); assertThat( target.getAnotherStringSet() ).containsOnly( "1", "2" ); }</code><technical_language>setAnotherIntegerSet=setAnotherIntegerStream (it a common issue in this test)</technical_language>

Target:
public void shouldMapIntegerSetToStringSet() { Source source = new Source(); source.setAnotherIntegerStream( new HashSet<Integer>( Arrays.asList( 1, 2 ) ).stream() ); Target target = SourceTargetMapper.INSTANCE.sourceToTarget( source ); assertThat( target ).isNotNull(); assertThat( target.getAnotherStringSet() ).containsOnly( "1", "2" ); }
Source:
<code>public void testBaseOutput() { String[] args = new String[]{ <START> "I=" + TEST_INPUT_VCF1.getAbsolutePath(), <END> "O=" + TEST_OUTPUT, "G=" + TEST_GENOTYPES_VCF1.getAbsolutePath(), "H=" + SUBSETTED_HAPLOTYPE_DATABASE_FOR_TESTING.getAbsolutePath() }; Assert.assertEquals(runPicardCommandLine(args), 0); Assert.assertTrue(MetricsFile.areMetricsEqual(new File(TEST_OUTPUT + ".fingerprinting_summary_metrics"), RESULT_EXAMPLE_SUMMARY)); Assert.assertTrue(MetricsFile.areMetricsEqual(new File(TEST_OUTPUT + ".fingerprinting_detail_metrics"), RESULT_EXAMPLE_DETAIL)); }</code><technical_language>is this necessary? move getAbsolutePath() TEST_INPUT_VCF1 is declared</technical_language>

Target:
public void testBaseOutput() { String[] args = new String[]{ "I=" + TEST_INPUT_VCF1, "O=" + TEST_OUTPUT, "G=" + TEST_GENOTYPES_VCF1, "H=" + SUBSETTED_HAPLOTYPE_DATABASE_FOR_TESTING }; Assert.assertEquals(runPicardCommandLine(args), 0); Assert.assertTrue(MetricsFile.areMetricsEqual(new File(TEST_OUTPUT + ".fingerprinting_summary_metrics"), RESULT_EXAMPLE_SUMMARY)); Assert.assertTrue(MetricsFile.areMetricsEqual(new File(TEST_OUTPUT + ".fingerprinting_detail_metrics"), RESULT_EXAMPLE_DETAIL)); }
Source:
<code>public Date getLogTime() { <START> return this.logTime; <END> }</code><technical_language>For immutable object getLogTime exposes mutable state. constructor accepts mutable Data state of ActualMeterReads</technical_language>

Target:
public Date getLogTime() { return new Date(this.logTime.getTime()); }
Source:
<code>VolumeChooser getDelegateChooser(VolumeChooserEnvironment env) { switch (env.getScope()) { <START> case TABLE: return getVolumeChooserForTable(env); default: return getVolumeChooserForScope(env); } <END> }</code><technical_language>this point, switch statement (here in other class) turned a simple conditional: java if (env.getScope() == ChooserScope.TABLE) { return getVolumeChooserForTable(env); } return getVolumeChooserForScope(env); This style also... work fine, conditional is slightly shorter easier read, choice is binary</technical_language>

Target:
VolumeChooser getDelegateChooser(VolumeChooserEnvironment env) { if (env.getScope() == ChooserScope.TABLE) { return getVolumeChooserForTable(env); } return getVolumeChooserForScope(env); }
Source:
<code>private IdJoinQuery populateBranchIdsJoinIdTable() throws OseeDataStoreException, OseeCoreException { IdJoinQuery branchJoin = JoinUtility.createIdJoinQuery(); BranchFilter branchFilter = new BranchFilter(BranchArchivedState.UNARCHIVED); <START> branchFilter.setNegatedBranchStates(BranchState.PURGED); <END> branchFilter.setNegatedBranchStates(BranchState.DELETED); for (Branch branch : BranchManager.getBranches(branchFilter)) { branchJoin.add(branch.getId()); } branchJoin.store(); return branchJoin; }</code><technical_language>Change a single call arguments</technical_language>

Target:
private IdJoinQuery populateBranchIdsJoinIdTable() throws OseeDataStoreException, OseeCoreException { IdJoinQuery branchJoin = JoinUtility.createIdJoinQuery(); BranchFilter branchFilter = new BranchFilter(BranchArchivedState.UNARCHIVED); branchFilter.setNegatedBranchStates(BranchState.PURGED, BranchState.DELETED); for (Branch branch : BranchManager.getBranches(branchFilter)) { branchJoin.add(branch.getId()); } branchJoin.store(); return branchJoin; }
Source:
<code>public static void closeAll() throws IOException { if (LOGGER.isDebugEnabled()) { <START> debugLogFileSystemClose("closeAll", null); <END> } CACHE.closeAll(); }</code><technical_language>pass in "" instead of null theres' need for ? :</technical_language>

Target:
public static void closeAll() throws IOException { debugLogFileSystemClose("closeAll", ""); CACHE.closeAll(); }
Source:
<code><START> public DeepLinkUri getUri() { <END> return deepLinkUri; }</code><technical_language>for 3 methods</technical_language>

Target:
DeepLinkUri getUri() { return deepLinkUri; }
Source:
<code>private void handleException(final Entry<Optional<Executor>, List<ExecutableFlow>> entry, final Executor executor, final ExecutorManagerException e, final ArrayList<ExecutableFlow> finalizeFlows) { logger.error("Failed to get update from executor " + executor.getHost(), e); boolean sendUnresponsiveEmail = false; final boolean executorRemoved = executorRemoved(executor.getId()); for (final ExecutableFlow flow : entry.getValue()) { final Pair<ExecutionReference, ExecutableFlow> pair = this.runningExecutions.get().get(flow.getExecutionId()); this.updaterStage .set("Failed to get update for flow " + pair.getSecond().getExecutionId()); if (executorRemoved) { logger.warn("Finalizing execution " + flow.getExecutionId() <START> + ". Executor id of this execution doesn't exist"); <END> finalizeFlows.add(flow); } else { final ExecutionReference ref = pair.getFirst(); ref.setNextCheckTime(DateTime.now().getMillis() + this.errorThreshold); ref.setNumErrors(ref.getNumErrors() + 1); if (ref.getNumErrors() == this.numErrorsBeforeUnresponsiveEmail || ref.getNumErrors() % this.numErrorsBetweenUnresponsiveEmail == 0) { sendUnresponsiveEmail = true; } } } if (sendUnresponsiveEmail) { final Alerter mailAlerter = this.alerterHolder.get("email"); mailAlerter.alertOnFailedUpdate(executor, entry.getValue(), e); } }</code><technical_language>logging message is accurate, please update reflect "executor is removed"</technical_language>

Target:
private void handleException(final Entry<Optional<Executor>, List<ExecutableFlow>> entry, final Executor executor, final ExecutorManagerException e, final ArrayList<ExecutableFlow> finalizeFlows) { logger.error("Failed to get update from executor " + executor.getHost(), e); boolean sendUnresponsiveEmail = false; final boolean executorRemoved = isExecutorRemoved(executor.getId()); for (final ExecutableFlow flow : entry.getValue()) { final Pair<ExecutionReference, ExecutableFlow> pair = this.runningExecutions.get().get(flow.getExecutionId()); this.updaterStage .set("Failed to get update for flow " + pair.getSecond().getExecutionId()); if (executorRemoved) { logger.warn("Finalizing execution " + flow.getExecutionId() + ". Executor is removed"); finalizeFlows.add(flow); } else { final ExecutionReference ref = pair.getFirst(); ref.setNextCheckTime(DateTime.now().getMillis() + this.errorThreshold); ref.setNumErrors(ref.getNumErrors() + 1); if (ref.getNumErrors() == this.numErrorsBeforeUnresponsiveEmail || ref.getNumErrors() % this.numErrorsBetweenUnresponsiveEmail == 0) { sendUnresponsiveEmail = true; } } } if (sendUnresponsiveEmail) { final Alerter mailAlerter = this.alerterHolder.get("email"); mailAlerter.alertOnFailedUpdate(executor, entry.getValue(), e); } }
Source:
<code>private void processComponentConfig(final EEApplicationClasses applicationClasses, final DeploymentReflectionIndex deploymentReflectionIndex, final ComponentDescription description, DeploymentUnit deploymentUnit) throws DeploymentUnitProcessingException { final Class<?> componentClass; try { componentClass = ClassLoadingUtils.loadClass(description.getComponentClassName(), deploymentUnit); handleAnnotations(applicationClasses, deploymentReflectionIndex, componentClass, description); } catch (Throwable e) { EeLogger.ROOT_LOGGER.debugf(e,"Ignoring failure to handle interceptor annotations for %s", description.getComponentClassName()); <START> return; <END> } }</code><technical_language>LGTM. Some minor points: remove throws DeploymentUnitProcessingException method catches exceptions. return; is redundant. final <class> componentClass combined assignment</technical_language>

Target:
private void processComponentConfig(final EEApplicationClasses applicationClasses, final DeploymentReflectionIndex deploymentReflectionIndex, final ComponentDescription description, DeploymentUnit deploymentUnit) { try { final Class<?> componentClass = ClassLoadingUtils.loadClass(description.getComponentClassName(), deploymentUnit); handleAnnotations(applicationClasses, deploymentReflectionIndex, componentClass, description); } catch (Throwable e) { EeLogger.ROOT_LOGGER.debugf(e,"Ignoring failure to handle interceptor annotations for %s", description.getComponentClassName()); } }