Mark ConnectorPageSource.getNextPage() deprecated for removal

Author: dain

core/trino-main/src/test/java/io/trino/connector/MockConnectorPageSource.java

@@ -61,6 +61,7 @@ public boolean isFinished()
     }
 
     @Override
+    @SuppressWarnings("removal")
     public Page getNextPage()
     {
         return delegate.getNextPage();

core/trino-spi/src/main/java/io/trino/spi/connector/ConnectorPageSource.java

@@ -55,7 +55,10 @@ default OptionalLong getCompletedPositions()
 
     /**
      * Gets the next page of data.  This method is allowed to return null.
+     *
+     * @deprecated Use {@link #getNextSourcePage()} instead
      */
+    @Deprecated(forRemoval = true)
     default Page getNextPage()
     {
         throw new UnsupportedOperationException();

core/trino-spi/src/main/java/io/trino/spi/connector/EmptyPageSource.java

@@ -37,6 +37,7 @@ public boolean isFinished()
     }
 
     @Override
+    @SuppressWarnings("removal")
     public Page getNextPage()
     {
         return null;

core/trino-spi/src/main/java/io/trino/spi/connector/FixedPageSource.java

@@ -74,6 +74,7 @@ public boolean isFinished()
     }
 
     @Override
+    @SuppressWarnings("removal")
     public Page getNextPage()
     {
         if (isFinished()) {

core/trino-spi/src/main/java/io/trino/spi/connector/RecordPageSource.java

@@ -81,6 +81,7 @@ public boolean isFinished()
     }
 
     @Override
+    @SuppressWarnings("removal")
     public Page getNextPage()
     {
         if (!closed) {