auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public RelNode align(Aggregate rel, List<RelFieldCollation> collations) { LinkedHashSet<Integer> aggregateColumnsOrder = new LinkedHashSet<>(); ImmutableList.Builder<RelFieldCollation> propagateCollations = ImmutableList.builder(); if (rel.getGroupType() == Group.SIMPLE && !collations.isEmpty()) { for (RelFieldCollation c : collations) { if (c.getFieldIndex() < rel.getGroupCount()) { if (aggregateColumnsOrder.add(c.getFieldIndex())) { propagateCollations.add(c.copy(rel.getGroupSet().nth(c.getFieldIndex()))); } } } } for (int i = 0; i < rel.getGroupCount(); i++) { if (!aggregateColumnsOrder.contains(i)) { propagateCollations.add(new RelFieldCollation(rel.getGroupSet().nth(i))); } } final RelNode child = dispatchAlign(rel.getInput(), propagateCollations.build()); <START> final Aggregate newAggregate = (Aggregate) rel.copy(rel.getTraitSet(), <END> ImmutableList.of(child)); if (newAggregate instanceof HiveAggregate) { ((HiveAggregate) newAggregate).setAggregateColumnsOrder(aggregateColumnsOrder); } return newAggregate; }</code><technical_language>need change this, in fact this logic need run for operators pushed JDBC. Instead, in method public RelNode align(RelNode rel, List<RelFieldCollation> collations) (the general case), check if rel is instance of HiveJdbcConverter return node in case</technical_language>

Target:
public RelNode align(Aggregate rel, List<RelFieldCollation> collations) { LinkedHashSet<Integer> aggregateColumnsOrder = new LinkedHashSet<>(); ImmutableList.Builder<RelFieldCollation> propagateCollations = ImmutableList.builder(); if (rel.getGroupType() == Group.SIMPLE && !collations.isEmpty()) { for (RelFieldCollation c : collations) { if (c.getFieldIndex() < rel.getGroupCount()) { if (aggregateColumnsOrder.add(c.getFieldIndex())) { propagateCollations.add(c.copy(rel.getGroupSet().nth(c.getFieldIndex()))); } } } } for (int i = 0; i < rel.getGroupCount(); i++) { if (!aggregateColumnsOrder.contains(i)) { propagateCollations.add(new RelFieldCollation(rel.getGroupSet().nth(i))); } } final RelNode child = dispatchAlign(rel.getInput(), propagateCollations.build()); final HiveAggregate newAggregate = (HiveAggregate) rel.copy(rel.getTraitSet(), ImmutableList.of(child)); newAggregate.setAggregateColumnsOrder(aggregateColumnsOrder); return newAggregate; }
Source:
<code>public void trade() { <START> <END> for (Entry<String, ArrayList<Order>> entry : buyOrders.entrySet()) { String stockKey = entry.getKey(); ArrayList<Order> stockBuyList = entry.getValue(); if (stockBuyList != null) { if (sellOrders.get(stockKey) != null) { ArrayList<Order> stockSellList = sellOrders.get(stockKey); for (Order buy : stockBuyList) { for (Order sell : stockSellList) { if (buy.getPrice() == sell.getPrice()) { Trader buyTrader = buy.getTrader(); Trader sellTrader = sell.getTrader(); try { buyTrader.tradePerformed(buy, buy.getPrice()); } catch (StockMarketExpection e) { e.printStackTrace(); } try { sellTrader.tradePerformed(sell, sell.getPrice()); } catch (StockMarketExpection e) { e.printStackTrace(); } try { market.updateStockPrice(buy.getStockSymbol(), buy.getPrice()); } catch (StockMarketExpection e) { e.printStackTrace(); } stockBuyList.remove(buy); stockSellList.remove(sell); buyOrders.put(stockKey, stockBuyList); sellOrders.put(stockKey, stockSellList); } } } } } } }</code><technical_language>Do need amount of for loops if statements?! If time try a algorithm</technical_language>

Target:
public void trade() { for (Entry<String, ArrayList<Order>> entry : buyOrders.entrySet()) { String stockKey = entry.getKey(); ArrayList<Order> stockBuyList = entry.getValue(); if (stockBuyList != null && sellOrders.get(stockKey) != null) { ArrayList<Order> stockSellList = sellOrders.get(stockKey); for (Order buy : stockBuyList) { for (Order sell : stockSellList) { if (buy.getPrice() == sell.getPrice()) { Trader buyTrader = buy.getTrader(); Trader sellTrader = sell.getTrader(); try { buyTrader.tradePerformed(buy, buy.getPrice()); } catch (StockMarketExpection e) { e.printStackTrace(); } try { sellTrader.tradePerformed(sell, sell.getPrice()); } catch (StockMarketExpection e) { e.printStackTrace(); } try { market.updateStockPrice(buy.getStockSymbol(), buy.getPrice()); } catch (StockMarketExpection e) { e.printStackTrace(); } stockBuyList.remove(buy); stockSellList.remove(sell); buyOrders.put(stockKey, stockBuyList); sellOrders.put(stockKey, stockSellList); } } } } } }
Source:
<code>private static void validatePlatformKeys( ImmutableList<ConfiguredTargetKey> platformKeys, Environment env) throws InterruptedException, InvalidPlatformException { <START> Map<ConfiguredTargetKey, PackageIdentifier> targetsToPackageIdentifiers = <END> platformKeys.stream() .distinct() .collect( toImmutableMap(Function.identity(), ctk -> ctk.getLabel().getPackageIdentifier())); ImmutableSet<PackageValue.Key> packageKeys = targetsToPackageIdentifiers.values().stream() .distinct() .map(PackageValue::key) .collect(toImmutableSet()); Map<PackageIdentifier, Package> packages = new HashMap<>(); Map<SkyKey, ValueOrException<NoSuchPackageException>> values = env.getValuesOrThrow(packageKeys, NoSuchPackageException.class); if (env.valuesMissing()) { return; } for (Map.Entry<SkyKey, ValueOrException<NoSuchPackageException>> value : values.entrySet()) { try { PackageValue packageValue = (PackageValue) value.getValue().get(); packages.put(packageValue.getPackage().getPackageIdentifier(), packageValue.getPackage()); } catch (NoSuchPackageException e) { } } HasPlatformInfo hasPlatformInfo = HasPlatformInfo.create(); for (ConfiguredTargetKey platformKey : platformKeys) { try { Label platformLabel = platformKey.getLabel(); Target target = packages.get(platformLabel.getPackageIdentifier()).getTarget(platformLabel.getName()); if (!hasPlatformInfo.hasPlatformInfo(target)) { throw new InvalidPlatformException(platformLabel); } } catch (NoSuchTargetException e) { } } }</code><technical_language>This map create another collection out of values. Consider merging stream operations and/or making this a set of package identifiers</technical_language>

