auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public SWTBotTreeItem doubleClick() { waitForEnabled(); log.debug(MessageFormat.format("Double-clicking on {0}", this)); notifyTree(SWT.MouseEnter, createMouseEvent(0, SWT.NONE, 0)); notifyTree(SWT.Activate, super.createEvent()); setFocus(); notifyTree(SWT.FocusIn, super.createEvent()); notifyTree(SWT.MouseDown, createMouseEvent(1, SWT.NONE, 1)); notifyTree(SWT.Selection, createSelectionEvent(SWT.NONE), selectRunnable(widget, false)); notifyTree(SWT.MouseUp, createMouseEvent(1, SWT.BUTTON1, 1)); notifyTree(SWT.MouseDown, createMouseEvent(1, SWT.NONE, 2)); notifyTree(SWT.MouseDoubleClick, createMouseEvent(1, SWT.NONE, 2)); <START> notifyTree(SWT.DefaultSelection, createSelectionEvent(SWT.NONE)); <END> notifyTree(SWT.MouseUp, createMouseEvent(1, SWT.BUTTON1, 2)); notifyTree(SWT.MouseExit, createMouseEvent(0, SWT.NONE, 0)); notifyTree(SWT.Deactivate, super.createEvent()); notifyTree(SWT.FocusOut, super.createEvent()); log.debug(MessageFormat.format("Double-clicked on {0}", this)); return this; }</code><technical_language>Statemask for DefaultSelection is follows- win(BUTTON1), linux(NONE), mac(NONE). BUTTON1 for macOS is case</technical_language>

Target:
public SWTBotTreeItem doubleClick() { waitForEnabled(); log.debug(MessageFormat.format("Double-clicking on {0}", this)); notifyTree(SWT.MouseEnter, createMouseEvent(0, SWT.NONE, 0)); notifyTree(SWT.Activate, super.createEvent()); setFocus(); notifyTree(SWT.FocusIn, super.createEvent()); notifyTree(SWT.MouseDown, createMouseEvent(1, SWT.NONE, 1)); notifyTree(SWT.Selection, createSelectionEvent(SWT.NONE), selectRunnable(widget, false)); notifyTree(SWT.MouseUp, createMouseEvent(1, SWT.BUTTON1, 1)); notifyTree(SWT.MouseDown, createMouseEvent(1, SWT.NONE, 2)); notifyTree(SWT.MouseDoubleClick, createMouseEvent(1, SWT.NONE, 2)); notifyTree(SWT.DefaultSelection, createDefaultSelectionEvent(SWT.NONE)); notifyTree(SWT.MouseUp, createMouseEvent(1, SWT.BUTTON1, 2)); notifyTree(SWT.MouseExit, createMouseEvent(0, SWT.NONE, 0)); notifyTree(SWT.Deactivate, super.createEvent()); notifyTree(SWT.FocusOut, super.createEvent()); log.debug(MessageFormat.format("Double-clicked on {0}", this)); return this; }
Source:
<code>public MessagePacker packExtendedTypeHeader(int extType, int payloadLen) throws IOException { if(payloadLen < (1 << 8)) { if(payloadLen == 1) { writeByteAndByte(FIXEXT1, (byte) extType); } else if(payloadLen == 2){ writeByteAndByte(FIXEXT2, (byte) extType); } else if(payloadLen == 4) { writeByteAndByte(FIXEXT4, (byte) extType); } else if(payloadLen == 8) { writeByteAndByte(FIXEXT8, (byte) extType); } else if(payloadLen == 16) { <START> writeByteAndByte(FIXEXT16, (byte) extType); <END> } else { writeByteAndByte(EXT8, (byte) payloadLen); writeByte((byte) extType); } } else if(payloadLen < (1 << 16)) { writeByteAndShort(EXT16, (short) payloadLen); writeByte((byte) extType); } else { writeByteAndInt(EXT32, payloadLen); writeByte((byte) extType); } return this; }</code><technical_language>changing this else if (payloadLen == 16) { ... } else { ... writeByteAndByte(EXT8, ... )}</technical_language>

Target:
public MessagePacker packExtendedTypeHeader(int extType, int payloadLen) throws IOException { if(payloadLen < (1 << 8)) { if(payloadLen > 0 && (payloadLen & (payloadLen - 1)) == 0) { if(payloadLen == 1) { writeByteAndByte(FIXEXT1, (byte) extType); } else if(payloadLen == 2){ writeByteAndByte(FIXEXT2, (byte) extType); } else if(payloadLen == 4) { writeByteAndByte(FIXEXT4, (byte) extType); } else if(payloadLen == 8) { writeByteAndByte(FIXEXT8, (byte) extType); } else if(payloadLen == 16) { writeByteAndByte(FIXEXT16, (byte) extType); } else { writeByteAndByte(EXT8, (byte) payloadLen); writeByte((byte) extType); } } else { writeByteAndByte(EXT8, (byte) payloadLen); writeByte((byte) extType); } } else if(payloadLen < (1 << 16)) { writeByteAndShort(EXT16, (short) payloadLen); writeByte((byte) extType); } else { writeByteAndInt(EXT32, payloadLen); writeByte((byte) extType); } return this; }
Source:
<code>private static LeaderPingResult getLeaderPingResult( UUID uuid, @Nullable Future<Map.Entry<LeaderPingerContext<PingableLeader>, PingResult>> pingFuture, <START> Optional<OrderableSlsVersion> timeLockVersion) throws ExecutionException { <END> if (pingFuture == null) { return LeaderPingResults.pingTimedOut(); } PingResult pingResult = Futures.getDone(pingFuture).getValue(); if (!pingResult.isLeader()) { return LeaderPingResults.pingReturnedFalse(); } return isAtLeastOurVersion(pingResult, timeLockVersion) ? LeaderPingResults.pingReturnedTrue( uuid, Futures.getDone(pingFuture).getKey().hostAndPort()) : LeaderPingResults.pingReturnedTrueWithOlderVersion(pingResult.timeLockVersion().get()); }</code><technical_language>nit: basically exceptions for control flow I think, is great. return PingCallFailure check a failure higher up? missed</technical_language>

