HyukjinKwon commented on a change in pull request #28395: URL: https://github.com/apache/spark/pull/28395#discussion_r417672569
########## File path: python/pyspark/rdd.py ########## @@ -877,6 +877,19 @@ def collect(self): sock_info = self.ctx._jvm.PythonRDD.collectAndServe(self._jrdd.rdd()) return list(_load_from_socket(sock_info, self._jrdd_deserializer)) + def collectWithJobGroup(self, groupId, description, interruptOnCancel=False): + """ + .. note:: Experimental + + When collect rdd, use this method to specify job group. + + .. versionadded:: 3.0.0 Review comment: I dont think we will ignore the existing definitions of API stability tags. Might be best to stick to what's documented there in tags ... Of course we should take it considering the ammended semver but I think it's a rather rubric to consider when we break. I kind of strongly think we should deprecate/remove it to stick to the standard approach for this one specifically later when we're able. I also agree with having this APIs as a step to migrate meanwhile. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org