From a3241bec1cf1ae5fd514923ba5f923abc5427762 Mon Sep 17 00:00:00 2001 From: ktimoney Date: Tue, 13 Apr 2021 14:40:06 +0100 Subject: Add a new key class which uses @GeneratedValue to base classes This will provide an alternative unique key. It will be used in place of PfTimestampKey in the PdpStatistics table. We make use of the @GeneratedValue annotation to provide this feature. Added DummyGeneratedIdEntity and unit tests Added PfFilterFactory class Added new filter classes Issue-ID: POLICY-3189 Change-Id: I097eab318d2cb958e1157e35825fd6d35d429a25 Signed-off-by: ktimoney --- .../java/org/onap/policy/models/dao/PfDao.java | 32 +++++ .../java/org/onap/policy/models/dao/PfFilter.java | 91 ++++++++++++ .../onap/policy/models/dao/PfFilterFactory.java | 77 ++++++++++ .../onap/policy/models/dao/impl/DefaultPfDao.java | 147 +++++++++---------- .../models/dao/impl/PfNonTimestampKeyFilter.java | 46 ++++++ .../dao/impl/PfReferenceTimestampKeyFilter.java | 46 ++++++ .../models/dao/impl/PfTimestampKeyFilter.java | 46 ++++++ .../policy/models/dao/DummyGeneratedIdEntity.java | 132 ++++++++++++++++++ .../org/onap/policy/models/dao/EntityTest.java | 155 +++++++++++++++++++++ .../src/test/resources/META-INF/persistence.xml | 1 + 10 files changed, 692 insertions(+), 81 deletions(-) create mode 100644 models-dao/src/main/java/org/onap/policy/models/dao/PfFilter.java create mode 100644 models-dao/src/main/java/org/onap/policy/models/dao/PfFilterFactory.java create mode 100644 models-dao/src/main/java/org/onap/policy/models/dao/impl/PfNonTimestampKeyFilter.java create mode 100644 models-dao/src/main/java/org/onap/policy/models/dao/impl/PfReferenceTimestampKeyFilter.java create mode 100644 models-dao/src/main/java/org/onap/policy/models/dao/impl/PfTimestampKeyFilter.java create mode 100644 models-dao/src/test/java/org/onap/policy/models/dao/DummyGeneratedIdEntity.java (limited to 'models-dao') diff --git a/models-dao/src/main/java/org/onap/policy/models/dao/PfDao.java b/models-dao/src/main/java/org/onap/policy/models/dao/PfDao.java index eb77be9c0..844759367 100644 --- a/models-dao/src/main/java/org/onap/policy/models/dao/PfDao.java +++ b/models-dao/src/main/java/org/onap/policy/models/dao/PfDao.java @@ -27,6 +27,7 @@ import java.util.List; import java.util.Map; import org.onap.policy.models.base.PfConcept; import org.onap.policy.models.base.PfConceptKey; +import org.onap.policy.models.base.PfGeneratedIdKey; import org.onap.policy.models.base.PfModelException; import org.onap.policy.models.base.PfReferenceKey; import org.onap.policy.models.base.PfReferenceTimestampKey; @@ -94,6 +95,15 @@ public interface PfDao { */ void delete(Class someClass, PfTimestampKey timeStampKey); + /** + * Delete an Policy Framework concept on the database. + * + * @param the type of the object to delete, a subclass of {@link PfConcept} + * @param someClass the class of the object to delete, a subclass of {@link PfConcept} + * @param idKey the PfConceptIdKey of the object to delete + */ + void delete(Class someClass, PfGeneratedIdKey idKey); + /** * Create a collection of objects in the database. * @@ -202,6 +212,16 @@ public interface PfDao { */ T get(Class someClass, PfTimestampKey timestampKey); + /** + * Get an object from the database, referred to by reference key. + * + * @param the type of the object to get, a subclass of {@link PfConcept} + * @param someClass the class of the object to get, a subclass of {@link PfConcept} + * @param idKey the PfConceptIdKey of the object to get + * @return the object that was retrieved from the database or null if the object was not retrieved + */ + T get(Class someClass, PfGeneratedIdKey idKey); + /** * Get an object from the database, referred to by reference timestamp key. * @@ -251,6 +271,18 @@ public interface PfDao { */ List getAllVersionsByParent(Class someClass, final String parentKeyName); + /** + * Get all the objects in the database of a given type. + * + * @param the type of the objects to get, a subclass of {@link PfConcept} + * @param someClass the class of the objects to get, a subclass of {@link PfConcept} + * @param key the key of the PfGeneratedIdKey to get + * @param timeStamp the timeStamp of the concepts to get + * @return the objects or null if no objects were retrieved + */ + List getByTimestamp(final Class someClass, + final PfGeneratedIdKey key, final Instant timeStamp); + /** * Get a concept from the database with the given concept key. * diff --git a/models-dao/src/main/java/org/onap/policy/models/dao/PfFilter.java b/models-dao/src/main/java/org/onap/policy/models/dao/PfFilter.java new file mode 100644 index 000000000..ef9d4f939 --- /dev/null +++ b/models-dao/src/main/java/org/onap/policy/models/dao/PfFilter.java @@ -0,0 +1,91 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2021 Nordix Foundation. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * SPDX-License-Identifier: Apache-2.0 + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.dao; + +import java.time.Instant; +import java.util.Map; +import lombok.Data; +/** + * This abstract class is used as a base for the filter implementations. + * + */ + +@Data +public abstract class PfFilter { + private static final String AND = " AND "; + private static final String ORDER = " ORDER BY "; + + private String nameFilter; + private String timeStampStartFilter; + private String timeStampEndFilter; + private String timeStampFilter; + private String nameParameter; + + /** + * Generates filter string. + * + * @param inputFilterString current filterString generated from FilterMap + * @param name the pdpInstance name for the PDP statistics to get + * @param startTime the start timeStamp to filter from database, filter rule: + * startTime <= filteredRecord timeStamp <= endTime. null for ignore end time + * @param endTime the end timeStamp to filter from database, filter rule: + * startTime <= filteredRecord timeStamp <= endTime. null for ignore end time + * @param filterMap Map store extra key/value used to filter from database, can be null * + * @param sortOrder sortOrder to query database + * @param getRecordNum Total query count from database + + */ + public String addFilter(final String inputFilterString, + final String name, final Instant startTime, final Instant endTime, + final Map filterMap, final String sortOrder, final int getRecordNum) { + StringBuilder filterQueryString = new StringBuilder(inputFilterString); + if (filterMap != null) { + for (String key : filterMap.keySet()) { + filterQueryString.append("c." + key + "= :" + key + AND); + } + } + + if (name != null) { + filterQueryString.append(getNameFilter() + AND); + } + + if (startTime != null) { + if (endTime != null) { + filterQueryString.append(getTimeStampStartFilter()); + filterQueryString.append(AND); + filterQueryString.append(getTimeStampEndFilter()); + } else { + filterQueryString.append(getTimeStampStartFilter()); + } + } else { + if (endTime != null) { + filterQueryString.append(getTimeStampEndFilter()); + } else { + filterQueryString.delete(filterQueryString.length() - AND.length(), filterQueryString.length()); + } + } + + if (getRecordNum > 0) { + filterQueryString.append(ORDER + getTimeStampFilter() + sortOrder); + } + return filterQueryString.toString(); + } +} diff --git a/models-dao/src/main/java/org/onap/policy/models/dao/PfFilterFactory.java b/models-dao/src/main/java/org/onap/policy/models/dao/PfFilterFactory.java new file mode 100644 index 000000000..643481f39 --- /dev/null +++ b/models-dao/src/main/java/org/onap/policy/models/dao/PfFilterFactory.java @@ -0,0 +1,77 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2021 Nordix Foundation. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * SPDX-License-Identifier: Apache-2.0 + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.dao; + +import java.time.Instant; +import java.util.Map; +import org.onap.policy.models.base.PfConcept; +import org.onap.policy.models.dao.impl.PfNonTimestampKeyFilter; +import org.onap.policy.models.dao.impl.PfReferenceTimestampKeyFilter; +import org.onap.policy.models.dao.impl.PfTimestampKeyFilter; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; + +/** + * This factory class creates a filter class for a key type. + * The filter returned depends on the type of class being used. + * + */ +public class PfFilterFactory { + // Get a reference to the logger + private static final Logger LOGGER = LoggerFactory.getLogger(PfFilterFactory.class); + + /** + * Return a filter class for the input class. + * + * @param the type of the object to get, a subclass of {@link PfConcept}* + * @return the filter type for the input class * + */ + public PfFilter createFilter(final Class someClass) { + PfFilter filter = null; + + switch (getKeyName(someClass)) { + case "PfTimestampKey": + filter = new PfTimestampKeyFilter(); + break; + case "PfReferenceTimestampKey": + filter = new PfReferenceTimestampKeyFilter(); + break; + default: + filter = new PfNonTimestampKeyFilter(); + } + return filter; + } + + /** + * Gets the name of the key class of the class invoking the DAO. + * @param someClass class that invoked Dao + * @return the name of the key class + */ + private String getKeyName(final Class someClass) { + try { + String fullClassName = someClass.getDeclaredField("key").getType().toString(); + return fullClassName.substring(fullClassName.lastIndexOf('.') + 1); + } catch (NoSuchFieldException e) { + LOGGER.error("Error getting the key:", e); + return "NON_TIMESTAMP_KEY"; + } + } +} diff --git a/models-dao/src/main/java/org/onap/policy/models/dao/impl/DefaultPfDao.java b/models-dao/src/main/java/org/onap/policy/models/dao/impl/DefaultPfDao.java index bdbc04e22..1bd2e09da 100644 --- a/models-dao/src/main/java/org/onap/policy/models/dao/impl/DefaultPfDao.java +++ b/models-dao/src/main/java/org/onap/policy/models/dao/impl/DefaultPfDao.java @@ -34,6 +34,7 @@ import javax.persistence.TypedQuery; import javax.ws.rs.core.Response; import org.onap.policy.models.base.PfConcept; import org.onap.policy.models.base.PfConceptKey; +import org.onap.policy.models.base.PfGeneratedIdKey; import org.onap.policy.models.base.PfModelException; import org.onap.policy.models.base.PfModelRuntimeException; import org.onap.policy.models.base.PfReferenceKey; @@ -42,6 +43,8 @@ import org.onap.policy.models.base.PfTimestampKey; import org.onap.policy.models.base.PfUtils; import org.onap.policy.models.dao.DaoParameters; import org.onap.policy.models.dao.PfDao; +import org.onap.policy.models.dao.PfFilter; +import org.onap.policy.models.dao.PfFilterFactory; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -56,6 +59,7 @@ public class DefaultPfDao implements PfDao { private static final String NAME = "name"; private static final String VERSION = "version"; private static final String TIMESTAMP = "timeStamp"; + private static final String GENERATEDID = "Id"; private static final String PARENT_NAME = "parentname"; private static final String PARENT_VERSION = "parentversion"; private static final String LOCAL_NAME = "localname"; @@ -73,8 +77,8 @@ public class DefaultPfDao implements PfDao { private static final String NAME_FILTER = "c.key.name = :name"; private static final String VERSION_FILTER = "c.key.version = :version"; private static final String TIMESTAMP_FILTER = "c.key.timeStamp = :timeStamp"; - private static final String TIMESTAMP_START_FILTER = "c.key.timeStamp >= :startTime"; - private static final String TIMESTAMP_END_FILTER = "c.key.timeStamp <= :endTime"; + private static final String TIMESTAMP_FILTER_NOKEY = "c.timeStamp = :timeStamp"; + private static final String GENERATED_ID_FILTER = "c.key.generatedId = :Id"; private static final String PARENT_NAME_FILTER = "c.key.parentKeyName = :parentname"; private static final String PARENT_VERSION_FILTER = "c.key.parentKeyVersion = :parentversion"; private static final String LOCAL_NAME_FILTER = "c.key.localName = :localname"; @@ -89,6 +93,9 @@ public class DefaultPfDao implements PfDao { private static final String DELETE_BY_TIMESTAMP_KEY = DELETE_FROM_TABLE + WHERE + NAME_FILTER + AND + VERSION_FILTER + AND + TIMESTAMP_FILTER; + private static final String DELETE_BY_GENERATED_ID_KEY = + DELETE_FROM_TABLE + WHERE + NAME_FILTER + AND + VERSION_FILTER + AND + GENERATED_ID_FILTER; + private static final String DELETE_BY_REFERENCE_KEY = DELETE_FROM_TABLE + WHERE + PARENT_NAME_FILTER + AND + PARENT_VERSION_FILTER + AND + LOCAL_NAME_FILTER; @@ -103,6 +110,9 @@ public class DefaultPfDao implements PfDao { private static final String SELECT_BY_CONCEPT_KEY = SELECT_FROM_TABLE + WHERE + NAME_FILTER + AND + VERSION_FILTER; + private static final String SELECT_BY_TIMESTAMP_NOKEY = + SELECT_FROM_TABLE + WHERE + NAME_FILTER + AND + VERSION_FILTER + AND + TIMESTAMP_FILTER_NOKEY; + private static final String SELECT_BY_REFERENCE_KEY = SELECT_FROM_TABLE + WHERE + PARENT_NAME_FILTER + AND + PARENT_VERSION_FILTER + AND + LOCAL_NAME_FILTER; // @formatter:on @@ -245,6 +255,27 @@ public class DefaultPfDao implements PfDao { } } + @Override + public void delete(final Class someClass, final PfGeneratedIdKey key) { + if (key == null) { + return; + } + final EntityManager mg = getEntityManager(); + try { + // @formatter:off + mg.getTransaction().begin(); + mg.createQuery(setQueryTable(DELETE_BY_GENERATED_ID_KEY, someClass), someClass) + .setParameter(NAME, key.getName()) + .setParameter(VERSION, key.getVersion()) + .setParameter(GENERATEDID, key.getGeneratedId()) + .executeUpdate(); + mg.getTransaction().commit(); + // @formatter:on + } finally { + mg.close(); + } + } + @Override public void createCollection(final Collection objs) { if (objs == null || objs.isEmpty()) { @@ -366,14 +397,11 @@ public class DefaultPfDao implements PfDao { String filterQueryString = SELECT_FROM_TABLE + WHERE; try { - if (filterMap != null) { - filterQueryString = buildFilter(filterMap, filterQueryString, isRefTimestampKey(someClass)); - } - filterQueryString = addKeyFilterString(filterQueryString, name, startTime, endTime, - isRefTimestampKey(someClass)); - if (getRecordNum > 0) { - filterQueryString += ORDER + " c.key.timeStamp " + sortOrder; - } + PfFilter timeStampFilter = new PfFilterFactory().createFilter(someClass); + filterQueryString = timeStampFilter.addFilter(filterQueryString, + name, startTime, endTime, filterMap, sortOrder, getRecordNum); + + TypedQuery query = mg.createQuery(setQueryTable(filterQueryString, someClass), someClass); if (filterMap != null) { @@ -382,11 +410,7 @@ public class DefaultPfDao implements PfDao { } } if (name != null) { - if (isRefTimestampKey(someClass)) { - query.setParameter("parentKeyName", name); - } else { - query.setParameter("name", name); - } + query.setParameter(timeStampFilter.getNameParameter(), name); } if (startTime != null) { if (endTime != null) { @@ -411,33 +435,6 @@ public class DefaultPfDao implements PfDao { } } - /** - * This method checks if the class invoking the DAO is using PfReferenceTimestamp Key. - * @param someClass class that invoked Dao - * @return true if the key is PfReferenceTimestampKey. - */ - private boolean isRefTimestampKey(final Class someClass) { - try { - return PfReferenceTimestampKey.class.isAssignableFrom(someClass.getDeclaredField("key").getType()); - } catch (NoSuchFieldException e) { - LOGGER.error("Error verifying the key for reference timestamp:", e); - return false; - } - } - - private String buildFilter(final Map filterMap, String filterQueryString, - boolean isRefTimestampKey) { - StringBuilder bld = new StringBuilder(filterQueryString); - for (String key : filterMap.keySet()) { - if (isRefTimestampKey) { - bld.append("c.key.referenceKey." + key + "= :" + key + AND); - } else { - bld.append("c." + key + "= :" + key + AND); - } - } - return bld.toString(); - } - @Override public T get(final Class someClass, final PfConceptKey key) { return genericGet(someClass, key); @@ -448,6 +445,11 @@ public class DefaultPfDao implements PfDao { return genericGet(someClass, key); } + @Override + public T get(final Class someClass, final PfGeneratedIdKey key) { + return genericGet(someClass, key); + } + @Override public T get(final Class someClass, final PfTimestampKey key) { return genericGet(someClass, key); @@ -539,6 +541,28 @@ public class DefaultPfDao implements PfDao { } } + @Override + public List getByTimestamp(final Class someClass, + final PfGeneratedIdKey key, + final Instant timeStamp) { + if (someClass == null || key == null || timeStamp == null) { + return Collections.emptyList(); + } + + final EntityManager mg = getEntityManager(); + try { + // @formatter:off + return mg.createQuery(setQueryTable(SELECT_BY_TIMESTAMP_NOKEY, someClass), someClass) + .setParameter(NAME, key.getName()) + .setParameter(VERSION, key.getVersion()) + .setParameter(TIMESTAMP, Timestamp.from(timeStamp)) + .getResultList(); + // @formatter:on + } finally { + mg.close(); + } + } + @Override public T getConcept(final Class someClass, final PfConceptKey key) { if (someClass == null || key == null) { @@ -643,45 +667,6 @@ public class DefaultPfDao implements PfDao { return ret.get(0); } - /** - * generate filter string with the filter value in TimestampKey. - * - * @param inputFilterString current filterString generated from FilterMap - * @param name the pdp name the start timeStamp to filter from database, filter rule: startTime <= filteredRecord - * timeStamp <= endTime. null for ignore start time. - * @param endTime the end timeStamp to filter from database, filter rule: startTime <= filteredRecord timeStamp <= - * endTime. null for ignore end time - * @param isRefTimestampKey boolean value, set to true if the query invoked for pfReferenceTimestampKey - * @return the filter string to query database - */ - private String addKeyFilterString(String inputFilterString, final String name, final Instant startTime, - final Instant endTime, final boolean isRefTimestampKey) { - String filterQueryString; - String inputFilter = inputFilterString; - if (name != null) { - if (isRefTimestampKey) { - inputFilter += PARENT_NAME_REF_FILTER + AND; - } else { - inputFilter += NAME_FILTER + AND; - } - } - if (startTime != null) { - if (endTime != null) { - filterQueryString = inputFilter + TIMESTAMP_START_FILTER + AND + TIMESTAMP_END_FILTER; - } else { - filterQueryString = inputFilter + TIMESTAMP_START_FILTER; - } - } else { - if (endTime != null) { - filterQueryString = inputFilter + TIMESTAMP_END_FILTER; - } else { - filterQueryString = inputFilter.substring(0, inputFilter.length() - AND.length()); - } - } - - return filterQueryString; - } - /** * check the result get from database and return the object. * diff --git a/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfNonTimestampKeyFilter.java b/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfNonTimestampKeyFilter.java new file mode 100644 index 000000000..9c9a3e844 --- /dev/null +++ b/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfNonTimestampKeyFilter.java @@ -0,0 +1,46 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2021 Nordix Foundation. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * SPDX-License-Identifier: Apache-2.0 + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.dao.impl; + +import org.onap.policy.models.dao.PfFilter; + +/** + * This class is used to set the values for a non timeStamp key query. + * + */ +public class PfNonTimestampKeyFilter extends PfFilter { + private static final String NAME_FILTER = "c.key.name = :name"; + private static final String TIMESTAMP_START_FILTER = "c.timeStamp >= :startTime"; + private static final String TIMESTAMP_END_FILTER = "c.timeStamp <= :endTime"; + private static final String TIMESTAMP_FILTER = " c.timeStamp "; + private static final String NAME_PARAMETER = "name"; + + /** + * The default constructor injects query strings. + */ + public PfNonTimestampKeyFilter() { + setNameFilter(NAME_FILTER); + setTimeStampStartFilter(TIMESTAMP_START_FILTER); + setTimeStampEndFilter(TIMESTAMP_END_FILTER); + setTimeStampFilter(TIMESTAMP_FILTER); + setNameParameter(NAME_PARAMETER); + } +} diff --git a/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfReferenceTimestampKeyFilter.java b/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfReferenceTimestampKeyFilter.java new file mode 100644 index 000000000..53802f306 --- /dev/null +++ b/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfReferenceTimestampKeyFilter.java @@ -0,0 +1,46 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2021 Nordix Foundation. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * SPDX-License-Identifier: Apache-2.0 + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.dao.impl; + +import org.onap.policy.models.dao.PfFilter; + +/** + * This class is used to set the values for a reference timeStamp key query. + * + */ +public class PfReferenceTimestampKeyFilter extends PfFilter { + private static final String PARENT_NAME_REF_FILTER = "c.key.referenceKey.parentKeyName = :parentKeyName"; + private static final String TIMESTAMP_START_FILTER = "c.key.timeStamp >= :startTime"; + private static final String TIMESTAMP_END_FILTER = "c.key.timeStamp <= :endTime"; + private static final String TIMESTAMP_FILTER = " c.key.timeStamp "; + private static final String NAME_PARAMETER = "parentKeyName"; + + /** + * The default constructor injects query strings. + */ + public PfReferenceTimestampKeyFilter() { + setNameFilter(PARENT_NAME_REF_FILTER); + setTimeStampStartFilter(TIMESTAMP_START_FILTER); + setTimeStampEndFilter(TIMESTAMP_END_FILTER); + setTimeStampFilter(TIMESTAMP_FILTER); + setNameParameter(NAME_PARAMETER); + } +} diff --git a/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfTimestampKeyFilter.java b/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfTimestampKeyFilter.java new file mode 100644 index 000000000..f01f80498 --- /dev/null +++ b/models-dao/src/main/java/org/onap/policy/models/dao/impl/PfTimestampKeyFilter.java @@ -0,0 +1,46 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2021 Nordix Foundation. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * SPDX-License-Identifier: Apache-2.0 + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.dao.impl; + +import org.onap.policy.models.dao.PfFilter; + +/** + * This class is used to set the values for a timeStamp key query. + * + */ +public class PfTimestampKeyFilter extends PfFilter { + private static final String NAME_FILTER = "c.key.name = :name"; + private static final String TIMESTAMP_START_FILTER = "c.key.timeStamp >= :startTime"; + private static final String TIMESTAMP_END_FILTER = "c.key.timeStamp <= :endTime"; + private static final String TIMESTAMP_FILTER = " c.key.timeStamp "; + private static final String NAME_PARAMETER = "name"; + + /** + * The default constructor injects query strings. + */ + public PfTimestampKeyFilter() { + setNameFilter(NAME_FILTER); + setTimeStampStartFilter(TIMESTAMP_START_FILTER); + setTimeStampEndFilter(TIMESTAMP_END_FILTER); + setTimeStampFilter(TIMESTAMP_FILTER); + setNameParameter(NAME_PARAMETER); + } +} diff --git a/models-dao/src/test/java/org/onap/policy/models/dao/DummyGeneratedIdEntity.java b/models-dao/src/test/java/org/onap/policy/models/dao/DummyGeneratedIdEntity.java new file mode 100644 index 000000000..6ba045c54 --- /dev/null +++ b/models-dao/src/test/java/org/onap/policy/models/dao/DummyGeneratedIdEntity.java @@ -0,0 +1,132 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2021 Nordix Foundation. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * SPDX-License-Identifier: Apache-2.0 + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.dao; + +import java.util.ArrayList; +import java.util.Date; +import java.util.List; +import javax.persistence.Column; +import javax.persistence.EmbeddedId; +import javax.persistence.Entity; +import javax.persistence.Table; +import javax.persistence.Temporal; +import javax.persistence.TemporalType; +import lombok.Data; +import lombok.EqualsAndHashCode; +import lombok.NonNull; +import org.onap.policy.common.parameters.BeanValidationResult; +import org.onap.policy.models.base.PfConcept; +import org.onap.policy.models.base.PfGeneratedIdKey; +import org.onap.policy.models.base.PfKey; + + +@Entity +@Table(name = "DummyGeneratedIdEntity") +@Data +@EqualsAndHashCode(callSuper = false) +public class DummyGeneratedIdEntity extends PfConcept { + private static final long serialVersionUID = -2962570563281067896L; + + @EmbeddedId() + @NonNull + private PfGeneratedIdKey key; + + @Column(precision = 3) + @Temporal(TemporalType.TIMESTAMP) + private Date timeStamp; + + @Column + private double doubleValue; + + /** + * Default constructor. + */ + public DummyGeneratedIdEntity() { + this.key = new PfGeneratedIdKey(); + this.timeStamp = new Date(); + } + + public DummyGeneratedIdEntity(DummyGeneratedIdEntity source) { + this.key = source.key; + this.timeStamp = source.timeStamp; + } + + /** + * Constructor. + * + * @param key the key + * @param timeStamp the date value + */ + public DummyGeneratedIdEntity(final PfGeneratedIdKey key, final Date timeStamp) { + this.key = key; + this.timeStamp = timeStamp; + } + + /** + * Constructor. + * + * @param key the key + * @param timeStamp the date value + * @param doubleValue the double value * + */ + public DummyGeneratedIdEntity(final PfGeneratedIdKey key, final Date timeStamp, final double doubleValue) { + this.key = key; + this.timeStamp = timeStamp; + this.doubleValue = doubleValue; + } + + @Override + public List getKeys() { + final List keyList = new ArrayList<>(); + keyList.add(getKey()); + return keyList; + } + + @Override + public BeanValidationResult validate(@NonNull String fieldName) { + BeanValidationResult result = new BeanValidationResult(fieldName, this); + result.addResult(key.validate("key")); + return result; + } + + @Override + public void clean() { + key.clean(); + } + + @Override + public int compareTo(@NonNull final PfConcept otherObj) { + if (this == otherObj) { + return 0; + } + + if (getClass() != otherObj.getClass()) { + return this.getClass().getName().compareTo(otherObj.getClass().getName()); + } + + final DummyGeneratedIdEntity other = (DummyGeneratedIdEntity) otherObj; + if (this.timeStamp != other.timeStamp) { + return timeStamp.compareTo(other.timeStamp); + } + return Double.compare(doubleValue, other.doubleValue); + } + +} diff --git a/models-dao/src/test/java/org/onap/policy/models/dao/EntityTest.java b/models-dao/src/test/java/org/onap/policy/models/dao/EntityTest.java index bc022be0a..80730743b 100644 --- a/models-dao/src/test/java/org/onap/policy/models/dao/EntityTest.java +++ b/models-dao/src/test/java/org/onap/policy/models/dao/EntityTest.java @@ -29,6 +29,7 @@ import static org.junit.Assert.assertNull; import java.time.Instant; import java.util.ArrayList; +import java.util.Date; import java.util.HashMap; import java.util.List; import java.util.Map; @@ -40,6 +41,7 @@ import org.eclipse.persistence.config.PersistenceUnitProperties; import org.junit.After; import org.junit.Test; import org.onap.policy.models.base.PfConceptKey; +import org.onap.policy.models.base.PfGeneratedIdKey; import org.onap.policy.models.base.PfModelException; import org.onap.policy.models.base.PfReferenceKey; import org.onap.policy.models.base.PfTimestampKey; @@ -62,6 +64,10 @@ public class EntityTest { private static final Instant TIMESTAMP0 = Instant.ofEpochSecond(1613494293); private static final Instant TIMESTAMP1 = Instant.ofEpochSecond(1613494293).plusSeconds(55); private static final Instant TIMESTAMP2 = Instant.ofEpochSecond(1613494293).plusSeconds(90); + private static final Long GENERATEDID0 = 10000L; + private static final Long GENERATEDID1 = 10001L; + private static final Long GENERATEDID2 = 10002L; + private PfDao pfDao; /** @@ -126,9 +132,13 @@ public class EntityTest { testAllOps(); + testGeneratedId(); + testVersionOps(); testgetFilteredOps(); + + testgetFilteredOps2(); } @Test @@ -367,6 +377,111 @@ public class EntityTest { assertEquals(0, pfDao.size(DummyTimestampEntity.class)); } + private void testGeneratedId() { + final PfGeneratedIdKey agKey0 = new PfGeneratedIdKey("AT-KEY0", VERSION001, GENERATEDID0); + final PfGeneratedIdKey agKey1 = new PfGeneratedIdKey("AT-KEY1", VERSION001, GENERATEDID1); + final PfGeneratedIdKey agKey2 = new PfGeneratedIdKey("AT-KEY2", VERSION001, GENERATEDID2); + final DummyGeneratedIdEntity gkeyInfo0 = new DummyGeneratedIdEntity(agKey0, Date.from(TIMESTAMP0)); + final DummyGeneratedIdEntity gkeyInfo1 = new DummyGeneratedIdEntity(agKey1, Date.from(TIMESTAMP1)); + final DummyGeneratedIdEntity gkeyInfo2 = new DummyGeneratedIdEntity(agKey2, Date.from(TIMESTAMP2)); + + pfDao.create(gkeyInfo0); + + final DummyGeneratedIdEntity gkeyInfoBack0 = pfDao.get(DummyGeneratedIdEntity.class, agKey0); + assertEquals(gkeyInfo0, gkeyInfoBack0); + + assertEquals(1, pfDao.getByTimestamp(DummyGeneratedIdEntity.class, agKey0, TIMESTAMP0).size()); + + final DummyGeneratedIdEntity gkeyInfoBackNull = + pfDao.get(DummyGeneratedIdEntity.class, PfGeneratedIdKey.getNullKey()); + assertNull(gkeyInfoBackNull); + + final Set gkeyInfoSetIn = new TreeSet<>(); + gkeyInfoSetIn.add(gkeyInfo1); + gkeyInfoSetIn.add(gkeyInfo2); + + pfDao.createCollection(gkeyInfoSetIn); + + Set gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + + gkeyInfoSetIn.add(gkeyInfo0); + assertEquals(gkeyInfoSetIn, gkeyInfoSetOut); + + pfDao.delete(gkeyInfo1); + gkeyInfoSetIn.remove(gkeyInfo1); + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + assertEquals(gkeyInfoSetIn, gkeyInfoSetOut); + + pfDao.deleteCollection(gkeyInfoSetIn); + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + assertEquals(0, gkeyInfoSetOut.size()); + + gkeyInfoSetIn.add(gkeyInfo2); + pfDao.createCollection(gkeyInfoSetIn); + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + assertEquals(gkeyInfoSetIn, gkeyInfoSetOut); + + pfDao.delete(DummyGeneratedIdEntity.class, agKey2); + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + assertEquals(gkeyInfoSetOut.size(), pfDao.size(DummyGeneratedIdEntity.class)); + + pfDao.deleteAll(DummyGeneratedIdEntity.class); + assertEquals(0, pfDao.size(DummyGeneratedIdEntity.class)); + + final PfGeneratedIdKey agKey3 = new PfGeneratedIdKey("AT-KEY0", VERSION001); + final PfGeneratedIdKey agKey4 = new PfGeneratedIdKey("AT-KEY1", VERSION001); + final PfGeneratedIdKey agKey5 = new PfGeneratedIdKey("AT-KEY2", VERSION001); + final DummyGeneratedIdEntity gkeyInfo3 = new DummyGeneratedIdEntity(agKey3, Date.from(TIMESTAMP0)); + final DummyGeneratedIdEntity gkeyInfo4 = new DummyGeneratedIdEntity(agKey4, Date.from(TIMESTAMP1)); + final DummyGeneratedIdEntity gkeyInfo5 = new DummyGeneratedIdEntity(agKey5, Date.from(TIMESTAMP2)); + + pfDao.create(gkeyInfo3); + + final DummyGeneratedIdEntity gkeyInfoBack3 = pfDao.get(DummyGeneratedIdEntity.class, agKey3); + assertEquals(gkeyInfo3, gkeyInfoBack3); + + assertEquals(1, pfDao.getByTimestamp(DummyGeneratedIdEntity.class, agKey3, TIMESTAMP0).size()); + + assertEquals(1, gkeyInfo3.getKeys().size()); + + assertEquals(1, gkeyInfo4.compareTo(gkeyInfo3)); + + assertNull(gkeyInfo4.validate(VERSION002).getResult()); + + + gkeyInfoSetIn.clear(); + gkeyInfoSetIn.add(gkeyInfo4); + gkeyInfoSetIn.add(gkeyInfo5); + + pfDao.createCollection(gkeyInfoSetIn); + + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + + gkeyInfoSetIn.add(gkeyInfo3); + assertEquals(gkeyInfoSetIn, gkeyInfoSetOut); + + pfDao.delete(gkeyInfo4); + gkeyInfoSetIn.remove(gkeyInfo4); + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + assertEquals(gkeyInfoSetIn, gkeyInfoSetOut); + + pfDao.deleteCollection(gkeyInfoSetIn); + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + assertEquals(0, gkeyInfoSetOut.size()); + + gkeyInfoSetIn.add(gkeyInfo5); + pfDao.createCollection(gkeyInfoSetIn); + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + assertEquals(gkeyInfoSetIn, gkeyInfoSetOut); + + pfDao.delete(DummyGeneratedIdEntity.class, agKey5); + gkeyInfoSetOut = new TreeSet<>(pfDao.getAll(DummyGeneratedIdEntity.class)); + assertEquals(gkeyInfoSetOut.size(), pfDao.size(DummyGeneratedIdEntity.class)); + + pfDao.deleteAll(DummyGeneratedIdEntity.class); + assertEquals(0, pfDao.size(DummyGeneratedIdEntity.class)); + } + private void testVersionOps() { final PfConceptKey aKey0 = new PfConceptKey("AAA0", VERSION001); final PfConceptKey aKey1 = new PfConceptKey("AAA0", VERSION002); @@ -466,6 +581,46 @@ public class EntityTest { filterMap.put("doubleValue", 200.1); assertEquals(1, pfDao.getFiltered(DummyTimestampEntity.class, null, null, null, null, filterMap, "DESC", 0).size()); + } + + private void testgetFilteredOps2() { + Map filterMap = new HashMap<>(); + filterMap.put("doubleValue", 200.1); + + final PfGeneratedIdKey agKey0 = new PfGeneratedIdKey("AT-KEY0", VERSION001); + final PfGeneratedIdKey agKey1 = new PfGeneratedIdKey("AT-KEY1", VERSION001); + final PfGeneratedIdKey agKey2 = new PfGeneratedIdKey("AT-KEY2", VERSION001); + final DummyGeneratedIdEntity gkeyInfo0 = new DummyGeneratedIdEntity(agKey0, Date.from(TIMESTAMP0), 200.0); + final DummyGeneratedIdEntity gkeyInfo1 = new DummyGeneratedIdEntity(agKey1, Date.from(TIMESTAMP1), 200.1); + final DummyGeneratedIdEntity gkeyInfo2 = new DummyGeneratedIdEntity(agKey2, Date.from(TIMESTAMP2), 200.2); + + pfDao.create(gkeyInfo0); + pfDao.create(gkeyInfo1); + pfDao.create(gkeyInfo2); + + assertEquals(1, pfDao + .getFiltered(DummyGeneratedIdEntity.class, "AT-KEY0", VERSION001, null, null, null, "DESC", 0).size()); + assertEquals(1, + pfDao.getFiltered(DummyGeneratedIdEntity.class, "AT-KEY0", null, null, null, null, "DESC", 0).size()); + assertEquals(3, pfDao + .getFiltered(DummyGeneratedIdEntity.class, null, VERSION001, TIMESTAMP0, TIMESTAMP2, null, "DESC", 0) + .size()); + assertEquals(1, pfDao + .getFiltered(DummyGeneratedIdEntity.class, "AT-KEY0", VERSION001, + TIMESTAMP0, TIMESTAMP2, null, "DESC", 0) + .size()); + assertEquals(3, pfDao + .getFiltered(DummyGeneratedIdEntity.class, null, VERSION001, null, TIMESTAMP2, null, "DESC", 0).size()); + assertEquals(3, pfDao + .getFiltered(DummyGeneratedIdEntity.class, null, VERSION001, + TIMESTAMP0, null, null, "DESC", 0).size()); + assertEquals(2, + pfDao.getFiltered(DummyGeneratedIdEntity.class, null, VERSION001, + TIMESTAMP0, TIMESTAMP2, null, "DESC", 2) + .size()); + + assertEquals(1, + pfDao.getFiltered(DummyGeneratedIdEntity.class, null, null, null, null, filterMap, "DESC", 0).size()); } } diff --git a/models-dao/src/test/resources/META-INF/persistence.xml b/models-dao/src/test/resources/META-INF/persistence.xml index 3d1f4ff27..fca31e2c6 100644 --- a/models-dao/src/test/resources/META-INF/persistence.xml +++ b/models-dao/src/test/resources/META-INF/persistence.xml @@ -29,6 +29,7 @@ org.onap.policy.models.dao.DummyConceptEntity org.onap.policy.models.dao.DummyReferenceEntity org.onap.policy.models.dao.DummyTimestampEntity + org.onap.policy.models.dao.DummyGeneratedIdEntity -- cgit 1.2.3-korg