auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>protected CalculatedStatus calculateStatus() { Maybe<SshMachineLocation> sshMachineOption = Machines.findUniqueSshMachineLocation(entity.getLocations()); if (sshMachineOption.isPresent()) { SshMachineLocation sshMachine = sshMachineOption.get(); try { Duration timeout = config().get(CONNECT_TIMEOUT); Map<String, ?> flags = ImmutableMap.of( SshTool.PROP_CONNECT_TIMEOUT.getName(), timeout.toMilliseconds(), SshTool.PROP_SESSION_TIMEOUT.getName(), timeout.toMilliseconds(), SshTool.PROP_SSH_TRIES.getName(), 1); int exitCode = sshMachine.execCommands(flags, SshMachineFailureDetector.class.getName(), ImmutableList.of("exit")); return new BasicCalculatedStatus(exitCode == 0, sshMachine.toString()); <START> } catch (Exception e) { <END> return new BasicCalculatedStatus(false, e.getMessage()); } } else { return new BasicCalculatedStatus(true, "no machine started, not complaining"); } }</code><technical_language>Risks swallowing InterruptedException. Worth Exceptions.propagateIfFatal(e). Do want log (at debug) time ssh exception? strong feelings. want log time (unless trace)</technical_language>

Target:
protected CalculatedStatus calculateStatus() { Maybe<SshMachineLocation> sshMachineOption = Machines.findUniqueSshMachineLocation(entity.getLocations()); if (sshMachineOption.isPresent()) { SshMachineLocation sshMachine = sshMachineOption.get(); try { Duration timeout = config().get(CONNECT_TIMEOUT); Map<String, ?> flags = ImmutableMap.of( SshTool.PROP_CONNECT_TIMEOUT.getName(), timeout.toMilliseconds(), SshTool.PROP_SESSION_TIMEOUT.getName(), timeout.toMilliseconds(), SshTool.PROP_SSH_TRIES.getName(), 1); int exitCode = sshMachine.execCommands(flags, SshMachineFailureDetector.class.getName(), ImmutableList.of("exit")); return new BasicCalculatedStatus(exitCode == 0, sshMachine.toString()); } catch (Exception e) { Exceptions.propagateIfFatal(e); boolean isFirstFailure = lastPublished != LastPublished.FAILED && currentFailureStartTime == null; if (isFirstFailure) { if (LOG.isDebugEnabled()) { LOG.debug("Failed connecting to machine " + sshMachine, e); } } else { if (LOG.isTraceEnabled()) { LOG.trace("Failed connecting to machine " + sshMachine, e); } } return new BasicCalculatedStatus(false, e.getMessage()); } } else { return new BasicCalculatedStatus(true, "no machine started, not complaining"); } }
Source:
<code><START> private MessageType createSchema() { <END> return new MessageType("schema", new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.INT32, INT32_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.INT64, INT64_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.FLOAT, FLOAT_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.DOUBLE, DOUBLE_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.BINARY, BINARY_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.FIXED_LEN_BYTE_ARRAY, 12, FIXED_LEN_BYTE_ARRAY_FIELD) ); }</code><technical_language>this static</technical_language>

Target:
private static MessageType createSchema() { return new MessageType("schema", new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.INT32, INT32_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.INT64, INT64_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.FLOAT, FLOAT_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.DOUBLE, DOUBLE_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.BINARY, BINARY_FIELD), new PrimitiveType(Type.Repetition.REQUIRED, PrimitiveType.PrimitiveTypeName.FIXED_LEN_BYTE_ARRAY, 12, FIXED_LEN_BYTE_ARRAY_FIELD) ); }
Source:
<code>public ProjectExporter(ComponentService.Iface componentClient, ProjectService.Iface projectClient, User user, List<Project> projects, boolean extendedByReleases) throws SW360Exception { super(new ProjectHelper(projectClient, user, extendedByReleases, <START> new ReleaseHelper(componentClient, false, null, null))); <END> preloadRelatedDataFor(projects, extendedByReleases, user); }</code><technical_language>overloaded constructor in ReleaseHelper do this(cClient, false, null, null). false, null, null part code readable this place, parameters mean. usage in ComponentExporter</technical_language>

Target:
public ProjectExporter(ComponentService.Iface componentClient, ProjectService.Iface projectClient, User user, List<Project> projects, boolean extendedByReleases) throws SW360Exception { super(new ProjectHelper(projectClient, user, extendedByReleases, new ReleaseHelper(componentClient))); preloadRelatedDataFor(projects, extendedByReleases, user); }
Source:
<code>private DemoAdapter() { for (int i = 0; i < <START> sSize; i++) <END> { mItems.add("Item " + i); } }</code><technical_language>Good point! Forgot this constructor</technical_language>

Target:
private DemoAdapter() { for (int i = 0; i < SIZE; i++) { mItems.add("Item " + i); } }
Source:
<code><START> public static List<IAnalysisModuleSource> getSources() { <END> List<IAnalysisModuleSource> sources = new ArrayList<>(); IConfigurationElement[] config = Platform.getExtensionRegistry().getConfigurationElementsFor(TMF_ANALYSIS_TYPE_ID); for (IConfigurationElement ce : config) { String elementName = ce.getName(); if (elementName.equals(SOURCE_ELEM)) { try { IAnalysisModuleSource source = (IAnalysisModuleSource) ce.createExecutableExtension(CLASS_ATTR); sources.add(source); } catch (InvalidRegistryObjectException e) { Activator.logError("Error creating module source", e); } catch (CoreException e) { Activator.logError("Error creating module source", e); } } } return sources; }</code><technical_language>Minor thing, is need expose this a List object (getSources().get(1) anything), return type a Collection<IAnalysisModuleSource></technical_language>

