auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>void initTable(ApplicationConstants constants) { TextColumnWithTooltip<event_subscriber> eventNameColumn = new EnumColumn<event_subscriber, AuditLogType>() { @Override protected AuditLogType getRawValue(event_subscriber object) { return Enum.valueOf(AuditLogType.class, object.getevent_up_name()); } }; eventNameColumn.makeSortable(new Comparator<event_subscriber>() { @Override public int compare(event_subscriber o1, event_subscriber o2) { <START> return new LexoNumericComparator().compare(o1.getevent_up_name(), o2.getevent_up_name()); <END> } }); getTable().addColumn(eventNameColumn, constants.eventNameEventNotifier()); getTable().addActionButton(new WebAdminButtonDefinition<event_subscriber>(constants.manageEventsEventNotifier()) { @Override protected UICommand resolveCommand() { return getDetailModel().getManageEventsCommand(); } }); }</code><technical_language>Please consider putting: private final LexoNumericComparator lexoNumeric = new LexoNumericComparator(); in scope of anonymous class declaration, compare() method</technical_language>

Target:
void initTable(ApplicationConstants constants) { TextColumnWithTooltip<event_subscriber> eventNameColumn = new EnumColumn<event_subscriber, AuditLogType>() { @Override protected AuditLogType getRawValue(event_subscriber object) { return Enum.valueOf(AuditLogType.class, object.getevent_up_name()); } }; eventNameColumn.makeSortable(new Comparator<event_subscriber>() { private final LexoNumericComparator lexoNumericComparator = new LexoNumericComparator(); @Override public int compare(event_subscriber o1, event_subscriber o2) { return lexoNumericComparator.compare(o1.getevent_up_name(), o2.getevent_up_name()); } }); getTable().addColumn(eventNameColumn, constants.eventNameEventNotifier()); getTable().addActionButton(new WebAdminButtonDefinition<event_subscriber>(constants.manageEventsEventNotifier()) { @Override protected UICommand resolveCommand() { return getDetailModel().getManageEventsCommand(); } }); }
Source:
<code><START> public void beginTrainingWithTrainingLabelsForJPGTrainingSet(HttpClient httpClient, <END> FormRecognizerServiceVersion serviceVersion) { client = getFormTrainingAsyncClient(httpClient, serviceVersion); beginTrainingLabeledRunner((trainingFilesUrl, useTrainingLabels) -> { SyncPoller<OperationResult, CustomFormModel> trainingPoller = client.beginTraining(trainingFilesUrl, useTrainingLabels).getSyncPoller(); trainingPoller.waitForCompletion(); validateCustomModelData(trainingPoller.getFinalResult(), true); }); }</code><technical_language>Is this a rename change</technical_language>

Target:
public void beginTrainingWithTrainingLabelsForJPGTrainingSet(HttpClient httpClient, FormRecognizerServiceVersion serviceVersion) { client = getFormTrainingAsyncClient(httpClient, serviceVersion); beginTrainingLabeledRunner((trainingFilesUrl, useTrainingLabels) -> { SyncPoller<OperationResult, CustomFormModel> trainingPoller = client.beginTraining(trainingFilesUrl, useTrainingLabels, null, durationTestMode).getSyncPoller(); trainingPoller.waitForCompletion(); validateCustomModelData(trainingPoller.getFinalResult(), true); }); }
Source:
<code>private boolean refineCoefficient(int coefIndex, double lowerBound, double upperBound, double[][] xValues, double[] yValues) throws Exception { boolean result = false; if (coefficient[coefIndex] < lowerBound || coefficient[coefIndex] > upperBound) { double value; if (coefficient[coefIndex] < lowerBound) { value = lowerBound; } else { value = upperBound; } int ptr = -1; if (validColumnIndices.size() >= 1 && <START> validColumnIndices.get(validColumnIndices.size() - 1) == coefIndex) { <END> ptr = validColumnIndices.size() - 1; } else if (validColumnIndices.size() >= 2 && validColumnIndices.get(validColumnIndices.size() - 2) == coefIndex) { ptr = validColumnIndices.size() - 2; } if (ptr != -1) { if (LOG.isDebugEnabled()) { LOG.debug("addRecord: coefficient at index " + coefIndex + " is wrong in the regression, setting it to " + value); } validColumnIndices.remove(ptr); fillXMatrix(dataSamples, validColumnIndices, xValues); for (int i = 0; i < memorySamples.size(); ++i) { yValues[i] -= value * dataSamples.get(i)[coefIndex]; } extreme[coefIndex] = value; mlr.newSampleData(yValues, xValues); calculateRegression(coefficient, validColumnIndices, mlr); result = true; } else { if (LOG.isDebugEnabled()) { LOG.debug( "addRecord: coefficient was not in the regression, " + "setting it to the lower bound"); } result = false; } coefficient[coefIndex] = value; } return result; }</code><technical_language>This is pretty hardcore, a loop instead? loose a CPU cycles, more maintainable</technical_language>

Target:
private boolean refineCoefficient(int coefIndex, double lowerBound, double upperBound, double[][] xValues, double[] yValues) throws Exception { boolean result = false; if (coefficient[coefIndex] < lowerBound || coefficient[coefIndex] > upperBound) { double value; if (coefficient[coefIndex] < lowerBound) { value = lowerBound; } else { value = upperBound; } int ptr = -1; for (int i = validColumnIndices.size() - 1; i >= 0; --i) { if (validColumnIndices.get(i) == coefIndex) { ptr = i; break; } } if (ptr != -1) { if (LOG.isDebugEnabled()) { LOG.debug("addRecord: coefficient at index " + coefIndex + " is wrong in the regression, setting it to " + value); } validColumnIndices.remove(ptr); fillXMatrix(dataSamples, validColumnIndices, xValues); for (int i = 0; i < memorySamples.size(); ++i) { yValues[i] -= value * dataSamples.get(i)[coefIndex]; } extreme[coefIndex] = value; mlr.newSampleData(yValues, xValues); calculateRegression(coefficient, validColumnIndices, mlr); result = true; } else { if (LOG.isDebugEnabled()) { LOG.debug( "addRecord: coefficient was not in the regression, " + "setting it to the extreme of the bound"); } result = false; } coefficient[coefIndex] = value; } return result; }
Source:
<code>String getUpdateLocation0(Module module) { ModuleRevision current = module.getCurrentRevision(); Generation generation = (Generation) current.getRevisionInfo(); String updateLocation = generation.getHeaders().get(Constants.BUNDLE_UPDATELOCATION); if (updateLocation == null) { updateLocation = module.getLocation(); } if (updateLocation.startsWith(INITIAL_LOCATION)) { updateLocation = updateLocation.substring(INITIAL_LOCATION.length()); } <START> return updateLocation == null ? module.getLocation() : updateLocation; <END> }</code><technical_language>Due null test above, do need retest for null in return statement</technical_language>

