auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void finish(final long dataFileLength) { if (indexes.isEmpty()) { throw new IllegalArgumentException("Cannot merge zero BAI files"); } final long[] offsets = partLengths.stream().mapToLong(i -> i).toArray(); Arrays.parallelPrefix(offsets, (a, b) -> a + b); <START> try (BinaryBAMIndexWriter writer = <END> new BinaryBAMIndexWriter(numReferences, out)) { for (int ref = 0; ref < numReferences; ref++) { final int r = ref; List<BAMIndexContent> bamIndexContentList = indexes.stream().map(index -> index.getQueryResults(r)).collect(Collectors.toList()); final BAMIndexContent bamIndexContent = mergeBAMIndexContent(ref, bamIndexContentList, offsets); writer.writeReference(bamIndexContent); } writer.writeNoCoordinateRecordCount(noCoordinateCount); } }</code><technical_language>extraneous new-line</technical_language>

Target:
public void finish(final long dataFileLength) { if (indexes.isEmpty()) { throw new IllegalArgumentException("Cannot merge zero BAI files"); } final long[] offsets = partLengths.stream().mapToLong(i -> i).toArray(); Arrays.parallelPrefix(offsets, Long::sum); try (BinaryBAMIndexWriter writer = new BinaryBAMIndexWriter(numReferences, out)) { for (int ref = 0; ref < numReferences; ref++) { final int r = ref; List<BAMIndexContent> bamIndexContentList = indexes.stream().map(index -> index.getQueryResults(r)).collect(Collectors.toList()); final BAMIndexContent bamIndexContent = mergeBAMIndexContent(ref, bamIndexContentList, offsets); writer.writeReference(bamIndexContent); } writer.writeNoCoordinateRecordCount(noCoordinateCount); } }
Source:
<code>private RemoteFileArtifactValue getRemoteInputMetadata(String execPathString) { FileArtifactValue m = inputArtifactData.getMetadata(execPathString); if (m != null && m.isRemote()) { return (RemoteFileArtifactValue) m; } RemoteFileArtifactValue rm = injectedMetadata.get(execPathString); if (rm != null) { return rm; <START> } <END> return null; }</code><technical_language>nit: return rm unconditionally</technical_language>

Target:
private RemoteFileArtifactValue getRemoteInputMetadata(String execPathString) { FileArtifactValue m = inputArtifactData.getMetadata(execPathString); if (m != null && m.isRemote()) { return (RemoteFileArtifactValue) m; } return injectedMetadata.get(execPathString); }
Source:
<code>public void setColumnWidth(Column<T, ?> column, String width) { <START> if (!initializedColumns.contains(column) && columnResizePersistenceEnabled) { <END> String persistedWidth = readColumnWidth(column); if (persistedWidth != null) { width = persistedWidth; } initializedColumns.add(column); } super.setColumnWidth(column, width); if (columnResizingEnabled) { int columnIndex = getColumnIndex(column); TableElement tableElement = getElement().cast(); for (TableCellElement cell : getTableBodyCells(tableElement, columnIndex)) { cell.getStyle().setProperty("width", width); } for (TableCellElement cell : getTableHeaderCells(tableElement, columnIndex)) { cell.getStyle().setProperty("width", width); } } }</code><technical_language>Minor thing: I if "columnResizePersistenceEnabled" in this expression (but a big deal initializedColumns is empty columnResizePersistenceEnabled==false): if (columnResizePersistenceEnabled && !initializedColumns.contains(column)) ..</technical_language>

Target:
public void setColumnWidth(Column<T, ?> column, String width) { if (columnResizePersistenceEnabled && !initializedColumns.contains(column)) { String persistedWidth = readColumnWidth(column); if (persistedWidth != null) { width = persistedWidth; } initializedColumns.add(column); } super.setColumnWidth(column, width); if (columnResizingEnabled) { int columnIndex = getColumnIndex(column); TableElement tableElement = getElement().cast(); for (TableCellElement cell : getTableBodyCells(tableElement, columnIndex)) { cell.getStyle().setProperty("width", width); } for (TableCellElement cell : getTableHeaderCells(tableElement, columnIndex)) { cell.getStyle().setProperty("width", width); } } }
Source:
<code>public void testNonExistentCipherSuite() throws Exception { <START> fail("IGNITE-10245"); <END> cipherSuites = new String[] { "TLS_RSA_WITH_AES_128_GCM_SHA256" }; startGrid(); checkClientStartFailure( new String[][] { new String[] { "TLC_FAKE_CIPHER", "TLS_DHE_RSA_WITH_AES_128_GCM_SHA256" } }, null, GridClientException.class, "Unsupported ciphersuite" ); }</code><technical_language>Add full link please <LINK_0></technical_language>

Target:
public void testNonExistentCipherSuite() throws Exception { cipherSuites = new String[] { "TLS_RSA_WITH_AES_128_GCM_SHA256" }; startGrid(); checkClientStartFailure( new String[] { "TLC_FAKE_CIPHER", "TLS_DHE_RSA_WITH_AES_128_GCM_SHA256" }, null, "Unsupported ciphersuite" ); }
Source:
<code><START> public AbstractArtifactCoordinates getArtifactCoordinates() <END> { return artifactCoordinates; }</code><technical_language>need change this</technical_language>

Target:
public ArtifactCoordinates getArtifactCoordinates() { return artifactCoordinates; }
Source:
<code>public ArtificialIntelligence(Level level) { this.level = level; <START> this.enemies = new ArrayList<Enemy>(); <END> this.units = new ArrayList<Unit>(); this.movementsX = new ArrayList<Integer>(); this.movementsY = new ArrayList<Integer>(); }</code><technical_language>remove explicit type argument. unnecessary. new code is new ArrayList<>(); < layout</technical_language>

Target:
public ArtificialIntelligence(Level level) { this.level = level; this.enemies = new ArrayList<>(); this.units = new ArrayList<>(); }
Source:
<code>public void updateOutput() { int o = (maximumIOSize() == EnumConnectionType.Size.SIXTEEN) ? size / 2 - 8 : 1; for (int i = 0; i < ((maximumIOSize() == EnumConnectionType.Size.SIXTEEN) ? 16 : 1); i++) { Vec2 pos1 = new Vec2(size - 1 - (i + o), 0); Vec2 pos2 = new Vec2(size - 1, size - 1 - (i + o)); Vec2 pos3 = new Vec2(i + o, size - 1); Vec2 pos4 = new Vec2(0, i + o); CircuitPart part1 = getPart(pos1); CircuitPart part2 = getPart(pos2); CircuitPart part3 = getPart(pos3); CircuitPart part4 = getPart(pos4); if (part1 instanceof PartIOBit) ((PartIOBit) part1).updateExternalOutput(pos1, parent); <START> if (part1 instanceof PartIOBit) ((PartIOBit) part2).updateExternalOutput(pos2, parent); <END> if (part1 instanceof PartIOBit) ((PartIOBit) part3).updateExternalOutput(pos3, parent); if (part1 instanceof PartIOBit) ((PartIOBit) part4).updateExternalOutput(pos4, parent); } }</code><technical_language>part2, right</technical_language>