Target:
public static Iterable<IAnalysisModuleSource> getSources() { List<IAnalysisModuleSource> sources = new ArrayList<>(); IConfigurationElement[] config = Platform.getExtensionRegistry().getConfigurationElementsFor(TMF_ANALYSIS_TYPE_ID); for (IConfigurationElement ce : config) { String elementName = ce.getName(); if (elementName.equals(SOURCE_ELEM)) { try { IAnalysisModuleSource source = (IAnalysisModuleSource) ce.createExecutableExtension(CLASS_ATTR); sources.add(source); } catch (InvalidRegistryObjectException e) { Activator.logError("Error creating module source", e); } catch (CoreException e) { Activator.logError("Error creating module source", e); } } } return sources; }
Source:
<code>public void canTriggerOpenCircuit() throws InterruptedException, TimeoutException { EventForwarder eventForwarder = mock(EventForwarder.class); NettyDnsResolver dnsResolver = new NettyDnsResolver(eventForwarder, new MetricRegistry()); dnsResolver.setNameservers(InetAddressUtils.str(InetAddressUtils.UNPINGABLE_ADDRESS)); <START> dnsResolver.setBreakerFailureRateThreshold(80); <END> dnsResolver.setBreakerWaitDurationInOpenState(15); dnsResolver.setBreakerRingBufferSizeInHalfOpenState(10); dnsResolver.setBreakerRingBufferSizeInClosedState(100); dnsResolver.init(); final int N = 2 * dnsResolver.getCircuitBreaker().getCircuitBreakerConfig().getRingBufferSizeInClosedState(); final CompletableFuture futures[] = new CompletableFuture[N]; for (int i = 0; i < N; i++) { futures[i] = dnsResolver.reverseLookup(InetAddressUtils.addr("fe80::")); } try { CompletableFuture.allOf(futures) .get(2 * dnsResolver.getQueryTimeoutMillis(), TimeUnit.MILLISECONDS); fail("Expected an ExecutionException to be thrown"); } catch (ExecutionException e) { } try { dnsResolver.reverseLookup(InetAddressUtils.addr("fe80::")).get(); fail("Expected an CallNotPermittedException to be thrown"); } catch (ExecutionException e) { assertThat(e.getCause(), is(instanceOf(CallNotPermittedException.class))); } }</code><technical_language>Move defaults class instead of user call setters</technical_language>

Target:
public void canTriggerOpenCircuit() throws InterruptedException, TimeoutException { EventForwarder eventForwarder = mock(EventForwarder.class); NettyDnsResolver dnsResolver = new NettyDnsResolver(eventForwarder, new MetricRegistry()); dnsResolver.setNameservers(InetAddressUtils.str(InetAddressUtils.UNPINGABLE_ADDRESS)); dnsResolver.init(); final int N = 2 * dnsResolver.getCircuitBreaker().getCircuitBreakerConfig().getRingBufferSizeInClosedState(); final CompletableFuture futures[] = new CompletableFuture[N]; for (int i = 0; i < N; i++) { futures[i] = dnsResolver.reverseLookup(InetAddressUtils.addr("fe80::")); } try { CompletableFuture.allOf(futures) .get(2 * dnsResolver.getQueryTimeoutMillis(), TimeUnit.MILLISECONDS); fail("Expected an ExecutionException to be thrown"); } catch (ExecutionException e) { } try { dnsResolver.reverseLookup(InetAddressUtils.addr("fe80::")).get(); fail("Expected an CallNotPermittedException to be thrown"); } catch (ExecutionException e) { assertThat(e.getCause(), is(instanceOf(CallNotPermittedException.class))); } }
Source:
<code>public URI getBaseUri() { if (baseURI == null) { <START> baseURI = delegate.getBaseUriBuilder().scheme(scheme).host(hostname).port(port).replacePath(contextPath).build(); <END> } return baseURI; }</code><technical_language>this change applied other methods of this class (getAbsolutePath, getRequestUri)</technical_language>

Target:
public URI getBaseUri() { if (baseURI == null) { baseURI = initUriBuilder(delegate.getBaseUriBuilder()).build(); } return baseURI; }
Source:
<code>KafkaConsumer<byte[], byte[]> newConsumer() { ClassLoader currCtxCl = Thread.currentThread().getContextClassLoader(); try { Thread.currentThread().setContextClassLoader(getClass().getClassLoader()); final Map<String, Object> consumerConfigs = KafkaConsumerConfigs.getConsumerProperties(); final Properties props = new Properties(); <START> props.putAll(consumerConfigs); <END> KafkaRecordSupplier.addConsumerPropertiesFromConfig( props, configMapper, ioConfig.getConsumerProperties() ); return new KafkaConsumer<>(props); } finally { Thread.currentThread().setContextClassLoader(currCtxCl); } }</code><technical_language>This called KafkaRecordSupplier.addConsumerPropertiesFromConfig</technical_language>

Target:
KafkaConsumer<byte[], byte[]> newConsumer() { ClassLoader currCtxCl = Thread.currentThread().getContextClassLoader(); try { Thread.currentThread().setContextClassLoader(getClass().getClassLoader()); final Map<String, Object> consumerConfigs = KafkaConsumerConfigs.getConsumerProperties(); final Properties props = new Properties(); KafkaRecordSupplier.addConsumerPropertiesFromConfig( props, configMapper, ioConfig.getConsumerProperties() ); props.putAll(consumerConfigs); return new KafkaConsumer<>(props); } finally { Thread.currentThread().setContextClassLoader(currCtxCl); } }
Source:
<code>private static Document toDocument(Node node, Map<String, String> namespaces) { Namespaces abbr = namespaces instanceof Namespaces ? (Namespaces) namespaces : new Namespaces(namespaces); DocumentBuilder builder; try { builder = DocumentBuilderFactory.newInstance().newDocumentBuilder(); } catch (ParserConfigurationException e) { String message = e.getMessage(); <START> throw new RuntimeException(message != null ? message : e.getClass().getSimpleName(), e); <END> } String qualifiedName = abbr.abbreviateElement(getTypeOrDefault(node)); Document document = builder.getDOMImplementation() .createDocument(Namespaces.namespaceOfForXMLFile(getTypeOrDefault(node)), qualifiedName, null); Element root = document.getDocumentElement(); if ((node instanceof Node) && (node instanceof IdentifiableNode)) { root.setAttribute(abbr.abbreviateAttribute(getTypeOrDefault(node), RDF.ABOUT), ((IdentifiableNode) node).getIdentifier()); } nodeToElement(node, document, abbr, root); for (Entry<String, String> nsAttr : abbr.namespaceSetForXMLFile()) { root.setAttribute(nsAttr.getKey(), nsAttr.getValue()); } return document; }</code><technical_language>if this is happen a reason this exception is thrown declared a fetched RuntimeException is thrown</technical_language>