Target:
private static LeaderPingResult getLeaderPingResult( UUID uuid, @Nullable Future<Map.Entry<LeaderPingerContext<PingableLeader>, PingResult>> pingFuture, Optional<OrderableSlsVersion> timeLockVersion) { if (pingFuture == null) { return LeaderPingResults.pingTimedOut(); } try { PingResult pingResult = Futures.getDone(pingFuture).getValue(); if (!pingResult.isLeader()) { return LeaderPingResults.pingReturnedFalse(); } return isAtLeastOurVersion(pingResult, timeLockVersion) ? LeaderPingResults.pingReturnedTrue( uuid, Futures.getDone(pingFuture).getKey().hostAndPort()) : LeaderPingResults.pingReturnedTrueWithOlderVersion(pingResult.timeLockVersion().get()); } catch (ExecutionException e) { return LeaderPingResults.pingCallFailedWithExecutionException(e.getCause()); } }
Source:
<code>public void setup() throws Exception { initMocks(this); md5 = "md5"; currentUser = new Username(new CaseInsensitiveString("user")); cruiseConfig = GoConfigMother.defaultCruiseConfig(); newPackageRepo = new PackageRepository(); oldPackageRepo = new PackageRepository(); newRepoId = oldRepoId = "npmOrg"; pluginId = "npm"; <START> newPackageRepo.setId(newRepoId); <END> newPackageRepo.setName(newRepoId); oldPackageRepo.setId(oldRepoId); oldPackageRepo.setName(oldRepoId); newPackageRepo.setPluginConfiguration(new PluginConfiguration(pluginId, "1")); oldPackageRepo.setPluginConfiguration(new PluginConfiguration(pluginId, "1")); result = new HttpLocalizedOperationResult(); cruiseConfig.setPackageRepositories(new PackageRepositories(oldPackageRepo)); }</code><technical_language>Move a factory introduce a constructor</technical_language>

Target:
public void setup() throws Exception { initMocks(this); currentUser = new Username(new CaseInsensitiveString("user")); cruiseConfig = GoConfigMother.defaultCruiseConfig(); repoId = "npmOrg"; newPackageRepo = new PackageRepository(repoId, repoId, new PluginConfiguration("npm", "1"), new Configuration(new ConfigurationProperty(new ConfigurationKey("REPO_URL"), new ConfigurationValue("http://bar")))); oldPackageRepo = new PackageRepository(repoId, repoId, new PluginConfiguration("npm", "1"), new Configuration(new ConfigurationProperty(new ConfigurationKey("REPO_URL"), new ConfigurationValue("http://foo")))); result = new HttpLocalizedOperationResult(); cruiseConfig.setPackageRepositories(new PackageRepositories(oldPackageRepo)); }
Source:
<code>public ByteBuf allocate(int capacity) { accountBytes.accept(capacity); <START> return Unpooled.wrappedBuffer(new byte[capacity]); <END> }</code><technical_language>reasons platform aware return Unpooled.buffer(capacity)</technical_language>

Target:
public ByteBuf allocate(int capacity) { accountBytes.accept(capacity); return Unpooled.buffer(capacity); }
Source:
<code>public void applyNewObjectPosition(@NonNull Object o, @NonNull LatLon position, @Nullable ApplyMovedObjectCallback callback) { boolean result = false; if (o instanceof FavouritePoint) { favorites.editFavourite((FavouritePoint) o, position.getLatitude(), position.getLongitude()); <START> favorites.lookupAddressAllPersonalPoints(); <END> result = true; } if (callback != null) { callback.onApplyMovedObject(result, o); } }</code><technical_language>?? personal points ?</technical_language>

Target:
public void applyNewObjectPosition(@NonNull Object o, @NonNull LatLon position, @Nullable ApplyMovedObjectCallback callback) { boolean result = false; if (o instanceof FavouritePoint) { favorites.editFavourite((FavouritePoint) o, position.getLatitude(), position.getLongitude()); favorites.lookupAddress((FavouritePoint) o); result = true; } if (callback != null) { callback.onApplyMovedObject(result, o); } }
Source:
<code>public GetObjectResponse( Headers headers, String bucket, String region, String object, InputStream body) { this.response = new BaseResponse(headers, bucket, region, object); <START> this.body = body; <END> }</code><technical_language>GetObjectResponse extend BufferedInputStream call super(body) constructor. this, class authentically InputStream need store body variable write wrapper methods for it</technical_language>

Target:
public GetObjectResponse( Headers headers, String bucket, String region, String object, InputStream body) { super(body); this.response = new GenericResponse(headers, bucket, region, object); }
Source:
<code>public String getName() { <START> String[] nameElements = ctClass.getSimpleName().replace('$', '.').split("\\."); <END> return nameElements[nameElements.length - 1]; }</code><technical_language>call getClassName here</technical_language>

Target:
public String getName() { return getClassName(); }
Source:
<code>private int addWriter( SinkTransformation<InputT, CommT, WriterStateT, GlobalCommT> sinkTransformation, int parallelism, Context context) { final boolean hasState = sinkTransformation .getSink() .getWriterStateSerializer() .isPresent(); checkState(sinkTransformation.getInputs().size() == 1); @SuppressWarnings("unchecked") final Transformation<InputT> input = (Transformation<InputT>) sinkTransformation .getInputs() .get(0); final TypeInformation<InputT> inputTypeInfo = input.getOutputType(); final StreamOperatorFactory<CommT> writer = hasState ? new StatefulWriterOperatorFactory<>(sinkTransformation.getSink()) : new StatelessWriterOperatorFactory<>( sinkTransformation.getSink()); final int writerId = addOperatorToStreamGraph( <START> writer, input.getId(), <END> inputTypeInfo, extractCommittableTypeInformation(sinkTransformation.getSink()), "Sink Writer:", parallelism, sinkTransformation.getMaxParallelism(), sinkTransformation, context); StreamGraphUtils.configureResourceProperties( context.getStreamGraph(), writerId, sinkTransformation); return writerId; }</code><technical_language>nit: put input.getId() in new line, other calls addOperatorToStreamGraph</technical_language>

