cloud-fan commented on a change in pull request #35657:
URL: https://github.com/apache/spark/pull/35657#discussion_r821708404
##########
File path:
sql/catalyst/src/main/java/org/apache/spark/sql/connector/read/partitioning/Distribution.java
##########
@@ -29,16 +28,20 @@
* partition(the output records of a single {@link PartitionReader}).
* <p>
* The instance of this interface is created and provided by Spark, then
consumed by
- * {@link Partitioning#satisfy(Distribution)}. This means data source
developers don't need to
+ * {@link Partitioning}. This means data source developers don't need to
* implement this interface, but need to catch as more concrete
implementations of this interface
- * as possible in {@link Partitioning#satisfy(Distribution)}.
+ * as possible in {@link Partitioning}.
* <p>
* Concrete implementations until now:
* <ul>
* <li>{@link ClusteredDistribution}</li>
* </ul>
*
+ * <b>NOTE</b>: this interface is deprecated in favor of
+ * {@link org.apache.spark.sql.connector.distributions.Distribution} and is
subject to
+ * future removal.
+ *
* @since 3.0.0
*/
-@Evolving
+@Deprecated
Review comment:
ditto
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