Target:
private static Document toDocument(Node node, Map<String, String> namespaces) { try { Namespaces abbr = namespaces instanceof Namespaces ? (Namespaces) namespaces : new Namespaces(namespaces); String qualifiedName = abbr.abbreviateElement(getTypeOrDefault(node)); Document document = DocumentBuilderFactory.newInstance().newDocumentBuilder().getDOMImplementation() .createDocument(Namespaces.namespaceOfForXMLFile(getTypeOrDefault(node)), qualifiedName, null); Element root = document.getDocumentElement(); if (node instanceof IdentifiableNode) { root.setAttribute(abbr.abbreviateAttribute(getTypeOrDefault(node), RDF.ABOUT), ((IdentifiableNode) node).getIdentifier()); } nodeToElement(node, document, abbr, root); abbr.namespaceSetForXMLFile().forEach(nsAttr -> root.setAttribute(nsAttr.getKey(), nsAttr.getValue())); return document; } catch (ParserConfigurationException e) { throw new IllegalStateException(e.getMessage(), e); } }
Source:
<code>public static String toPortableString(Path path) { Objects.requireNonNull(path, "The given path must not be null"); if (path.isAbsolute()) { throw new IllegalArgumentException("The given path must not be absolute"); } String localPathString = path.toString(); if (File.separator.equals(PORTABLE_SEPARATOR)) { <START> if (localPathString.contains(WINDOWS_SEPARATOR)) { <END> log.warn( "The given local path contains occurrences of the Windows separator character '" + WINDOWS_SEPARATOR + "'. As this character is not seen as a separator by the local filesystem, this " + "will most likely lead to issues when the path is transferred to a Saros " + "instance running on a Windows system."); } return localPathString; } return localPathString.replace(File.separator, PORTABLE_SEPARATOR); }</code><technical_language>a bunch more of characters directory names reserved Windows systems</technical_language>

Target:
public static String toPortableString(Path path) { Objects.requireNonNull(path, "The given path must not be null"); Path normalizedPath = normalize(path); String normalizedLocalPathString = normalizedPath.toString(); if (File.separator.equals(PORTABLE_SEPARATOR)) { if (normalizedLocalPathString.contains(WINDOWS_SEPARATOR)) { log.warn( "The given local path contains occurrences of the Windows separator character '" + WINDOWS_SEPARATOR + "'. This character is not seen as a separator by the local filesystem, meaning " + " resolving the path on a Windows system will lead to a different path. path: " + normalizedLocalPathString); } return normalizedLocalPathString; } return normalizedLocalPathString.replace(File.separator, PORTABLE_SEPARATOR); }
Source:
<code>private static void merge(String key, BiFunction<Long,Long,Long> mergeFunc, Map<String, Long> stats1, Map<String,Long> stats2) { <START> if (stats2.containsKey(key) && (stats1.containsKey(key) == false)) { <END> stats1.put(key, stats2.get(key)); } else if (stats2.containsKey(key) && stats1.containsKey(key)){ stats1.merge(key, stats2.get(key), mergeFunc); } }</code><technical_language>I Java's merge() function map handles case a key is present. I code insert stats2 if stats1 a (w/o calling mergeFunc). java Long val2 = stats2.get(key); if(val2 != null) { stats1.merge(key, val2, mergeFunc); } This avoid calling contains() get() map is lookups</technical_language>

Target:
private static void merge(String key, BiFunction<Long,Long,Long> mergeFunc, Map<String, Long> stats1, Map<String,Long> stats2) { Long mergeVal = stats2.get(key); if(mergeVal != null) { stats1.merge(key, mergeVal, mergeFunc); } }
Source:
<code>private CacheStoreHolder getHolder(int grpId) throws IgniteCheckedException { try { return cleanupAsyncExecutor.afterAsyncCompletion( () -> idxCacheStores.computeIfAbsent(grpId, (key) -> { CacheGroupDescriptor gDesc = cctx.cache().cacheGroupDescriptors().get(grpId); CacheStoreHolder holder0 = null; <START> if (gDesc != null) { <END> if (CU.isPersistentCache(gDesc.config(), cctx.gridConfig().getDataStorageConfiguration())) { try { holder0 = initForCache(gDesc, gDesc.config()); } catch (IgniteCheckedException e) { throw new IgniteException(e); } } } return holder0; }) ); } catch (IgniteException ex) { if (X.hasCause(ex, IgniteCheckedException.class)) throw ex.getCause(IgniteCheckedException.class); else throw ex; } }</code><technical_language>Megre if statement single if</technical_language>