Target:
private int addWriter( SinkTransformation<InputT, CommT, WriterStateT, GlobalCommT> sinkTransformation, int parallelism, Context context) { final boolean hasState = sinkTransformation .getSink() .getWriterStateSerializer() .isPresent(); checkState(sinkTransformation.getInputs().size() == 1); @SuppressWarnings("unchecked") final Transformation<InputT> input = (Transformation<InputT>) sinkTransformation .getInputs() .get(0); final TypeInformation<InputT> inputTypeInfo = input.getOutputType(); final StreamOperatorFactory<CommT> writer = hasState ? new StatefulWriterOperatorFactory<>(sinkTransformation.getSink()) : new StatelessWriterOperatorFactory<>( sinkTransformation.getSink()); final ChainingStrategy chainingStrategy = sinkTransformation.getChainingStrategy(); if (chainingStrategy != null) { writer.setChainingStrategy(chainingStrategy); } return addOperatorToStreamGraph( writer, input.getId(), inputTypeInfo, extractCommittableTypeInformation(sinkTransformation.getSink()), "Sink Writer:", parallelism, sinkTransformation.getMaxParallelism(), sinkTransformation, context); }
Source:
<code>protected void assertNotSupported(String projection, String message) { try { functionAssertions.executeProjectionWithFullEngine(projection); <START> fail("Expected to thrown a NOT_SUPPORTED exception"); <END> } catch (PrestoException e) { assertEquals(e.getErrorCode(), NOT_SUPPORTED.toErrorCode()); assertEquals(e.getMessage(), message); } }</code><technical_language>this java fail("expected exception");</technical_language>

Target:
protected void assertNotSupported(String projection, String message) { try { functionAssertions.executeProjectionWithFullEngine(projection); fail("expected exception"); } catch (PrestoException e) { assertEquals(e.getErrorCode(), NOT_SUPPORTED.toErrorCode()); assertEquals(e.getMessage(), message); } }
Source:
<code>public boolean addChildAtLoading(INode node) { int pos; if (!node.isReference()) { pos = (children == null) ? (-1) : (-children.size() - 1); } else { <START> pos = searchChildren(node.getLocalNameBytes()); <END> if (pos >= 0) { return false; } } addChild(node, pos); return true; }</code><technical_language>reuse addChild() cleaner? understanding is in case of INodeReference, fall original behavior. is, } else { return addChild(node); }</technical_language>

Target:
public boolean addChildAtLoading(INode node) { int pos; if (!node.isReference()) { pos = (children == null) ? (-1) : (-children.size() - 1); addChild(node, pos); return true; } else { return addChild(node); } }
Source:
<code>public PageEntity(String unlocalizedName, String entity, int size) { super(unlocalizedName); Class EntityClass=(Class) EntityList.stringToClassMapping.get(entity); this.size=size; try { dummyEntity = (Entity)EntityClass.getConstructor(new Class[] {World.class}).newInstance(new Object[] {null}); <START> } catch (InstantiationException e) { <END> e.printStackTrace(); } catch (IllegalAccessException e) { e.printStackTrace(); } catch (IllegalArgumentException e) { e.printStackTrace(); } catch (InvocationTargetException e) { e.printStackTrace(); } catch (NoSuchMethodException e) { e.printStackTrace(); } catch (SecurityException e) { e.printStackTrace(); } }</code><technical_language>pokemon-catching these</technical_language>

Target:
public PageEntity(String unlocalizedName, String entity, int size) { super(unlocalizedName); Class EntityClass=(Class) EntityList.stringToClassMapping.get(entity); this.size=size; try { dummyEntity = (Entity)EntityClass.getConstructor(new Class[] {World.class}).newInstance(new Object[] {null}); } catch (Exception e) { e.printStackTrace(); } }
Source:
<code>private ByteBuffer parsePacketIDB(ByteBuffer pcapPacketHeader, int blockLength) throws IOException { ByteBuffer pcapPacketData; pcapPacketData = ByteBuffer.allocate(blockLength - PcapFileValues.PCAPNG_PACKET_HEADER_SIZE); pcapPacketHeader.getShort(); pcapPacketData.clear(); SeekableByteChannel <START> fFileChannel; <END> try { fFileChannel = getFileChannel(); fFileChannel.read(pcapPacketData); } catch (BadPcapFileException e1) { } pcapPacketData.flip(); return pcapPacketData; }</code><technical_language>This is a local variable, camelCase fCamelCase</technical_language>

Target:
private ByteBuffer parsePacketIDB(ByteBuffer pcapPacketHeader, int blockLength) throws IOException { ByteBuffer pcapPacketData; pcapPacketData = ByteBuffer.allocate(blockLength - PcapFileValues.PCAPNG_PACKET_HEADER_SIZE); pcapPacketHeader.getShort(); pcapPacketData.clear(); SeekableByteChannel fileChannel; try { fileChannel = getFileChannel(); fileChannel.read(pcapPacketData); } catch (BadPcapFileException e1) { } pcapPacketData.flip(); return pcapPacketData; }
Source:
<code><START> public void createOutboundLocalSocketBinding() throws IOException, CommandFailedException { <END> ConfigFragment fragment = page.switchToOutboundLocal(); WizardWindow wizard = fragment.getResourceManager().addResource(); Editor editor = wizard.getEditor(); editor.text("name", OUTBOUND_LOCAL_NAME); editor.text("socketBinding", "https"); boolean result = wizard.finish(); assertTrue("Window should be closed", result); assertTrue("Socket Binding should be present in table", fragment.resourceIsPresent(OUTBOUND_LOCAL_NAME)); verifier.verifyResource(DMR_OUTBOUND_LOCAL, true); fragment.getResourceManager().removeResourceAndConfirm(OUTBOUND_LOCAL_NAME); verifier.verifyResource(DMR_OUTBOUND_LOCAL, false); }</code><technical_language>throwing exceptions here, code require throwing</technical_language>

Target:
public void createOutboundLocalSocketBinding() { ConfigFragment fragment = page.switchToOutboundLocal(); WizardWindow wizard = fragment.getResourceManager().addResource(); Editor editor = wizard.getEditor(); editor.text("name", OUTBOUND_LOCAL_NAME); editor.text("socketBinding", "https"); boolean result = wizard.finish(); assertTrue("Window should be closed", result); assertTrue("Socket Binding should be present in table", fragment.resourceIsPresent(OUTBOUND_LOCAL_NAME)); verifier.verifyResource(DMR_OUTBOUND_LOCAL, true); fragment.getResourceManager().removeResourceAndConfirm(OUTBOUND_LOCAL_NAME); verifier.verifyResource(DMR_OUTBOUND_LOCAL, false); }
Source:
<code>private File getDhclientFile(String interfaceName) throws KuraException { File ret; File globalDhClientFile = new File(formGlobalDhclientLeasesFilename()); File interfaceDhClientFile = new File(formInterfaceDhclientLeasesFilename(interfaceName)); if (interfaceDhClientFile.exists()) { ret = interfaceDhClientFile; } else if (globalDhClientFile.exists()) { ret = globalDhClientFile; } else { try { if (interfaceDhClientFile.createNewFile()) { <START> logger.debug("The doesn't exist, created new empty file ...", interfaceDhClientFile.getName()); <END> } } catch (Exception e) { throw new KuraException(KuraErrorCode.CONFIGURATION_ERROR, e); } ret = interfaceDhClientFile; } return ret; }</code><technical_language>info</technical_language>