Target:
private static void validatePlatformKeys( ImmutableList<ConfiguredTargetKey> platformKeys, Environment env) throws InterruptedException, InvalidPlatformException { ImmutableSet<PackageValue.Key> packageKeys = platformKeys.stream() .map(ConfiguredTargetKey::getLabel) .map(Label::getPackageIdentifier) .distinct() .map(PackageValue::key) .collect(toImmutableSet()); Map<SkyKey, ValueOrException<NoSuchPackageException>> values = env.getValuesOrThrow(packageKeys, NoSuchPackageException.class); boolean valuesMissing = env.valuesMissing(); Map<PackageIdentifier, Package> packages = valuesMissing ? null : new HashMap<>(); for (Map.Entry<SkyKey, ValueOrException<NoSuchPackageException>> value : values.entrySet()) { try { PackageValue packageValue = (PackageValue) value.getValue().get(); if (!valuesMissing && packageValue != null) { packages.put(packageValue.getPackage().getPackageIdentifier(), packageValue.getPackage()); } } catch (NoSuchPackageException e) { throw new InvalidPlatformException(e); } } if (valuesMissing) { return; } for (ConfiguredTargetKey platformKey : platformKeys) { try { Label platformLabel = platformKey.getLabel(); Target target = packages.get(platformLabel.getPackageIdentifier()).getTarget(platformLabel.getName()); if (!hasPlatformInfo(target)) { throw new InvalidPlatformException(platformLabel); } } catch (NoSuchTargetException e) { throw new InvalidPlatformException(e); } } }
Source:
<code>public Datastream getDatastream(String datastreamName) throws DatastreamException { GetRequest request = _builders.get().id(datastreamName).build(); ResponseFuture<Datastream> datastreamResponseFuture = _restClient.sendRequest(request); Response<Datastream> response; try { response = datastreamResponseFuture.getResponse(); return response.getEntity(); } catch (RemoteInvocationException e) { if(e instanceof RestLiResponseException && ((RestLiResponseException) e).getStatus() == HttpStatus.S_404_NOT_FOUND.getCode()) { throw new DatastreamNotFoundException(datastreamName, e); } else { <START> throw new DatastreamException(String.format("Get Datastream {%s} failed with error %s", datastreamName, e.toString()), e); <END> } } }</code><technical_language>innerexception do need print exception? exception printed print outer exception</technical_language>

Target:
public Datastream getDatastream(String datastreamName) throws DatastreamException { GetRequest request = _builders.get().id(datastreamName).build(); ResponseFuture<Datastream> datastreamResponseFuture = _restClient.sendRequest(request); Response<Datastream> response; try { response = datastreamResponseFuture.getResponse(); return response.getEntity(); } catch (RemoteInvocationException e) { if(e instanceof RestLiResponseException && ((RestLiResponseException) e).getStatus() == HttpStatus.S_404_NOT_FOUND.getCode()) { throw new DatastreamNotFoundException(datastreamName, e); } else { throw new DatastreamException(String.format("Get Datastream {%s} failed with error.", datastreamName), e); } } }
Source:
<code>private void executeTaskListener(TaskExecution taskExecution, Set<Method> methods, Map<Method, Object> instances){ for (Method method : methods) { try { method.invoke(instances.get(method),taskExecution); } catch (IllegalAccessException e) { throw new IllegalStateException("@BeforeTask and @AfterTask annotated methods must be public."); } catch (InvocationTargetException e) { <START> throw new IllegalStateException("Failed to process @BeforeTask or @AfterTask" + <END> "annotation because: ", e); } catch (IllegalArgumentException e){ throw new IllegalStateException("taskExecution parameter is required for @BeforeTask and @AfterTask annotated methods"); } } }</code><technical_language>IllegalStateException sense here? Is a state issue</technical_language>

Target:
private void executeTaskListener(TaskExecution taskExecution, Set<Method> methods, Map<Method, Object> instances){ for (Method method : methods) { try { method.invoke(instances.get(method),taskExecution); } catch (IllegalAccessException e) { throw new TaskExecutionException("@BeforeTask and @AfterTask annotated methods must be public.", e); } catch (InvocationTargetException e) { throw new TaskExecutionException("Failed to process @BeforeTask or @AfterTask" + "annotation because: ", e); } catch (IllegalArgumentException e){ throw new TaskExecutionException("taskExecution parameter is required for @BeforeTask and @AfterTask annotated methods", e); } } }
Source:
<code>public abstract String retrieveSubjectToken() throws IOException; <START> public HttpTransportFactory getTransportFactory() { <END> return transportFactory; }</code><technical_language>this need public</technical_language>

