auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public List<Entity> getBatch(Repository<Entity> repository, Iterable<Object> ids) { try { return getEntityCache(repository).getAll(ids) .values() .stream() .filter(Optional::isPresent) .map(Optional::get) .map(e -> entityHydration.hydrate(e, repository.getEntityType())) .collect(Collectors.toList()); } catch (ExecutionException exception) { <START> throw new RuntimeException(exception); <END> } }</code><technical_language>this change</technical_language>

Target:
public List<Entity> getBatch(Repository<Entity> repository, Iterable<Object> ids) { try { return getEntityCache(repository).getAll(ids) .values() .stream() .filter(Optional::isPresent) .map(Optional::get) .map(e -> entityHydration.hydrate(e, repository.getEntityType())) .collect(Collectors.toList()); } catch (ExecutionException exception) { if (exception.getCause() != null && exception.getCause() instanceof RuntimeException) { throw (RuntimeException) exception.getCause(); } throw new RuntimeException(exception); } }
Source:
<code>default int lastIndexOf(T element, int end) { int result = -1, index = 0; <START> for (Stream<T> stream = this; !stream.isEmpty(); stream = stream.tail(), index++) { <END> if (index <= end && Objects.equals(stream.head(), element)) { result = index; } } return result; }</code><technical_language>for condition !stream.isEmpty() extended index <= end && !stream.isEmpty()? ..</technical_language>

Target:
default int lastIndexOf(T element, int end) { int result = -1, index = 0; for (Stream<T> stream = this; index <= end && !stream.isEmpty(); stream = stream.tail(), index++) { if (Objects.equals(stream.head(), element)) { result = index; } } return result; }
Source:
<code>public VmDevice createSysprepPayloadDevice(String sysPrepContent, VM vm) { <START> boolean cdromPayload = vm.getCompatibilityVersion().greaterOrEquals(Version.v4_4); <END> VmPayload vmPayload = new VmPayload(); if (cdromPayload) { vmPayload.setDeviceType(VmDeviceType.CDROM); } else { vmPayload.setDeviceType(VmDeviceType.FLOPPY); } vmPayload.getFiles().put( osRepository.getSysprepFileName(vm.getOs(), vm.getCompatibilityVersion()), new String(BASE_64.encode(sysPrepContent.getBytes()), StandardCharsets.UTF_8)); return new VmDevice(new VmDeviceId(Guid.newGuid(), vm.getId()), VmDeviceGeneralType.DISK, cdromPayload ? VmDeviceType.CDROM.getName() : VmDeviceType.FLOPPY.getName(), "", vmPayload.getSpecParams(), true, true, true, "", null, null, null); }</code><technical_language>talked in earlier PS - osinfo a toggle for floppy support. it</technical_language>

Target:
public VmDevice createSysprepPayloadDevice(String sysPrepContent, VM vm) { boolean cdromPayload = !osRepository.isFloppySupported(vm.getOs(), vm.getCompatibilityVersion()); VmPayload vmPayload = new VmPayload(); if (cdromPayload) { vmPayload.setDeviceType(VmDeviceType.CDROM); } else { vmPayload.setDeviceType(VmDeviceType.FLOPPY); } vmPayload.getFiles().put( osRepository.getSysprepFileName(vm.getOs(), vm.getCompatibilityVersion()), new String(BASE_64.encode(sysPrepContent.getBytes()), StandardCharsets.UTF_8)); return new VmDevice(new VmDeviceId(Guid.newGuid(), vm.getId()), VmDeviceGeneralType.DISK, cdromPayload ? VmDeviceType.CDROM.getName() : VmDeviceType.FLOPPY.getName(), "", vmPayload.getSpecParams(), true, true, true, "", null, null, null); }
Source:
<code>private void validateNetworkExclusiveOnIface(VdsNetworkInterface iface, NetworkType networkType) { String ifaceName = NetworkUtils.stripVlan(iface); List<NetworkType> networksOnIface = ifacesWithExclusiveNetwork.get(ifaceName); if (networksOnIface == null) { networksOnIface = new ArrayList<>(); ifacesWithExclusiveNetwork.put(ifaceName, networksOnIface); } <START> if ((networkType == NetworkType.VLAN && networksOnIface.contains(NetworkType.VM)) <END> || (networkType == NetworkType.VM && !networksOnIface.isEmpty())) { addViolation(EngineMessage.NETWORK_INTERFACES_NOT_EXCLUSIVELY_USED_BY_NETWORK, ifaceName); } networksOnIface.add(networkType); }</code><technical_language>invoke 'networkExclusivenessValidator'</technical_language>

Target:
private void validateNetworkExclusiveOnIface(VdsNetworkInterface iface, NetworkType networkType) { String ifaceName = NetworkUtils.stripVlan(iface); List<NetworkType> networksOnIface = ifacesWithExclusiveNetwork.get(ifaceName); if (networksOnIface == null) { networksOnIface = new ArrayList<>(); ifacesWithExclusiveNetwork.put(ifaceName, networksOnIface); } networksOnIface.add(networkType); if (!networkExclusivenessValidator.isNetworkExclusive(networksOnIface)) { addViolation(networkExclusivenessValidator.getViolationMessage(), ifaceName); } }
Source:
<code>public void assertValueAtIndexNoMatch() { TestObserver<Integer> ts = new TestObserver<Integer>(); Observable.just(1, 2, 3).subscribe(ts); thrown.expect(AssertionError.class); thrown.expectMessage("Expected: 2 (class: Integer), Actual: 3 (class: Integer) (latch = 0, values = 3, errors = 0, completions = 1)"); <START> ts.assertValueAt(2, 2); <END> }</code><technical_language>please compare else integers (String ok)? Silly code change in assert method break logic pass this test index 2 :)</technical_language>

Target:
public void assertValueAtIndexNoMatch() { TestObserver<String> ts = new TestObserver<String>(); Observable.just("a", "b", "c").subscribe(ts); thrown.expect(AssertionError.class); thrown.expectMessage("Expected: b (class: String), Actual: c (class: String) (latch = 0, values = 3, errors = 0, completions = 1)"); ts.assertValueAt(2, "b"); }
Source:
<code>public RegistryAuth authForSwarm() throws DockerException { final AuthorizationData accessToken; try { accessToken = getAccessToken(); } catch (IOException e) { <START> log.warn("unable to get access token for Google Container Registry due to exception, " <END> + "configuration for Swarm will not contain RegistryAuth for GCR", e); return null; } return authForAuthorizationData(accessToken); }</code><technical_language>change error message AWS GCR</technical_language>