Target:
String getUpdateLocation0(Module module) { ModuleRevision current = module.getCurrentRevision(); Generation generation = (Generation) current.getRevisionInfo(); String updateLocation = generation.getHeaders().get(Constants.BUNDLE_UPDATELOCATION); if (updateLocation == null) { updateLocation = module.getLocation(); } if (updateLocation.startsWith(INITIAL_LOCATION)) { updateLocation = updateLocation.substring(INITIAL_LOCATION.length()); } return updateLocation; }
Source:
<code>public static Container readContainer(final Version version, final InputStream inputStream, final long containerByteOffset) { <START> Container container = readContainerInternal(version.major, inputStream, containerByteOffset); <END> if (container == null) { CramVersionPolicies.eofNotFound(version); return readContainerInternal(version.major, new ByteArrayInputStream(CramIO.ZERO_B_EOF_MARKER), containerByteOffset); } if (container.isEOF()) { log.debug("EOF marker found, file/stream is complete."); } return container; }</code><technical_language>This readContainer method private now</technical_language>

Target:
private static Container readContainer(final Version version, final InputStream inputStream, final long containerByteOffset) { Container container = readContainerInternal(version.major, inputStream, containerByteOffset); if (container == null) { CramVersionPolicies.eofNotFound(version); return readContainerInternal(version.major, new ByteArrayInputStream(CramIO.ZERO_B_EOF_MARKER), containerByteOffset); } if (container.isEOF()) { log.debug("EOF marker found, file/stream is complete."); } return container; }
Source:
<code>static ExecutorService startClientService(SystemMain system, ClientType clientType, AtomicInteger jobCounter) { switch (clientType) { case ALERT: return startAlertClientService(system, jobCounter); case COMMIT_SCHEMA: return startCommitSchemaClientService(system, jobCounter); case COMMIT_ANNOTATIONS: return startCommitAnnotationsClientService(system, jobCounter); <START> case PROCESS_METRICS: <END> return startProcessMetricsClientService(system, jobCounter); default: return startCommitMetricsClientService(system, jobCounter); } }</code><technical_language>Is this name a reflection of client does? I more lines of PROCESS_QUERIES BATCH_QUERY</technical_language>

Target:
static ExecutorService startClientService(SystemMain system, ClientType clientType, AtomicInteger jobCounter) { switch (clientType) { case ALERT: return startAlertClientService(system, jobCounter); case COMMIT_SCHEMA: return startCommitSchemaClientService(system, jobCounter); case COMMIT_ANNOTATIONS: return startCommitAnnotationsClientService(system, jobCounter); case PROCESS_QUERIES: return startProcessMetricsClientService(system, jobCounter); default: return startCommitMetricsClientService(system, jobCounter); } }
Source:
<code>public UfsClient getJournal(URI location) { synchronized (this) { if (mJournalUfsClientMap == null) { mJournalUfsClientMap = new HashMap<>(); } <START> UfsClient ufsClient = mJournalUfsClientMap.get(location.getScheme()); <END> if (ufsClient == null) { addMount(IdUtils.UFS_JOURNAL_MOUNT_ID, new AlluxioURI(location.toString()), UfsJournal.getJournalUfsConf()); try { ufsClient = get(IdUtils.UFS_JOURNAL_MOUNT_ID); mJournalUfsClientMap.put(location.getScheme(), ufsClient); } catch (NotFoundException | UnavailableException e) { throw new RuntimeException("We should never reach here", e); } } return ufsClient; } }</code><technical_language>I confused. multiple journal locations? this similar root ufs client, is 1 of</technical_language>

Target:
public UfsClient getJournal(URI location) { synchronized (this) { if (mJournalUfsClient == null) { addMount(IdUtils.UFS_JOURNAL_MOUNT_ID, new AlluxioURI(location.toString()), UfsJournal.getJournalUfsConf()); try { mJournalUfsClient = get(IdUtils.UFS_JOURNAL_MOUNT_ID); } catch (NotFoundException | UnavailableException e) { throw new RuntimeException("We should never reach here", e); } } return mJournalUfsClient; } }
Source:
<code>public boolean isPending() { <START> return fScenarioInfo.getStatus().equals(ScenarioStatusType.PENDING); <END> }</code><technical_language>enums, == instead</technical_language>

Target:
public boolean isPending() { return fScenarioInfo.getStatus() == ScenarioStatusType.PENDING; }
Source:
<code>public String docs() { <START> return "boolean {locationArray} Returns the name of the Command Block at the given location."; <END> }</code><technical_language>beginning recognize a pattern ;)</technical_language>

Target:
public String docs() { return "string {locationArray} Returns the name of the Command Block at the given location."; }
Source:
<code>public <C extends Context> C unwrap(TypeToken<C> type) { <START> checkArgument(checkNotNull(type, "type").isSupertypeOf(backendType), "backend type: %s not assignable to %s", backendType, type); <END> return (C) backend; }</code><technical_language>[minor] change message type: %s is a supertype of %s... match code? more comprehensible for users, ;-)</technical_language>

Target:
public <C extends Context> C unwrap(TypeToken<C> type) { checkArgument(TypeTokenUtils.isSupertypeOf(checkNotNull(type, "type"), backendType), "%s is not a supertype of backend type %s", type, backendType); return (C) backend; }
Source:
<code>public static void withCustomerGroup(final BlockingSphereClient client, final UnaryOperator<CustomerGroup> consumer) { final String name = randomString(); final String key = randomKey(); final CustomerGroupDraft customerGroupDraft = CustomerGroupDraft.of(name,key); final CustomerGroup customerGroup = client.executeBlocking(CustomerGroupCreateCommand.of(customerGroupDraft)); <START> CustomerGroup updatedCustomerGroup = consumer.apply(customerGroup); <END> final Optional<CustomerGroup> customerGroupOptional = Optional.ofNullable(client.executeBlocking(CustomerGroupByIdGet.of(updatedCustomerGroup.getId()))); customerGroupOptional.ifPresent(group -> client.executeBlocking(CustomerGroupDeleteCommand.of(updatedCustomerGroup))); }</code><technical_language>final modifier is missing</technical_language>

