PARQUET-2384: Mark toOriginalType as deprecated (#1202)

The OriginalType itself has been marked as deprecated
for a long time, would be good to mark these as deprecated
as well.

Author: Fokko

parquet-column/src/main/java/org/apache/parquet/schema/LogicalTypeAnnotation.java

@@ -151,8 +151,11 @@ protected LogicalTypeAnnotation fromString(List<String> params) {
    *
    * API should be considered private
    *
+   * Deprecated: Please use the LogicalTypeAnnotation itself
+   *
    * @return the OriginalType representation of the new logical type, or null if there's none
    */
+  @Deprecated
   public abstract OriginalType toOriginalType();
 
   /**
@@ -314,9 +317,12 @@ private StringLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.UTF8;
     }
@@ -357,9 +363,12 @@ private MapLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.MAP;
     }
@@ -395,9 +404,12 @@ private ListLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.LIST;
     }
@@ -433,9 +445,12 @@ private EnumLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.ENUM;
     }
@@ -489,9 +504,12 @@ public int getScale() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.DECIMAL;
     }
@@ -546,9 +564,12 @@ private DateLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.DATE;
     }
@@ -598,9 +619,12 @@ private TimeLogicalTypeAnnotation(boolean isAdjustedToUTC, TimeUnit unit) {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       switch (unit) {
         case MILLIS:
@@ -681,9 +705,12 @@ private TimestampLogicalTypeAnnotation(boolean isAdjustedToUTC, TimeUnit unit) {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       switch (unit) {
         case MILLIS:
@@ -771,9 +798,12 @@ private IntLogicalTypeAnnotation(int bitWidth, boolean isSigned) {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       switch (bitWidth) {
         case 8:
@@ -847,9 +877,12 @@ private JsonLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.JSON;
     }
@@ -890,9 +923,12 @@ private BsonLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.BSON;
     }
@@ -934,9 +970,12 @@ private UUIDLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       // No OriginalType for UUID
       return null;
@@ -974,9 +1013,12 @@ private IntervalLogicalTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.INTERVAL;
     }
@@ -1029,9 +1071,12 @@ private MapKeyValueTypeAnnotation() {
     /**
      * API Should be considered private
      *
+     * Deprecated: Please use the LogicalTypeAnnotation itself
+     *
      * @return the original type
      */
     @Override
+    @Deprecated
     public OriginalType toOriginalType() {
       return OriginalType.MAP_KEY_VALUE;
     }