Target:
public RegistryAuth authForSwarm() throws DockerException { final AuthorizationData accessToken; try { accessToken = getAccessToken(); } catch (IOException e) { log.warn("unable to get access token for AWS Elastic Container Registry due to exception, " + "configuration for Swarm will not contain RegistryAuth for ECR", e); return null; } return authForAuthorizationData(accessToken); }
Source:
<code>public String updateFilter(Request request, Response response) throws IOException { long id = getIdFromRequest(request.params("id")); NotificationFilter notificationFilter = buildEntityFromRequestBody(request); User user = userService.findUserByName(currentUsernameString()); Optional<NotificationFilter> optionalFilter = user.getNotificationFilters().stream() .filter(filter -> filter.getId() == id) <END> .findFirst(); if (optionalFilter.isEmpty()) { throw new RecordNotFoundException(EntityType.NotificationFilter, id); } <START> try { notificationFilter.setId(id); user.getNotificationFilters().remove(optionalFilter.get()); user.addNotificationFilter(notificationFilter); userService.saveOrUpdate(user); setEtagHeader(response, etagFor(notificationFilter)); return writerForTopLevelObject(request, response, writer -> NotificationFilterRepresenter.toJSON(writer, notificationFilter)); } catch (ValidationException e) { throw new UnprocessableEntityException(e.getMessage()); } }</code><technical_language>move updation logic service create update? easier test</technical_language>

Target:
public String updateFilter(Request request, Response response) throws IOException { long id = parseIdToLong(request.params("id")); NotificationFilter notificationFilter = buildEntityFromRequestBody(request); notificationFilter.setId(id); try { userService.updateNotificationFilter(currentUserId(request), notificationFilter); setEtagHeader(response, etagFor(notificationFilter)); return writerForTopLevelObject(request, response, writer -> NotificationFilterRepresenter.toJSON(writer, notificationFilter)); } catch (UncheckedValidationException e) { throw new UnprocessableEntityException(e.getMessage()); } }
Source:
<code>public void testIndexProperlySetAfterLoad() throws Exception { File directory = folder.newFolder("testIndexAfterLoad"); String holderName = "myHolder"; <START> try (FileOutputStream fos = new FileOutputStream(new File(directory, HOLDER_FILE_NAME))) { <END> try (ObjectOutputStream oos = new ObjectOutputStream(fos)) { oos.writeObject(holderName); oos.writeObject(new FileBasedStateRepository.Tuple(0, new TransientStateHolder<Long, String>())); } } FileBasedStateRepository stateRepository = new FileBasedStateRepository(directory); stateRepository.getPersistentStateHolder("otherHolder", Long.class, Long.class); stateRepository.close(); File[] files = directory.listFiles((dir, name) -> name.contains("otherHolder") && name.contains("-1-")); assertThat(files.length, is(1)); }</code><technical_language>Collapse</technical_language>

Target:
public void testIndexProperlySetAfterLoad() throws Exception { File directory = folder.newFolder("testIndexAfterLoad"); String holderName = "myHolder"; try (FileOutputStream fos = new FileOutputStream(new File(directory, HOLDER_FILE_NAME)); ObjectOutputStream oos = new ObjectOutputStream(fos)) { oos.writeObject(holderName); oos.writeObject(new FileBasedStateRepository.Tuple(0, new TransientStateHolder<Long, String>())); } FileBasedStateRepository stateRepository = new FileBasedStateRepository(directory); stateRepository.getPersistentStateHolder("otherHolder", Long.class, Long.class); stateRepository.close(); File[] files = directory.listFiles((dir, name) -> name.contains("otherHolder") && name.contains("-1-")); assertThat(files.length, is(1)); }
Source:
<code>public void closesDecoratedResponseInputStream() throws IOException { final String xml = Joiner.on(' ').join( "<?xml-stylesheet href='/b.xsl' type='text/xsl'?>", "<subject>World</subject>" ); final String xsl = Joiner.on(' ').join( "<stylesheet xmlns='<LINK_1>' ", " xmlns:x='<LINK_0>' version='2.0' >", "<output method='text'/><template match='/'> ", "Hello, <value-of select='/subject'/>!</template></stylesheet>" ); final StateAwareInputStream inputStream = new StateAwareInputStream(IOUtils.toInputStream(xml)); MatcherAssert.assertThat( new RsPrint( new RsXSLT( new RsText(inputStream), new URIResolver() { @Override public Source resolve(final String href, <START> final String base) { <END> return new StreamSource(new StringReader(xsl)); } } ) ).print(), Matchers.endsWith("Hello, World!") ); MatcherAssert.assertThat( inputStream.isClosed(), Matchers.is(true) ); }</code><technical_language>@Happy-Neko indentation, 4 lines indented line 153 only</technical_language>

Target:
public void closesDecoratedResponseInputStream() throws Exception { final String xml = Joiner.on(' ').join( "<?xml-stylesheet href='/b.xsl' type='text/xsl'?>", "<subject>World</subject>" ); final String xsl = Joiner.on(' ').join( "<stylesheet xmlns='<LINK_1>' ", " xmlns:x='<LINK_0>' version='2.0' >", "<output method='text'/><template match='/'> ", "Hello, <value-of select='/subject'/>!</template></stylesheet>" ); final StateAwareInputStream stream = new StateAwareInputStream(IOUtils.toInputStream(xml)); MatcherAssert.assertThat( new RsPrint( new RsXSLT( new RsText(stream), new URIResolver() { @Override public Source resolve(final String href, final String base) { return new StreamSource(new StringReader(xsl)); } } ) ).print(), Matchers.endsWith("Hello, World!") ); MatcherAssert.assertThat( stream.isClosed(), Matchers.is(true) ); }
Source:
<code>private static String getHostAddress(final int port) throws IOException { <START> String hostAddress = "INVALID"; <END> final Enumeration e = NetworkInterface.getNetworkInterfaces(); while (e.hasMoreElements()) { final NetworkInterface n = (NetworkInterface) e.nextElement(); if (n.isLoopback() || n.isVirtual() || !n.isUp()) { continue; } final Enumeration ee = n.getInetAddresses(); while (ee.hasMoreElements()) { final InetAddress i = (InetAddress) ee.nextElement(); if (i.isLinkLocalAddress()) { continue; } hostAddress = i.getHostAddress(); break; } } try { (new Socket(hostAddress, port)).close(); LOG.log(Level.WARNING, "Port number already in use."); throw new IOException("Port number already in use."); } catch (ConnectException connectException) { LOG.log(Level.INFO, "URL found: {0}:{1}", new Object[]{hostAddress, port}); return hostAddress; } }</code><technical_language>INVALID_HOST_ADDRESS</technical_language>