Target:
private File getDhclientFile(String interfaceName) throws KuraException { File ret; File globalDhClientFile = new File(formGlobalDhclientLeasesFilename()); File interfaceDhClientFile = new File(formInterfaceDhclientLeasesFilename(interfaceName)); if (interfaceDhClientFile.exists()) { ret = interfaceDhClientFile; } else if (globalDhClientFile.exists()) { ret = globalDhClientFile; } else { try { if (interfaceDhClientFile.createNewFile()) { logger.info("The doesn't exist, created new empty file ...", interfaceDhClientFile.getName()); } } catch (Exception e) { throw new KuraException(KuraErrorCode.CONFIGURATION_ERROR, e); } ret = interfaceDhClientFile; } return ret; }
Source:
<code><START> public void sendMessage_shouldSaveTheSentMessage() throws MessageException { <END> Message message = ms.createMessage("recipient@example.com", "sender@example.com", "subject", "content"); message.setId(123); try { ms.sendMessage(message); SentMessage sentMessage = Context.getSentMessageService().getSentMessage(123); Assert.assertNotNull(sentMessage); Assert.assertEquals(123, (int)sentMessage.getMessageId()); } catch (MessageException e) { if (!e.getMessage().contains(NO_SMTP_SERVER_ERROR)) { e.printStackTrace(); fail(); } } }</code><technical_language>![Codacy](<LINK_0> Issue found: [Method names underscores](<LINK_1></technical_language>

Target:
public void sendMessage_shouldSaveTheSentMessage() throws MessageException { Message message = ms.createMessage("recipient@example.com", "sender@example.com", "subject", "content"); message.setId(123); try { ms.sendMessage(message); SentMessage sentMessage = Context.getSentMessageService().getSentMessage(123); assertNotNull(sentMessage); assertEquals(123, (int)sentMessage.getMessageId()); } catch (MessageException e) { if (!e.getMessage().contains(NO_SMTP_SERVER_ERROR)) { e.printStackTrace(); fail(); } } }
Source:
<code>public SpectralSearchlight(UUID ownerId, CardSetInfo setInfo) { super(ownerId, setInfo, new CardType[]{CardType.ARTIFACT}, "{3}"); ManaEffect effect = new AddManaOfAnyColorToManaPoolTargetControllerEffect("chosen player"); effect.setText("Choose a player. That player adds one mana of any color he or she chooses to his or her mana pool"); Ability ability = new SimpleManaAbility(Zone.BATTLEFIELD, effect, new TapSourceCost()); <START> ability.addTarget(new TargetPlayer(1, 1, true)); <END> this.addAbility(ability); }</code><technical_language>existing ChoosePlayerEffect do this. If a target player is defined cast time defined time abilitiy resolves</technical_language>

Target:
public SpectralSearchlight(UUID ownerId, CardSetInfo setInfo) { super(ownerId, setInfo, new CardType[]{CardType.ARTIFACT}, "{3}"); ManaEffect effect = new AddManaOfAnyColorToManaPoolTargetPlayerEffect("chosen player"); effect.setText("Choose a player. That player adds one mana of any color he or she chooses to his or her mana pool"); Ability ability = new SimpleManaAbility(Zone.BATTLEFIELD, effect, new TapSourceCost()); ability.getEffects().add(0, new ChoosePlayerEffect(Outcome.PutManaInPool)); this.addAbility(ability); }
Source:
<code>public void testArrayLengthWithAllSupportedTypes() throws Exception { for (DataType<?> type : DataTypeTesting.ALL_TYPES_EXCEPT_ARRAYS) { <START> if(type.equals(DataTypes.INTERVAL)) { <END> continue; } Supplier dataGenerator = DataTypeTesting.getDataGenerator(type); Object val1 = dataGenerator.get(); Object val2 = dataGenerator.get(); Object[] arr = {val1, val2}; Object[] values = new Object[] { arr }; resetClusterService(); try (QueryTester tester = new QueryTester.Builder( createTempDir(), THREAD_POOL, clusterService, Version.CURRENT, "create table \"t_"+ type.getName() + "\" (xs array(\"" + type.getName() + "\"))" ).indexValues("xs", values).build()) { System.out.println(type); List<Object> result = tester.runQuery("xs", "array_length(xs, 1) >= 2"); assertThat(result.size(), is(1)); ArrayType arrayType = new ArrayType<>(type); if (type.id() != ObjectType.ID) { assertThat(arrayType.compareValueTo((List) result.get(0), Arrays.asList(arr)), is(0)); } } } }</code><technical_language>DataTypes.STORAGE_UNSUPPORTED here, support interval storage test reflect change automatically (otherwise forget this "temporary" skipping)</technical_language>

Target:
public void testArrayLengthWithAllSupportedTypes() throws Exception { for (DataType<?> type : DataTypeTesting.ALL_TYPES_EXCEPT_ARRAYS) { if(DataTypes.STORAGE_UNSUPPORTED.contains(type)) { continue; } Supplier dataGenerator = DataTypeTesting.getDataGenerator(type); Object val1 = dataGenerator.get(); Object val2 = dataGenerator.get(); Object[] arr = {val1, val2}; Object[] values = new Object[] { arr }; resetClusterService(); try (QueryTester tester = new QueryTester.Builder( createTempDir(), THREAD_POOL, clusterService, Version.CURRENT, "create table \"t_"+ type.getName() + "\" (xs array(\"" + type.getName() + "\"))" ).indexValues("xs", values).build()) { System.out.println(type); List<Object> result = tester.runQuery("xs", "array_length(xs, 1) >= 2"); assertThat(result.size(), is(1)); ArrayType arrayType = new ArrayType<>(type); if (type.id() != ObjectType.ID) { assertThat(arrayType.compareValueTo((List) result.get(0), Arrays.asList(arr)), is(0)); } } } }
Source:
<code>public Range<Double> getMzRange() { Range<Double> mzRange = <START> MsSpectrumUtil.getMzRange(mzValues, getMzBinaryDataInfo().getArrayLength()); <END> return mzRange; }</code><technical_language>CV terms for mz range: <cvParam cvRef="MS" accession="MS:1000528" name="lowest observed m/z" value="116.825372650742" unitCvRef="MS" unitAccession="MS:1000040" unitName="m/z"/> <cvParam cvRef="MS" accession="MS:1000527" name="highest observed m/z" value="1767.680685613041" unitCvRef="MS" unitAccession="MS:1000040" unitName="m/z"/> MsSpectrumUtil method if present in mzML file. Save result for reuse (same TIC)</technical_language>

Target:
public Range<Double> getMzRange() { Range<Double> mzRange = null; try { mzRange = Range.closed(Double.valueOf(getCVValue(MzMLCV.cvLowestMz)), Double.valueOf(getCVValue(MzMLCV.cvHighestMz))); } catch (Exception e) { e.printStackTrace(); } if (mzRange == null) mzRange = MsSpectrumUtil.getMzRange(getMzValues(), getMzBinaryDataInfo().getArrayLength()); return mzRange; }
Source:
<code>private Map<String, List<String>> createTestData(){ Map<String, List<String>> testMap = new HashMap<>(); List<String> nameList = newArrayList("Dave", "Jeff"); List<String> jobList = newArrayList("Plumber", "Builder"); List<String> cityList = newArrayList("Dover", "Boston"); <START> testMap.put("name", nameList); <END> testMap.put("job", jobList); testMap.put("city", cityList); return testMap; }</code><technical_language>inline lists</technical_language>

Target:
private Map<String, List<String>> createTestData(){ Map<String, List<String>> testMap = new HashMap<>(); testMap.put("name", Arrays.asList("Dave", "Jeff")); testMap.put("job", Arrays.asList("Plumber", "Builder")); testMap.put("city", Arrays.asList("Dover", "Boston")); return testMap; }
Source:
<code>private static void loadProperties( Properties props, String propertyFileName) { InputStream propertyFileInputStream = null; try { try { propertyFileInputStream = new DefaultResourceLoader(ClassLoaderUtils.getDefaultClassLoader()).getResource(propertyFileName).getInputStream(); props.load(propertyFileInputStream); } finally { if (propertyFileInputStream != null) { propertyFileInputStream.close(); } } } catch (IOException e) { <START> e.printStackTrace(); <END> throw new RuntimeException("PropertyLoadingFactoryBean unable to load property file: " + propertyFileName); } }</code><technical_language>logging stack trace, dumping System.err</technical_language>

Target:
private static void loadProperties( Properties props, String propertyFileName) { InputStream propertyFileInputStream = null; try { try { propertyFileInputStream = new DefaultResourceLoader(ClassLoaderUtils.getDefaultClassLoader()).getResource(propertyFileName).getInputStream(); props.load(propertyFileInputStream); } finally { if (propertyFileInputStream != null) { propertyFileInputStream.close(); } } } catch (IOException e) { LOG.error(e); throw new RuntimeException("PropertyLoadingFactoryBean unable to load property file: " + propertyFileName); } }
Source:
<code>public CloseableIterator<SAMRecord> queryAlignmentStart(String sequence, int start) { long[] filePointers = null; final SAMFileHeader fileHeader = getFileHeader(); final int referenceIndex = fileHeader.getSequenceIndex(sequence); if (referenceIndex != -1) { final BAMIndex fileIndex = getIndex(); final BAMFileSpan fileSpan = fileIndex.getSpanOverlapping( referenceIndex, start, -1); filePointers = fileSpan != null ? fileSpan.toCoordinateArray() : null; } if (filePointers == null || filePointers.length == 0) return emptyIterator; SeekableStream s = null; if (file != null) { try { s = new SeekableFileStream(file); } catch (FileNotFoundException e) { throw new RuntimeException(e); } } else if (is instanceof SeekableStream) s = (SeekableStream) is; <START> <END> SAMIteratorForCRAM si = null; try { s.seek(0); si = new SAMIteratorForCRAM(s, referenceSource); si.setValidationStringency(validationStringency); it = si; } catch (IOException e) { throw new RuntimeEOFException(e); } Container c = null; for (int i = 0; i < filePointers.length; i += 2) { long containerOffset = filePointers[i] >>> 16; int sliceIndex = (int) ((filePointers[i] << 48) >>> 48); try { s.seek(containerOffset); c = CramIO.readContainerHeader(s); if (c.alignmentStart + c.alignmentSpan > start) { s.seek(containerOffset); return si; } } catch (IOException e) { throw new RuntimeException(e); } } it = emptyIterator; return it; }</code><technical_language>pattern serveral times. This extracted a private method</technical_language>

Target:
public CloseableIterator<SAMRecord> queryAlignmentStart(String sequence, int start) { long[] filePointers = null; final SAMFileHeader fileHeader = getFileHeader(); final int referenceIndex = fileHeader.getSequenceIndex(sequence); if (referenceIndex != -1) { final BAMIndex fileIndex = getIndex(); final BAMFileSpan fileSpan = fileIndex.getSpanOverlapping( referenceIndex, start, -1); filePointers = fileSpan != null ? fileSpan.toCoordinateArray() : null; } if (filePointers == null || filePointers.length == 0) return emptyIterator; SeekableStream s = getSeekableStreamOrFailWithRTE(); CRAMIterator si = null; try { s.seek(0); si = new CRAMIterator(s, referenceSource); si.setValidationStringency(validationStringency); it = si; } catch (IOException e) { throw new RuntimeEOFException(e); } Container c = null; for (int i = 0; i < filePointers.length; i += 2) { long containerOffset = filePointers[i] >>> 16; int sliceIndex = (int) ((filePointers[i] << 48) >>> 48); try { s.seek(containerOffset); c = CramIO.readContainerHeader(s); if (c.alignmentStart + c.alignmentSpan > start) { s.seek(containerOffset); return si; } } catch (IOException e) { throw new RuntimeException(e); } } it = null; return it; }
Source:
<code>public IType getType() { if (type == null) { IASTNode parent= getPrimaryDeclaration().getParent(); while (parent != null) { if (parent instanceof ICPPASTParameterDeclaration) { type= CPPVisitor.createType((ICPPASTParameterDeclaration) parent, true); break; } parent= parent.getParent(); } } <START> if (type instanceof CPPPlaceholderType) { <END> CPPDependentEvaluation eval = new EvalBinding(this, null, getPrimaryDeclaration()); TypeOfDependentExpression replacementType = new TypeOfDependentExpression(eval); replacementType.setForTemplateAuto(true); type = replacementType; } return type; }</code><technical_language>Move this 'if' block above, do this extra check initial computation caching of type</technical_language>

Target:
public IType getType() { if (type == null) { IASTNode parent= getPrimaryDeclaration().getParent(); while (parent != null) { if (parent instanceof ICPPASTParameterDeclaration) { type= CPPVisitor.createType((ICPPASTParameterDeclaration) parent, true); break; } parent= parent.getParent(); } if (type instanceof CPPPlaceholderType) { CPPDependentEvaluation eval = new EvalBinding(this, null, getPrimaryDeclaration()); TypeOfDependentExpression replacementType = new TypeOfDependentExpression(eval); replacementType.setForTemplateAuto(true); type = replacementType; } } return type; }
Source:
<code>private void maybeCollapseAssignShorthand(Node n, Node parent) { <START> if (n.isAssign() && n.getFirstChild().isName() && n.getLastChild().hasChildren() <END> && n.getLastChild().getFirstChild().isName()) { Node op = n.getLastChild(); int assignOp = getAssignOpFromOp(op); if (n.getFirstChild().getString().equals(op.getFirstChild().getString())) { op.setType(assignOp); Node opDetached = op.detachFromParent(); opDetached.setJSDocInfo(n.getJSDocInfo()); parent.replaceChild(n, opDetached); compiler.reportCodeChange(); } } }</code><technical_language>This create invalid AST contains error token. Instead of hasChildren, need check child is operator converted assignment operator, eg, a helper method this: js static boolean hasCorrespondingAssignmentOp(Node n) { switch (n.getType()) { case Token.BITOR: case Token.BITXOR: case Token.BITAND: case Token.LSH: case Token.RSH: case Token.URSH: case Token.ADD: case Token.SUB: case Token.MUL: case Token.DIV: case Token.MOD: return true; default: return false; } }</technical_language>

Target:
private void maybeCollapseAssignShorthand(Node n, Node parent) { if (n.isAssign() && n.getFirstChild().isName() && NodeUtil.hasCorrespondingAssignmentOp(n.getLastChild()) && n.getLastChild().getFirstChild().isName()) { Node op = n.getLastChild(); int assignOp = NodeUtil.getAssignOpFromOp(op); if (n.getFirstChild().getString().equals(op.getFirstChild().getString())) { op.setType(assignOp); Node opDetached = op.detachFromParent(); opDetached.setJSDocInfo(n.getJSDocInfo()); parent.replaceChild(n, opDetached); compiler.reportCodeChange(); } } }
Source:
<code>public void run() { JsonRpcRequest request = null; try { request = JsonRpcRequest.fromByteArray(message); <START> if(request.getMethod().contains("VM")){ <END> setReactorsMap(request.getParams().get("vmID").asText(), client); } ContextHolder.init(); if (client.getRetryPolicy().getIdentifier() != null){ ContextHolder.setServerName(client.getRetryPolicy().getIdentifier()); log.debug("client policy identifier {}", client.getRetryPolicy().getIdentifier()); }else{ ContextHolder.setServerName(Integer.toString(client.hashCode())); log.error("client identifier were not found, using hash"); } reactorClient = client; ResponseBuilder builder = new ResponseBuilder(request.getId()); String methodName = request.getMethod(); builder = CommandFactory.createCommand(methodName).run(request.getParams(), builder); send(builder.build(), request.getMethod()); } catch (Throwable e) { log.error("Failure in processing request", e); Map<String, Object> error = new HashMap<>(); error.put("code", 100); error.put("message", e.getMessage()); if (request != null) { send(new ResponseBuilder(request.getId()).withError(error).build(), request.getMethod()); } } }</code><technical_language>command VM in name please run add for specific commands. code needs generic soon other events</technical_language>

Target:
public void run() { JsonRpcRequest request = null; try { request = JsonRpcRequest.fromByteArray(message); if(eventSupportedMethods.contains(request.getMethod())){ setReactorsMap(request.getParams().get("vmID").asText(), client); } ContextHolder.init(); if (client.getRetryPolicy().getIdentifier() != null){ ContextHolder.setServerName(client.getRetryPolicy().getIdentifier()); log.debug("client policy identifier {}", client.getRetryPolicy().getIdentifier()); }else{ ContextHolder.setServerName(Integer.toString(client.hashCode())); log.error("client identifier were not found, using hash"); } ResponseBuilder builder = new ResponseBuilder(request.getId()); String methodName = request.getMethod(); builder = CommandFactory.createCommand(methodName).run(request.getParams(), builder); send(builder.build(), request.getMethod()); } catch (Throwable e) { log.error("Failure in processing request", e); Map<String, Object> error = new HashMap<>(); error.put("code", 100); error.put("message", e.getMessage()); if (request != null) { send(new ResponseBuilder(request.getId()).withError(error).build(), request.getMethod()); } } }
Source:
<code>public EnrichmentTransformRuntime(EnrichmentTransform enrichmentTransform) { this.enrichmentTransform = enrichmentTransform; <START> cachedDataProvider = new CachedDataProvider<Object, Object>(enrichmentTransform.getDataProvider(), enrichmentTransform.getMaxCacheSize(), <END> enrichmentTransform.getEntryExpirationInterval(), enrichmentTransform.getEntryRefreshInterval()); cachedDataProvider.prepare(); }</code><technical_language>Right this return null for field data providers is this supposed work</technical_language>

Target:
public EnrichmentTransformRuntime(EnrichmentTransform enrichmentTransform) { this.enrichmentTransform = enrichmentTransform; final TransformDataProvider transformDataProvider = enrichmentTransform.getTransformDataProvider(); cachedDataProvider = new CachedTransformDataProviderRuntime(TransformDataProviderRuntimeService.get().get(transformDataProvider), enrichmentTransform.getMaxCacheSize(), enrichmentTransform.getEntryExpirationInterval(), enrichmentTransform.getEntryRefreshInterval()); cachedDataProvider.prepare(); }
Source:
<code>public ThreadMeta update(ThreadMeta newMeta) { if (newMeta.threadName != null && !newMeta.threadName.isEmpty()) { threadName = newMeta.threadName; } return this; <START> } <END></code><technical_language>I this class' immutability following. immutable mutable! java public ThreadMeta update(ThreadMeta newMeta) { if (newMeta.threadName != null && !newMeta.threadName.isEmpty()) { return newMeta; } return this; }</technical_language>

Target:
public ThreadMeta update(ThreadMeta newMeta) { if (newMeta.threadName != null && !newMeta.threadName.isEmpty()) { return new ThreadMeta(newMeta.threadId, newMeta.threadName); } return this; }
Source:
<code>private static byte[] generateAdditionalAttributes(SignerConfig signerConfig) { if (signerConfig.apkSignerLineage == null) { return new byte[0]; } <START> throw new NotImplementedException(); <END> }</code><technical_language>UnsupportedOperation instead</technical_language>

Target:
private static byte[] generateAdditionalAttributes(SignerConfig signerConfig) { if (signerConfig.mSigningCertificateLineage == null) { return new byte[0]; } throw new UnsupportedOperationException(); }
Source:
<code>public void testInsertWithColumnNames() throws Exception { <START> execute("create table test (\"firstName\" string, \"lastName\" string)"); <END> execute("insert into test (\"firstName\", \"lastName\") values('Youri', 'Zoon')"); assertEquals(1, response.rowCount()); refresh(); execute("select * from test where \"firstName\" = 'Youri'"); assertEquals(1, response.rowCount()); assertEquals("Youri", response.rows()[0][0]); assertEquals("Zoon", response.rows()[0][1]); }</code><technical_language>lets new name text</technical_language>

Target:
public void testInsertWithColumnNames() throws Exception { execute("create table test (\"firstName\" text, \"lastName\" text)"); execute("insert into test (\"firstName\", \"lastName\") values ('Youri', 'Zoon')"); assertEquals(1, response.rowCount()); refresh(); execute("select * from test where \"firstName\" = 'Youri'"); assertEquals(1, response.rowCount()); assertEquals("Youri", response.rows()[0][0]); assertEquals("Zoon", response.rows()[0][1]); }
Source:
<code>private void registerWithSubDirectories(AbstractWatchService watchService, Path toWatch) throws IOException { Files.walkFileTree(toWatch, EnumSet.of(FileVisitOption.FOLLOW_LINKS), Integer.MAX_VALUE, new SimpleFileVisitor<Path>() { @Override public FileVisitResult preVisitDirectory(Path subDir, BasicFileAttributes attrs) throws IOException { Kind<?>[] kinds = watchService.getWatchEventKinds(subDir); if (kinds != null) { registerDirectoryInternal(watchService, kinds, subDir); } return FileVisitResult.CONTINUE; } @Override public FileVisitResult visitFileFailed(Path file, IOException exc) throws IOException { if (exc instanceof AccessDeniedException) { <START> logger.warn("Do not have permission to access directory '{}', skipping it", <END> file.toAbsolutePath().toString()); } return FileVisitResult.SKIP_SUBTREE; } }); }</code><technical_language>"Access folder '{}' denied, skipping it."</technical_language>

Target:
private void registerWithSubDirectories(AbstractWatchService watchService, Path toWatch) throws IOException { Files.walkFileTree(toWatch, EnumSet.of(FileVisitOption.FOLLOW_LINKS), Integer.MAX_VALUE, new SimpleFileVisitor<Path>() { @Override public FileVisitResult preVisitDirectory(Path subDir, BasicFileAttributes attrs) throws IOException { Kind<?>[] kinds = watchService.getWatchEventKinds(subDir); if (kinds != null) { registerDirectoryInternal(watchService, kinds, subDir); } return FileVisitResult.CONTINUE; } @Override public FileVisitResult visitFileFailed(Path file, IOException exc) throws IOException { if (exc instanceof AccessDeniedException) { logger.warn("Access to folder '{}' was denied, therefore skipping it.", file.toAbsolutePath().toString()); } return FileVisitResult.SKIP_SUBTREE; } }); }
Source:
<code>private void endCurrentPlayerTurn() { <START> if (this.currentPlayerIndex < this.playerList.size()) <END> this.currentPlayerIndex++; else this.currentPlayerIndex = 0; }</code><technical_language>Please a mod here..</technical_language>

Target:
private void endCurrentPlayerTurn() { this.currentPlayerIndex = (this.currentPlayerIndex + 1) % NUMBER_OF_PLAYERS; }
Source:
<code>private static void validatePKConstraints(TableSchema schema) { if (!schema.getPrimaryKey().isPresent()) { <START> throw new ValidationException("PK constraints must be defined in the 'upsert-kafka' connector."); <END> } for (String key: schema.getPrimaryKey().get().getColumns()) { if (!schema.getTableColumn(key).get().isPhysical()) { throw new ValidationException( "Define the PK on the physical columns rather than metadata in the 'upsert-kafka' connector."); } } }</code><technical_language>suggestion throw new ValidationException( "'upsert-kafka' tables require define a PRIMARY KEY constraint. " + "The PRIMARY KEY specifies columns read write Kafka message key. " + "The PRIMARY KEY defines records in 'upsert-kafka' table update delete keys.");</technical_language>

Target:
private static void validatePKConstraints(TableSchema schema) { if (!schema.getPrimaryKey().isPresent()) { throw new ValidationException( "'upsert-kafka' tables require to define a PRIMARY KEY constraint. " + "The PRIMARY KEY specifies which columns should be read from or write to the Kafka message key. " + "The PRIMARY KEY also defines records in the 'upsert-kafka' table should update or delete on which keys."); } }
Source:
<code>public void getLocale_noContext() { final ActivityController<MainActivity> controller = Robolectric.buildActivity(MainActivity.class); controller.setup(); final Locale locale = Tools.getLocale(null); Assert.assertThat("If no context supplied, should always be the same", locale, is(equalTo(Locale.getDefault()))); RuntimeEnvironment.setQualifiers("de-rDE"); controller.configurationChange(); final Locale localeAfterChange = Tools.getLocale(null); <START> Assert.assertThat("If no context supplied, should always be the same", locale, <END> is(equalTo(Locale.getDefault()))); }</code><technical_language>Yes, intentional, is "reason" string beginning of assertThat is same. Context is supplied, Locale.getDefault is stays app runtime; test is for: Showing this is happens</technical_language>

Target:
public void getLocale_noContext() { final ActivityController<MainActivity> controller = Robolectric.buildActivity(MainActivity.class); controller.setup(); final Locale locale = Tools.getLocale(null); Assert.assertThat("If no context supplied, should always be the same", locale, is(equalTo(Locale.getDefault()))); RuntimeEnvironment.setQualifiers("de-rDE"); controller.configurationChange(); final Locale localeAfterChange = Tools.getLocale(null); Assert.assertThat("If no context supplied, should always be the same", localeAfterChange, is(equalTo(Locale.getDefault()))); }
Source:
<code>private void setCache(String lastModified, String cacheControl, String expires, String etag, String url, String date, Pair<String, String> varyHeader) { if (log.isDebugEnabled()){ log.debug("setCache(" + lastModified + "," + cacheControl + "," + expires + "," + etag + "," + url + "," + date + ")"); } Date expiresDate = null; if (useExpires) { final String maxAge = "max-age="; if(cacheControl != null && cacheControl.contains("no-store")) { return; } if (expires != null) { expiresDate = extractExpiresDateFromExpires(expires); } if(cacheControl != null && !cacheControl.contains("no-cache")) { expiresDate = extractExpiresDateFromCacheControl(lastModified, cacheControl, expires, etag, url, date, maxAge); } } if (varyHeader != null) { log.debug("Set entry into cache for url and vary ({})", url, varyHeader, <START> varyUrl(url, varyHeader.getLeft(), varyHeader.getRight())); <END> getCache().put(url, new CacheEntry(lastModified, expiresDate, etag, varyHeader.getLeft())); getCache().put(varyUrl(url, varyHeader.getLeft(), varyHeader.getRight()), new CacheEntry(lastModified, expiresDate, etag, null)); } else { getCache().put(url, new CacheEntry(lastModified, expiresDate, etag, null)); } }</code><technical_language>Please add isDebugEnabled check prevent varyUrl computation in debug disabled case</technical_language>

Target:
private void setCache(String lastModified, String cacheControl, String expires, String etag, String url, String date, Pair<String, String> varyHeader) { if (log.isDebugEnabled()){ log.debug("setCache(" + lastModified + "," + cacheControl + "," + expires + "," + etag + "," + url + "," + date + ")"); } Date expiresDate = null; if (useExpires) { final String maxAge = "max-age="; if(cacheControl != null && cacheControl.contains("no-store")) { return; } if (expires != null) { expiresDate = extractExpiresDateFromExpires(expires); } if(cacheControl != null && !cacheControl.contains("no-cache")) { expiresDate = extractExpiresDateFromCacheControl(lastModified, cacheControl, expires, etag, url, date, maxAge, expiresDate); } } if (varyHeader != null) { if (log.isDebugEnabled()) { log.debug("Set entry into cache for url and vary ({})", url, varyHeader, varyUrl(url, varyHeader.getLeft(), varyHeader.getRight())); } getCache().put(url, new CacheEntry(lastModified, expiresDate, etag, varyHeader.getLeft())); getCache().put(varyUrl(url, varyHeader.getLeft(), varyHeader.getRight()), new CacheEntry(lastModified, expiresDate, etag, null)); } else { getCache().put(url, new CacheEntry(lastModified, expiresDate, etag, null)); } }
Source:
<code>public void testPathRegexDoesntMatchPartially() throws Exception { final DefaultConfiguration checkConfig = createCheckConfig(ImportControlCheck.class); checkConfig.addAttribute("file", getResourcePath("import-control_one.xml")); <START> checkConfig.addAttribute("path", "main"); <END> final String[] expected = CommonUtils.EMPTY_STRING_ARRAY; verify(checkConfig, getPath("InputImportControl.java"), expected); }</code><technical_language>careful single words in a regular expression directory name match. I similar RegExpOnFileName CI worked fine, failed locally for @romani of specific directory structure naming of checkstyle project. slashes in this name match full word main in it. Ex: **main**tain, **main**tenance Do need MatchPartially tests for coverage? If not, romani disagree, I remove them</technical_language>

Target:
public void testPathRegexDoesntMatchPartially() throws Exception { final DefaultConfiguration checkConfig = createCheckConfig(ImportControlCheck.class); checkConfig.addAttribute("file", getResourcePath("import-control_one.xml")); checkConfig.addAttribute("path", "[\\\\/]NoMatch\\.java"); final String[] expected = CommonUtils.EMPTY_STRING_ARRAY; verify(checkConfig, getPath("InputImportControl.java"), expected); }
Source:
<code>public void shouldConflictIfImmutableTimestampLockExpiresEvenIfNoWritesOnThoroughSweptTable() { TimelockService timelock = mock(TimelockService.class); LockService mockLockService = mock(LockService.class); TransactionManager txnManagerWithMocks = new SerializableTransactionManager(metricsManager, keyValueService, timelock, mockLockService, transactionService, () -> AtlasDbConstraintCheckingMode.FULL_CONSTRAINT_CHECKING_THROWS_EXCEPTIONS, conflictDetectionManager, sweepStrategyManager, NoOpCleaner.INSTANCE, () -> AtlasDbConstants.DEFAULT_TIMESTAMP_CACHE_SIZE, false, () -> AtlasDbConstants.DEFAULT_TRANSACTION_LOCK_ACQUIRE_TIMEOUT_MS, AbstractTransactionTest.GET_RANGES_THREAD_POOL_SIZE, AbstractTransactionTest.DEFAULT_GET_RANGES_CONCURRENCY, MultiTableSweepQueueWriter.NO_OP, MoreExecutors.newDirectExecutorService()); <START> setupTimeLock(timelock); <END> assertThatThrownBy(() -> txnManagerWithMocks.runTaskThrowOnConflict(txn -> { get(txn, TEST_TABLE_THOROUGH, "row1", "col1"); return null; })) .isInstanceOf(TransactionFailedRetriableException.class); }</code><technical_language>of setup is duplicated, setup arguments return a TransactionManager instead</technical_language>

Target:
public void shouldConflictIfImmutableTimestampLockExpiresEvenIfNoWritesOnThoroughSweptTable() { TransactionManager txnManagerWithMocks = setupTransactionManager(); assertThatThrownBy(() -> txnManagerWithMocks.runTaskThrowOnConflict(txn -> { get(txn, TEST_TABLE_THOROUGH, "row1", "col1"); return null; })) .isInstanceOf(TransactionFailedRetriableException.class); }