s1ck commented on issue #24851: [SPARK-27303][GRAPH] Add Spark Graph API URL: https://github.com/apache/spark/pull/24851#issuecomment-536180801 @dongjoon-hyun @WeichenXu123 Thanks for your feedback. I'm fine with making the trait `sealed` for now to unblock the work. We can have another round of discussion once the implementation PR is up. I agree that the remaining comments are minor and can be discussed / applied as part of the next PRs. I sincerely hope that we can move on with the SPIP work now. @WeichenXu123 Since we're not sure if we want to add the `graph` method, I'd like to avoid it for now.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