Target:
private static String getHostAddress(final int port) throws IOException { String hostAddress = INVALID_HOST_ADDRESS; final Enumeration e = NetworkInterface.getNetworkInterfaces(); while (e.hasMoreElements()) { final NetworkInterface n = (NetworkInterface) e.nextElement(); if (n.isLoopback() || n.isVirtual() || !n.isUp()) { continue; } final Enumeration ee = n.getInetAddresses(); while (ee.hasMoreElements()) { final InetAddress i = (InetAddress) ee.nextElement(); if (i.isLinkLocalAddress()) { continue; } hostAddress = i.getHostAddress(); break; } } try { (new Socket(hostAddress, port)).close(); LOG.log(Level.WARNING, "Port number already in use."); throw new IOException("Port number already in use."); } catch (ConnectException connectException) { LOG.log(Level.INFO, "URL found: {0}:{1}", new Object[]{hostAddress, port}); return hostAddress; } }
Source:
<code>public void postRun() throws Exception { ui.header("Database Primary Keys"); if (ui.yesno(true, "Fix primary keys column order")) { <START> ui.message("fixing primary keys..."); <END> fix.run(); ui.message("done"); } }</code><technical_language>column order is correct, Gerrit site created 2.6. perform dry-run mode of PrimaryKeyFix step, detect if site suffers wrong PK column order problem, list tables for PKs fixed run in dry-run mode silently exit otherwise, e. g.: Collection<String> wrongPKColumnOrderTables = fix.dryRun(); if (!wrongPKColumnOrderTables.isEmpty()) { if (ui.yesno(true, "Fix primary keys column order for tables: " + wrongPKColumnOrderTables)) { fix.run(); } } This avoid unnecessary noice</technical_language>

Target:
public void postRun() throws Exception { db = dbFactory.open(); try { conn = ((JdbcSchema) db).getConnection(); Map<String, List<String>> corrections = findPKUpdates(); if (corrections.isEmpty()) { return; } ui.header("Database Primary Keys"); if (ui.yesno(true, "Fix primary keys column order")) { ui.message("fixing primary keys...\n"); JdbcExecutor executor = new JdbcExecutor(conn); for (Map.Entry<String, List<String>> c : corrections.entrySet()) { ui.message(" table: %s ... ", c.getKey()); recreatePK(executor, c.getKey(), c.getValue()); ui.message("done\n"); } ui.message("done\n"); } } finally { db.close(); } }
Source:
<code>protected void generateTableSpecConfig(Map<String, String> tableSpecConfig) { super.generateTableSpecConfig(tableSpecConfig); if (writeBatchSize != null) { <START> addInMemoryConfig(tableSpecConfig, WRITE_BATCH_SIZE, writeBatchSize.toString()); <END> } if (objectCacheSize != null) { addInMemoryConfig(tableSpecConfig, OBJECT_CACHE_SIZE, objectCacheSize.toString()); } }</code><technical_language>properties of CachedStore, in memory store. this in BaseStore Descriptor</technical_language>

Target:
protected void generateTableSpecConfig(Map<String, String> tableSpecConfig) { super.generateTableSpecConfig(tableSpecConfig); }
Source:
<code>public void updateHealthState(final Stack stack) { if (stack == null) { return; } List<Service> services = objectManager.find(Service.class, SERVICE.STACK_ID, stack.getId(), SERVICE.REMOVED, null); List<? extends Host> ActiveHosts = allocatorDao.getActiveHosts(stack.getAccountId()); <START> HashSet<String> activeHosts = new HashSet<String>(); <END> for(Host host: ActiveHosts){ activeHosts.add(host.getId().toString()); } setServiceHealthState(services, activeHosts); setStackHealthState(stack); setEnvironmentHealthState(objectManager.loadResource(Account.class, stack.getAccountId())); }</code><technical_language>reason convert String here? long</technical_language>

Target:
public void updateHealthState(final Stack stack) { if (stack == null) { return; } List<Service> services = objectManager.find(Service.class, SERVICE.STACK_ID, stack.getId(), SERVICE.REMOVED, null); List<? extends Host> ActiveHosts = allocatorDao.getActiveHosts(stack.getAccountId()); HashSet<Long> activeHosts = new HashSet<Long>(); for(Host host: ActiveHosts){ activeHosts.add(host.getId()); } setServiceHealthState(services, activeHosts); setStackHealthState(stack); setEnvironmentHealthState(objectManager.loadResource(Account.class, stack.getAccountId())); }
Source:
<code>public void onError(Task task, Throwable throwable) { boolean alreadyRegisteredError = _errorsReported.getAndSet(true); <START> if (!alreadyRegisteredError) { <END> _analysisListener.errorUnknown(_analysisJob, throwable); } cleanup(); if (_nextTaskListener != null) { _nextTaskListener.onError(task, throwable); } }</code><technical_language>introduce a variable here</technical_language>

Target:
public void onError(Task task, Throwable throwable) { final boolean alreadyRegisteredError = _errorsReported.getAndSet(true); if (!alreadyRegisteredError) { _analysisListener.errorUnknown(_analysisJob, throwable); } cleanup(); if (_nextTaskListener != null) { _nextTaskListener.onError(task, throwable); } }
Source:
<code>public String toString() { return "FilterEntry{" + "eventName='" + eventName + '\'' + ", exclude=" + exclude + ", transportCandidate='" + transportCandidate + '\'' + ", address='" + address + '\'' + '}'; <START> } <END></code><technical_language>please string.format</technical_language>

Target:
public String toString() { return "FilterEntry{" + "eventName='" + eventName + '\'' + ", exclude=" + exclude + ", recipient=" + recipient + '}'; }
Source:
<code><START> public static void filePermissionAndKeyHelper(String filePermission, String filePermissionKey) { <END> if (filePermission != null && filePermissionKey != null) { throw new IllegalArgumentException(FileConstants.MessageConstants.FILE_PERMISSION_FILE_PERMISSION_KEY_INVALID); } if (filePermission != null) { Utility.assertInBounds("filePermission", filePermission.getBytes().length, 0, 8* Constants.KB); } }</code><technical_language>I this removed allow a request service fail there. If service changed allow values a larger file permission size this need updated inappropriately fail requests</technical_language>