Target:
public void updateOutput() { int o = supportsBundled() ? size / 2 - 8 : 1; for (int i = 0; i < (supportsBundled() ? 16 : 1); i++) { Vec2 pos1 = new Vec2(size - 1 - (i + o), 0); Vec2 pos2 = new Vec2(size - 1, size - 1 - (i + o)); Vec2 pos3 = new Vec2(i + o, size - 1); Vec2 pos4 = new Vec2(0, i + o); CircuitPart part1 = getPart(pos1); CircuitPart part2 = getPart(pos2); CircuitPart part3 = getPart(pos3); CircuitPart part4 = getPart(pos4); if (part1 instanceof PartIOBit) ((PartIOBit) part1).updateExternalOutput(pos1, parent); if (part2 instanceof PartIOBit) ((PartIOBit) part2).updateExternalOutput(pos2, parent); if (part3 instanceof PartIOBit) ((PartIOBit) part3).updateExternalOutput(pos3, parent); if (part4 instanceof PartIOBit) ((PartIOBit) part4).updateExternalOutput(pos4, parent); } }
Source:
<code>public String convert(Object tuple) { try { StringWriter stringWriter = new StringWriter(); ICsvBeanWriter beanWriter = new CsvBeanWriter(stringWriter, preference); beanWriter.write(tuple, nameMapping,processors); beanWriter.flush(); beanWriter.close(); return stringWriter.toString(); } catch (SuperCsvException e) { logger.error("Error while converting tuple ", tuple + " " + e.getMessage()); <START> }catch (IOException e) { <END> DTThrowable.rethrow(e); } return null; }</code><technical_language>space in { catch</technical_language>

Target:
public String convert(Object tuple) { try { StringWriter stringWriter = new StringWriter(); ICsvBeanWriter beanWriter = new CsvBeanWriter(stringWriter, preference); beanWriter.write(tuple, nameMapping, processors); beanWriter.flush(); beanWriter.close(); return stringWriter.toString(); } catch (SuperCsvException e) { logger.debug("Error while converting tuple {}",tuple,e.getMessage()); } catch (IOException e) { DTThrowable.rethrow(e); } return null; }
Source:
<code>private void addHeaders(Span span, SpanTextMap textMap) { addHeader(textMap, TraceMessageHeaders.TRACE_ID_NAME, Span.idToHex(span.getTraceId())); addHeader(textMap, TraceMessageHeaders.SPAN_ID_NAME, Span.idToHex(span.getSpanId())); if (span.isExportable()) { addAnnotations(this.traceKeys, textMap, span); Long parentId = getFirst(span.getParents()); if (parentId != null) { addHeader(textMap, TraceMessageHeaders.PARENT_ID_NAME, Span.idToHex(parentId)); } addHeader(textMap, TraceMessageHeaders.SPAN_NAME_NAME, span.getName()); addHeader(textMap, TraceMessageHeaders.PROCESS_ID_NAME, span.getProcessId()); addHeader(textMap, TraceMessageHeaders.SAMPLED_NAME, Span.SPAN_SAMPLED); } else { addHeader(textMap, TraceMessageHeaders.SAMPLED_NAME, Span.SPAN_NOT_SAMPLED); } <START> addHeader(textMap, TraceMessageHeaders.SPAN_HEADER, storeSpanAsJson(span)); <END> }</code><technical_language>eh? literally sending span in headers, right? basically of intent of in-band/reporting duo of dapper systems</technical_language>

Target:
private void addHeaders(Span span, SpanTextMap textMap) { addHeader(textMap, TraceMessageHeaders.TRACE_ID_NAME, Span.idToHex(span.getTraceId())); addHeader(textMap, TraceMessageHeaders.SPAN_ID_NAME, Span.idToHex(span.getSpanId())); if (span.isExportable()) { addAnnotations(this.traceKeys, textMap, span); Long parentId = getFirst(span.getParents()); if (parentId != null) { addHeader(textMap, TraceMessageHeaders.PARENT_ID_NAME, Span.idToHex(parentId)); } addHeader(textMap, TraceMessageHeaders.SPAN_NAME_NAME, span.getName()); addHeader(textMap, TraceMessageHeaders.PROCESS_ID_NAME, span.getProcessId()); addHeader(textMap, TraceMessageHeaders.SAMPLED_NAME, Span.SPAN_SAMPLED); } else { addHeader(textMap, TraceMessageHeaders.SAMPLED_NAME, Span.SPAN_NOT_SAMPLED); } }
Source:
<code>public void run() { logger.info( "START ClusterEventProcessor thread for cluster " + _clusterName + ", processor name: " + _processorName); while (!isInterrupted()) { try { ClusterEvent newClusterEvent = _eventBlockingQueue.take(); <START> String currentThreadName = this.getName().split("_")[0]; <END> this.setName(currentThreadName + "_" + newClusterEvent.getEventId()); handleEvent(newClusterEvent, _cache); } catch (InterruptedException e) { logger.warn("ClusterEventProcessor interrupted " + _processorName, e); interrupt(); } catch (ZkInterruptedException e) { logger .warn("ClusterEventProcessor caught a ZK connection interrupt " + _processorName, e); interrupt(); } catch (ThreadDeath death) { logger.error("ClusterEventProcessor caught a ThreadDeath " + _processorName, death); throw death; } catch (Throwable t) { logger.error("ClusterEventProcessor failed while running the controller pipeline " + _processorName, t); } } logger.info("END ClusterEventProcessor thread " + _processorName); }</code><technical_language>thread name fixed, regenerate name name convention</technical_language>

Target:
public void run() { logger.info( "START ClusterEventProcessor thread for cluster " + _clusterName + ", processor name: " + _processorName); while (!isInterrupted()) { try { ClusterEvent newClusterEvent = _eventBlockingQueue.take(); String threadName = String.format( "HelixController-pipeline-%s-(%s)", _processorName, newClusterEvent.getEventId()); this.setName(threadName); handleEvent(newClusterEvent, _cache); } catch (InterruptedException e) { logger.warn("ClusterEventProcessor interrupted " + _processorName, e); interrupt(); } catch (ZkInterruptedException e) { logger .warn("ClusterEventProcessor caught a ZK connection interrupt " + _processorName, e); interrupt(); } catch (ThreadDeath death) { logger.error("ClusterEventProcessor caught a ThreadDeath " + _processorName, death); throw death; } catch (Throwable t) { logger.error("ClusterEventProcessor failed while running the controller pipeline " + _processorName, t); } } logger.info("END ClusterEventProcessor thread " + _processorName); }
Source:
<code>public TasksView(String authority, ContentProviderClient client) { <START> super(new BaseView<TaskContract.Tasks>(client, TaskContract.Properties.getContentUri(authority))); <END> }</code><technical_language>This creates a Properties table. Copy'n'paste? ;-)</technical_language>

