mark getDataType(String) as deprecated

* internal code can use getDataTypeInternal when necessary
* external code should always get their data types in other ways

2 files changed, 12 insertions, 12 deletions

diff --git a/document/src/test/java/com/yahoo/document/DataTypeTestCase.java b/document/src/test/java/com/yahoo/document/DataTypeTestCase.java
index 4e522af527f..c81018c723f 100644
--- a/document/src/test/java/com/yahoo/document/DataTypeTestCase.java
+++ b/document/src/test/java/com/yahoo/document/DataTypeTestCase.java
@@ -34,9 +34,9 @@ public class DataTypeTestCase {
     public void testTagType() {
         DocumentTypeManager manager = new DocumentTypeManager();
         //assertEquals(DataType.getWeightedSet(DataType.STRING,true,true),DataType.TAG.getBaseType());
-        assertNotNull(manager.getDataType("tag"));
-        assertEquals(DataType.TAG, manager.getDataType("tag"));
-        assertEquals(manager.getDataType("tag").getCode(), 18);
+        assertNotNull(manager.getDataTypeInternal("tag"));
+        assertEquals(DataType.TAG, manager.getDataTypeInternal("tag"));
+        assertEquals(manager.getDataTypeInternal("tag").getCode(), 18);
         assertEquals(DataType.getWeightedSet(DataType.STRING,true,true).getCode(), 18);
     }
 
diff --git a/document/src/test/java/com/yahoo/document/DocumentTypeManagerTestCase.java b/document/src/test/java/com/yahoo/document/DocumentTypeManagerTestCase.java
index 5770b998c69..6829c7e6862 100644
--- a/document/src/test/java/com/yahoo/document/DocumentTypeManagerTestCase.java
+++ b/document/src/test/java/com/yahoo/document/DocumentTypeManagerTestCase.java
@@ -46,16 +46,16 @@ public class DocumentTypeManagerTestCase {
     @Test
     public void testBasicTypes() {
         DocumentTypeManager dtm = new DocumentTypeManager();
-        DataType intType = dtm.getDataType("int");
+        DataType intType = dtm.getDataTypeInternal("int");
         assertSame(DataType.INT, intType);
 
-        DataType stringType = dtm.getDataType("string");
+        DataType stringType = dtm.getDataTypeInternal("string");
         assertSame(DataType.STRING, stringType);
 
-        DataType longType = dtm.getDataType("long");
+        DataType longType = dtm.getDataTypeInternal("long");
         assertSame(DataType.LONG, longType);
 
-        DataType doubleType = dtm.getDataType("double");
+        DataType doubleType = dtm.getDataTypeInternal("double");
         assertSame(DataType.DOUBLE, doubleType);
     }
 
@@ -78,12 +78,12 @@ public class DocumentTypeManagerTestCase {
         manager.register(docType2);
 
         assertEquals(struct, manager.getDataType(struct.getId()));
-        assertEquals(struct, manager.getDataType("mystruct"));
+        assertEquals(struct, manager.getDataTypeInternal("mystruct"));
         assertEquals(wset1, manager.getDataType(wset1.getId()));
         assertEquals(wset2, manager.getDataType(wset2.getId()));
         assertEquals(array, manager.getDataType(array.getId()));
-        assertEquals(docType, manager.getDataType("mydoc"));
-        assertEquals(docType2, manager.getDataType("myotherdoc"));
+        assertEquals(docType, manager.getDataTypeInternal("mydoc"));
+        assertEquals(docType2, manager.getDataTypeInternal("myotherdoc"));
         assertEquals(docType, manager.getDocumentType(new DataTypeName("mydoc")));
         assertEquals(docType2, manager.getDocumentType(new DataTypeName("myotherdoc")));
     }
@@ -418,13 +418,13 @@ search annotationsimplicitstruct {
 
         DocumentTypeManager manager = createConfiguredManager("file:src/test/document/documentmanager.structsanyorder.cfg");
 
-        StructDataType foo = (StructDataType) manager.getDataType("foo");
+        StructDataType foo = (StructDataType) manager.getDataTypeInternal("foo");
         assertNotNull(foo);
         assertEquals(1, foo.getFields().size());
         Field foos1 = foo.getField("s1");
         assertSame(DataType.INT, foos1.getDataType());
 
-        StructDataType sct = (StructDataType) manager.getDataType("sct");
+        StructDataType sct = (StructDataType) manager.getDataTypeInternal("sct");
         assertNotNull(sct);
         assertEquals(4, sct.getFields().size());
         Field s1 = sct.getField("s1");