Revisited internals

git-svn-id: https://svn.d4science.research-infrastructures.eu/gcube/trunk/data-publishing/document-store-lib@125518 82a268e6-3cf1-43bd-a215-b396298e98cf

src/main/java/org/gcube/documentstore/records/AggregatedRecord.java

@@ -46,7 +46,9 @@ public interface AggregatedRecord<A extends AggregatedRecord<A,R>, R extends Rec
 	public static final String END_TIME = "endTime";
 	
 	/**
-	 * @return a Set containing the keys of required fields
+	 * @return a Set containing the keys of aggregated fields
+	 * The returned Set MUST be a copy of the internal representation. 
+	 * Any modification to the returned Set MUST not affect the object
 	 */
 	public Set<String> getAggregatedFields();
 	

src/main/java/org/gcube/documentstore/records/Record.java

@@ -39,9 +39,18 @@ public interface Record extends Comparable<Record>, Serializable {
 	
 	/**
 	 * @return a Set containing the keys of required fields
+	 * The returned Set MUST be a copy of the internal representation. 
+	 * Any modification to the returned Set MUST not affect the object
 	 */
 	public Set<String> getRequiredFields();
 	
+	/**
+	 * @return a Set containing the keys of computed fields
+	 * The returned Set MUST be a copy of the internal representation. 
+	 * Any modification to the returned Set MUST not affect the object
+	 */
+	public Set<String> getComputedFields();
+	
 	/**
 	 * Return the {@link Record} Type
 	 * @return {@link Record} Type

src/main/java/org/gcube/documentstore/records/implementation/AbstractRecord.java

@@ -190,7 +190,22 @@ public abstract class AbstractRecord implements Record {
 	 */
 	@Override
 	public Set<String> getRequiredFields() {
-		return requiredFields;
+		return new HashSet<String>(requiredFields);
+	}
+	
+	/**
+	 * {@inheritDoc}
+	 */
+	@Override
+	public Set<String> getComputedFields() {
+		return new HashSet<String>(computedFields);
+	}
+
+	/**
+	 * {@inheritDoc}
+	 */
+	public Set<String> getAggregatedFields() {
+		return new HashSet<String>(aggregatedFields);
 	}
 	
 	@Override