Target:
public static void filePermissionAndKeyHelper(String filePermission, String filePermissionKey) { if (filePermission != null && filePermissionKey != null) { throw new IllegalArgumentException(FileConstants.MessageConstants.FILE_PERMISSION_FILE_PERMISSION_KEY_INVALID); } if (filePermission != null) { Utility.assertInBounds("filePermission", filePermission.getBytes(StandardCharsets.UTF_8).length, 0, 8 * Constants.KB); } }
Source:
<code>public Set<PingableLeader> getPotentialLeaders() { if (leaderEligible.get()) { return Sets.union(ImmutableSet.of(localPingable), ImmutableSet.copyOf(otherPingables)); } else { return ImmutableSet.copyOf(otherPingables); } <START> } <END></code><technical_language>I this return original Sets.union(...) - this is health checks in TimeLock in LargeInternalProduct check a quorum of nodes is reachable, other - removing picture _could_ LargeInternalProduct healthchecks report quorum is quorum, do run lock services in numbers</technical_language>

Target:
public Set<PingableLeader> getPotentialLeaders() { return Sets.union(ImmutableSet.of(localPingable), ImmutableSet.copyOf(otherPingables)); }
Source:
<code>public static StringCredentials lookupCredentials(String credentialId, Item item) { <START> List<StringCredentials> credentials = com.cloudbees.plugins.credentials.CredentialsProvider.lookupCredentials(StringCredentials.class, item, ACL.SYSTEM, Collections.emptyList()); <END> CredentialsMatcher matcher = CredentialsMatchers.withId(credentialId); return CredentialsMatchers.firstOrNull(credentials, matcher); }</code><technical_language>Please import, 187 characters hurts readability</technical_language>

Target:
public static StringCredentials lookupCredentials(String credentialId, Item item) { List<StringCredentials> credentials = CredentialsProvider.lookupCredentials(StringCredentials.class, item, ACL.SYSTEM, Collections.emptyList()); return getCredentialWithId(credentialId, credentials); }
Source:
<code>InputStream startUpload() throws IOException { super.startUpload(); dataSize = buffer.size(); ByteArrayInputStream bufferData = new ByteArrayInputStream( buffer.toByteArray()); <START> buffer.reset(); <END> buffer = null; return bufferData; }</code><technical_language>I thinking remove buffer.reset(), next line is dropping reference buffer anyway</technical_language>

Target:
InputStream startUpload() throws IOException { super.startUpload(); dataSize = buffer.size(); ByteArrayInputStream bufferData = new ByteArrayInputStream( buffer.toByteArray()); buffer = null; return bufferData; }
Source:
<code>public List<Object> next() throws IOException, ParseException { List<Object> lineTuple = super.next(); String jsonLine = (String) lineTuple.get(0); <START> if ( jsonLine==null ) <END> return null; if ( jsonLine.trim().isEmpty() ) return next(); try { HashMap<String, Object> jsonMap = new ObjectMapper().readValue(jsonLine, HashMap.class); StreamlineEventImpl slEvent = new StreamlineEventImpl(jsonMap, "HdfsSpout"); ArrayList<Object> tuple = new ArrayList<>(1); tuple.add(slEvent); return tuple; } catch (JsonProcessingException e) { throw new ParseException("Json parsing error at location : " + getFileOffset().toString(), e); } }</code><technical_language>If avoid multiple return points. I suggest replace if (jsonLine != null) { ... }</technical_language>

Target:
public List<Object> next() throws IOException, ParseException { List<Object> lineTuple = super.next(); String jsonLine = (String) lineTuple.get(0); if ( jsonLine==null ) return null; if ( jsonLine.trim().isEmpty() ) return next(); try { HashMap<String, Object> jsonMap = new ObjectMapper().readValue(jsonLine, HashMap.class); StreamlineEventImpl slEvent = new StreamlineEventImpl(jsonMap, "HdfsSpout"); return Collections.singletonList(slEvent); } catch (JsonProcessingException e) { throw new ParseException("Json parsing error at location : " + getFileOffset().toString(), e); } }
Source:
<code>public void testDoesNotAllowCircularDependencies() throws Exception { <START> Bootstrap bootstrap = new Bootstrap((Module) binder -> { <END> binder.bind(InstanceA.class); binder.bind(InstanceB.class); }); try { bootstrap.initialize().getInstance(InstanceA.class); fail("should not allow circular dependencies"); } catch (ProvisionException e) { assertContains(e.getErrorMessages().iterator().next().getMessage(), "circular dependencies are disabled"); } }</code><technical_language>Is (Module) needed</technical_language>

Target:
public void testDoesNotAllowCircularDependencies() throws Exception { Bootstrap bootstrap = new Bootstrap(binder -> { binder.bind(InstanceA.class); binder.bind(InstanceB.class); }); try { bootstrap.initialize().getInstance(InstanceA.class); fail("should not allow circular dependencies"); } catch (ProvisionException e) { assertContains(e.getErrorMessages().iterator().next().getMessage(), "circular dependencies are disabled"); } }
Source:
<code>public void setProperties(EntityDraft draft, Entity entity, String username) { draft.setParentEntity(entity); draft.setParentVersion(entity.getEntityVersion()); draft.setDraftOwnerUsername(username); draft.setLastModificationDate(DateUtil.nowUTC()); <START> draft.setSuperClass(entity.getSuperClass()); <END> draft.setName(entity.getName()); draft.setClassName(entity.getClassName()); draft.setNamespace(entity.getNamespace()); draft.setModule(entity.getModule()); draft.setSecurityMode(entity.getSecurityMode()); draft.setSecurityMembers(new HashSet<>(entity.getSecurityMembers())); draft.setReadOnlySecurityMode(entity.getReadOnlySecurityMode()); draft.setReadOnlySecurityMembers(new HashSet<>(entity.getReadOnlySecurityMembers())); draft.getFields().clear(); for (Field field : entity.getFields()) { draft.addField(field.copy()); } draft.getLookups().clear(); for (Lookup lookup : entity.getLookups()) { Lookup newLookup = lookup.copy(draft.getFields()); draft.addLookup(newLookup); } if (entity.getRestOptions() != null) { RestOptions restOptions = entity.getRestOptions().copy(); restOptions.setEntity(draft); draft.setRestOptions(restOptions); } if (entity.getTracking() != null) { Tracking tracking = entity.getTracking().copy(); tracking.setEntity(draft); draft.setTracking(tracking); } }</code><technical_language>check in git history if this line existed in ? wondering here, worth check git history</technical_language>