Target:
public abstract String retrieveSubjectToken() throws IOException; public String getAudience() { return audience; }
Source:
<code>public RangeBubble() { <START> getModel().setValue(50); <END> getModel().setEnd(80); JsonArray values = Json.createArray(); for (int i = 0; i < 100; i += 5) { values.set(values.length(), String.valueOf(i)); } rangeEnd.getElement().setPropertyJson("items", values); rangeEnd.getElement().setProperty("value", Integer.toString(getModel().getEnd())); value.getElement().setPropertyJson("items", values); value.getElement().setProperty("value", Integer.toString(getModel().getValue())); rangeEnd.getElement().addEventListener("value-changed", e -> { getModel().setEnd(Integer .parseInt(e.getEventData().getString("element.value"))); }, "element.value"); value.getElement().addEventListener("value-changed", e -> { getModel().setValue(Integer .parseInt(e.getEventData().getString("element.value"))); }, "element.value"); }</code><technical_language>![MAJOR](<LINK_1> Remove this call a constructor overridable "getModel" method. [![rule](<LINK_2>](<LINK_0></technical_language>

Target:
public RangeBubble() { getModel().setValue(50); getModel().setEnd(80); List<String> values = new ArrayList<>(); for (int i = 0; i < 100; i += 5) { values.add(String.valueOf(i)); } rangeEnd.setItems(values); value.setItems(values); rangeEnd.setValue(Integer.toString(getModel().getEnd())); value.setValue(Integer.toString(getModel().getValue())); rangeEnd.getElement().addEventListener("value-changed", e -> getModel().setEnd(Integer .parseInt(e.getEventData().getString(ELEMENT_VALUE))), ELEMENT_VALUE); value.getElement().addEventListener("value-changed", e -> getModel().setValue(Integer .parseInt(e.getEventData().getString(ELEMENT_VALUE))), ELEMENT_VALUE); }
Source:
<code>private List<Person> makeWriters(Element cast) { List<Person> people = new ArrayList<>(); Elements writers = cast.getChildElements("actor"); for (int i = 0; i < writers.size(); i++) { Element writer = writers.get(i); people.add(makeWriter(writer)); } <START> return people; <END> }</code><technical_language>stream & collect</technical_language>

Target:
private List<Person> makeWriters(Element cast) { List<Person> people = Lists.newArrayList(); Elements writers = cast.getChildElements(ACTOR); for (int i = 0; i < writers.size(); i++) { Element writer = writers.get(i); people.add(makeWriter(writer)); } return people; }
Source:
<code>private ChangeControl control(ChangeData cd) throws OrmException { if (lastControl != null && cd.getId().equals(lastControl.getChange().getId())) { return lastControl; } ChangeControl ctrl; ProjectControl projectCtrl; try { projectCtrl = projectControls.get(cd.change().getProject()); if (projectCtrl != null) { ctrl = projectCtrl.controlFor(cd.change()); } else { <START> ctrl = cd.changeControl(); <END> } } catch (NoSuchChangeException | ExecutionException e) { throw new OrmException(e); } lastControl = ctrl; return ctrl; }</code><technical_language>build control for change owner, caller is running this ChangeJson</technical_language>

Target:
private ChangeControl control(ChangeData cd) throws OrmException { if (lastControl != null && cd.getId().equals(lastControl.getChange().getId())) { return lastControl; } ChangeControl ctrl; ProjectControl projectCtrl; try { projectCtrl = projectControls.get(cd.change().getProject()); if (projectCtrl != null) { ctrl = projectCtrl.controlFor(cd.change()); } else { ctrl = cd.changeControl().forUser(userProvider.get()); } } catch (NoSuchChangeException | ExecutionException e) { throw new OrmException(e); } lastControl = ctrl; return ctrl; }
Source:
<code>public void saveToBarFolder(final BusinessArchive businessArchive, final File barFolder) throws IOException { final ActorMapping actorMapping = businessArchive.getActorMapping(); if (actorMapping != null) { try { <START> final byte[] fileContent = new ActorMappingConverter().serializeToXML(actorMapping); <END> final File file = new File(barFolder, ACTOR_MAPPING_FILE); IOUtil.write(file, fileContent); } catch (JAXBException | org.xml.sax.SAXException e) { throw new IOException("Cannot write Actor Mapping to Bar folder", e); } } }</code><technical_language>IMHO ActorMappingConverter passed ActorMappingContribution constructor</technical_language>

Target:
public void saveToBarFolder(final BusinessArchive businessArchive, final File barFolder) throws IOException { final ActorMapping actorMapping = businessArchive.getActorMapping(); if (actorMapping != null) { try { final byte[] fileContent = new ActorMappingMarshaller().serializeToXML(actorMapping); final File file = new File(barFolder, ACTOR_MAPPING_FILE); IOUtil.write(file, fileContent); } catch (XmlMarshallException e) { throw new IOException("Cannot write Actor Mapping to Bar folder", e); } } }
Source:
<code>public void testNullFileSystemOptions() { VertxOptions options = new VertxOptions().setFileSystemOptions(null); try { options.isFileResolverCachingEnabled(); options.setFileResolverCachingEnabled(true); } catch (NullPointerException e) { <START> fail("Should handle null FileSystemOptions"); <END> } }</code><technical_language>NPE fail test</technical_language>

Target:
public void testNullFileSystemOptions() { VertxOptions options = new VertxOptions().setFileSystemOptions(null); options.isFileResolverCachingEnabled(); options.setFileResolverCachingEnabled(true); }
Source:
<code>public void setFont (Font font) { checkWidget(); if (((state & FONT) == 0) && font == null) return; this.font = font; long fontDesc; if (font == null) { fontDesc = defaultFont ().handle; } else { if (font.isDisposed ()) error(SWT.ERROR_INVALID_ARGUMENT); fontDesc = font.handle; } if (font == null) { state &= ~FONT; } else { state |= FONT; } <START> if (OS.GTK_VERSION<OS.VERSION(3, 22, 0)){ <END> setFontDescription (fontDesc); } else { long context = OS.gtk_widget_get_style_context (handle); OS.gtk_style_context_add_provider (context, provider, OS.GTK_STYLE_PROVIDER_PRIORITY_APPLICATION); long family = OS.pango_font_description_get_family(fontDesc); String name; if (family == 0) { name = "*"; } else { int length = OS.strlen (family); byte buffer = new byte [length]; OS.memmove (buffer, family, length); name = new String (Converter.mbcsToWcs (null, buffer)); } String cssName = OS.GTK_VERSION >= OS.VERSION(3, 20, 0) ? display.gtk_widget_class_get_css_name(handle) : display.gtk_widget_get_name(handle); int height = OS.pango_font_description_get_size(fontDesc) / OS.PANGO_SCALE; String css = cssName + " { font: "+ name +" "+height+"; "+"}"; gtk_css_provider_load_from_css(context,css); } }</code><technical_language>Spaces side of "<" sign</technical_language>

Target:
public void setFont (Font font) { checkWidget(); if (((state & FONT) == 0) && font == null) return; this.font = font; long fontDesc; if (font == null) { fontDesc = defaultFont ().handle; } else { if (font.isDisposed ()) error(SWT.ERROR_INVALID_ARGUMENT); fontDesc = font.handle; } if (font == null) { state &= ~FONT; } else { state |= FONT; } if (OS.GTK_VERSION < OS.VERSION(3, 22, 0)){ setFontDescription (fontDesc); } else { long context = OS.gtk_widget_get_style_context (handle); long family = OS.pango_font_description_get_family(fontDesc); String name; if (family == 0) { return; } else { int length = OS.strlen (family); byte buffer = new byte [length]; OS.memmove (buffer, family, length); name = new String (Converter.mbcsToWcs (null, buffer)); } String cssName = OS.GTK_VERSION >= OS.VERSION(3, 20, 0) ? display.gtk_widget_class_get_css_name(handle) : display.gtk_widget_get_name(handle); int height = OS.pango_font_description_get_size(fontDesc) / OS.PANGO_SCALE; String css = cssName + " {font: "+ name +" "+height+";}"; gtk_css_provider_load_from_css(context,css); } }
Source:
<code>public void toggleSortOrder(int column) { <START> LOGGER.debug("toggleSortOrder"+column); <END> if(isSortable(column)) { SortKey sortRequest=new SortKey(column, SortOrder.ASCENDING); for(int i=0;i<sortedColumns.size();i++) { SortKey col = sortedColumns.get(i); if(col.getColumn()==column) { SortOrder order; if(col.getSortOrder().equals(SortOrder.ASCENDING)) { order = SortOrder.DESCENDING; } else { order = SortOrder.ASCENDING; } sortRequest = new SortKey(column, order); break; } } launchSortProcess(sortRequest); } }</code><technical_language>nicer actual sentence here</technical_language>

Target:
public void toggleSortOrder(int column) { if(isSortable(column)) { SortKey sortRequest=new SortKey(column, SortOrder.ASCENDING); for(int i=0;i<sortedColumns.size();i++) { SortKey col = sortedColumns.get(i); if(col.getColumn()==column) { SortOrder order; if(col.getSortOrder().equals(SortOrder.ASCENDING)) { order = SortOrder.DESCENDING; } else { order = SortOrder.ASCENDING; } sortRequest = new SortKey(column, order); break; } } launchSortProcess(sortRequest); } }
Source:
<code>public void connectToSNIProxyDocker() { Properties gemFireProps = new Properties(); gemFireProps.setProperty(SSL_ENABLED_COMPONENTS, "all"); gemFireProps.setProperty(SSL_KEYSTORE_TYPE, "jks"); gemFireProps.setProperty(SSL_REQUIRE_AUTHENTICATION, "false"); gemFireProps.setProperty(SSL_KEYSTORE, keyStorePath); gemFireProps.setProperty(SSL_KEYSTORE_PASSWORD, "geode"); gemFireProps.setProperty(SSL_TRUSTSTORE, trustStorePath); gemFireProps.setProperty(SSL_TRUSTSTORE_PASSWORD, "geode"); ENDPOINT_IDENTIFICATION_ENABLED, "true"); ClientCache cache = new ClientCacheFactory(gemFireProps) .addPoolLocator( "localhost", 55221) .create(); Region<String, String> region = <START> cache.<String, String>createClientRegionFactory(ClientRegionShortcut.CACHING_PROXY) <END> .create("jellyfish"); region.destroy("hello"); region.put("hello", "world"); assertThat(region.get("hello")).isEqualTo("world"); }</code><technical_language>empty region here. a CACHING_PROXY get() operation is find locally</technical_language>

Target:
public void connectToSNIProxyDocker() { Properties gemFireProps = new Properties(); gemFireProps.setProperty(SSL_ENABLED_COMPONENTS, "all"); gemFireProps.setProperty(SSL_KEYSTORE_TYPE, "jks"); gemFireProps.setProperty(SSL_REQUIRE_AUTHENTICATION, "false"); gemFireProps.setProperty(SSL_KEYSTORE, keyStorePath); gemFireProps.setProperty(SSL_KEYSTORE_PASSWORD, "geode"); gemFireProps.setProperty(SSL_TRUSTSTORE, trustStorePath); gemFireProps.setProperty(SSL_TRUSTSTORE_PASSWORD, "geode"); gemFireProps.setProperty(SSL_ENDPOINT_IDENTIFICATION_ENABLED, "true"); ClientCache cache = new ClientCacheFactory(gemFireProps) .addPoolLocator( "locator", 10334) .create(); Region<String, String> region = cache.<String, String>createClientRegionFactory(ClientRegionShortcut.PROXY) .create("jellyfish"); region.destroy("hello"); region.put("hello", "world"); assertThat(region.get("hello")).isEqualTo("world"); }
Source:
<code>@Test public void typeMustBeSupported() { JavaFileObject source = JavaFileObjects.forSourceString("test.Test", Joiner.on('\n').join( "package test;", "import android.app.Activity;", "import butterknife.BindArray;", "public class Test extends Activity {", " @BindArray(1) String one;", "}" )); ASSERT.about(javaSource()).that(source) .processedWith(new ButterKnifeProcessor()) .failsToCompile() .withErrorContaining( "@BindArray field type must be one of: 'String[], int[], CharSequence[], " <START> + "android.content.res.TypedArray'. (test.Test.one)") <END> .in(source).onLine(5); }</code><technical_language>I single quotes in this error message add anything. Drop 'em</technical_language>

Target:
@Test public void typeMustBeSupported() { JavaFileObject source = JavaFileObjects.forSourceString("test.Test", Joiner.on('\n').join( "package test;", "import android.app.Activity;", "import butterknife.BindArray;", "public class Test extends Activity {", " @BindArray(1) String one;", "}" )); ASSERT.about(javaSource()).that(source) .processedWith(new ButterKnifeProcessor()) .failsToCompile() .withErrorContaining( "@BindArray field type must be one of: String[], int[], CharSequence[], " + "android.content.res.TypedArray. (test.Test.one)") .in(source).onLine(5); }
Source:
<code>public TitleSearchGenerator( SearchResolver searchResolver, Class<? extends T> cls, Iterable<Publisher> publishers, Function<String,String> titleTransform, int searchLimit, Score exactMatchScore, Score partialMatchBound, boolean includeSelfPublisher, boolean useContentSpecialization, boolean changeComponentName ) { this.searchResolver = searchResolver; this.cls = cls; this.searchLimit = searchLimit; this.searchPublishers = ImmutableSet.copyOf(publishers); this.titleTransform = titleTransform; <START> this.name = prependIfDeerInstance( <END> changeComponentName ? "Title Search Generator" : "Title" ); this.titleScorer = new ContentTitleScorer<>( name, titleTransform, exactMatchScore, partialMatchBound ); this.titleExpander = new ExpandingTitleTransformer(); this.includeSelfPublisher = includeSelfPublisher; this.useContentSpecialization = useContentSpecialization; }</code><technical_language>If Deer call "Deer Title Search Generator" in cases in opinion</technical_language>

Target:
public TitleSearchGenerator( SearchResolver searchResolver, Class<? extends T> cls, Iterable<Publisher> publishers, Function<String,String> titleTransform, int searchLimit, Score exactMatchScore, Score partialMatchBound, boolean includeSelfPublisher, boolean useContentSpecialization, boolean changeComponentName ) { this.searchResolver = searchResolver; this.cls = cls; this.searchLimit = searchLimit; this.searchPublishers = ImmutableSet.copyOf(publishers); this.titleTransform = titleTransform; this.name = getName(changeComponentName); this.titleScorer = new ContentTitleScorer<>( name, titleTransform, exactMatchScore, partialMatchBound ); this.titleExpander = new ExpandingTitleTransformer(); this.includeSelfPublisher = includeSelfPublisher; this.useContentSpecialization = useContentSpecialization; }
Source:
<code>public Object abort(@PathParam("executionId") String executionId) throws IOException, MessagingException { if (exists(executionId) && !isCompleted(executionId)) { if (isAsync(executionId)) { Serializable commandId = getCommandId(executionId); return getAsyncService(executionId).abort(commandId); } <START> return ResponseHelper.getResponse("RUNNING", request, Response.Status.OK.getStatusCode()); <END> } throw new WebResourceNotFoundException("Execution with id=" + executionId + " has completed"); }</code><technical_language>I for error case. allowed abort async async adapter</technical_language>

Target:
public Object abort(@PathParam("executionId") String executionId) throws IOException, MessagingException { if (exists(executionId) && !isCompleted(executionId)) { if (isAsync(executionId)) { Serializable taskId = getTaskId(executionId); return getAsyncService(executionId).abort(taskId); } return ResponseHelper.getResponse(RUNNING_STATUS, request, HttpServletResponse.SC_OK); } throw new WebResourceNotFoundException("Execution with id=" + executionId + " has completed"); }
Source:
<code>private void doInit(ExtMap inputMap, ExtMap outputMap) { context = inputMap.<ExtMap> get(Base.InvokeKeys.CONTEXT); configuration = context.<Properties> get(Base.ContextKeys.CONFIGURATION); <START> Utils.putDefaultsIfAbsent(configuration); <END> broker = LdapFactory.getInstance(getAuthzName()); context.<List<String>> get( Base.ContextKeys.CONFIGURATION_SENSITIVE_KEYS ).add("config.authn.user.password"); context.mput( Base.ContextKeys.AUTHOR, "The oVirt Project").mput( Base.ContextKeys.EXTENSION_NAME, "Kerberos/Ldap Authn (Built-in)" ).mput( Base.ContextKeys.LICENSE, "ASL 2.0" ).mput( Base.ContextKeys.HOME_URL, "<LINK_0>" ).mput( Base.ContextKeys.VERSION, "N/A" ).mput( Authn.ContextKeys.CAPABILITIES, Authn.Capabilities.AUTHENTICATE_PASSWORD ).mput( Base.ContextKeys.BUILD_INTERFACE_VERSION, Base.INTERFACE_VERSION_CURRENT); try { Utils.handleApplicationInit(System.getProperty("application.name")); } catch (Exception e) { outputMap.mput( Base.InvokeKeys.MESSAGE, e.getMessage() ).mput( Base.InvokeKeys.RESULT, Base.InvokeResult.FAILED ); } }</code><technical_language>set defaults name</technical_language>

Target:
private void doInit(ExtMap inputMap, ExtMap outputMap) { context = inputMap.<ExtMap> get(Base.InvokeKeys.CONTEXT); configuration = context.<Properties> get(Base.ContextKeys.CONFIGURATION); Utils.setDefaults(configuration); broker = LdapFactory.getInstance(getAuthzName()); context.<List<String>> get( Base.ContextKeys.CONFIGURATION_SENSITIVE_KEYS ).add("config.authn.user.password"); context.mput( Base.ContextKeys.AUTHOR, "The oVirt Project").mput( Base.ContextKeys.EXTENSION_NAME, "Kerberos/Ldap Authn (Built-in)" ).mput( Base.ContextKeys.LICENSE, "ASL 2.0" ).mput( Base.ContextKeys.HOME_URL, "<LINK_0>" ).mput( Base.ContextKeys.VERSION, "N/A" ).mput( Authn.ContextKeys.CAPABILITIES, Authn.Capabilities.AUTHENTICATE_PASSWORD ).mput( Base.ContextKeys.BUILD_INTERFACE_VERSION, Base.INTERFACE_VERSION_CURRENT); try { Utils.handleApplicationInit(context.<ExtMap>get(Base.ContextKeys.GLOBAL_CONTEXT).<String>get(Base.GlobalContextKeys.APPLICATION_NAME)); } catch (Exception e) { outputMap.mput( Base.InvokeKeys.MESSAGE, e.getMessage() ).mput( Base.InvokeKeys.RESULT, Base.InvokeResult.FAILED ); } }
Source:
<code>public void handleTransientDeployment(AuthenticatedContext authenticatedContext, List<CloudResource> resources) { <START> Optional<String> deploymentNameOpt = resources.stream() .filter(cloudResource -> cloudResource.getType() == ResourceType.ARM_TEMPLATE) .map(CloudResource::getName) .findFirst(); Optional<String> resourceGroupNameOpt = resources.stream() .filter(cloudResource -> cloudResource.getType() == ResourceType.AZURE_RESOURCE_GROUP) .map(CloudResource::getName) .findFirst(); <END> AzureClient client = authenticatedContext.getParameter(AzureClient.class); LOGGER.debug("Database template saved: {}, resource group saved: {}", deploymentNameOpt, resourceGroupNameOpt); if (deploymentNameOpt.isPresent() && resourceGroupNameOpt.isPresent()) { String resourceGroupName = resourceGroupNameOpt.get(); String deploymentName = deploymentNameOpt.get(); LOGGER.debug("Checking if database deployment {}.{} status is transient", resourceGroupName, deploymentName); ResourceStatus deploymentStatus = client.getTemplateDeploymentStatus(resourceGroupName, deploymentName); if (deploymentStatus.isTransient()) { LOGGER.info("Template deployment has transient status , cancelling it now.", deploymentName, deploymentStatus); Deployment deployment = client.getTemplateDeployment(resourceGroupName, deploymentName); deployment.cancel(); List<CloudResource> deployedResources = azureCloudResourceService.getDeploymentCloudResources(deployment); LOGGER.info("Found resources to be removed: {}", deployedResources); resources.addAll(deployedResources); } } }</code><technical_language>please extract streams a private method</technical_language>

Target:
public void handleTransientDeployment(AuthenticatedContext authenticatedContext, List<CloudResource> resources) { Optional<String> deploymentNameOpt = getFirstResourceName(resources, ARM_TEMPLATE); Optional<String> resourceGroupNameOpt = getFirstResourceName(resources, AZURE_RESOURCE_GROUP); LOGGER.debug("Database template saved: {}, resource group saved: {}", deploymentNameOpt, resourceGroupNameOpt); if (deploymentNameOpt.isPresent() && resourceGroupNameOpt.isPresent()) { AzureClient client = authenticatedContext.getParameter(AzureClient.class); String resourceGroupName = resourceGroupNameOpt.get(); String deploymentName = deploymentNameOpt.get(); LOGGER.debug("Checking if database deployment {}.{} status is transient", resourceGroupName, deploymentName); resources.addAll(azureTransientDeploymentService.handleTransientDeployment(client, resourceGroupName, deploymentName)); } }
Source:
<code>public ApplyDeltaCommand(Object deltaAwareValueKey, Delta delta, Collection<Object> keys, CommandInvocationId commandInvocationId) { <START> super(deltaAwareValueKey, -1, 0, commandInvocationId); <END> }</code><technical_language>segment = -1 throw IllegalArgumentException in AbstractDataCommand</technical_language>

Target:
public ApplyDeltaCommand(Object deltaAwareValueKey, Delta delta, Collection<Object> keys, CommandInvocationId commandInvocationId) { throw new UnsupportedOperationException(); }
Source:
<code>public void emitData(final I input) { if (isWatermark.test(input)) { latestWatermarkTimestamp = parseTimestamp.apply(input); outputEmitter.emitWatermark(new MistWatermarkEvent(latestWatermarkTimestamp)); } else if (generateEvent(input) != null) { <START> outputEmitter.emitData(generateEvent(input)); <END> } }</code><technical_language>invoking generateEvent (101, 102)</technical_language>

Target:
public void emitData(final I input) { if (isWatermark.test(input)) { latestWatermarkTimestamp = parseTimestamp.apply(input); outputEmitter.emitWatermark(new MistWatermarkEvent(latestWatermarkTimestamp)); } else { MistDataEvent newInputEvent = generateEvent(input); if (newInputEvent != null) { outputEmitter.emitData(newInputEvent); } } }
Source:
<code>@Override protected void onCreate(@Nullable Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_main); mMapView = findViewById(R.id.mapView); ArcGISMap map = new ArcGISMap(Basemap.Type.STREETS, 40, -95, 4); mFeatureTable = new ServiceFeatureTable(getString(R.string.feature_layer_url)); mFeatureLayer = new FeatureLayer(mFeatureTable); map.getOperationalLayers().add(mFeatureLayer); mMapView.setOnTouchListener(new DefaultMapViewOnTouchListener(this, mMapView) { @Override public boolean onSingleTapConfirmed(MotionEvent event) { <START> android.graphics.Point point = new android.graphics.Point((int) event.getX(), (int) event.getY()); <END> Point mapPoint = mMapView.screenToLocation(point); Point normalizedMapPoint = (Point) GeometryEngine.normalizeCentralMeridian(mapPoint); ListenableFuture<IdentifyLayerResult> results = mMapView.identifyLayerAsync(mFeatureLayer, point, 1, false); results.addDoneListener(() -> { try { IdentifyLayerResult layer = results.get(); if (layer.getElements().size() > 0 && layer.getElements().get(0) instanceof Feature) { inflateCallout(mMapView, layer.getElements().get(0), normalizedMapPoint).show(); } } catch (InterruptedException | ExecutionException e) { logToUser(getString(R.string.error_getting_identify_result, e.getCause().getMessage())); } }); return super.onSingleTapConfirmed(event); } }); mMapView.setMap(map); }</code><technical_language>call this a screenPoint for clarity</technical_language>

Target:
@Override protected void onCreate(@Nullable Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_main); mMapView = findViewById(R.id.mapView); ArcGISMap map = new ArcGISMap(Basemap.Type.STREETS, 40, -95, 4); mFeatureTable = new ServiceFeatureTable(getString(R.string.feature_layer_url)); mFeatureLayer = new FeatureLayer(mFeatureTable); map.getOperationalLayers().add(mFeatureLayer); mMapView.setOnTouchListener(new DefaultMapViewOnTouchListener(this, mMapView) { @Override public boolean onSingleTapConfirmed(MotionEvent event) { android.graphics.Point screenPoint = new android.graphics.Point((int) event.getX(), (int) event.getY()); Point mapPoint = mMapView.screenToLocation(screenPoint); ListenableFuture<IdentifyLayerResult> identifyLayerFuture = mMapView .identifyLayerAsync(mFeatureLayer, screenPoint, 1, false); identifyLayerFuture.addDoneListener(() -> { try { IdentifyLayerResult layer = identifyLayerFuture.get(); if (!layer.getElements().isEmpty() && layer.getElements().get(0) instanceof Feature) { inflateCallout(mMapView, layer.getElements().get(0), mapPoint).show(); } } catch (InterruptedException | ExecutionException e) { logToUser(true, getString(R.string.error_getting_identify_result, e.getCause().getMessage())); } }); return super.onSingleTapConfirmed(event); } }); mMapView.setMap(map); }
Source:
<code>public void setRefreshRate(int interval) { if (getRefreshRate() == interval) { return; } reset(); logger.fine("GridTimer[" + name + "]: Refresh Rate set to: " + interval); normalInterval = interval; <START> ValueChangeEvent.fire(this, String.valueOf(getRefreshRate())); <END> }</code><technical_language>Hm, a nit-picky question: GridTimer implement HasValueChangeHandlers<Integer> instead of current HasValueChangeHandlers<String></technical_language>