Target:
public TasksView(String authority, ContentProviderClient client) { super(new BaseView<TaskContract.Tasks>(client, TaskContract.Tasks.getContentUri(authority))); }
Source:
<code>protected byte[] prepareCompressedData(byte[] uncompressed) { try { <START> byte[] compressed = new byte[org.xerial.snappy.Snappy.maxCompressedLength(uncompressed.length)]; <END> int compressedSize = org.xerial.snappy.Snappy.compress( uncompressed, 0, uncompressed.length, compressed, 0); return Arrays.copyOf(compressed, compressedSize); } catch (IOException e) { throw new RuntimeException(e); } }</code><technical_language>snappy</technical_language>

Target:
protected byte[] prepareCompressedData(byte[] uncompressed) { return LZ4Factory.fastestInstance().fastCompressor().compress(uncompressed); }
Source:
<code>public static org.apache.parquet.schema.Type getParquetType(Type prestoType, MessageType messageType, boolean useParquetColumnNames, String columnName, int columnHiveIndex, HiveType hiveType) { org.apache.parquet.schema.Type type = null; if (useParquetColumnNames) { type = getParquetTypeByName(columnName, messageType); } if (columnHiveIndex < messageType.getFieldCount()) { type = messageType.getType(columnHiveIndex); } if (type == null) { return null; } <START> if (useParquetColumnNames) { <END> return type; } if (!schemaMatch(type, prestoType)) { String parquetTypeName = null; if (type.isPrimitive()) { parquetTypeName = type.asPrimitiveType().getPrimitiveTypeName().toString(); } else { GroupType group = type.asGroupType(); StringBuilder builder = new StringBuilder(); group.writeToStringBuilder(builder, " "); parquetTypeName = builder.toString(); } throw new PrestoException(HIVE_PARTITION_SCHEMA_MISMATCH, format("The column %s is declared as type %s, but the Parquet file declares the column as type %s", columnName, hiveType, parquetTypeName)); } return type; }</code><technical_language>check for this case? please elaborate a bit more schema evolution mean? Is table -> partition schema migration? (changing columns for a table existing partitions)? If a mismatch? fail reading</technical_language>

Target:
public static org.apache.parquet.schema.Type getParquetType(Type prestoType, MessageType messageType, boolean useParquetColumnNames, String columnName, int columnHiveIndex, HiveType hiveType) { org.apache.parquet.schema.Type type = null; if (useParquetColumnNames) { type = getParquetTypeByName(columnName, messageType); } else if (columnHiveIndex < messageType.getFieldCount()) { type = messageType.getType(columnHiveIndex); } if (type == null) { return null; } if (!checkSchemaMatch(type, prestoType)) { String parquetTypeName; if (type.isPrimitive()) { parquetTypeName = type.asPrimitiveType().getPrimitiveTypeName().toString(); } else { GroupType group = type.asGroupType(); StringBuilder builder = new StringBuilder(); group.writeToStringBuilder(builder, ""); parquetTypeName = builder.toString(); } throw new PrestoException(HIVE_PARTITION_SCHEMA_MISMATCH, format("The column %s is declared as type %s, but the Parquet file declares the column as type %s", columnName, hiveType, parquetTypeName)); } return type; }
Source:
<code>public void store(Map<TopicPartition, Long> partitionsAndOffsets) { <START> if (rdfClient == null) return; <END> UpdateBuilder ub = new UpdateBuilder(UPDATE_OFFSETS); StringBuilder sb = new StringBuilder(); for (Map.Entry<TopicPartition, Long> partitionAndOffset : partitionsAndOffsets.entrySet()) { TopicPartition tp = partitionAndOffset.getKey(); Long offset = partitionAndOffset.getValue(); sb.append(String.format(Locale.ROOT, "<%s> wikibase:kafka ( \"%s:%d\" %d ) .\n", root, tp.topic(), tp.partition(), offset)); } ub.bindUri("root", root); ub.bind("data", sb.toString()); rdfClient.update(ub.toString()); }</code><technical_language>{} IMHO</technical_language>

Target:
public void store(Map<TopicPartition, Long> partitionsAndOffsets) { if (rdfClient == null) { return; } UpdateBuilder ub = new UpdateBuilder(UPDATE_OFFSETS); StringBuilder sb = new StringBuilder(); for (Map.Entry<TopicPartition, Long> partitionAndOffset : partitionsAndOffsets.entrySet()) { TopicPartition tp = partitionAndOffset.getKey(); Long offset = partitionAndOffset.getValue(); sb.append(String.format(Locale.ROOT, "<%s> wikibase:kafka ( \"%s:%d\" %d ) .\n", root, tp.topic(), tp.partition(), offset)); } ub.bindUri("root", root); ub.bind("data", sb.toString()); rdfClient.update(ub.toString()); }
Source:
<code>protected void ExecuteVdsBrokerCommand() { boolean filteringLUNsEnabled = Config.<Boolean> GetValue( ConfigValues.FilteringLUNsEnabled, getParameters().getcompatibility_version().getValue()); XmlRpcStruct options = new XmlRpcStruct(); <START> if (!filteringLUNsEnabled) { <END> options.add(VdsProperties.includePartitioned, Boolean.TRUE.toString()); } int storageType = (int) getParameters().getStorageType().getValue(); _result = getBroker().getDeviceList(storageType, options); ProceedProxyReturnValue(); setReturnValue(ParseLUNList(_result.lunList)); }</code><technical_language>if just: options.add(VdsProperties.includePartitioned, filteringLUNsEnabled.toString());</technical_language>