Target:
public void setProperties(EntityDraft draft, Entity entity, String username) { draft.setParentEntity(entity); draft.setParentVersion(entity.getEntityVersion()); draft.setDraftOwnerUsername(username); draft.setLastModificationDate(DateUtil.nowUTC()); draft.setName(entity.getName()); draft.setClassName(entity.getClassName()); draft.setNamespace(entity.getNamespace()); draft.setModule(entity.getModule()); draft.setSecurityMode(entity.getSecurityMode()); draft.setSecurityMembers(new HashSet<>(entity.getSecurityMembers())); draft.setReadOnlySecurityMode(entity.getReadOnlySecurityMode()); draft.setReadOnlySecurityMembers(new HashSet<>(entity.getReadOnlySecurityMembers())); draft.getFields().clear(); for (Field field : entity.getFields()) { draft.addField(field.copy()); } draft.getLookups().clear(); for (Lookup lookup : entity.getLookups()) { Lookup newLookup = lookup.copy(draft.getFields()); draft.addLookup(newLookup); } if (entity.getRestOptions() != null) { RestOptions restOptions = entity.getRestOptions().copy(); restOptions.setEntity(draft); draft.setRestOptions(restOptions); } if (entity.getTracking() != null) { Tracking tracking = entity.getTracking().copy(); tracking.setEntity(draft); draft.setTracking(tracking); } }
Source:
<code>public void onMapReady(MapboxMap mapboxMap) { MultipleHeatmapStylingActivity.this.mapboxMap = mapboxMap; CameraPosition cameraPositionForFragmentMap = new CameraPosition.Builder() .target(new LatLng(34.056684, -118.254002)) .zoom(11.047) .build(); mapboxMap.animateCamera( CameraUpdateFactory.newCameraPosition(cameraPositionForFragmentMap), 2600); addHeatmapDataSource(); initHeatmapColors(); initHeatmapRadiusStops(); initHeatmapIntensityStops(); addHeatmapLayer(); findViewById(R.id.switch_heatmap_style_fab).setOnClickListener(new View.OnClickListener() { @Override public void onClick(View view) { if (index == listOfHeatmapColors.length - 1) { index = 0; } <START> if (firstLoad) { <END> index = 1; firstLoad = false; } mapboxMap.getLayer(HEATMAP_LAYER_ID).setProperties( heatmapColor(listOfHeatmapColors[index]), heatmapRadius(listOfHeatmapRadiusStops[index]), heatmapIntensity(listOfHeatmapIntensityStops[index]) ); index++; } }); }</code><technical_language>remove this boolean move index++ top of this method</technical_language>

Target:
public void onMapReady(MapboxMap mapboxMap) { MultipleHeatmapStylingActivity.this.mapboxMap = mapboxMap; CameraPosition cameraPositionForFragmentMap = new CameraPosition.Builder() .target(new LatLng(34.056684, -118.254002)) .zoom(11.047) .build(); mapboxMap.animateCamera( CameraUpdateFactory.newCameraPosition(cameraPositionForFragmentMap), 2600); addHeatmapDataSource(); initHeatmapColors(); initHeatmapRadiusStops(); initHeatmapIntensityStops(); addHeatmapLayer(); findViewById(R.id.switch_heatmap_style_fab).setOnClickListener(new View.OnClickListener() { @Override public void onClick(View view) { index++; if (index == listOfHeatmapColors.length - 1) { index = 0; } if (mapboxMap.getLayer(HEATMAP_LAYER_ID) != null) { mapboxMap.getLayer(HEATMAP_LAYER_ID).setProperties( heatmapColor(listOfHeatmapColors[index]), heatmapRadius(listOfHeatmapRadiusStops[index]), heatmapIntensity(listOfHeatmapIntensityStops[index]) ); } } }); }
Source:
<code><START> public void revsDiff_emptyInput_returnEmpty() throws Exception { <END> Map<String, List<String>> revs = datastore.revsDiff(new ValueListMap<String, String>()); Assert.assertTrue(revs.size() == 0); }</code><technical_language>Ok, true, if want change it</technical_language>

Target:
public void revsDiff_emptyInput_returnEmpty() throws Exception { thrown.expect(IllegalArgumentException.class); thrown.expectMessage("revisions cannot be empty"); datastore.revsDiff(new ValueListMap<String, String>()); }
Source:
<code>protected WrappedByteBuf(ByteBuf buf) { ObjectUtil.checkNotNull(buf, "buf"); <START> this.buf = buf; <END> }</code><technical_language>nit: merge lines checkNotNull return argument</technical_language>

