Repository: asterixdb
Updated Branches:
  refs/heads/master 053552693 -> 8b3e67af4


[ASTERIXDB-2114][MTD] Fixed incorrectly persisted FeedPolicy

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- New FeedPolicy was incorrectly persisting PolicyName & Description fields

Change-Id: I04beb24c861b116525a9da467fafc1742f4a73d4
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2035
Contrib: Jenkins <[email protected]>
Reviewed-by: Steven Jacobs <[email protected]>
Sonar-Qube: Jenkins <[email protected]>
Integration-Tests: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/asterixdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/asterixdb/commit/8b3e67af
Tree: http://git-wip-us.apache.org/repos/asf/asterixdb/tree/8b3e67af
Diff: http://git-wip-us.apache.org/repos/asf/asterixdb/diff/8b3e67af

Branch: refs/heads/master
Commit: 8b3e67af4a7c761f8087e41baa66c0b33a38378e
Parents: 0535526
Author: Ildar Absalyamov <[email protected]>
Authored: Thu Sep 28 18:02:59 2017 -0700
Committer: Ildar Absalyamov <[email protected]>
Committed: Fri Sep 29 16:25:40 2017 -0700

----------------------------------------------------------------------
 .../feeds/create-policy-from-file/create-policy-from-file.1.adm    | 2 +-
 .../metadata/entitytupletranslators/FeedPolicyTupleTranslator.java | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/8b3e67af/asterixdb/asterix-app/src/test/resources/runtimets/results/feeds/create-policy-from-file/create-policy-from-file.1.adm
----------------------------------------------------------------------
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/results/feeds/create-policy-from-file/create-policy-from-file.1.adm
 
b/asterixdb/asterix-app/src/test/resources/runtimets/results/feeds/create-policy-from-file/create-policy-from-file.1.adm
index 7c4c53b..f006dcb 100644
--- 
a/asterixdb/asterix-app/src/test/resources/runtimets/results/feeds/create-policy-from-file/create-policy-from-file.1.adm
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/results/feeds/create-policy-from-file/create-policy-from-file.1.adm
@@ -1 +1 @@
-{ "DataverseName": "experiments", "PolicyName": "someString", "Description": 
"experiments", "Properties": {{ { "Name": "name", "Value": "testname" }, { 
"Name": "value", "Value": "testvalue" }, { "Name": "key", "Value": "testkey" } 
}} }
+{ "DataverseName": "experiments", "PolicyName": "testPolicy", "Description": 
"someString", "Properties": {{ { "Name": "name", "Value": "testname" }, { 
"Name": "value", "Value": "testvalue" }, { "Name": "key", "Value": "testkey" } 
}} }

http://git-wip-us.apache.org/repos/asf/asterixdb/blob/8b3e67af/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/FeedPolicyTupleTranslator.java
----------------------------------------------------------------------
diff --git 
a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/FeedPolicyTupleTranslator.java
 
b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/FeedPolicyTupleTranslator.java
index 8ddac7a..ec22735 100644
--- 
a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/FeedPolicyTupleTranslator.java
+++ 
b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/FeedPolicyTupleTranslator.java
@@ -141,7 +141,7 @@ public class FeedPolicyTupleTranslator extends 
AbstractTupleTranslator<FeedPolic
         fieldValue.reset();
         aString.setValue(feedPolicy.getDescription());
         stringSerde.serialize(aString, fieldValue.getDataOutput());
-        
recordBuilder.addField(MetadataRecordTypes.FEED_POLICY_ARECORD_POLICY_NAME_FIELD_INDEX,
 fieldValue);
+        
recordBuilder.addField(MetadataRecordTypes.FEED_POLICY_ARECORD_DESCRIPTION_FIELD_INDEX,
 fieldValue);
 
         // write field 3 (properties)
         Map<String, String> properties = feedPolicy.getProperties();

Reply via email to