Target:
private CacheStoreHolder getHolder(int grpId) throws IgniteCheckedException { try { return idxCacheStores.computeIfAbsent(grpId, (key) -> { CacheGroupDescriptor gDesc = cctx.cache().cacheGroupDescriptors().get(grpId); CacheStoreHolder holder0 = null; if (gDesc != null && CU.isPersistentCache(gDesc.config(), cctx.gridConfig().getDataStorageConfiguration())) { try { holder0 = initForCache(gDesc, gDesc.config()); } catch (IgniteCheckedException e) { throw new IgniteException(e); } } return holder0; }); } catch (IgniteException ex) { if (X.hasCause(ex, IgniteCheckedException.class)) throw ex.getCause(IgniteCheckedException.class); else throw ex; } }
Source:
<code>protected void run() throws Exception { final AmazonS3 s3 = new AmazonS3(properties()); if (null == op) { throw die(MessageFormat.format(CLIText.get().unsupportedOperation, op)); } else switch (op) { case <START> "get": END> final URLConnection c = s3.get(bucket, key); int len = c.getContentLength(); try (InputStream in = c.getInputStream()) { outw.flush(); final byte[] tmp = new byte[2048]; while (len > 0) { final int n = in.read(tmp); if (n < 0) throw new EOFException(MessageFormat.format( CLIText.get().expectedNumberOfbytes, valueOf(len))); outs.write(tmp, 0, n); len -= n; } outs.flush(); } break; case "ls"://$NON-NLS-1$ case "list"://$NON-NLS-1$ for (String k : s3.list(bucket, key)) outw.println(k); break; case "rm"://$NON-NLS-1$ case "delete"://$NON-NLS-1$ s3.delete(bucket, key); break; case "put"://$NON-NLS-1$ try (OutputStream os = s3.beginPut(bucket, key, null, null)) { final byte[] tmp = new byte[2048]; int n; while ((n = ins.read(tmp)) > 0) os.write(tmp, 0, n); } break; default: throw die(MessageFormat.format(CLIText.get().unsupportedOperation, op)); } }</code><technical_language>Spurious TAB, a blank</technical_language>

Target:
protected void run() throws Exception { final AmazonS3 s3 = new AmazonS3(properties()); if (op == null) { throw die(MessageFormat.format(CLIText.get().unsupportedOperation, op)); } switch (op) { case "get": final URLConnection c = s3.get(bucket, key); int len = c.getContentLength(); try (InputStream in = c.getInputStream()) { outw.flush(); final byte[] tmp = new byte[2048]; while (len > 0) { final int n = in.read(tmp); if (n < 0) throw new EOFException(MessageFormat.format( CLIText.get().expectedNumberOfbytes, valueOf(len))); outs.write(tmp, 0, n); len -= n; } outs.flush(); } break; case "ls": case "list": for (String k : s3.list(bucket, key)) outw.println(k); break; case "rm": case "delete": s3.delete(bucket, key); break; case "put": try (OutputStream os = s3.beginPut(bucket, key, null, null)) { final byte[] tmp = new byte[2048]; int n; while ((n = ins.read(tmp)) > 0) os.write(tmp, 0, n); } break; default: throw die(MessageFormat.format(CLIText.get().unsupportedOperation, op)); } }
Source:
<code>static boolean isSupportedOnVersion(ListChangesOption listChangesOption, String gerritVersion) { ListChangesOption maxOption = maxChangeOptionByVersion.get(gerritVersion); <START> if(maxOption == null) { <END> return true; } return listChangesOption.getValue() <= maxOption.getValue(); }</code><technical_language>please add a space if for consistency (here in other cases)</technical_language>

Target:
static boolean isSupportedOnVersion(ListChangesOption listChangesOption, String gerritVersion) { Matcher versionMatcher = VERSION_PATTERN.matcher(gerritVersion); if (!versionMatcher.matches()) { return true; } ListChangesOption maxOption = MAX_CHANGE_OPTION_BY_VERSION.get(versionMatcher.group(1) + "." + versionMatcher.group(2)); if (maxOption == null) { return true; } return listChangesOption.getValue() <= maxOption.getValue(); }
Source:
<code><START> protected Route(BlackboardArtifact artifact) throws TskCoreException { <END> points = new ArrayList<>(); Waypoint point = getRouteStartPoint(artifact); if (point != null) { points.add(point); } point = getRouteEndPoint(artifact); if (point != null) { points.add(point); } altitude = getRouteAltitude(artifact); timestamp = getRouteTimestamp(artifact); otherAttributesMap = initalizeOtherAttributes(artifact); }</code><technical_language>ArtifactWaypoint, this class is final a package access constructor</technical_language>

Target:
Route(BlackboardArtifact artifact) throws GeoLocationDataException { points = new ArrayList<>(); Map<BlackboardAttribute.ATTRIBUTE_TYPE, BlackboardAttribute> attributeMap = ArtifactWaypoint.getAttributesFromArtifactAsMap(artifact); points.add(getRouteStartPoint(attributeMap)); points.add(getRouteEndPoint(attributeMap)); BlackboardAttribute attribute = attributeMap.get(BlackboardAttribute.ATTRIBUTE_TYPE.TSK_GEO_ALTITUDE); altitude = attribute != null ? attribute.getValueDouble() : null; attribute = attributeMap.get(BlackboardAttribute.ATTRIBUTE_TYPE.TSK_DATETIME); timestamp = attribute != null ? attribute.getValueLong() : null; immutablePropertiesList = Collections.unmodifiableList(Waypoint.createGeolocationProperties(attributeMap)); }
Source:
<code>public VdcActionParametersBase getParameters(Template incoming, VmTemplate entity) { VmTemplate updated = getMapper(modelType, VmTemplate.class).map(incoming, entity); updated.setUsbPolicy(VmMapper.getUsbPolicyOnUpdate(incoming.getUsb(), entity.getUsbPolicy(), lookupCluster(updated.getVdsGroupId()))); UpdateVmTemplateParameters params = new UpdateVmTemplateParameters(updated); if (incoming.isSetConsoleDeviceEnabled()) { params.setConsoleEnabled(incoming.isConsoleDeviceEnabled()); <START> } <END> return params; }</code><technical_language>create a dedicated mapper for UpdateVmTemplateParameters (see [1]), [1] VmMapper.RunVmOnceParams map(VM vm, RunVmOnceParams template)</technical_language>

Target:
public VdcActionParametersBase getParameters(Template incoming, VmTemplate entity) { VmTemplate updated = getMapper(modelType, VmTemplate.class).map(incoming, entity); updated.setUsbPolicy(VmMapper.getUsbPolicyOnUpdate(incoming.getUsb(), entity.getUsbPolicy(), lookupCluster(updated.getVdsGroupId()))); return getMapper(modelType, UpdateVmTemplateParameters.class).map(incoming, new UpdateVmTemplateParameters(updated)); }
Source:
<code>private ArrayList<CramCompressionRecord> getRecords(final Slice slice, final CompressionHeader header, final ValidationStringency validationStringency) { String seqName = SAMRecord.NO_ALIGNMENT_REFERENCE_NAME; switch (slice.sequenceId) { case SAMRecord.NO_ALIGNMENT_REFERENCE_INDEX: case -2: break; default: final SAMSequenceRecord sequence = samFileHeader .getSequence(slice.sequenceId); seqName = sequence.getSequenceName(); break; } final CramRecordReader reader = slice.createCramRecordReader(header, validationStringency); final ArrayList<CramCompressionRecord> records = new ArrayList<>(slice.nofRecords); int prevAlignmentStart = slice.alignmentStart; for (int i = 0; i < slice.nofRecords; i++) { final CramCompressionRecord record = new CramCompressionRecord(); record.sliceIndex = slice.index; record.index = i; reader.read(record, prevAlignmentStart); <START> prevAlignmentStart = record.alignmentStart; <END> if (record.sequenceId == slice.sequenceId) { record.sequenceName = seqName; record.sequenceId = slice.sequenceId; } else { if (record.sequenceId == SAMRecord.NO_ALIGNMENT_REFERENCE_INDEX) { record.sequenceName = SAMRecord.NO_ALIGNMENT_REFERENCE_NAME; } else { record.sequenceName = samFileHeader.getSequence(record.sequenceId) .getSequenceName(); } } records.add(record); } return records; }</code><technical_language>alternative suggestion: CramRecordReader.read return new alignment start</technical_language>

Target:
private ArrayList<CramCompressionRecord> getRecords(final Slice slice, final CompressionHeader header, final ValidationStringency validationStringency) { String seqName = SAMRecord.NO_ALIGNMENT_REFERENCE_NAME; switch (slice.sequenceId) { case SAMRecord.NO_ALIGNMENT_REFERENCE_INDEX: case -2: break; default: final SAMSequenceRecord sequence = samFileHeader .getSequence(slice.sequenceId); seqName = sequence.getSequenceName(); break; } final CramRecordReader reader = slice.createCramRecordReader(header, validationStringency); final ArrayList<CramCompressionRecord> records = new ArrayList<>(slice.nofRecords); int prevAlignmentStart = slice.alignmentStart; for (int i = 0; i < slice.nofRecords; i++) { final CramCompressionRecord record = new CramCompressionRecord(); record.sliceIndex = slice.index; record.index = i; prevAlignmentStart = reader.read(record, prevAlignmentStart); if (record.sequenceId == slice.sequenceId) { record.sequenceName = seqName; record.sequenceId = slice.sequenceId; } else { if (record.sequenceId == SAMRecord.NO_ALIGNMENT_REFERENCE_INDEX) { record.sequenceName = SAMRecord.NO_ALIGNMENT_REFERENCE_NAME; } else { record.sequenceName = samFileHeader.getSequence(record.sequenceId) .getSequenceName(); } } records.add(record); } return records; }
Source:
<code>public void selectionChanged(final SelectionChangedEvent event) { final ISelection selection = event.getSelection(); depopulateMenus(); Collection<?> newChildDescriptors = null; if (selection instanceof IStructuredSelection && ((IStructuredSelection) selection).size() == 1) { final Object object = ((IStructuredSelection) selection).getFirstElement(); final EditingDomain domain = ((IEditingDomainProvider) activeEditorPart).getEditingDomain(); newChildDescriptors = domain.getNewChildDescriptors(object, null); <START> } else if (((IStructuredSelection) selection).size() > 1) { <END> newChildDescriptors = Lists.newArrayList(); } if (newChildDescriptors != null) { updateChildDescriptorMenus(selection, newChildDescriptors); } populateMenus(); }</code><technical_language>Joao, selection is a IStructuredSelection here, if test return false becauseof selection type. Please review entire if block: if (sel instanceof IStruc..ion) { if (size ==1) { ... } else if (size > 1) { ... } }</technical_language>

Target:
public void selectionChanged(final SelectionChangedEvent event) { final ISelection selection = event.getSelection(); depopulateMenus(); Collection<?> newChildDescriptors = null; if (selection instanceof IStructuredSelection) { IStructuredSelection stsel = (IStructuredSelection) selection; if (stsel.size() == 1) { final Object object = ((IStructuredSelection) selection).getFirstElement(); final EditingDomain domain = ((IEditingDomainProvider) activeEditorPart).getEditingDomain(); newChildDescriptors = domain.getNewChildDescriptors(object, null); } else if (stsel.size() > 1) { newChildDescriptors = Lists.newArrayList(); } } if (newChildDescriptors != null) { updateChildDescriptorMenus(selection, newChildDescriptors); } populateMenus(); }
Source:
<code>public boolean apply(Game game, Ability source) { Player controller = game.getPlayer(source.getControllerId()); Permanent sourcePermanent = game.getPermanentOrLKIBattlefield(source.getSourceId()); Spell spell = (Spell) game.getStack().getStackObject(targetPointer.getFirst(game, source)); if(sourcePermanent != null && spell != null) { Cost cost = new ManaCostsImpl(spell.getSpellAbility().getManaCosts().getText()); <START> for (UUID playerId : game.getState().getPlayerList(source.getControllerId())) { <END> Player player = game.getPlayer(playerId); if(player.getId() != spell.getControllerId()) { cost.clearPaid(); if (cost.canPay(source, source.getSourceId(), player.getId(), game) && player.chooseUse(outcome, "Pay " + cost.getText() + " to counter " + spell.getIdName() + '?', source, game)) { if (cost.pay(source, game, source.getSourceId(), playerId, false, null)) { game.informPlayers(player.getLogName() + " pays" + cost.getText() + " to counter " + spell.getIdName() + '.'); game.getStack().counter(spell.getId(), source.getSourceId(), game); } } } } } return true; }</code><technical_language>getPlayersInRange(source.getControllerId()) get a list of players is controller interact ability</technical_language>

Target:
public boolean apply(Game game, Ability source) { Player controller = game.getPlayer(source.getControllerId()); Permanent sourcePermanent = game.getPermanentOrLKIBattlefield(source.getSourceId()); Spell spell = (Spell) game.getStack().getStackObject(targetPointer.getFirst(game, source)); if(sourcePermanent != null && spell != null && controller != null) { Player spellController = game.getPlayer(spell.getControllerId()); Cost cost = new ManaCostsImpl(spell.getSpellAbility().getManaCosts().getText()); if(spellController != null) { for (UUID playerId : game.getState().getPlayersInRange(source.getControllerId(), game)) { Player player = game.getPlayer(playerId); if(player != null && player != spellController) { cost.clearPaid(); if(cost.canPay(source, source.getSourceId(), player.getId(), game) && player.chooseUse(outcome, "Pay " + cost.getText() + " to counter " + spell.getIdName() + '?', source, game)) { if(cost.pay(source, game, source.getSourceId(), playerId, false, null)) { game.informPlayers(player.getLogName() + " pays" + cost.getText() + " to counter " + spell.getIdName() + '.'); game.getStack().counter(spell.getId(), source.getSourceId(), game); return true; } } } } } } return true; }
Source:
<code>public void clickOnWorkspaceStopStartButton(String workspaceName) { String buttonXpath = String.format(WORKSPACE_ITEM_STOP_START_WORKSPACE_BUTTON, workspaceName); <START> waitProgressBarInvisibility(); <END> moveCursorToWorkspaceRamSection(workspaceName); waitWorkspaceActionTooltipDisappearance(); seleniumWebDriverHelper.waitAndClick(By.xpath(buttonXpath)); waitProgressBarInvisibility(); }</code><technical_language>Do need wait for progress bar invisibility start of method? is expected workspace is completely ready perform operation, it</technical_language>

Target:
public void clickOnWorkspaceStopStartButton(String workspaceName) { String buttonXpath = String.format(WORKSPACE_ITEM_STOP_START_WORKSPACE_BUTTON, workspaceName); moveCursorToWorkspaceRamSection(workspaceName); waitWorkspaceActionTooltipDisappearance(); seleniumWebDriverHelper.waitAndClick(By.xpath(buttonXpath)); waitProgressBarInvisibility(); }
Source:
<code>public void handle(final I input) { <START> final S state = (S)ssm.read(queryId, operatorId).getState(); <END> final S intermediateState = updateState(input, state); ssm.update(queryId, operatorId, new OperatorState<>(intermediateState)); final O output = generateOutput(intermediateState); LOG.log(Level.FINE, "{0} updates the state {1} with input {2} to {3}, and generates {4}", new Object[]{getOperatorClassName(), state, input, intermediateState, output}); if (output != null) { outputEmitter.emit(output); } }</code><technical_language><S> for state, for consistency, I change OperatorState <S> <I>, is is right now</technical_language>

Target:
public void handle(final I input) { final S state = (S)ssm.read(queryId, operatorId).getState(); final S intermediateState = updateState(input, state); ssm.update(queryId, operatorId, new OperatorState<>(intermediateState)); final O output = generateOutput(intermediateState); LOG.log(Level.FINE, "{0} updates the state {1} with input {2} to {3}, and generates {4}", new Object[]{getOperatorIdentifier(), state, input, intermediateState, output}); if (output != null) { outputEmitter.emit(output); } }
Source:
<code>public void visitArrayAccess(ArrayAccessTree tree) { if (tree.object().is(Tree.Kind.VARIABLE_IDENTIFIER) && tree.offset() != null && tree.offset().is(Tree.Kind.REGULAR_STRING_LITERAL)) { String variable = ((VariableIdentifierTree) tree.object()).text(); String indexValue = trimQuotes((LiteralTree) tree.offset()); if ("argv".equals(indexValue) && SUSPICIOUS_ARRAY_ACCESSES.contains(variable)) { <START> context().newIssue(this, tree, MESSAGE); <END> } } super.visitArrayAccess(tree); }</code><technical_language>method I talking about. simplify of adding a method in NamespaceAwareVisitor. strange chained calls, while method: void reportIssue(Tree tree, String message) { context().newIssue(this, tree, message); }</technical_language>

Target:
public void visitArrayAccess(ArrayAccessTree tree) { ExpressionTree offset = tree.offset(); if (tree.object().is(Tree.Kind.VARIABLE_IDENTIFIER) && offset != null && offset.is(Tree.Kind.REGULAR_STRING_LITERAL)) { String variable = ((VariableIdentifierTree) tree.object()).text(); String indexValue = trimQuotes((LiteralTree) offset); if ("argv".equals(indexValue) && SUSPICIOUS_ARRAY_ACCESSES.contains(variable)) { context().newIssue(this, tree, MESSAGE); } } super.visitArrayAccess(tree); }
Source:
<code><START> public PartitionClientResponse[] callAllPartitionProcedure(String procedureName, Object... params) <END> throws IOException, NoConnectionsException, ProcCallException { Object[] args = new Object[params.length + 1]; System.arraycopy(params, 0, args, 1, params.length); setPartitions(); int partitionCount = m_partitionKeys.size(); PartitionClientResponse[] responses = new PartitionClientResponse[partitionCount]; for (int key : m_partitionKeys) { args[0] = key; partitionCount--; ClientResponse response = callProcedure(procedureName, args); responses[partitionCount] = new PartitionClientResponse(key, response); } return responses; }</code><technical_language>I suggest sync version call async version block callback. This reduces duplicated code, serialize calls, want</technical_language>

Target:
public ClientResponseWithPartitionKey[] callAllPartitionProcedure(String procedureName, Object... params) throws IOException, NoConnectionsException, ProcCallException { SyncAllPartitionProcedureCallback callBack = new SyncAllPartitionProcedureCallback(); callAllPartitionProcedure(callBack, procedureName, params); return callBack.getResponse(); }
Source:
<code>protected void setLoadedHostsEntries(Collection<HostEntryPair> keys) { <START> keysSupplier = () -> keys; <END> }</code><technical_language>Update of keysSupplier field is mutually exclusive..</technical_language>

Target:
protected void setLoadedHostsEntries(Collection<HostEntryPair> keys) { keysSupplier.set(() -> keys); }
Source:
<code><START> public void copyPixelsFromBuffer (Buffer dst) { <END> if (isRecycled()) { throw new IllegalStateException("Can't call copyPixelsFromBuffer() on a recycled bitmap"); } if (getBytesPerPixel(config) != 4) { throw new RuntimeException( "Not implemented: only Bitmaps with 4 bytes per pixel are supported"); } if (!(dst instanceof ByteBuffer)) { throw new RuntimeException("Not implemented: unsupported Buffer subclass"); } ByteBuffer byteBuffer = (ByteBuffer) dst; if (byteBuffer.remaining() < colors.length * 4) { throw new RuntimeException("Buffer not large enough for pixels"); } for (int i = 0; i < colors.length; i++) { colors[i] = byteBuffer.getInt(); } }</code><technical_language>remove whitespace parenthesis. below</technical_language>

Target:
public void copyPixelsFromBuffer(Buffer dst) { if (isRecycled()) { throw new IllegalStateException("Can't call copyPixelsFromBuffer() on a recycled bitmap"); } if (getBytesPerPixel(config) != INTERNAL_BYTES_PER_PIXEL) { throw new RuntimeException("Not implemented: only Bitmaps with " + INTERNAL_BYTES_PER_PIXEL + " bytes per pixel are supported"); } if (!(dst instanceof ByteBuffer)) { throw new RuntimeException("Not implemented: unsupported Buffer subclass"); } ByteBuffer byteBuffer = (ByteBuffer) dst; if (byteBuffer.remaining() < colors.length * INTERNAL_BYTES_PER_PIXEL) { throw new RuntimeException("Buffer not large enough for pixels"); } for (int i = 0; i < colors.length; i++) { colors[i] = byteBuffer.getInt(); } }
Source:
<code><START> public ResponseEntity<?> charge(@RequestBody @Valid ChargeDTO chargeDTO) { <END> LOG.info(" Charge controller new chargeDTO: " + chargeDTO.getDescription()); ChargeDTO resultDTO; try { resultDTO = paymentService.charge(chargeDTO, PaymentType.USER_LICENSE); } catch (ServiceException e) { LOG.log(Level.SEVERE, e.getMessage()); throw new BadRequestException(e.getMessage()); } return new ResponseEntity<>(resultDTO, HttpStatus.OK); }</code><technical_language>type ChargeDTO in return responseEntity instead of</technical_language>

Target:
public ResponseEntity<ChargeDTO> charge(@RequestBody @Valid ChargeDTO chargeDTO) { LOG.info(" Charge controller new chargeDTO: " + chargeDTO.getDescription()); ChargeDTO resultDTO; try { resultDTO = paymentService.charge(chargeDTO, PaymentType.USER_LICENSE); } catch (ServiceException e) { LOG.log(Level.SEVERE, e.getMessage()); throw new BadRequestException(e.getMessage()); } return new ResponseEntity<>(resultDTO, HttpStatus.OK); }
Source:
<code>public void schedule(Work work, Scheduling scheduling, boolean afterCommit) { String workId = work.getId(); String queueId = getCategoryQueueId(work.getCategory()); if (!isQueuingEnabled(queueId)) { return; } if (afterCommit && scheduleAfterCommit(work, scheduling)) { return; } work.setWorkInstanceState(State.SCHEDULED); WorkSchedulePath.newInstance(work); switch (scheduling) { case ENQUEUE: break; case CANCEL_SCHEDULED: getExecutor(queueId).removeScheduled(workId); WorkStateHelper.setCanceled(work.getId()); break; case IF_NOT_SCHEDULED: case IF_NOT_RUNNING_OR_SCHEDULED: boolean disabled = Boolean.TRUE.booleanValue(); if (!disabled && hasWorkInState(workId, scheduling.state)) { log.debug("Canceling schedule because found: {}", scheduling); return; } break; } if (work.isGroupJoin()) { WorkStateHelper.addGroupJoinWork(work.getPartitionKey()); <START> } <END> queuing.workSchedule(queueId, work); }</code><technical_language>point logging for StreamWorkManager</technical_language>

Target:
public void schedule(Work work, Scheduling scheduling, boolean afterCommit) { String workId = work.getId(); String queueId = getCategoryQueueId(work.getCategory()); if (!isQueuingEnabled(queueId)) { return; } if (afterCommit && scheduleAfterCommit(work, scheduling)) { return; } work.setWorkInstanceState(State.SCHEDULED); WorkSchedulePath.newInstance(work); switch (scheduling) { case ENQUEUE: break; case CANCEL_SCHEDULED: getExecutor(queueId).removeScheduled(workId); WorkStateHelper.setCanceled(work.getId()); break; case IF_NOT_SCHEDULED: case IF_NOT_RUNNING_OR_SCHEDULED: boolean disabled = Boolean.TRUE.booleanValue(); if (!disabled && hasWorkInState(workId, scheduling.state)) { log.debug("Canceling schedule because found: {}", scheduling); return; } break; } if (work.isGroupJoin()) { log.debug("Submit Work: to GroupJoin: {}", work::getId, work::getPartitionKey); WorkStateHelper.addGroupJoinWork(work.getPartitionKey()); } queuing.workSchedule(queueId, work); }
Source:
<code>public LeaderPingResult pingLeaderWithUuid(UUID uuid) { Optional<LeaderPingerContext<PingableLeader>> suspectedLeader = getSuspectedLeader(uuid); if (!suspectedLeader.isPresent()) { return LeaderPingResults.pingReturnedFalse(); } LeaderPingerContext<PingableLeader> leader = suspectedLeader.get(); MultiplexingCompletionService<LeaderPingerContext<PingableLeader>, Boolean> multiplexingCompletionService = MultiplexingCompletionService.create(leaderPingExecutors); try { multiplexingCompletionService.submit(leader, () -> leader.pinger().ping()); Future<Map.Entry<LeaderPingerContext<PingableLeader>, Boolean>> pingFuture = multiplexingCompletionService .poll(leaderPingResponseWait.toMillis(), TimeUnit.MILLISECONDS); return getLeaderPingResult(uuid, pingFuture); } catch (InterruptedException ex) { Thread.currentThread().interrupt(); return LeaderPingResults.pingCallFailure(ex); <START> } catch (RejectedExecutionException ex) { <END> log.warn("Could not ping the leader, because the executor used to talk to that node is overloaded", ex); return LeaderPingResults.pingCallFailure(ex); } }</code><technical_language>nit: while this above, normal usage is e instead of ex, including in other places in this PR</technical_language>

Target:
public LeaderPingResult pingLeaderWithUuid(UUID uuid) { Optional<LeaderPingerContext<PingableLeader>> suspectedLeader = getSuspectedLeader(uuid); if (!suspectedLeader.isPresent()) { return LeaderPingResults.pingReturnedFalse(); } LeaderPingerContext<PingableLeader> leader = suspectedLeader.get(); MultiplexingCompletionService<LeaderPingerContext<PingableLeader>, Boolean> multiplexingCompletionService = MultiplexingCompletionService.create(leaderPingExecutors); try { multiplexingCompletionService.submit(leader, () -> leader.pinger().ping()); Future<Map.Entry<LeaderPingerContext<PingableLeader>, Boolean>> pingFuture = multiplexingCompletionService .poll(leaderPingResponseWait.toMillis(), TimeUnit.MILLISECONDS); return getLeaderPingResult(uuid, pingFuture); } catch (InterruptedException e) { Thread.currentThread().interrupt(); return LeaderPingResults.pingCallFailure(e); } catch (RejectedExecutionException e) { log.warn("Could not ping the leader, because the executor used to talk to that node is overloaded", e); return LeaderPingResults.pingCallFailure(e); } }
Source:
<code>private void shareShortAppInviteLink(String longUrl) { FirebaseDynamicLinksHub firebaseDynamicLinksHub = App.from(activity).getFirebaseDynamicLinksHub(); String requestString = "{\"longDynamicLink\": \"" + longUrl + "\",\"suffix\": {\"option\": \"SHORT\"}}"; RequestBody body = RequestBody.create(okhttp3.MediaType.parse("application/json; charset=utf-8"), requestString); firebaseDynamicLinksHub.getShortenedUrl(BuildConfig.FIREBASE_WEB_API_KEY, body).enqueue(new Callback<FirebaseDynamicLinksResponse>() { @Override public void onSuccess(FirebaseDynamicLinksResponse response) { createChooser(HttpUrl.parse(response.getShortLink())); <START> shareProgressDialog.dismiss(); <END> } @Override public void onError() { createChooser(NON_SIGNED_IN_INVITE_URL); shareProgressDialog.dismiss(); } }); }</code><technical_language>try decouple UI concerns actual link shortening. link creation is asynchronous. Ideally a Listener a method onDynamicLinkReady(String url). Listener's method called shortened link is successful. place implement this Listener handle progress show/dismiss. do think</technical_language>

Target:
private void shareShortAppInviteLink(String longUrl) { FirebaseDynamicLinks firebaseDynamicLinks = App.from(activity).getFirebaseDynamicLinks(); FirebaseDynamicLinksRequest firebaseDynamicLinksRequest = new FirebaseDynamicLinksRequest(longUrl, new Suffix()); firebaseDynamicLinks.shortenUrl(BuildConfig.IP_SIMPLE_API_ACCESS_KEY, firebaseDynamicLinksRequest) .enqueue(new Callback<FirebaseDynamicLinksResponse>() { @Override public void onSuccess(FirebaseDynamicLinksResponse response) { createChooser(HttpUrl.parse(response.getShortLink())); shareProgressDialog.dismiss(); } @Override public void onError() { createChooser(NON_SIGNED_IN_INVITE_URL); shareProgressDialog.dismiss(); } }); }
Source:
<code>protected String getNonNullRequestParamValue(String paramName) { String value = getRequestParamValue(paramName); Assumption.assertPostParamNotNull(paramName, value); <START> return value; <END> }</code><technical_language>this instead? return getNonNullRequestParamValues(paramName)[0]</technical_language>

Target:
protected String getNonNullRequestParamValue(String paramName) { return getNonNullRequestParamValues(paramName)[0]; }
Source:
<code>protected void analyzeBFB(BasicFBType basicFBType) { ECC ecc = basicFBType.getECC(); int <START> nrstates = <END> ecc.getECState().size(); int nrtransitions = ecc.getECTransition().size(); double ccfb = (nrtransitions - nrstates + 2); for (ECState state : ecc.getECState()) { for (ECAction action : state.getECAction()) { if (null != action.getAlgorithm()) { ccfb += analyzeAlgorithm(action.getAlgorithm()); } } } ccapp += ccfb; metrics.add(new MetricData("Cyclomatic Number " + basicFBType.getName(), ccfb)); }</code><technical_language>camel casing</technical_language>

Target:
protected void analyzeBFB(BasicFBType basicFBType) { ECC ecc = basicFBType.getECC(); double ccfb = (ecc.getECTransition().size() - ecc.getECState().size() + 2); for (ECState state : ecc.getECState()) { for (ECAction action : state.getECAction()) { if (null != action.getAlgorithm()) { ccfb += analyzeAlgorithm(action.getAlgorithm()); } } } ccapp += ccfb; metrics.add(new MetricData("Cyclomatic Number " + basicFBType.getName(), ccfb)); }
Source:
<code>private KafkaIndexTask createTask( final String taskId, final DataSchema dataSchema, final KafkaIOConfig ioConfig ) { final KafkaTuningConfig tuningConfig = new KafkaTuningConfig( <START> 1000, null, <END> maxRowsPerSegment, new Period("P1Y"), null, null, null, true, reportParseExceptions, handoffConditionTimeout, resetOffsetAutomatically, null, intermediateHandoffPeriod ); final Map<String, Object> context = isIncrementalHandoffSupported ? ImmutableMap.of(KafkaSupervisor.IS_INCREMENTAL_HANDOFF_SUPPORTED, true) : null; final KafkaIndexTask task = new KafkaIndexTask( taskId, null, cloneDataSchema(dataSchema), tuningConfig, ioConfig, context, null, null ); task.setPollRetryMs(POLL_RETRY_MS); return task; }</code><technical_language>nit: Druid format convention is 1000, null,</technical_language>

Target:
private KafkaIndexTask createTask( final String taskId, final DataSchema dataSchema, final KafkaIOConfig ioConfig ) { final KafkaTuningConfig tuningConfig = new KafkaTuningConfig( 1000, null, maxRowsPerSegment, new Period("P1Y"), null, null, null, true, reportParseExceptions, handoffConditionTimeout, resetOffsetAutomatically, null, intermediateHandoffPeriod, logParseExceptions, maxParseExceptions, maxSavedParseExceptions ); final Map<String, Object> context = isIncrementalHandoffSupported ? ImmutableMap.of(KafkaSupervisor.IS_INCREMENTAL_HANDOFF_SUPPORTED, true) : null; final KafkaIndexTask task = new KafkaIndexTask( taskId, null, cloneDataSchema(dataSchema), tuningConfig, ioConfig, context, null, null ); task.setPollRetryMs(POLL_RETRY_MS); return task; }
Source:
<code>public String toString() { <START> return prow + " " + pcol + " " + (isWrite ? "WRITE" : "NOT_WRITE") + " " + (isDelete ? "DELETE" : "NOT_DELETE") <END> + " " + (isTrigger ? "TRIGGER" : "NOT_TRIGGER") + " " + transactor; }</code><technical_language>Personal preference, feel free ignore. printing longs in situations I prefer hex decimal</technical_language>

Target:
public String toString() { return prow + " " + pcol + " " + (isWrite ? "WRITE" : "NOT_WRITE") + " " + (isDelete ? "DELETE" : "NOT_DELETE") + " " + (isTrigger ? "TRIGGER" : "NOT_TRIGGER") + " " + TransactorID.longToString(transactor); }