Target:
protected void ExecuteVdsBrokerCommand() { boolean filteringLUNsEnabled = getParameters().getcompatibility_version().compareTo(new Version(VERSION_3_1)) < 0; XmlRpcStruct options = new XmlRpcStruct(); options.add(VdsProperties.includePartitioned, Boolean.toString(filteringLUNsEnabled)); int storageType = (int) getParameters().getStorageType().getValue(); _result = getBroker().getDeviceList(storageType, options); ProceedProxyReturnValue(); setReturnValue(ParseLUNList(_result.lunList)); }
Source:
<code>public boolean shouldThrottle() { return shouldThrotle; <START> } <END></code><technical_language>license</technical_language>

Target:
public boolean shouldThrottle() { return shouldThrottle; }
Source:
<code><START> private boolean hardDelete(boolean throttle) throws IOException, StoreException, InterruptedException { <END> if (indexes.size() > 0) { final Timer.Context context = metrics.hardDeleteTime.time(); try { FindInfo info = findDeletedEntriesSince(startToken, scanSizeInBytes, SystemTime.getInstance().seconds() - messageRetentionSeconds); endToken = info.getFindToken(); if (!endToken.equals(startToken)) { persistCleanupToken(); if (!info.getMessageEntries().isEmpty()) { performHardDeletes(info.getMessageEntries(), throttle); } startToken = endToken; return true; } } finally { context.stop(); } } return false; }</code><technical_language>do need differentiate IOException StoreException ioerror</technical_language>

Target:
private boolean hardDelete(boolean throttle) throws StoreException, InterruptedException { if (indexes.size() > 0) { final Timer.Context context = metrics.hardDeleteTime.time(); try { FindInfo info = findDeletedEntriesSince(startToken, scanSizeInBytes, SystemTime.getInstance().seconds() - messageRetentionSeconds); endToken = info.getFindToken(); if (!endToken.equals(startToken)) { persistCleanupToken(); if (!info.getMessageEntries().isEmpty()) { performHardDeletes(info.getMessageEntries(), throttle); } startToken = endToken; return true; } } catch (InterruptedException e) { throw e; } catch (StoreException e) { metrics.hardDeleteExceptionsCount.inc(); throw e; } catch (IOException e) { metrics.hardDeleteExceptionsCount.inc(); throw new StoreException(e, StoreErrorCodes.IOError); } catch (Exception e) { metrics.hardDeleteExceptionsCount.inc(); throw new StoreException(e, StoreErrorCodes.Unknown_Error); } finally { context.stop(); } } return false; }
Source:
<code>public CompletableFuture<Void> close() { List<CompletableFuture<Void>> futures = partitions.values().stream() .map(RaftPartition::close) .collect(Collectors.toList()); return CompletableFuture.allOf(futures.toArray(new CompletableFuture[futures.size()])).thenRun(() -> { try { threadContextFactory.close(); <START> } catch (Exception e) { <END> } LOGGER.info("Stopped"); }); }</code><technical_language>ignoring here? Debug</technical_language>

Target:
public CompletableFuture<Void> close() { List<CompletableFuture<Void>> futures = partitions.values().stream() .map(RaftPartition::close) .collect(Collectors.toList()); return CompletableFuture.allOf(futures.toArray(new CompletableFuture[futures.size()])).thenRun(() -> { threadContextFactory.close(); LOGGER.info("Stopped"); }); }
Source:
<code>public void snapshotAllMetrics(MetricRegistry metricRegistry, MetricsRecordBuilder builder) { Map<String, Metric> metrics = metricRegistry.getMetrics(); <START> Iterator i$ = metrics.entrySet().iterator(); <END> while(i$.hasNext()) { Entry<String, Metric> e = (Entry)i$.next(); String name = StringUtils.capitalize((String)e.getKey()); Metric metric = (Metric)e.getValue(); if (metric instanceof Gauge) { this.addGauge(name, (Gauge)metric, builder); } else if (metric instanceof Counter) { this.addCounter(name, (Counter)metric, builder); } else if (metric instanceof Histogram) { this.addHistogram(name, (Histogram)metric, builder); } else if (metric instanceof Meter) { this.addMeter(name, (Meter)metric, builder); } else if (metric instanceof Timer) { this.addTimer(name, (Timer)metric, builder); } else { LOG.info("Ignoring unknown Metric class " + metric.getClass().getName()); } } }</code><technical_language>do need this? i$</technical_language>

Target:
private void snapshotAllMetrics(MetricRegistry metricRegistry, MetricsRecordBuilder builder) { Map<String, Metric> metrics = metricRegistry.getMetrics(); Iterator iterator = metrics.entrySet().iterator(); while(iterator.hasNext()) { Entry<String, Metric> e = (Entry)iterator.next(); String name = StringUtils.capitalize(e.getKey()); Metric metric = e.getValue(); if (metric instanceof Gauge) { this.addGauge(name, (Gauge)metric, builder); } else if (metric instanceof Counter) { this.addCounter(name, (Counter)metric, builder); } else if (metric instanceof Histogram) { this.addHistogram(name, (Histogram)metric, builder); } else if (metric instanceof Meter) { this.addMeter(name, (Meter)metric, builder); } else if (metric instanceof Timer) { this.addTimer(name, (Timer)metric, builder); } else { LOG.info("Ignoring unknown Metric class " + metric.getClass().getName()); } } }
Source:
<code>public Supplier<IntIterator> makeDimensionProcessor(DimensionSelector selector, boolean multiValue) { if (selector.getValueCardinality() == DimensionDictionarySelector.CARDINALITY_UNKNOWN) { return () -> { final IndexedInts row = selector.getRow(); if (row.size() == 1) { int dimensionId = row.get(0); IntList rowNumbers = getRowNumbers(selector, dimensionId); return rowNumbers.iterator(); } else { throw new QueryException("Unsupported query", "Unsupported query on multi-valued column", QueryException.class.getName(), null); } }; } else { return () -> { final IndexedInts row = selector.getRow(); if (row.size() == 1) { int dimensionId = row.get(0); IntList rowNumbers = getAndCacheRowNumbers(selector, dimensionId); return rowNumbers.iterator(); } else { throw new QueryException("Unsupported query", <START> "Unsupported query on multi-valued column", <END> QueryException.class.getName(), null); } }; } }</code><technical_language>suggestion "Joining a multi-value dimension is supported.",</technical_language>