Target:
public static void withCustomerGroup(final BlockingSphereClient client, final UnaryOperator<CustomerGroup> consumer) { final String name = randomString(); final String key = randomKey(); final CustomerGroupDraft customerGroupDraft = CustomerGroupDraft.of(name,key); final CustomerGroup customerGroup = client.executeBlocking(CustomerGroupCreateCommand.of(customerGroupDraft)); final CustomerGroup updatedCustomerGroup = consumer.apply(customerGroup); final Optional<CustomerGroup> customerGroupOptional = Optional.ofNullable(client.executeBlocking(CustomerGroupByIdGet.of(updatedCustomerGroup.getId()))); customerGroupOptional.ifPresent(group -> client.executeBlocking(CustomerGroupDeleteCommand.of(updatedCustomerGroup))); }
Source:
<code>public void testVolatilityModelAdjointRhoP1() { final double eps = 1e-5; final double tol = 1e-6; final SABRFormulaData data = DATA.withRho(1.0); testVolatilityModelAdjoint(F, CALL_ATM, data, eps, tol); double z = 0.975; double strike = strikeForZ(z, F, ALPHA, BETA, NU); double[] adjModel = FUNCTION.getVolatilityModelAdjoint(CALL_ATM.withStrike(strike), F, data); <START> double[] adj = FUNCTION.getVolatilityAdjoint(CALL_ATM.withStrike(strike), F, data); <END> testVolatilityModelAdjoint(F, CALL_ATM.withStrike(strike), data, eps, 5e-2); z = -2.0; strike = strikeForZ(z, F, ALPHA, BETA, NU); testVolatilityModelAdjoint(F, CALL_ATM.withStrike(strike), data, eps, 50 * tol); z = 2.0; strike = strikeForZ(z, F, ALPHA, BETA, NU); testVolatilityModelAdjoint(F, CALL_ATM.withStrike(strike), DATA.withRho(1 - 1e-3), eps, 5e-5); testVolatilityModelAdjoint(F, CALL_ATM.withStrike(strike), data, 1e-6, 1.0); }</code><technical_language>used</technical_language>

Target:
public void testVolatilityModelAdjointRhoP1() { final double eps = 1e-5; final double tol = 1e-6; final SABRFormulaData data = DATA.withRho(1.0); testVolatilityModelAdjoint(F, CALL_ATM, data, eps, tol); double z = 0.975; double strike = strikeForZ(z, F, ALPHA, BETA, NU); testVolatilityModelAdjoint(F, CALL_ATM.withStrike(strike), data, eps, 5e-2); z = -2.0; strike = strikeForZ(z, F, ALPHA, BETA, NU); testVolatilityModelAdjoint(F, CALL_ATM.withStrike(strike), data, eps, 50 * tol); z = 2.0; strike = strikeForZ(z, F, ALPHA, BETA, NU); testVolatilityModelAdjoint(F, CALL_ATM.withStrike(strike), DATA.withRho(1 - 1e-3), eps, 5e-5); testVolatilityModelAdjoint(F, CALL_ATM.withStrike(strike), data, 1e-6, 1.0); }
Source:
<code>public void success(PageCombo pageCombo, Response response) { L.v(response.getUrl()); if (pageCombo.hasError()) { onError(); return; } final Page page = pageCombo.toPage(title); <START> new Handler(Looper.getMainLooper()).post(new Runnable() { <END> @Override public void run() { new SavePageTask(WikipediaApp.getInstance(), page.getTitle(), page).execute(); } }); L.d("Downloaded page " + title.getDisplayText()); onComplete(); }</code><technical_language>This block a odd. thread success is called</technical_language>

Target:
public void success(PageCombo pageCombo, Response response) { L.v(response.getUrl()); if (pageCombo.hasError()) { onError(); return; } final Page page = pageCombo.toPage(title); new SavePageTask(WikipediaApp.getInstance(), page.getTitle(), page).execute(); L.d("Downloaded page " + title.getDisplayText()); onComplete(); }
Source:
<code><START> private static List<String> parseExcludedRules(Configuration configuration) { <END> String excludeProperty = configuration.getProperty(GeneralOption.KTLINT_EXCLUDE); if (excludeProperty == null) { return new ArrayList<>(); } else { return Arrays.asList(excludeProperty.split(",")); } }</code><technical_language>static</technical_language>

Target:
private List<String> parseExcludedRules(Configuration configuration) { String excludeProperty = configuration.getProperty(GeneralOption.KTLINT_EXCLUDE); if (excludeProperty == null) { return new ArrayList<>(); } return Arrays.asList(excludeProperty.split(",")); }
Source:
<code>private TableScanNode tableScan(String id, String... symbols) { List<Symbol> symbolsList = Arrays.stream(symbols).map(Symbol::new).collect(toImmutableList()); ImmutableMap.Builder<Symbol, ColumnHandle> assignments = ImmutableMap.builder(); for (Symbol symbol : symbolsList) { assignments.put(symbol, new TpchColumnHandle("orderkey", BIGINT)); <START> } <END> TpchTableHandle tableHandle = new TpchTableHandle("orders", 1.0); return new TableScanNode( new PlanNodeId(id), new TableHandle( new ConnectorId("tpch"), tableHandle, TpchTransactionHandle.INSTANCE, Optional.of(new TpchTableLayoutHandle(tableHandle, TupleDomain.all()))), symbolsList, assignments.build(), TupleDomain.all(), TupleDomain.all()); }</code><technical_language>signature remain unchanged</technical_language>