Target:
public void setRefreshRate(int interval) { if (getRefreshRate() == interval) { return; } reset(); logger.fine("GridTimer[" + name + "]: Refresh Rate set to: " + interval); normalInterval = interval; ValueChangeEvent.fire(this, getRefreshRate()); }
Source:
<code>public BAMIndexMetaData getMetaData(int reference) { <START> throw new RuntimeException("Getting of BAM index metadata for SRA is not implemented"); <END> }</code><technical_language>UnsupportedOperationException</technical_language>

Target:
public BAMIndexMetaData getMetaData(int reference) { throw new UnsupportedOperationException("Getting of BAM index metadata for SRA is not implemented"); }
Source:
<code>public void whenFirstCursorIsNotCommittedThenNextCursorsAreNotSkipped() throws Exception { final NakadiCursor c1 = new NakadiCursor(timeline, P1, OLDEST_OFFSET); final NakadiCursor c2 = new NakadiCursor(timeline, P2, NEW_OFFSET); testCursors = ImmutableList.of(c1, c2); final List<Boolean> result = cursorsService.commitCursors(streamId, sid, testCursors); assertEquals(Boolean.FALSE, result.get(0)); <START> assertEquals(Boolean.TRUE, result.get(1)); <END> checkCurrentOffsetInZk(P1, OLD_OFFSET); checkCurrentOffsetInZk(P2, NEW_OFFSET); }</code><technical_language>if hamcrest-style anyway, assertTrue assertFalse</technical_language>