Target:
public Supplier<IntIterator> makeDimensionProcessor(DimensionSelector selector, boolean multiValue) { if (selector.getValueCardinality() == DimensionDictionarySelector.CARDINALITY_UNKNOWN) { return () -> { final IndexedInts row = selector.getRow(); if (row.size() == 1) { int dimensionId = row.get(0); IntList rowNumbers = getRowNumbers(selector, dimensionId); return rowNumbers.iterator(); } else if (row.size() == 0) { return IntIterators.EMPTY_ITERATOR; } else { throw new QueryUnsupportedException("Joining against a multi-value dimension is not supported."); } }; } else { return () -> { final IndexedInts row = selector.getRow(); if (row.size() == 1) { int dimensionId = row.get(0); IntList rowNumbers = getAndCacheRowNumbers(selector, dimensionId); return rowNumbers.iterator(); } else if (row.size() == 0) { return IntIterators.EMPTY_ITERATOR; } else { throw new QueryUnsupportedException("Joining against a multi-value dimension is not supported."); } }; } }
Source:
<code>public RangeSet<String> getDimensionRangeSet(String dimension) { <START> if (field instanceof AndDimFilter) { <END> List<DimFilter> fields = ((AndDimFilter) field).getFields(); return Druids.newOrDimFilterBuilder().fields(Lists.transform(fields, negate)).build().getDimensionRangeSet(dimension); } if (field instanceof OrDimFilter) { List<DimFilter> fields = ((OrDimFilter) field).getFields(); return Druids.newAndDimFilterBuilder().fields(Lists.transform(fields, negate)).build().getDimensionRangeSet(dimension); } if (field instanceof NotDimFilter) { return ((NotDimFilter) field).getField().getDimensionRangeSet(dimension); } RangeSet<String> rangeSet = field.getDimensionRangeSet(dimension); return rangeSet == null ? null : rangeSet.complement(); }</code><technical_language>is special casing for And/Or/Not needed complement of field's rangeSet</technical_language>

Target:
public RangeSet<String> getDimensionRangeSet(String dimension) { if (field instanceof AndDimFilter) { List<DimFilter> fields = ((AndDimFilter) field).getFields(); return Druids.newOrDimFilterBuilder().fields(Lists.transform(fields, NEGATE)).build().getDimensionRangeSet(dimension); } if (field instanceof OrDimFilter) { List<DimFilter> fields = ((OrDimFilter) field).getFields(); return Druids.newAndDimFilterBuilder().fields(Lists.transform(fields, NEGATE)).build().getDimensionRangeSet(dimension); } if (field instanceof NotDimFilter) { return ((NotDimFilter) field).getField().getDimensionRangeSet(dimension); } RangeSet<String> rangeSet = field.getDimensionRangeSet(dimension); return rangeSet == null ? null : rangeSet.complement(); }
Source:
<code>protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); <START> setContentView(R.layout.workouts_activity); <END> prepareGestureListener(); addListeners(); TabHost tabHost = (TabHost)findViewById(R.id.tabhost); tabHost.setup(); TabSpec tabSpecs = tabHost.newTabSpec("TabSpec"); tabSpecs.setContent(R.id.tab1); tabSpecs.setIndicator("Workouts"); tabHost.addTab(tabSpecs); tabSpecs = tabHost.newTabSpec("TabSpec2"); tabSpecs.setContent(R.id.tab2); tabSpecs.setIndicator("Workouts"); tabHost.addTab(tabSpecs); }</code><technical_language>name change</technical_language>

Target:
protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.planing_activity); prepareGestureListener(); addListeners(); TabHost tabHost = (TabHost)findViewById(R.id.tabhost); tabHost.setup(); TabSpec tabSpecs = tabHost.newTabSpec(TAB_SPEC_1_TAG); tabSpecs.setContent(R.id.tab1); tabSpecs.setIndicator("Workouts"); tabHost.addTab(tabSpecs); tabSpecs = tabHost.newTabSpec(TAB_SPEC_2_TAG); tabSpecs.setContent(R.id.tab2); tabSpecs.setIndicator("Workouts"); tabHost.addTab(tabSpecs); }
Source:
<code><START> public void testToStringDefault() { <END> DateFormat df = new SimpleDateFormat("HH:mm:ss.SSS"); df.setTimeZone(TimeZone.getTimeZone("UTC")); int timeOffset = TimeZone.getDefault().getOffset(0); Date d0 = new Date(0 + timeOffset); Date d1 = new Date(12345000L + timeOffset); Date d2 = new Date(1234500L + timeOffset); Date d3 = new Date(1234500000L + timeOffset); assertEquals("toString", df.format(d0) + " 000 000", ts0.toString()); assertEquals("toString", df.format(d1) + " 000 000", ts1.toString()); assertEquals("toString", df.format(d2) + " 000 000", ts2.toString()); assertEquals("toString", df.format(d3) + " 000 000", ts3.toString()); }</code><technical_language>simpler get TmfTimestampFormat.getDefaulTimeFormat() set time zone UTC, in setUp method</technical_language>

Target:
public void testToStringDefault() { DateFormat df = new SimpleDateFormat("HH:mm:ss.SSS"); Date d0 = new Date(0); Date d1 = new Date(12345000L); Date d2 = new Date(1234500L); Date d3 = new Date(1234500000L); assertEquals("toString", df.format(d0) + " 000 000", ts0.toString()); assertEquals("toString", df.format(d1) + " 000 000", ts1.toString()); assertEquals("toString", df.format(d2) + " 000 000", ts2.toString()); assertEquals("toString", df.format(d3) + " 000 000", ts3.toString()); }
Source:
<code>private void reviewRecursive(Path src, Path dest, Session session) throws IOException { <START> if (src.toString().contains("/dataset/")) { <END> warn().collectionID(this) .uri(dest.toString()) .user(session) .log("warning received recursive review request for dataset content uri - this is a " + "potentially destructive action and could delete the updated previous version in the " + "collection"); } FileUtils.deleteDirectory(dest.getParent().toFile()); FileUtils.moveDirectory(src.getParent().toFile(), dest.getParent().toFile()); zebedee.getCollections().removeEmptyCollectionDirectories(src.getParent()); }</code><technical_language>/dataset/ plural /datasets/</technical_language>

