[ATLAS-1879] Updating classification removes some properties

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.8-incubating
• Fix Version/s: 0.8.1, 1.0.0
• Component/s: atlas-core
• Assignee: Péter Gergő Barna
• Reporter: Laura Ngo
• Created: 16/Jun/17 14:32
• Updated: 27/Jul/17 23:32
• Resolved: 27/Jul/17 23:32

Description

• Created classification via POST.
• Updated via PUT
• Lost properties

POST http://127.0.0.1:21000/api/atlas/v2/types/typedefs

{
 "classificationDefs": [{
  "name": "test_classification_11",
  "description": "",
  "createdBy" : "admin",
  "superTypes": [],
  "attributeDefs": [{
    "name" : "test_class_11",
    "typeName" : "string",
    "isOptional" : true,
    "isUnique" : true,
    "isIndexable" : true,
    "cardinality": "SINGLE",
    "valuesMinCount": 0,
    "valuesMaxCount": 1
}]
 }],
 "entityDefs": [],
 "enumDefs": [],
 "structDefs": []
}

GET http://127.0.0.1:21000/api/atlas/v2/types/classification/name/test_classification_11

{
"category": "CLASSIFICATION",
"guid": "83162fe1-4bb4-4a87-b2b8-364e751a1265",
"createdBy": "admin",
"createTime": 1497485890857,
"updateTime": 1497485890857,
"version": 1,
"name": "test_classification_11",
"description": "",
"typeVersion": "1.0",
"attributeDefs": [
  {
"name": "test_class_11",
"typeName": "string",
"isOptional": true,
"cardinality": "SINGLE",
"valuesMinCount": 0,
"valuesMaxCount": 1,
"isUnique": true,
"isIndexable": true
}
],
"superTypes": [],
}

PUT http://127.0.0.1:21000/api/atlas/v2/types/typedefs

Update attribute.

GET http://127.0.0.1:21000/api/atlas/v2/types/classification/name/test_classification_11

{
"category": "CLASSIFICATION",
"createdBy": "admin",
"name": "test_classification_11",
"description": "",
"attributeDefs": [
  {
"name": "test_class_11",
"typeName": "string",
"isOptional": true,
"cardinality": "SINGLE",
"valuesMinCount": 0,
"valuesMaxCount": 1,
"isUnique": true,
"isIndexable": false
}
],
"superTypes": [],
}

Some properties are missing after PUT update of attribute "isIndexable"

Issue Links

• links to: GitHub Pull Request #39