Target:
public void whenFirstCursorIsNotCommittedThenNextCursorsAreNotSkipped() throws Exception { final NakadiCursor c1 = new NakadiCursor(timeline, P1, OLDEST_OFFSET); final NakadiCursor c2 = new NakadiCursor(timeline, P2, NEW_OFFSET); testCursors = ImmutableList.of(c1, c2); final List<Boolean> result = cursorsService.commitCursors(streamId, sid, testCursors); assertFalse(result.get(0)); assertTrue(result.get(1)); checkCurrentOffsetInZk(P1, OLD_OFFSET); checkCurrentOffsetInZk(P2, NEW_OFFSET); }
Source:
<code>public IValue getValue(IASTNode point) { if (isValueDependent()) return IntegralValue.create(this); if (isTypeDependent()) return IntegralValue.create(this); if (fRepresentsNewExpression) return IntegralValue.UNKNOWN; if (fInputType instanceof ICPPClassType) { ICPPClassType classType = (ICPPClassType)fInputType; IBinding ctor = getConstructor(null); if(EvalUtil.isCompilerGeneratedCtor(ctor)) { return CompositeValue.create(classType); } else { EvalConstructor evalCtor = new EvalConstructor(classType, (ICPPConstructor)ctor, fArguments, getTemplateDefinition()); ICPPEvaluation computedEvalCtor = evalCtor.computeForFunctionCall(new ActivationRecord(), new ConstexprEvaluationContext(point)); return computedEvalCtor.getValue(point); } } if (fArguments.length == 0 || isEmptyInitializerList(fArguments)) { if (fInputType instanceof ICPPBasicType) { switch(((ICPPBasicType) fInputType).getKind()) { case eInt: case eInt128: case eDouble: case eBoolean: case eFloat: case eFloat128: case eNullPtr: return IntegralValue.create(0l); case eChar: case eChar16: case eChar32: case eUnspecified: case eVoid: case eWChar: default: <START> return IntegralValue.UNKNOWN; <END> } } } if (fArguments.length == 1) { return fArguments[0].getValue(point); } return IntegralValue.UNKNOWN; }</code><technical_language>character types value-initialize zero</technical_language>

