You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/02/01 12:06:13 UTC
[GitHub] srowen commented on issue #23607: [SPARK-26681][SQL] Support
Ammonite inner-class scopes.
srowen commented on issue #23607: [SPARK-26681][SQL] Support Ammonite inner-class scopes.
URL: https://github.com/apache/spark/pull/23607#issuecomment-459702010
I also can't feel strongly either way. If the issue is simply, "what if 10 more projects want something like this" or "what if this turns into 1000 lines of code" then, it won't. What's the alternative in the near-term, a small SPI to implement some interface? yeah that could be OK too. If the worry is complexity, that's more complex. It's more flexible to support N more such callers, but are there any?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org