Target:
protected WrappedByteBuf(ByteBuf buf) { this.buf = ObjectUtil.checkNotNull(buf, "buf"); }
Source:
<code>public boolean run(CommandSender sender, Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { if (args.length == 1) { if (!args[0].equals("reload")) { return false; } if (!TFM_AdminList.isSuperAdmin(sender)) { playerMsg(TFM_Command.MSG_NO_PERMS); return true; } TFM_MainConfig.load(); TFM_AdminList.load(); TFM_PermbanList.load(); TFM_PlayerList.load(); TFM_BanManager.load(); TFM_CommandBlocker.load(); final String message = String.format("%s v%s.%s reloaded.", TotalFreedomMod.pluginName, TotalFreedomMod.pluginVersion, TotalFreedomMod.buildNumber); playerMsg(message); TFM_Log.info(message); return true; } playerMsg("TotalFreedomMod for 'Total Freedom', the original all-op server.", ChatColor.GOLD); playerMsg(String.format("Version " + ChatColor.BLUE + "%s.%s" + ChatColor.GOLD + ", built " + ChatColor.BLUE + "%s" + ChatColor.GOLD + " by " + ChatColor.BLUE + "%s" + ChatColor.GOLD + ".", TotalFreedomMod.pluginVersion, TotalFreedomMod.buildNumber, TotalFreedomMod.buildDate, TotalFreedomMod.buildCreator), ChatColor.GOLD); playerMsg("Running on " + TFM_ConfigEntry.SERVER_NAME.getString() + ".", ChatColor.GOLD); playerMsg("Created by Madgeek1450 and Prozza.", ChatColor.GOLD); <START> playerMsg("If you wish to contribute to the development of this plugin, go to " + ChatColor.AQUA + "<LINK_0>", ChatColor.GREEN); <END> playerMsg("Visit " + ChatColor.AQUA + "<LINK_1>" + ChatColor.GREEN + " for more information.", ChatColor.GREEN); return true; }</code><technical_language>about: "This plugin is opensource: <LINK_2>" instead</technical_language>

Target:
public boolean run(CommandSender sender, Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { if (args.length == 1) { if (!args[0].equals("reload")) { return false; } if (!TFM_AdminList.isSuperAdmin(sender)) { playerMsg(TFM_Command.MSG_NO_PERMS); return true; } TFM_MainConfig.load(); TFM_AdminList.load(); TFM_PermbanList.load(); TFM_PlayerList.load(); TFM_BanManager.load(); TFM_CommandBlocker.load(); final String message = String.format("%s v%s.%s reloaded.", TotalFreedomMod.pluginName, TotalFreedomMod.pluginVersion, TotalFreedomMod.buildNumber); playerMsg(message); TFM_Log.info(message); return true; } playerMsg("TotalFreedomMod for 'Total Freedom', the original all-op server.", ChatColor.GOLD); playerMsg(String.format("Version " + ChatColor.BLUE + "%s.%s" + ChatColor.GOLD + ", built " + ChatColor.BLUE + "%s" + ChatColor.GOLD + " by " + ChatColor.BLUE + "%s" + ChatColor.GOLD + ".", TotalFreedomMod.pluginVersion, TotalFreedomMod.buildNumber, TotalFreedomMod.buildDate, TotalFreedomMod.buildCreator), ChatColor.GOLD); playerMsg("Running on " + TFM_ConfigEntry.SERVER_NAME.getString() + ".", ChatColor.GOLD); playerMsg("Created by Madgeek1450 and Prozza.", ChatColor.GOLD); playerMsg("This Plugin is opensource: " + ChatColor.AQUA + "<LINK_0>", ChatColor.GREEN); playerMsg("Visit " + ChatColor.AQUA + "<LINK_1>" + ChatColor.GREEN + " for more information.", ChatColor.GREEN); return true; }
Source:
<code>public ActionResponseDto execute(final ClientConnection conn, final DlmsDevice device, final GetFirmwareVersionRequestDataDto getFirmwareVersionRequestDataDto) { List<FirmwareVersionDto> resultList; try { <START> resultList = this.getFirmwareVersionsCommandExecutor.execute(conn, device, null); <END> } catch (final ProtocolAdapterException e) { LOGGER.error("Error while getting firmware versions from device: " + device.getDeviceIdentification(), e); return new ActionResponseDto(e, "Error while getting firmware versions from device: " + device.getDeviceIdentification()); } return new FirmwareVersionResponseDataDto(resultList); }</code><technical_language>better: return new FirmwareVersionResponseDataDto(this.getFirmwareVersionsCommandExecutor.execute(conn, device, null));</technical_language>

Target:
public ActionResponseDto execute(final ClientConnection conn, final DlmsDevice device, final GetFirmwareVersionRequestDataDto getFirmwareVersionRequestDataDto) { try { return new FirmwareVersionResponseDataDto(this.getFirmwareVersionsCommandExecutor.execute(conn, device, null)); } catch (final ProtocolAdapterException e) { LOGGER.error("Error while getting firmware versions from device: " + device.getDeviceIdentification(), e); return new ActionResponseDto(e, "Error while getting firmware versions from device: " + device.getDeviceIdentification()); } }
Source:
<code>private boolean hasMissingRequiredFieldInGroupType(GroupType requested, GroupType fullSchema) { for (Type field : fullSchema.getFields()) { Type requestedType = null; try { requestedType = requested.getType(field.getName()); <START> } catch (InvalidRecordException e) { <END> if (field.getRepetition() == Type.Repetition.REQUIRED) return true; else continue; } if (!field.isPrimitive()) { if (hasMissingRequiredFieldInGroupType(requestedType.asGroupType(), field.asGroupType())) return true; } } return false; }</code><technical_language>this is try/catch for flow control... kind of hacky generally frowned upon. Is a check if requested schema this field</technical_language>

Target:
private boolean hasMissingRequiredFieldInGroupType(GroupType requested, GroupType fullSchema) { for (Type field : fullSchema.getFields()) { if (requested.containsField(field.getName())) { Type requestedType = requested.getType(field.getName()); if (!field.isPrimitive()) { if (hasMissingRequiredFieldInGroupType(requestedType.asGroupType(), field.asGroupType())) { return true; } else { continue; } } } else { if (field.getRepetition() == Type.Repetition.REQUIRED) { return true; } else { continue; } } } return false; }
Source:
<code>private DataManager getDataManager() { <START> return (DataManager)Services.getService(DataManager.class); <END> }</code><technical_language>cast is necessary, generics for ;-)</technical_language>

Target:
private DataManager getDataManager() { return Services.getService(DataManager.class); }
Source:
<code>public WorkloadStatus getStatus() { int started = samplesCountStartedFromTerminatedThreads; int finished = samplesCountFinishedFromTerminatedThreads; <START> long emptyTrx = emptyTransactionsFromTerminatedThreads; <END> int runningThreads = 0; for (WorkloadService thread : threads) { started += thread.getStartedSamples(); finished += thread.getFinishedSamples(); emptyTrx += thread.getEmptyTransactions(); if (thread.isRunning()) { runningThreads ++; } } return new WorkloadStatus(started, finished, runningThreads, emptyTrx); }</code><technical_language>Pls change name, I catch Trx mean</technical_language>

Target:
public WorkloadStatus getStatus() { int started = samplesCountStartedFromTerminatedThreads; int finished = samplesCountFinishedFromTerminatedThreads; long emptyTrn = emptyTransactionsFromTerminatedThreads; int runningThreads = 0; for (WorkloadService thread : threads) { started += thread.getStartedSamples(); finished += thread.getFinishedSamples(); emptyTrn += thread.getEmptyTransactions(); if (thread.isRunning()) { runningThreads ++; } } return new WorkloadStatus(started, finished, runningThreads, emptyTrn); }
Source:
<code>public WCheckBoxFieldUI() { panel.add(messages); WFieldLayout fieldLayout = new WFieldLayout(LAYOUT_FLAT); fieldLayout.setTitle("this is a bunch of fields"); WCheckBox wCheckBox = new WCheckBox(); fieldLayout.addField("this is a checkbox", wCheckBox); panel.add(fieldLayout); add(panel); <START> } <END></code><technical_language>Is this sufficient test WCheckBoxWebElement? check box is sufficient if is exposed a getter tests set properties it</technical_language>

Target:
public WCheckBoxFieldUI() { panel.add(messages); WFieldLayout fieldLayout = new WFieldLayout(LAYOUT_FLAT); fieldLayout.setTitle("this is a bunch of fields"); fieldLayout.addField("this is a checkbox", wCheckBox); panel.add(fieldLayout); add(panel); }
Source:
<code>public void getObject() throws Exception { final String bucketName = "test_get_object"; try { final String expected = "SUCCESS: Finished downloading object. The object was written to: " <START> + SterilizeString.pathToWindows(Util.DOWNLOAD_BASE_NAME) + "beowulf.txt"; <END> Util.createBucket(client, bucketName); Util.loadBookTestData(client, bucketName); final Arguments args = new Arguments(new String[]{"--http", "-c", "get_object", "-b", bucketName, "-o", "beowulf.txt", "-d", Util.DOWNLOAD_BASE_NAME}); final CommandResponse response = Util.command(client, args); assertThat(response.getMessage(), is(expected)); } finally { Util.deleteBucket(client, bucketName); Util.deleteLoadedFile("beowulf.txt"); } }</code><technical_language>this work non-windows systems</technical_language>

Target:
public void getObject() throws Exception { final String bucketName = "test_get_object"; try { final String expected = "SUCCESS: Finished downloading object. The object was written to: " + SterilizeString.osSpecificPath(Util.DOWNLOAD_BASE_NAME) + "beowulf.txt"; Util.createBucket(client, bucketName); Util.loadBookTestData(client, bucketName); final Arguments args = new Arguments(new String[]{"--http", "-c", "get_object", "-b", bucketName, "-o", "beowulf.txt", "-d", Util.DOWNLOAD_BASE_NAME}); final CommandResponse response = Util.command(client, args); assertThat(response.getMessage(), is(expected)); } finally { Util.deleteBucket(client, bucketName); Util.deleteLoadedFile("beowulf.txt"); } }
Source:
<code>public List<ResolvedReferenceType> getAncestors(boolean acceptIncompleteList) { List<ResolvedReferenceType> ancestors = new ArrayList<>(); ResolvedReferenceType enumClass = ReflectionFactory.typeUsageFor(Enum.class, typeSolver).asReferenceType(); ResolvedTypeParameterDeclaration eTypeParameter = enumClass.getTypeDeclaration().getTypeParameters().get(0); enumClass = enumClass.deriveTypeParameters(new ResolvedTypeParametersMap.Builder().setValue(eTypeParameter, new ReferenceTypeImpl(this, typeSolver)).build()); ancestors.add(enumClass); if (wrappedNode.getImplementedTypes() != null) { for (ClassOrInterfaceType implementedType : wrappedNode.getImplementedTypes()) { ResolvedReferenceType ancestor; try { <START> ancestor = toReferenceType(implementedType); <END> } catch (UnsolvedSymbolException e) { if (acceptIncompleteList) { ancestor = null; } else { throw e; } } if (ancestor != null) { ancestors.add(ancestor); } } } return ancestors; }</code><technical_language>do add ancestors here, avoiding temp var null assignment</technical_language>

Target:
public List<ResolvedReferenceType> getAncestors(boolean acceptIncompleteList) { List<ResolvedReferenceType> ancestors = new ArrayList<>(); ResolvedReferenceType enumClass = ReflectionFactory.typeUsageFor(Enum.class, typeSolver).asReferenceType(); ResolvedTypeParameterDeclaration eTypeParameter = enumClass.getTypeDeclaration().getTypeParameters().get(0); enumClass = enumClass.deriveTypeParameters(new ResolvedTypeParametersMap.Builder().setValue(eTypeParameter, new ReferenceTypeImpl(this, typeSolver)).build()); ancestors.add(enumClass); if (wrappedNode.getImplementedTypes() != null) { for (ClassOrInterfaceType implementedType : wrappedNode.getImplementedTypes()) { try { ancestors.add(toReferenceType(implementedType)); } catch (UnsolvedSymbolException e) { if (!acceptIncompleteList) { throw e; } } } } return ancestors; }
Source:
<code>public ActivityCalendar(final String calendarName, final Date activatePassiveCalendarTime, final List<SeasonProfile> seasonProfileList) { <START> super(); <END> this.calendarName = calendarName; this.activatePassiveCalendarTime = new Date(activatePassiveCalendarTime.getTime()); this.seasonProfileList = new ArrayList<SeasonProfile>(seasonProfileList); }</code><technical_language>include unnecessary calls</technical_language>

Target:
public ActivityCalendar(final String calendarName, final Date activatePassiveCalendarTime, final List<SeasonProfile> seasonProfileList) { this.calendarName = calendarName; this.activatePassiveCalendarTime = new Date(activatePassiveCalendarTime.getTime()); this.seasonProfileList = new ArrayList<SeasonProfile>(seasonProfileList); }
Source:
<code>public List<MatchmakerQueueMapPool> getMatchmakerPools(int matchmakerQueueId) { <START> return getAll("/data/matchmakerQueueMapPool", ImmutableMap.of( <END> "include", "matchmakerQueue,mapPool,mapPool.mapVersion," + "mapPool.mapVersion.map,mapPool.mapVersion.map.latestVersion," + "mapPool.mapVersion.map.author,mapPool.mapVersion.map.statistics," + "mapPool.mapVersion.map.reviewsSummary,mapPool.mapVersion.map.versions.reviews," + "mapPool.mapVersion.map.versions.reviews.player", "filter", rsql(qBuilder().string("matchmakerQueue.id").eq(String.valueOf(matchmakerQueueId))))); }</code><technical_language>Map.of()</technical_language>

Target:
public List<MatchmakerQueueMapPool> getMatchmakerPools(int matchmakerQueueId) { return getAll("/data/matchmakerQueueMapPool", java.util.Map.of( INCLUDE, "matchmakerQueue,mapPool,mapPool.mapVersions," + "mapPool.mapVersions.map,mapPool.mapVersions.map.latestVersion," + "mapPool.mapVersions.map.author,mapPool.mapVersions.map.statistics," + "mapPool.mapVersions.map.reviewsSummary,mapPool.mapVersions.map.versions.reviews," + "mapPool.mapVersions.map.versions.reviews.player", FILTER, rsql(qBuilder().string("matchmakerQueue.id").eq(String.valueOf(matchmakerQueueId))))); }
Source:
<code>public static String docs() { <START> return "Allow to manipulate the Bukkit metadata."; <END> }</code><technical_language>Change this to: "This class allows manipulation of entity metadata."</technical_language>

Target:
public static String docs() { return "This class allows manipulation of entity metadata."; }
Source:
<code>private IQueryResultProvider getResultProviderInternal(IQuerySpecification<?> query, QueryEvaluationHint hint) throws QueryProcessingException, ViatraQueryException { <START> IQueryResultProvider resultProvider = resultProviders.get(query); <END> if (resultProvider == null){ resultProvider = createResultProvider(query, hint); resultProviders.put(query, resultProvider); } return resultProvider; }</code><technical_language>is result provider query, result provider query hint? If this is true, getResultProvider() calls restructured, is field 'hints' removes, is point</technical_language>

Target:
private IQueryResultProvider getResultProviderInternal(IQuerySpecification<?> query, QueryEvaluationHint hint) throws ViatraQueryException, QueryProcessingException{ final IQueryBackend backend = getQueryBackend(query.getInternalQueryRepresentation()); return backend.getResultProvider(query.getInternalQueryRepresentation(), hint); }
Source:
<code>public void testTranslationWithRawKeyVersion2() { DocumentTranslator translator = new DocumentTranslator(TestUtils.createHBaseConfigWithRawKeyV2()); Table table = new Table(); table.setName(UUID.randomUUID().toString()); Document document = new Document(); document.setId(UUID.randomUUID().toString()); document.setTimestamp(System.currentTimeMillis()); document.setData(mapper.createObjectNode().put("name", "rishabh")); Document translatedDocument = translator.translate(table, document); <START> assert (translatedDocument.getMetadata() != null); <END> assert (translatedDocument.getId().equals(translatedDocument.getMetadata().getRawStorageId())); assert (translatedDocument.getMetadata().getId().equals(document.getId())); assert (translatedDocument.getMetadata().getRawStorageId().endsWith(Constants.rawKeyVersionToSuffixMap.get("2.0"))); }</code><technical_language>junit assert</technical_language>

Target:
public void testTranslationWithRawKeyVersion2() { DocumentTranslator translator = new DocumentTranslator(TestUtils.createHBaseConfigWithRawKeyV2()); Table table = new Table(); table.setName(UUID.randomUUID().toString()); Document document = new Document(); document.setId(UUID.randomUUID().toString()); document.setTimestamp(System.currentTimeMillis()); document.setData(mapper.createObjectNode().put("name", "rishabh")); Document translatedDocument = translator.translate(table, document); assertNotNull(translatedDocument.getMetadata()); assertEquals(translatedDocument.getId(), translatedDocument.getMetadata().getRawStorageId()); assertEquals(translatedDocument.getMetadata().getId(), document.getId()); assertTrue(translatedDocument.getMetadata().getRawStorageId().endsWith(Constants.rawKeyVersionToSuffixMap.get("2.0"))); }
Source:
<code>public MockUserState(String inPersistKey, boolean load) { super(inPersistKey, load); persistKey = inPersistKey; <START> if (load) <END> loadState(); else { dependValues = new JSONObject(); syncValues = new JSONObject(); } }</code><technical_language>code style, consistent, if else block lets put if block if is a single line</technical_language>

Target:
public MockUserState(String inPersistKey, boolean load) { super(inPersistKey, load); persistKey = inPersistKey; if (load) { loadState(); } else { dependValues = new JSONObject(); syncValues = new JSONObject(); } }
Source:
<code>@Override public boolean lock(Session ses, boolean exclusive, boolean force) { Long res = sessions.get(ses); if (res != null) { if (EXCLUSIVE_LOCK == res) return true; if(ver.longValue() != res) throw new QueryRetryException(getName()); return false; } lock(exclusive); if (destroyed) { unlock(exclusive); throw new IllegalStateException("Table " + identifierString() + " already destroyed."); } <START> sessions.put(ses, exclusive ? EXCLUSIVE_LOCK : ver.longValue()); <END> ses.addLock(this); return false; }</code><technical_language>ver.longValue() -> ver.get() (it more usual)</technical_language>

Target:
@Override public boolean lock(Session ses, boolean exclusive, boolean force) { SessionLock sesLock = sessions.get(ses); if (sesLock != null) { if (sesLock.isExclusive()) return true; if (ver.get() != sesLock.version()) throw new QueryRetryException(getName()); return false; } lock(exclusive); if (destroyed) { unlock(exclusive); throw new IllegalStateException("Table " + identifierString() + " already destroyed."); } sessions.put(ses, exclusive ? SessionLock.exclusiveLock() : SessionLock.sharedLock(ver.longValue())); ses.addLock(this); return false; }
Source:
<code>public ThriftSessionProperties(SessionPropertyProvider sessionPropertyProvider) { requireNonNull(sessionPropertyProvider, "sessionPropertyProvider is null"); <START> headerProperties = requireNonNull(sessionPropertyProvider.getHeaderProperties(), "sessionPropertyProvider returns null header properties"); <END> headerProperties.values().stream().forEach(sessionProperty -> checkIfTypeSupported(sessionProperty.getJavaType())); sessionProperties = ImmutableList.copyOf(headerProperties.values()); }</code><technical_language>"returns" -> "returned"</technical_language>

Target:
public ThriftSessionProperties(SessionPropertyProvider sessionPropertyProvider) { requireNonNull(sessionPropertyProvider, "sessionPropertyProvider is null"); headerProperties = requireNonNull(sessionPropertyProvider.getHeaderProperties(), "sessionPropertyProvider returned null header properties"); headerProperties.values().stream().forEach(property -> checkIfTypeSupported(property.getJavaType())); sessionProperties = ImmutableList.copyOf(headerProperties.values()); }