Target:
private void reviewRecursive(Path src, Path dest, Session session) throws IOException { if (src.toString().contains(DATASETS_URI)) { warn().collectionID(this) .uri(dest.toString()) .user(session) .log("warning received recursive review request for dataset content uri - this is a " + "potentially destructive action and could delete the updated previous version in the " + "collection"); } FileUtils.deleteDirectory(dest.getParent().toFile()); FileUtils.moveDirectory(src.getParent().toFile(), dest.getParent().toFile()); zebedee.getCollections().removeEmptyCollectionDirectories(src.getParent()); }
Source:
<code><START> public boolean mappingExists(String alias, Map<String, Tag> idMap) { <END> return alias != null && idMap != null && idMap.containsKey(alias); }</code><technical_language>this a static method</technical_language>

Target:
public static boolean mappingExists(String alias, Map<String, Tag> idMap) { return alias != null && idMap != null && idMap.containsKey(alias); }
Source:
<code><START> public void convert_skip_null_abstract() throws IOException { <END> Result sourceDataset = minimalEntityBuilder(ID, InfoSpaceConstants.SEMANTIC_CLASS_INSTANCE_TYPE_DATASET); addDescription(sourceDataset, "null"); addDescription(sourceDataset, ABSTRACT); DataSetReference metadata = converter.convert(sourceDataset); assertNotNull(metadata); assertEquals(ABSTRACT, metadata.getDescription()); }</code><technical_language>Exception is thrown</technical_language>

Target:
public void convert_skip_null_abstract() { Result sourceDataset = minimalEntityBuilder(ID, InfoSpaceConstants.SEMANTIC_CLASS_INSTANCE_TYPE_DATASET); addDescription(sourceDataset, "null"); addDescription(sourceDataset, ABSTRACT); DataSetReference metadata = converter.convert(sourceDataset); assertNotNull(metadata); assertEquals(ABSTRACT, metadata.getDescription()); }
Source:
<code>public DroolsAbstractTriConstraintStream(DroolsConstraintFactory<Solution_> constraintFactory, DroolsAbstractTriConstraintStream<Solution_, A, B, C> parent) { super(constraintFactory); if (parent == null && !(this instanceof DroolsJoinTriConstraintStream)) { <START> throw new IllegalArgumentException("Parent of stream (" + this + ") must not be null, or not join stream."); <END> } this.parent = parent; }</code><technical_language>typo: "not a join stream" Actually, this sentence is incorrect: "Parent ... null, join stream." implies "parent is a join stream", while "this is a join stream" Suggestion "The stream (" + this + ") a parent (null), a join stream."</technical_language>

Target:
public DroolsAbstractTriConstraintStream(DroolsConstraintFactory<Solution_> constraintFactory, DroolsAbstractTriConstraintStream<Solution_, A, B, C> parent) { super(constraintFactory); if (parent == null && !(this instanceof DroolsJoinTriConstraintStream)) { throw new IllegalArgumentException("The stream (" + this + ") must have a parent (null), " + "unless it's a join stream."); } this.parent = parent; }
Source:
<code>public ITimeEvent splitBefore(long splitTime) { <START> throw new UnsupportedOperationException(); <END> }</code><technical_language>return (splitTime > fTime ? new TimeEvent(fEntry, fTime, Math.min(fDuration, splitTime - fTime)) : null); }</technical_language>

Target:
public ITimeEvent splitBefore(long splitTime) { return (splitTime > fTime ? new TimeEvent(getEntry(), fTime, Math.min(fDuration, splitTime - fTime)) : null); }
Source:
<code>private org.mule.maven.client.api.model.RemoteRepository toRemoteRepo(RemoteRepository remoteRepository) { String id = remoteRepository.getId(); Optional<Authentication> authentication = getAuthentication(remoteRepository); URL url = null; try { url = getURL(remoteRepository); } catch (MavenExecutionException e) { <START> e.printStackTrace(); <END> } org.mule.maven.client.api.model.RemoteRepository.RemoteRepositoryBuilder builder = new org.mule.maven.client.api.model.RemoteRepository.RemoteRepositoryBuilder(); authentication.ifPresent(builder::withAuthentication); return builder .withId(id) .withUrl(url) .build(); }</code><technical_language>throw exception halt execution here? if continue a null url instead of print stack trace log</technical_language>

Target:
private org.mule.maven.client.api.model.RemoteRepository toRemoteRepo(RemoteRepository remoteRepository) { String id = remoteRepository.getId(); Optional<Authentication> authentication = getAuthentication(remoteRepository); URL remoteRepositoryUrl = getURL(remoteRepository); org.mule.maven.client.api.model.RemoteRepository.RemoteRepositoryBuilder builder = new org.mule.maven.client.api.model.RemoteRepository.RemoteRepositoryBuilder(); authentication.ifPresent(builder::withAuthentication); return builder .withId(id) .withUrl(remoteRepositoryUrl) .build(); }
Source:
<code><START> public boolean isEnabledForMultipleContexts() { <END> return enabledForMultipleContexts; }</code><technical_language>This protected (more internal detail for external consumption.)</technical_language>

Target:
protected boolean isEnabledForMultipleContexts() { return enabledForMultipleContexts; }
Source:
<code>public EventProcessor buildEventProcessor() { EventPosition initialEventPosition = this.initialEventPosition == null ? EventPosition.earliest() : this.initialEventPosition; <START> return new EventProcessor(buildAsyncClient(), this.consumerGroupName, <END> this.partitionProcessorFactory, initialEventPosition, partitionManager); }</code><technical_language>I conventions this, this. is inconsistently reference member variables. (ie. partitionManager this). prefer this. if conflicting local variables (makes more succinct), I lose sleep if this change in</technical_language>

Target:
public EventProcessor buildEventProcessor() { EventPosition initialEventPosition = this.initialEventPosition == null ? EventPosition.earliest() : this.initialEventPosition; final TracerProvider tracerProvider = new TracerProvider(ServiceLoader.load(Tracer.class)); return new EventProcessor(buildAsyncClient(), consumerGroupName, partitionProcessorFactory, initialEventPosition, partitionManager, tracerProvider); }
Source:
<code><START> private RSDL getRSDL(Application application) { <END> for (Object obj : application.getSingletons()) { if (obj instanceof BackendApiResource) { BackendApiResource resource = (BackendApiResource) obj; return resource.getRSDL(); } } return null; }</code><technical_language>RSDL is useful in context I deserves a class manage it. do agree add a RSDLManager class? this: public class BackendRSDLManager { private RSDL rsdl = ...; public static RSDL getRSDL() { return rsdl; } } put in this class specifics of building RSDL for backend, avoid loop up. reuse this class in other places need RSDL</technical_language>

