LuciferYang commented on PR #51959: URL: https://github.com/apache/spark/pull/51959#issuecomment-3301462088
At present, internal package names are excluded during the MIMA check, such as `org.apache.spark.sql.execution.*`. This makes it difficult for us to recognize when we've inadvertently broken something important while modifying the code. Therefore, should we refine the relevant rules? For example, only exclude `org.apache.spark.sql.execution.xxx.*` while not excluding `org.apache.spark.sql.execution.streaming.*` (this would require relevant experts to handle the refinement). This approach might enable developers and reviewers to notice related issues before code merging and then further consult with experts in relevant fields to determine whether these internal APIs are suitable for modification. What are your thoughts on this? @cloud-fan @dongjoon-hyun @HeartSaVioR @anishshri-db -- 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: reviews-unsubscr...@spark.apache.org 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