Target:
private TableScanNode tableScan(String id, String... symbols) { List<VariableReferenceExpression> variables = Arrays.stream(symbols) .map(symbol -> new VariableReferenceExpression(symbol, BIGINT)) .collect(toImmutableList()); return tableScan(id, variables); }
Source:
<code>public boolean tryLock() { if (!isInUse.compareAndSet(false, true)) { return false; } final String path; try { path = createAndGetLockPath(); } catch (InterruptedException e) { log.log( java.util.logging.Level.WARNING, "InterruptedException while trying to get lock path " + absoluteLockPath, e); return false; } catch (KeeperException e) { log.log( java.util.logging.Level.WARNING, "KeeperException while trying to get lock path " + absoluteLockPath, e); return false; } try { absoluteLockPath = zk.create( <START> path.toString(), <END> null, ZooDefs.Ids.OPEN_ACL_UNSAFE, CreateMode.EPHEMERAL_SEQUENTIAL); log.log( java.util.logging.Level.INFO, "Created lock node with path: " + absoluteLockPath); final long noTimeout = 0; final boolean noWait = false; return attemptToLock(noTimeout, noWait); } catch (InterruptedException e) { log.log( java.util.logging.Level.INFO, "InterruptedException while trying to create lock " + absoluteLockPath, e); return false; } catch (KeeperException e) { log.log( java.util.logging.Level.INFO, "KeeperException while trying to create lock " + absoluteLockPath, e); return false; } }</code><technical_language>path is a String</technical_language>

Target:
public boolean tryLock() { if (!isInUse.compareAndSet(false, true)) { return false; } final String path; try { path = createAndGetLockPath(); } catch (InterruptedException e) { log.log( java.util.logging.Level.WARNING, "InterruptedException while trying to get lock path " + absoluteLockPath, e); return false; } catch (KeeperException e) { log.log( java.util.logging.Level.WARNING, "KeeperException while trying to get lock path " + absoluteLockPath, e); return false; } try { absoluteLockPath = zk.create( path, null, ZooDefs.Ids.OPEN_ACL_UNSAFE, CreateMode.EPHEMERAL_SEQUENTIAL); log.log( java.util.logging.Level.INFO, "Created lock node with path: " + absoluteLockPath); final long noTimeout = 0; final boolean noWait = false; return attemptToLock(noTimeout, noWait); } catch (InterruptedException e) { log.log( java.util.logging.Level.INFO, "InterruptedException while trying to create lock " + absoluteLockPath, e); return false; } catch (KeeperException e) { log.log( java.util.logging.Level.INFO, "KeeperException while trying to create lock " + absoluteLockPath, e); return false; } }
Source:
<code>public void builderWorks() { CriteriaContext context = new CriteriaContext.Builder() .withStudyIdentifier(TestConstants.TEST_STUDY) .withUserId(USER_ID) .withClientInfo(CLIENT_INFO) .withUserDataGroups(TestConstants.USER_DATA_GROUPS) .withUserSubstudyIds(TestConstants.USER_SUBSTUDY_IDS).build(); assertEquals(CLIENT_INFO, context.getClientInfo()); assertEquals(TestConstants.USER_DATA_GROUPS, context.getUserDataGroups()); assertEquals(TestConstants.USER_SUBSTUDY_IDS, context.getUserSubstudyIds()); CriteriaContext copy = new CriteriaContext.Builder().withContext(context).build(); <START> assertEquals(CLIENT_INFO, copy.getClientInfo()); <END> assertEquals(TestConstants.USER_DATA_GROUPS, copy.getUserDataGroups()); }</code><technical_language>this part of test test copy substudies</technical_language>

Target:
public void builderWorks() { CriteriaContext context = new CriteriaContext.Builder() .withStudyIdentifier(TestConstants.TEST_STUDY) .withUserId(USER_ID) .withClientInfo(CLIENT_INFO) .withUserDataGroups(TestConstants.USER_DATA_GROUPS) .withUserSubstudyIds(TestConstants.USER_SUBSTUDY_IDS).build(); assertEquals(CLIENT_INFO, context.getClientInfo()); assertEquals(TestConstants.USER_DATA_GROUPS, context.getUserDataGroups()); assertEquals(TestConstants.USER_SUBSTUDY_IDS, context.getUserSubstudyIds()); CriteriaContext copy = new CriteriaContext.Builder().withContext(context).build(); assertEquals(CLIENT_INFO, copy.getClientInfo()); assertEquals(TestConstants.TEST_STUDY, copy.getStudyIdentifier()); assertEquals(USER_ID, copy.getUserId()); assertEquals(TestConstants.USER_DATA_GROUPS, copy.getUserDataGroups()); assertEquals(TestConstants.USER_SUBSTUDY_IDS, copy.getUserSubstudyIds()); }
Source:
<code>private static Format getFormat(String delimiter) { if (",".equals(delimiter)) { return Format.CSV; <START> } else if (delimiter != null && delimiter.length() > 0) { <END> Format.CustomizeSV.setDelimiter(delimiter, null); return Format.CustomizeSV; } else { return Format.TSV; } }</code><technical_language>single character delimiters supported, suggest adding a check for this DelimitedInputFormat constructor throws exception if a multichar delimiter is provided, behavior mysterious users (specifying a delimiter silently ignored)</technical_language>

Target:
private static Format getFormat(String delimiter) { if (",".equals(delimiter)) { return Format.CSV; } else if ("\t".equals(delimiter)) { return Format.TSV; } else { Format.CustomizeSV.setDelimiter(delimiter, null); return Format.CustomizeSV; } }
Source:
<code>public void updateSystemMemoryUsage(long deltaMemoryInBytes) { if (taskContext == null) { <START> Optional<TaskContext> taskContext = queryContext.getTaskContextByTaskId(taskId); <END> verify(taskContext.isPresent(), "task does not exist"); this.taskContext = taskContext.get(); } if (deltaMemoryInBytes > 0) { taskContext.reserveSystemMemory(deltaMemoryInBytes); } else { taskContext.freeSystemMemory(-deltaMemoryInBytes); } }</code><technical_language>Consider adding queryContext.getRequiredTaskContext(taskId) do this in shot Optional</technical_language>

