added one more question

hapi-fhir-storage/src/test/java/ca/uhn/fhir/jpa/interceptor/validation/SearchParameterValidatingInterceptorTest.java

@@ -150,6 +150,7 @@ public class SearchParameterValidatingInterceptorTest {
 		List<ResourcePersistentId> resourcePersistentIds = theSearchParams
 			.stream()
 			.map(SearchParameter::getId)
+			//TODO: JpaPid only takes the Long value as a constructor parameter. Should I add a different constructor or use a different data type?
 			.map(theS -> new ResourcePersistentId(theS))
 			.collect(Collectors.toList());
 		when(myIFhirResourceDao.searchForIds(any(), any())).thenReturn(resourcePersistentIds);