Target:
public IValue getValue(IASTNode point) { if (isValueDependent()) return IntegralValue.create(this); if (isTypeDependent()) return IntegralValue.create(this); if (fRepresentsNewExpression) return IntegralValue.UNKNOWN; if (fInputType instanceof ICPPClassType) { ICPPClassType classType = (ICPPClassType)fInputType; IBinding ctor = getConstructor(null); if(EvalUtil.isCompilerGeneratedCtor(ctor)) { return CompositeValue.create(classType); } else if (ctor == AGGREGATE_INITIALIZATION) { return IntegralValue.UNKNOWN; } else if (ctor != null) { EvalConstructor evalCtor = new EvalConstructor(classType, (ICPPConstructor)ctor, fArguments, getTemplateDefinition()); ICPPEvaluation computedEvalCtor = evalCtor.computeForFunctionCall(new ActivationRecord(), new ConstexprEvaluationContext(point)); return computedEvalCtor.getValue(point); } else { return IntegralValue.ERROR; } } if (fArguments.length == 0 || isEmptyInitializerList(fArguments)) { if (fInputType instanceof ICPPBasicType) { switch(((ICPPBasicType) fInputType).getKind()) { case eInt: case eInt128: case eDouble: case eBoolean: case eFloat: case eFloat128: case eNullPtr: return IntegralValue.create(0l); case eChar: case eChar16: case eChar32: case eUnspecified: case eVoid: case eWChar: default: return IntegralValue.UNKNOWN; } } } if (fArguments.length == 1) { return fArguments[0].getValue(point); } return IntegralValue.UNKNOWN; }
Source:
<code>public void handleCommand(ChannelUID channelUID, Command command) { if (command instanceof RefreshType) { updateChannel(channelUID.getId()); } else { <START> logger.info("The Weather Underground binding is a read-only binding and can not handle command {}", <END> command); } }</code><technical_language>></technical_language>