Target:
public void updateSystemMemoryUsage(long deltaMemoryInBytes) { if (taskContext == null) { taskContext = queryContext.getTaskContextByTaskId(taskId); } if (deltaMemoryInBytes > 0) { taskContext.reserveSystemMemory(deltaMemoryInBytes); } else { taskContext.freeSystemMemory(-deltaMemoryInBytes); } }
Source:
<code>public void testUserManagedDestination() throws Exception { int numberOfPartitions = 2; String destinationTopic = "userManaged_" + UUID.randomUUID().toString(); _datastreamCluster = initializeTestDatastreamServerWithFileConnector(1, BROADCAST_STRATEGY_FACTORY, numberOfPartitions); int totalEvents = 10; _datastreamCluster.startup(); String fileName1 = "/tmp/testFile1_" + UUID.randomUUID().toString(); ZkClient zkClient = new ZkClient(_datastreamCluster.getZkConnection()); ZkConnection zkConnection = new ZkConnection(_datastreamCluster.getZkConnection()); ZkUtils zkUtils = new ZkUtils(zkClient, zkConnection, false); AdminUtils.createTopic(zkUtils, destinationTopic, numberOfPartitions, 1, new Properties()); Datastream fileDatastream1 = createFileDatastream(fileName1, destinationTopic, 2); Assert.assertEquals((int) fileDatastream1.getDestination().getPartitions(), 2); Collection<String> eventsWritten1 = TestUtils.generateStrings(totalEvents); FileUtils.writeLines(new File(fileName1), eventsWritten1); <START> Collection<String> eventsReceived1 = readFileDatastreamEvents(fileDatastream1, 0, 9); <END> Collection<String> eventsReceived2 = readFileDatastreamEvents(fileDatastream1, 1, 1); eventsReceived1.addAll(eventsReceived2); LOG.info("Events Received " + eventsReceived1); LOG.info("Events Written to file " + eventsWritten1); Assert.assertTrue(eventsReceived1.containsAll(eventsWritten1)); }</code><technical_language>curious: do number of events changed in test</technical_language>

Target:
public void testUserManagedDestination() throws Exception { int numberOfPartitions = 2; String destinationTopic = "userManaged_" + UUID.randomUUID().toString(); _datastreamCluster = initializeTestDatastreamServerWithFileConnector(1, BROADCAST_STRATEGY_FACTORY, numberOfPartitions); int totalEvents = 10; _datastreamCluster.startup(); String fileName1 = "/tmp/testFile1_" + UUID.randomUUID().toString(); ZkClient zkClient = new ZkClient(_datastreamCluster.getZkConnection()); ZkConnection zkConnection = new ZkConnection(_datastreamCluster.getZkConnection()); ZkUtils zkUtils = new ZkUtils(zkClient, zkConnection, false); AdminUtils.createTopic(zkUtils, destinationTopic, numberOfPartitions, 1, new Properties()); Datastream fileDatastream1 = createFileDatastream(fileName1, destinationTopic, 2); Assert.assertEquals((int) fileDatastream1.getDestination().getPartitions(), 2); Collection<String> eventsWritten1 = TestUtils.generateStrings(totalEvents); FileUtils.writeLines(new File(fileName1), eventsWritten1); Collection<String> eventsReceived1 = readFileDatastreamEvents(fileDatastream1, 0, 5); Collection<String> eventsReceived2 = readFileDatastreamEvents(fileDatastream1, 1, 5); eventsReceived1.addAll(eventsReceived2); LOG.info("Events Received " + eventsReceived1); LOG.info("Events Written to file " + eventsWritten1); Assert.assertTrue(eventsReceived1.containsAll(eventsWritten1)); }
Source:
<code>public void testPeanutButterDidntTurnIntoWine() { Wine wine = new Wine(); assertNotEquals(wine.getBackendType(), typeToken(PeanutButter.class)); try { wine.unwrap(typeToken(PeanutButter.class)); fail(); } catch (IllegalArgumentException e) { <START> assertEquals(e.getMessage(), "backend type: org.jclouds.internal.BaseViewTest$Water is not a supertype of org.jclouds.internal.BaseViewTest$PeanutButter"); <END> } }</code><technical_language>this message depend Guava version used? In case need change assertion, builds check versions of Guava fail</technical_language>