Target:
private RSDL getRSDL(Application application) { RSDL rsdl = null; for (Object obj : application.getSingletons()) { if (obj instanceof BackendApiResource) { BackendApiResource resource = (BackendApiResource) obj; rsdl = resource.getRSDL(); break; } } assert (rsdl != null) : "Resource that generates RSDL, BackendApiResource, not found (this should never happen)"; return rsdl; }
Source:
<code>private BlameScmResult doShellAnnotate( IntegrityScmProviderRepository iRepo, ScmFileSet workingDirectory, String filename ) { BlameScmResult result; Commandline shell = new Commandline(); shell.setWorkingDirectory( workingDirectory.getBasedir() ); shell.setExecutable( "si" ); shell.createArg().setValue( "annotate" ); shell.createArg().setValue( "--hostname=" + iRepo.getHost() ); shell.createArg().setValue( "--port=" + iRepo.getPort() ); shell.createArg().setValue( "--user=" + iRepo.getUser() ); shell.createArg().setValue( "--fields=date,revision,author" ); shell.createArg().setValue( '"' + filename + '"' ); IntegrityBlameConsumer shellConsumer = new IntegrityBlameConsumer( getLogger() ); String commandLine = CommandLineUtils.toString( shell.getCommandline() ); try { <START> if ( getLogger().isDebugEnabled() ) { <END> getLogger().debug( "Executing: " + commandLine ); } int exitCode = CommandLineUtils.executeCommandLine( shell, shellConsumer, new CommandLineUtils.StringStreamConsumer() ); boolean success = ( exitCode == 0 ? true : false ); ScmResult scmResult = new ScmResult( commandLine, "", "Exit Code: " + exitCode, success ); return new BlameScmResult( shellConsumer.getBlameList(), scmResult ); } catch ( CommandLineException cle ) { getLogger().error( "Command Line Exception: " + cle.getMessage() ); result = new BlameScmResult( commandLine, cle.getMessage(), "", false ); } return result; }</code><technical_language>This necessary anymore soon SLF4J</technical_language>

Target:
private BlameScmResult doShellAnnotate( IntegrityScmProviderRepository iRepo, ScmFileSet workingDirectory, String filename ) { BlameScmResult result; Commandline shell = new Commandline(); shell.setWorkingDirectory( workingDirectory.getBasedir() ); shell.setExecutable( "si" ); shell.createArg().setValue( "annotate" ); shell.createArg().setValue( "--hostname=" + iRepo.getHost() ); shell.createArg().setValue( "--port=" + iRepo.getPort() ); shell.createArg().setValue( "--user=" + iRepo.getUser() ); shell.createArg().setValue( "--fields=date,revision,author" ); shell.createArg().setValue( '"' + filename + '"' ); IntegrityBlameConsumer shellConsumer = new IntegrityBlameConsumer( getLogger() ); String commandLine = CommandLineUtils.toString( shell.getCommandline() ); try { getLogger().debug( "Executing: " + commandLine ); int exitCode = CommandLineUtils.executeCommandLine( shell, shellConsumer, new CommandLineUtils.StringStreamConsumer() ); boolean success = ( exitCode == 0 ? true : false ); ScmResult scmResult = new ScmResult( commandLine, "", "Exit Code: " + exitCode, success ); return new BlameScmResult( shellConsumer.getBlameList(), scmResult ); } catch ( CommandLineException cle ) { getLogger().error( "Command Line Exception: " + cle.getMessage() ); result = new BlameScmResult( commandLine, cle.getMessage(), "", false ); } return result; }
Source:
<code>public static List<String> getLocalesKeys(BackendLocal backend, ConfigurationValues configValues) { if (!configValues.equals(ConfigurationValues.UnsupportedLocalesFilter) && !configValues.equals(ConfigurationValues.UnsupportedDisplayedLocales)) { throw new IllegalArgumentException("Passed in config value not related to locales"); } List<String> result = new ArrayList<String>(); String locales = backend.runPublicQuery(VdcQueryType.GetConfigurationValue, new GetConfigurationValueParameters(configValues, ConfigCommon.defaultConfigurationVersion)).getReturnValue(); if (locales != null && !locales.isEmpty()) { String[] localeKeys = locales.split(" *, *"); for (String localeKey: localeKeys) { try { org.apache.commons.lang.LocaleUtils.toLocale(localeKey.trim()); <START> result.add(localeKey.trim()); <END> } catch (IllegalArgumentException iae) { } } } return result; }</code><technical_language>I claim trim is needed :)) split</technical_language>

Target:
public static List<String> getLocalesKeys(BackendLocal backend, ConfigurationValues configValues) { if (!configValues.equals(ConfigurationValues.UnsupportedLocalesFilter) && !configValues.equals(ConfigurationValues.UnsupportedDisplayedLocales)) { throw new IllegalArgumentException("Passed in config value not related to locales"); } List<String> result = new ArrayList<String>(); String locales = backend.runPublicQuery(VdcQueryType.GetConfigurationValue, new GetConfigurationValueParameters(configValues, ConfigCommon.defaultConfigurationVersion)).getReturnValue(); if (locales != null && !locales.isEmpty()) { String[] localeKeys = locales.trim().split(" *, *"); for (String localeKey: localeKeys) { try { org.apache.commons.lang.LocaleUtils.toLocale(localeKey); result.add(localeKey); } catch (IllegalArgumentException iae) { } } } return result; }
Source:
<code>protected ByteStreamSplitValuesReader(int elementSizeInBytes) { <START> numStreams = elementSizeInBytes; <END> this.elementSizeInBytes = elementSizeInBytes; indexInStream = 0; numberOfValues = 0; }</code><technical_language>Please add "this." numStreams, indexInStream, numberOfValues consistent for fields</technical_language>

Target:
protected ByteStreamSplitValuesReader(int elementSizeInBytes) { this.elementSizeInBytes = elementSizeInBytes; this.indexInStream = 0; this.valuesCount = 0; }
Source:
<code>public void onPartitionsAssigned(Collection<TopicPartition> partitions) { log.debug("controlLoop partitions assigned"); <START> acknowlegmentHandler.addPartitions(partitions); <END> for (TopicPartition partition : partitions) { flowControl.merge(partition, new AtomicInteger(), (oldValue, newValue) -> oldValue == null ? newValue : oldValue); } }</code><technical_language>This block is updateAssignment; call that</technical_language>