Target:
public void handleCommand(ChannelUID channelUID, Command command) { if (command instanceof RefreshType) { updateChannel(channelUID.getId()); } else { logger.debug("The Weather Underground binding is a read-only binding and cannot handle command {}", command); } }
Source:
<code>public void onAccountChanged(OnAccountChanged event) { if (event.isError()) { if (mIsUpdatingDisplayName) { mIsUpdatingDisplayName = false; AnalyticsTracker.track(mIsEmailSignup ? AnalyticsTracker.Stat.SIGNUP_EMAIL_EPILOGUE_UPDATE_DISPLAY_NAME_FAILED : AnalyticsTracker.Stat.SIGNUP_SOCIAL_EPILOGUE_UPDATE_DISPLAY_NAME_FAILED); } else if (mIsUpdatingPassword) { mIsUpdatingPassword = false; } AppLog.e(T.API, "SignupEpilogueFragment.onAccountChanged: " + event.error.type + " - " + event.error.message); endProgress(); if (isPasswordInErrorMessage(event.error.message)) { showErrorDialogWithCloseButton(event.error.message); } else { showErrorDialog(getString(R.string.signup_epilogue_error_generic)); } } else if (mIsEmailSignup && event.causeOfChange == AccountAction.FETCH_ACCOUNT && !TextUtils.isEmpty(mAccountStore.getAccount().getEmail())) { endProgress(); populateViews(); <START> } else if (event.causeOfChange == AccountAction.PUSH_SETTINGS && mSignupEpilogueListener != null) { <END> mHasMadeUpdates = true; if (mIsUpdatingDisplayName) { mIsUpdatingDisplayName = false; AnalyticsTracker.track(mIsEmailSignup ? AnalyticsTracker.Stat.SIGNUP_EMAIL_EPILOGUE_UPDATE_DISPLAY_NAME_SUCCEEDED : AnalyticsTracker.Stat.SIGNUP_SOCIAL_EPILOGUE_UPDATE_DISPLAY_NAME_SUCCEEDED); } else if (mIsUpdatingPassword) { mIsUpdatingPassword = false; mHasUpdatedPassword = true; } updateAccountOrContinue(); } }</code><technical_language>suggestion } else if (event.causeOfChange == AccountAction.PUSH_SETTINGS) { mSignupEpilogueListener.onContinue(); statement moved, remove mSignupEpilogueListener != null condition else if statement? mSignupEpilogueListener is in updateAccountOrContinue() method, mSignupEpilogueListener != null check is made</technical_language>

Target:
public void onAccountChanged(OnAccountChanged event) { if (event.isError()) { if (mIsUpdatingDisplayName) { mIsUpdatingDisplayName = false; AnalyticsTracker.track(mIsEmailSignup ? AnalyticsTracker.Stat.SIGNUP_EMAIL_EPILOGUE_UPDATE_DISPLAY_NAME_FAILED : AnalyticsTracker.Stat.SIGNUP_SOCIAL_EPILOGUE_UPDATE_DISPLAY_NAME_FAILED); } else if (mIsUpdatingPassword) { mIsUpdatingPassword = false; } AppLog.e(T.API, "SignupEpilogueFragment.onAccountChanged: " + event.error.type + " - " + event.error.message); endProgress(); if (isPasswordInErrorMessage(event.error.message)) { showErrorDialogWithCloseButton(event.error.message); } else { showErrorDialog(getString(R.string.signup_epilogue_error_generic)); } } else if (mIsEmailSignup && event.causeOfChange == AccountAction.FETCH_ACCOUNT && !TextUtils.isEmpty(mAccountStore.getAccount().getEmail())) { endProgress(); populateViews(); } else if (event.causeOfChange == AccountAction.PUSH_SETTINGS) { mHasMadeUpdates = true; if (mIsUpdatingDisplayName) { mIsUpdatingDisplayName = false; AnalyticsTracker.track(mIsEmailSignup ? AnalyticsTracker.Stat.SIGNUP_EMAIL_EPILOGUE_UPDATE_DISPLAY_NAME_SUCCEEDED : AnalyticsTracker.Stat.SIGNUP_SOCIAL_EPILOGUE_UPDATE_DISPLAY_NAME_SUCCEEDED); } else if (mIsUpdatingPassword) { mIsUpdatingPassword = false; mHasUpdatedPassword = true; } updateAccountOrContinue(); } }
Source:
<code>public ChannelFuture writeSettingsAck(ChannelHandlerContext ctx, ChannelPromise promise) { if (outstandingRemoteSettingsQueue == null) { return frameWriter.writeSettingsAck(ctx, promise); } SimpleChannelPromiseAggregator aggregator = new SimpleChannelPromiseAggregator(promise, ctx.channel(), ctx.executor()); frameWriter.writeSettingsAck(ctx, aggregator); Http2Settings settings = outstandingRemoteSettingsQueue.poll(); if (settings == null) { <START> aggregator.setFailure(connectionError(PROTOCOL_ERROR, <END> "SETTINGS ACK sent with no pending SETTINGS frame")); } else { try { remoteSettings(settings); } catch (Throwable e) { aggregator.setFailure(e); lifecycleManager.onError(ctx, true, e); } } return aggregator.doneAllocatingPromises(); }</code><technical_language>do need call lifecycleManager.onError(....) this case consistent</technical_language>

Target:
public ChannelFuture writeSettingsAck(ChannelHandlerContext ctx, ChannelPromise promise) { if (outstandingRemoteSettingsQueue == null) { return frameWriter.writeSettingsAck(ctx, promise); } Http2Settings settings = outstandingRemoteSettingsQueue.poll(); if (settings == null) { return promise.setFailure(new Http2Exception(INTERNAL_ERROR, "attempted to write a SETTINGS ACK with no " + " pending SETTINGS")); } SimpleChannelPromiseAggregator aggregator = new SimpleChannelPromiseAggregator(promise, ctx.channel(), ctx.executor()); frameWriter.writeSettingsAck(ctx, aggregator); ChannelPromise applySettingsPromise = aggregator.newPromise(); try { remoteSettings(settings); applySettingsPromise.setSuccess(); } catch (Throwable e) { applySettingsPromise.setFailure(e); lifecycleManager.onError(ctx, true, e); } return aggregator.doneAllocatingPromises(); }
Source:
<code><START> private void setupGetExistingSensorTest(final Metrics metrics, <END> final String level) { final String fullSensorName = INTERNAL_PREFIX + SENSOR_PREFIX_DELIMITER + level + SENSOR_NAME_DELIMITER + sensorName1; expect(metrics.getSensor(fullSensorName)).andStubReturn(sensor); replay(metrics); }</code><technical_language>This test replaces shouldGetThreadLevelSensor(). Thus, safely remove shouldGetThreadLevelSensor()</technical_language>

Target:
private void setupGetExistingSensorTest(final Metrics metrics, final String level) { final String fullSensorName = fullSensorName(level); expect(metrics.getSensor(fullSensorName)).andStubReturn(sensor); replay(metrics); }
Source:
<code><START> protected TmfAbstractAnalysisModule createStatisticsAnalysiModule() { <END> SystemCallLatencyStatisticsAnalysisModule module = new SystemCallLatencyStatisticsAnalysisModule(); return module; }</code><technical_language>put "protected @Nullable TmfAbstractAnalysisModule", auto-wrap too</technical_language>

Target:
protected @Nullable TmfAbstractAnalysisModule createStatisticsAnalysiModule() { SystemCallLatencyStatisticsAnalysisModule module = new SystemCallLatencyStatisticsAnalysisModule(); return module; }