Target:
public void testPeanutButterDidntTurnIntoWine() { Wine wine = new Wine(); assertNotEquals(wine.getBackendType(), typeToken(PeanutButter.class)); try { wine.unwrap(typeToken(PeanutButter.class)); fail(); } catch (IllegalArgumentException e) { assertEquals(e.getMessage(), "org.jclouds.internal.BaseViewTest$PeanutButter is not a supertype of backend type org.jclouds.internal.BaseViewTest$Water"); } }
Source:
<code>public static void refreshMetadata(String username, String email) { AnalyticsMetadata metadata = new AnalyticsMetadata(); SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(WordPress.getContext()); metadata.setSessionCount(preferences.getInt(AnalyticsTrackerMixpanel.SESSION_COUNT, 0)); metadata.setUserConnected(AccountHelper.isSignedIn()); metadata.setWordPressComUser(AccountHelper.isSignedInWordPressDotCom()); metadata.setJetpackUser(AccountHelper.isJetPackUser()); metadata.setNumBlogs(WordPress.wpDB.getNumBlogs()); <START> metadata.setVersionCode(PackageUtils.getVersionCode(WordPress.getContext())); <END> metadata.setUsername(username); metadata.setEmail(email); AnalyticsTracker.refreshMetadata(metadata); }</code><technical_language>Version code is automatically tracked in Nosara [here](<LINK_1> Mixpanel [here](<LINK_0> - drop metadata. I remember added supported in Mixpanel some point ref <LINK_2></technical_language>

Target:
public static void refreshMetadata(String username, String email) { AnalyticsMetadata metadata = new AnalyticsMetadata(); SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(WordPress.getContext()); metadata.setSessionCount(preferences.getInt(AnalyticsTrackerMixpanel.SESSION_COUNT, 0)); metadata.setUserConnected(AccountHelper.isSignedIn()); metadata.setWordPressComUser(AccountHelper.isSignedInWordPressDotCom()); metadata.setJetpackUser(AccountHelper.isJetPackUser()); metadata.setNumBlogs(WordPress.wpDB.getNumBlogs()); metadata.setUsername(username); metadata.setEmail(email); AnalyticsTracker.refreshMetadata(metadata); }
Source:
<code>private boolean audioDescribedOrFalse(Encoding encoding) { <START> return Optional.fromNullable(encoding.getAudioDescribed()).or(false); <END> }</code><technical_language>return Boolean.TRUE.equals(encoding.getAudioDescribed()); is a tad more obvious in terms of intent - of defaulting false if checked is null</technical_language>

Target:
private boolean audioDescribedOrFalse(Encoding encoding) { return Boolean.TRUE.equals(encoding.getAudioDescribed()); }
Source:
<code>public void downloadURL(URL url, int index) { <START> sleep(500); <END> if (getHost().contains("buttsmithy.com") == true || getHost().contains("www.totempole666.com") == true || getHost().contains("themonsterunderthebed.net") == true) { addURLToDownload(url, pageTitle + "_"); } addURLToDownload(url, getPrefix(index)); }</code><technical_language>Is rate limiting in play a delay useful? (If not, slowing rip for reason.)</technical_language>

Target:
public void downloadURL(URL url, int index) { if (getHost().contains("buttsmithy.com") || getHost().contains("www.totempole666.com") || getHost().contains("themonsterunderthebed.net")) { addURLToDownload(url, pageTitle + "_"); } addURLToDownload(url, getPrefix(index)); }
Source:
<code>public DataWord shiftLeft(DataWord arg) { <START> BigInteger result = value().shiftLeft(arg.value().intValue()); <END> return new DataWord(ByteUtil.copyToArray(result.and(MAX_VALUE))); }</code><technical_language>intValueSafe() avoid undesirable overflow</technical_language>

Target:
public DataWord shiftLeft(DataWord arg) { if (arg.value().compareTo(BigInteger.valueOf(MAX_POW)) >= 0) { return DataWord.ZERO; } BigInteger result = value().shiftLeft(arg.intValueSafe()); return new DataWord(ByteUtil.copyToArray(result.and(MAX_VALUE))); }
Source:
<code>public void stopMonitoring() { super.stopMonitoring(); try { <START> prometheusClient.close(); <END> } catch (IOException e) { } }</code><technical_language>pls check prometheusClient null</technical_language>

Target:
public void stopMonitoring() { super.stopMonitoring(); try { if (prometheusClient != null) { prometheusClient.close(); } } catch (IOException e) { } }
Source:
<code>private UpdaterConfiguration makeFiveConfiguration() { return UpdaterConfiguration.builder() .withSource(FIVE) .withItemEquivalenceUpdater( FIVE_ITEM, MoreSets.add(TARGET_SOURCES, LOVEFILM) ) .withTopLevelContainerEquivalenceUpdater( STANDARD_TOP_LEVEL_CONTAINER, MoreSets.add(TARGET_SOURCES, LOVEFILM) ) .withNonTopLevelContainerEquivalenceUpdater( STANDARD_SERIES, <START> TARGET_SOURCES <END> ) .build(); }</code><technical_language>Is a reason want target sources for non-top-level-containers rest? If consistent</technical_language>

Target:
private UpdaterConfiguration makeFiveConfiguration() { return UpdaterConfiguration.builder() .withSource(FIVE) .withItemEquivalenceUpdater( STRICT_ITEM, TARGET_SOURCES ) .withTopLevelContainerEquivalenceUpdater( STANDARD_TOP_LEVEL_CONTAINER, TARGET_SOURCES ) .withNonTopLevelContainerEquivalenceUpdater( STANDARD_SERIES, TARGET_SOURCES ) .build(); }
Source:
<code>void putUnlessValueExists(long sequenceNumber, T value) { Preconditions.checkState( sequenceNumber > 0, "Only positive sequence numbers are supported, but found %s", sequenceNumber); try { kvs.putUnlessExists(AtlasDbConstants.COORDINATION_TABLE, ImmutableMap.of(getCellForSequence(sequenceNumber), serializeData(value, VALUE_DESCRIPTION))); } catch (KeyAlreadyExistsException e) { <START> throw new IllegalStateException("The coordination store failed a putUnlessExists. This is unexpected" <END> + " as it implies timestamps may have been reused, or a writer to the store behaved badly.", e); } }</code><technical_language>leaking assume sequence numbers timestamps, this is right place do so, severity across</technical_language>

Target:
void putUnlessValueExists(long sequenceNumber, T value) { Preconditions.checkState( sequenceNumber > 0, "Only positive sequence numbers are supported, but found %s", sequenceNumber); try { kvs.putUnlessExists(AtlasDbConstants.COORDINATION_TABLE, ImmutableMap.of(getCellForSequence(sequenceNumber), serializeValue(value))); } catch (KeyAlreadyExistsException e) { throw new SafeIllegalStateException("The coordination store failed a putUnlessExists. This is unexpected" + " as it implies timestamps may have been reused, or a writer to the store behaved badly." + " The offending sequence number was {}. " + " Please contact support - DO NOT ATTEMPT TO FIX THIS YOURSELF.", e, SafeArg.of("sequenceNumber", sequenceNumber)); } }
Source:
<code>public int warmUpConnections(List<DataNodeId> dataNodeIds, int connectionWarmUpPercentagePerDataNode, long timeForWarmUp, List<ResponseInfo> responseInfoList) { NetworkClient client = childNetworkClients.get(ReplicaType.DISK_BACKED); if (client != null) { return client.warmUpConnections(dataNodeIds, connectionWarmUpPercentagePerDataNode, timeForWarmUp, responseInfoList); } else { <START> return 0; <END> } }</code><technical_language>This possible. call method for clients? LocalNetworkClient returns 0 anyway</technical_language>

Target:
public int warmUpConnections(List<DataNodeId> dataNodeIds, int connectionWarmUpPercentagePerDataNode, long timeForWarmUp, List<ResponseInfo> responseInfoList) { return childNetworkClients.values() .stream() .mapToInt(client -> client.warmUpConnections(dataNodeIds, connectionWarmUpPercentagePerDataNode, timeForWarmUp, responseInfoList)) .sum(); }
Source:
<code><START> public LanguageServerWrapper(@Nullable IProject project, @NonNull LanguageServerDefinition serverDefinition, <END> @Nullable IPath initialPath) { this.initialProject = project; this.initialPath = initialPath; this.allWatchedProjects = new HashSet<>(); this.serverDefinition = serverDefinition; this.connectedDocuments = new HashMap<>(); }</code><technical_language>In this case, getrid of project parameter (if project, extra initial path, explicitly initial path</technical_language>

Target:
private LanguageServerWrapper(@Nullable IProject project, @NonNull LanguageServerDefinition serverDefinition, @Nullable IPath initialPath) { this.initialProject = project; this.initialPath = initialPath; this.allWatchedProjects = new HashSet<>(); this.serverDefinition = serverDefinition; this.connectedDocuments = new HashMap<>(); }
Source:
<code>public static ColumnStatisticsObj createMetastoreColumnStatistics(String columnName, HiveType columnType, HiveColumnStatistics statistics, OptionalLong rowCount) { TypeInfo typeInfo = columnType.getTypeInfo(); if (typeInfo.getCategory() == PRIMITIVE) { switch (((PrimitiveTypeInfo) typeInfo).getPrimitiveCategory()) { case BOOLEAN: return createBooleanStatistics(columnName, columnType, statistics); case BYTE: case SHORT: case INT: case LONG: return createLongStatistics(columnName, columnType, statistics); case FLOAT: case DOUBLE: return createDoubleStatistics(columnName, columnType, statistics); case STRING: case VARCHAR: case CHAR: return createStringStatistics(columnName, columnType, statistics, rowCount); case DATE: return createDateStatistics(columnName, columnType, statistics); case TIMESTAMP: return createLongStatistics(columnName, columnType, statistics); case BINARY: return createBinaryStatistics(columnName, columnType, statistics, rowCount); case DECIMAL: return createDecimalStatistics(columnName, columnType, statistics); default: throw new IllegalArgumentException(format("unsupported type: %s", columnType)); } } <START> else if (typeInfo.getCategory() == STRUCT || typeInfo.getCategory() == MAP) { <END> return new ColumnStatisticsObj(); } else { throw new IllegalArgumentException(format("unsupported type: %s", columnType)); } }</code><technical_language>this if, a return in top if condition</technical_language>

Target:
public static ColumnStatisticsObj createMetastoreColumnStatistics(String columnName, HiveType columnType, HiveColumnStatistics statistics, OptionalLong rowCount) { TypeInfo typeInfo = columnType.getTypeInfo(); checkArgument(typeInfo.getCategory() == PRIMITIVE, "unsupported type: %s", columnType); switch (((PrimitiveTypeInfo) typeInfo).getPrimitiveCategory()) { case BOOLEAN: return createBooleanStatistics(columnName, columnType, statistics); case BYTE: case SHORT: case INT: case LONG: return createLongStatistics(columnName, columnType, statistics); case FLOAT: case DOUBLE: return createDoubleStatistics(columnName, columnType, statistics); case STRING: case VARCHAR: case CHAR: return createStringStatistics(columnName, columnType, statistics, rowCount); case DATE: return createDateStatistics(columnName, columnType, statistics); case TIMESTAMP: return createLongStatistics(columnName, columnType, statistics); case BINARY: return createBinaryStatistics(columnName, columnType, statistics, rowCount); case DECIMAL: return createDecimalStatistics(columnName, columnType, statistics); default: throw new IllegalArgumentException(format("unsupported type: %s", columnType)); } }
Source:
<code>public void startTest(Test test) { this.current = test; message("%TESTS ", test); try { <START> BundleContext context = targetBundle.getBundleContext(); <END> MethodHandle mh; try { Method m = test.getClass() .getMethod("setBundleContext", BundleContext.class); m.setAccessible(true); mh = publicLookup().unreflect(m); if (!Modifier.isStatic(m.getModifiers())) { mh = mh.bindTo(test); } } catch (NoSuchMethodException | IllegalAccessException e) { try { Field f = test.getClass() .getField("context"); f.setAccessible(true); mh = publicLookup().unreflectSetter(f); if (!Modifier.isStatic(f.getModifiers())) { mh = mh.bindTo(test); } } catch (NoSuchFieldException | IllegalAccessException e1) { mh = null; } } if (mh != null) { mh.invoke(context); } } catch (Error e) { throw e; } catch (Throwable t) { } }</code><technical_language>I a more robust fix for this I PR. remove this commit this PR</technical_language>

Target:
public void startTest(Test test) { this.current = test; message("%TESTS ", test); BundleContext context = targetBundle.getBundleContext(); try { MethodHandle mh; try { Method m = test.getClass() .getMethod("setBundleContext", BundleContext.class); m.setAccessible(true); mh = publicLookup().unreflect(m); if (!Modifier.isStatic(m.getModifiers())) { mh = mh.bindTo(test); } } catch (NoSuchMethodException | IllegalAccessException e) { try { Field f = test.getClass() .getField("context"); f.setAccessible(true); mh = publicLookup().unreflectSetter(f); if (!Modifier.isStatic(f.getModifiers())) { mh = mh.bindTo(test); } } catch (NoSuchFieldException | IllegalAccessException e1) { mh = null; } } if (mh != null) { mh.invoke(context); } } catch (Error e) { throw e; } catch (Throwable t) { } }
Source:
<code>protected void destroyStore(RecordStore store) { assertRunningOnPartitionThread(); <START> ((DefaultRecordStore) store).clearAndDestroyStorage(false, true); <END> }</code><technical_language>a difference in behaviour. Previously: clearMapStore(); storage.clear(false); mutationObserver.onClear(); storage.destroy(false); mutationObserver.onDestroy(true); Now: storage.clear(false); mutationObserver.onClear(); storage.destroy(false); mutationObserver.onDestroy(true); difference is clear mutation observers. This ok if observer data merged if observer longer a reference data. Previously observer data cleared destroyed. Also, clear map store anymore</technical_language>

Target:
protected void destroyStore(RecordStore store) { assertRunningOnPartitionThread(); ((DefaultRecordStore) store).destroyStorageAfterClear(false, true); }
Source:
<code>protected void drawMarker(IMarkerEvent marker, Rectangle bounds, ITimeDataProvider timeProvider, int nameSpace, GC gc) { Rectangle rect = Utils.clone(bounds); if (marker.getEntry() != null) { int index = fItemData.findItemIndex(marker.getEntry()); if (index == -1) { return; } rect = getStatesRect(bounds, index, nameSpace); if (rect.y < 0 || rect.y > bounds.height) { return; } } int x0 = getXForTime(marker.getTime()); int x1 = getXForTime(marker.getTime() + marker.getDuration()); if (x0 > bounds.width || x1 < nameSpace) { return; } rect.x = Math.max(nameSpace, Math.min(bounds.width, x0)); rect.width = Math.max(1, Math.min(bounds.width, x1) - rect.x); gc.setBackground(marker.getColor()); gc.setAlpha(marker.getColor().getAlpha()); gc.fillRectangle(rect); gc.setAlpha(255); String label = marker.getLabel(); if (label != null && marker.getEntry() != null) { <START> label = label.substring(0, Math.min(256, label.length())); <END> gc.setForeground(marker.getColor()); Utils.drawText(gc, label, rect.x - gc.textExtent(label).x, rect.y, true); } }</code><technical_language>extract constant please</technical_language>

Target:
protected void drawMarker(IMarkerEvent marker, Rectangle bounds, ITimeDataProvider timeProvider, int nameSpace, GC gc) { Rectangle rect = Utils.clone(bounds); if (marker.getEntry() != null) { int index = fItemData.findItemIndex(marker.getEntry()); if (index == -1) { return; } rect = getStatesRect(bounds, index, nameSpace); if (rect.y < 0 || rect.y > bounds.height) { return; } } int x0 = getXForTime(marker.getTime()); int x1 = getXForTime(marker.getTime() + marker.getDuration()); if (x0 > bounds.width || x1 < nameSpace) { return; } rect.x = Math.max(nameSpace, Math.min(bounds.width, x0)); rect.width = Math.max(1, Math.min(bounds.width, x1) - rect.x); gc.setBackground(marker.getColor()); gc.setAlpha(marker.getColor().getAlpha()); gc.fillRectangle(rect); gc.setAlpha(255); String label = marker.getLabel(); if (label != null && marker.getEntry() != null) { label = label.substring(0, Math.min(label.indexOf('\n') != -1 ? label.indexOf('\n') : label.length(), MAX_LABEL_LENGTH)); gc.setForeground(marker.getColor()); Utils.drawText(gc, label, rect.x - gc.textExtent(label).x, rect.y, true); } }
Source:
<code>private void putBlob(MessageInfo messageInfo, ByteBuffer messageBuf, long size) throws CloudStorageException { boolean performUpload = messageInfo.getExpirationTimeInMs() == Utils.Infinite_Time && !messageInfo.isDeleted(); if (performUpload) { BlobId blobId = (BlobId) messageInfo.getStoreKey(); CloudBlobMetadata blobMetadata = new CloudBlobMetadata(blobId, messageInfo.getOperationTimeMs(), messageInfo.getSize()); blobMetadata.setExpirationTime(messageInfo.getExpirationTimeInMs()); <START> cloudDestination.uploadBlob(blobId, size, blobMetadata, new ByteBufferInputStream(messageBuf)); <END> } }</code><technical_language>NIce add metrics log for upload success cases. aware of potential race conditions. deleteBlob updateBlobExpiration</technical_language>

Target:
private void putBlob(MessageInfo messageInfo, ByteBuffer messageBuf, long size) throws CloudStorageException { boolean performUpload = messageInfo.getExpirationTimeInMs() == Utils.Infinite_Time && !messageInfo.isDeleted(); if (performUpload) { BlobId blobId = (BlobId) messageInfo.getStoreKey(); CloudBlobMetadata blobMetadata = new CloudBlobMetadata(blobId, messageInfo.getOperationTimeMs(), messageInfo.getExpirationTimeInMs(), messageInfo.getSize()); cloudDestination.uploadBlob(blobId, size, blobMetadata, new ByteBufferInputStream(messageBuf)); } }
Source:
<code>private Table readTableOrView(String tableName) { <START> final List<Schema> schemas = new ArrayList<>(); <END> if (schemaName != null) { schemas.add(getSchema()); Table table = schemas.get(0).resolveTableOrView(session, tableName); if (table != null) { return table; } } else { schemas.add(database.getSchema(session.getCurrentSchemaName())); Table table = schemas.get(0).resolveTableOrView(session, tableName); if (table != null) { return table; } String[] schemaNames = session.getSchemaSearchPath(); if (schemaNames != null) { for (String name : schemaNames) { Schema s = database.getSchema(name); table = s.resolveTableOrView(session, tableName); if (table != null) { return table; } schemas.add(s); } } } if (isDualTable(tableName)) { return new DualTable(database); } throw getTableOrViewNotFoundDbException(schemas, tableName); }</code><technical_language>Please, remove this list, additional complications for main code path unwanted. build a list if found. Pass name of schema String getTableOrViewNotFoundDbException(), pass null unknown iterate getCurrentSchemaName() getSchemaSearchPath() in method</technical_language>

Target:
private Table readTableOrView(String tableName) { if (schemaName != null) { Table table = getSchema().resolveTableOrView(session, tableName); if (table != null) { return table; } } else { Table table = database.getSchema(session.getCurrentSchemaName()) .resolveTableOrView(session, tableName); if (table != null) { return table; } String[] schemaNames = session.getSchemaSearchPath(); if (schemaNames != null) { for (String name : schemaNames) { Schema s = database.getSchema(name); table = s.resolveTableOrView(session, tableName); if (table != null) { return table; } } } } if (isDualTable(tableName)) { return new DualTable(database); } throw getTableOrViewNotFoundDbException(tableName); }
Source:
<code>void clean(K key) { List<V> data = getDataMap().get(key); if (data != null) { <START> data.clear(); <END> } getDataMap().remove(key); }</code><technical_language>I this is necessary GC remove soon is more reference Object</technical_language>

Target:
void clean(K key) { getDataMap().remove(key); }
Source:
<code>protected boolean synchronizeModel(IDocument doc) { if (doc == null) { return false; } IFeatureModel model = (IFeatureModel) getModel(); String text = doc.get(); try (InputStream stream = new ByteArrayInputStream(text.getBytes(StandardCharsets.UTF_8))) { model.reload(stream, false); } catch (CoreException e) { return false; <START> } catch (IOException e) { <END> } return true; }</code><technical_language>return false in this case multi-catch catch (CoreException | IOException e) { return false; }</technical_language>

Target:
protected boolean synchronizeModel(IDocument doc) { if (doc == null) { return false; } IFeatureModel model = (IFeatureModel) getModel(); String text = doc.get(); try (InputStream stream = new ByteArrayInputStream(text.getBytes(StandardCharsets.UTF_8))) { model.reload(stream, false); } catch (CoreException | IOException e) { return false; } return true; }
Source:
<code><START> public void shouldThrowExceptionForInvalidOrganisation() throws Exception{ <END> expectedEx.expect(IllegalArgumentException.class); expectedEx.expectMessage("Invalid syntax"); validator.validateEntity("ORG-Test"); }</code><technical_language>Do need throws Exception? Is a checked exception</technical_language>

Target:
public void shouldThrowExceptionForInvalidOrganisation() { when(rdapExceptionMapper.badRequest("Invalid syntax.")) .thenReturn(new BadRequestException("Invalid syntax.")); expectedEx.expect(BadRequestException.class); expectedEx.expectMessage("Invalid syntax"); validator.validateEntity("ORG-Test"); }