Target:
public void onPartitionsAssigned(Collection<TopicPartition> partitions) { log.debug("controlLoop partitions assigned"); acknowledgmentHandler.addPartitions(partitions); for (TopicPartition partition : partitions) { flowControl.merge(partition, new AtomicInteger(), (oldValue, newValue) -> oldValue == null ? newValue : oldValue); } }
Source:
<code>private Profile[] findExistingProfiles( List<PagedFile> pagedFiles ) { return pagedFiles.stream() .map( pf -> pf.file().getParentFile() ) .distinct() .flatMap( dir -> Profile.findProfilesInDirectory( fs, dir ) ) <START> .sorted() <END> .toArray( Profile[]::new ); }</code><technical_language>This adds nothing</technical_language>

Target:
private Profile[] findExistingProfiles( List<PagedFile> pagedFiles ) { return pagedFiles.stream() .map( pf -> pf.file().getParentFile() ) .distinct() .flatMap( dir -> Profile.findProfilesInDirectory( fs, dir ) ) .toArray( Profile[]::new ); }
Source:
<code>public ProtocolVersion getLowerSupported() { switch (this) { case V5: return V4; case V4: return V3; case V3: return V2; case V2: return V1; } return null; <START> } <END></code><technical_language>return INT_TO_VERSION.get(asInt - 1);</technical_language>

Target:
public ProtocolVersion getLowerSupported() { return lowerSupported; }
Source:
<code>public int getMaxSnapshotID() { <START> return (1 << SNAPSHOT_ID_BIT_WIDTH); <END> }</code><technical_language>I leave this max index -1, snapshot id here</technical_language>

Target:
public int getMaxSnapshotID() { return ((1 << SNAPSHOT_ID_BIT_WIDTH) - 1); }
Source:
<code>static String findProjectBaseDir(Map<String, Object> properties) { Path rootBaseDir = Paths.get(properties.get("sonar.projectBaseDir").toString()).toAbsolutePath().normalize(); List<Path> allProjectsBaseDir = properties.entrySet().stream() .filter(e -> e.getKey().endsWith(".projectBaseDir")) .map(e -> Paths.get(e.getValue().toString())) .collect(Collectors.toList()); for (Path baseDir : allProjectsBaseDir) { if (!baseDir.toAbsolutePath().normalize().startsWith(rootBaseDir)) { int c1 = baseDir.getNameCount(); int c2 = rootBaseDir.getNameCount(); Path newBaseDir = rootBaseDir.getRoot(); for (int i = 0; i < c1 && i < c2 && baseDir.getName(i).equals(rootBaseDir.getName(i)); i++) { newBaseDir = newBaseDir.resolve(rootBaseDir.getName(i)); } rootBaseDir = newBaseDir; } <START> } <END> return rootBaseDir.toString(); }</code><technical_language>suggestion if (!rootBaseDir.getRoot().equals(baseDir.getRoot())) { LOG.warn("Unable analyze module " + baseDir + " is in root"); continue; } Path newBaseDir = rootBaseDir.getRoot();</technical_language>

Target:
static String findProjectBaseDir(Map<String, Object> properties) { Path rootBaseDir = Paths.get(properties.get("sonar.projectBaseDir").toString()).toAbsolutePath().normalize(); List<Path> allProjectsBaseDir = properties.entrySet().stream() .filter(e -> e.getKey().endsWith(".projectBaseDir")) .map(e -> Paths.get(e.getValue().toString())) .collect(Collectors.toList()); for (Path baseDir : allProjectsBaseDir) { Path normalizedBaseDir = baseDir.toAbsolutePath().normalize(); if (!normalizedBaseDir.getRoot().equals(rootBaseDir.getRoot())) { continue; } if (!normalizedBaseDir.startsWith(rootBaseDir)) { int c1 = normalizedBaseDir.getNameCount(); int c2 = rootBaseDir.getNameCount(); Path newBaseDir = rootBaseDir.getRoot(); for (int i = 0; i < c1 && i < c2 && normalizedBaseDir.getName(i).equals(rootBaseDir.getName(i)); i++) { newBaseDir = newBaseDir.resolve(rootBaseDir.getName(i)); } rootBaseDir = newBaseDir; } } return rootBaseDir.toString(); }
Source:
<code>public IContentProvider getContentProvider() { <START> if (useDynamic) <END> return dynamicProvider; return super.getContentProvider(); }</code><technical_language>Please brackets, for one-line blocks</technical_language>

Target:
public IContentProvider getContentProvider() { if (useDynamic) { return dynamicProvider; } return super.getContentProvider(); }
Source:
<code>private <T> T executeRawQuery( SQLiteDatabase database, String query, ExecuteResultHandler<T> handler) { Cursor cursor = database.rawQuery(query, null); try { <START> return handler.handleResult(cursor); <END> } finally { cursor.close(); } }</code><technical_language>Out of curiosity, this UPDATE al passed? cursor</technical_language>

Target:
private <T> T executeRawQuery( SQLiteDatabase database, String query, ExecuteResultHandler<T> handler) { database.execSQL(query); return handler.handleRawQuery(); }
Source:
<code>public void registerHandlers() { SessionFactory session = gov.hhs.fha.nhinc.event.persistence.HibernateUtil.getSessionFactory(); session = gov.hhs.fha.nhinc.direct.messagemonitoring.persistence.HibernateUtil.getSessionFactory(); LOG.debug("Registering event Loggers"); EventLoggerFactory.getInstance().registerLoggers(); <START> LOG.debug("Registering handlers..."); <END> getDirectReceiver(); }</code><technical_language>debug statements changed trace removed</technical_language>

Target:
public void registerHandlers() { SessionFactory session = gov.hhs.fha.nhinc.event.persistence.HibernateUtil.getSessionFactory(); session = gov.hhs.fha.nhinc.direct.messagemonitoring.persistence.HibernateUtil.getSessionFactory(); LOG.trace("Registering event Loggers"); EventLoggerFactory.getInstance().registerLoggers(); LOG.trace("Registering handlers..."); getDirectReceiver(); }
Source:
<code>public void testSerialTypes() { List<Row> results = Lists.newArrayList( tEnv.sqlQuery(String.format("select * from %s", TABLE_SERIAL_TYPE)).execute().collect()); assertEquals("[" + <START> "2767," + <END> "2147483647," + "32767," + "2147483647," + "9223372036854775807," + "9223372036854775807]", results.toString()); }</code><technical_language>suggestion "32767," +</technical_language>

Target:
public void testSerialTypes() { List<Row> results = Lists.newArrayList( tEnv.sqlQuery(String.format("select * from %s", TABLE_SERIAL_TYPE)).execute().collect()); assertEquals("[" + "32767," + "2147483647," + "32767," + "2147483647," + "9223372036854775807," + "9223372036854775807]", results.toString()); }