Github user weiqingy commented on the issue:
https://github.com/apache/spark/pull/15960
Hi, @rxin @srowen Thanks for the prompt feedbacks and suggestions. Yes, I
understand and agree with your concerns. The motivation of creating this PR is
I think the redundant string interpolators are unnecessary and misleading, so
itâs good to clean up the code and make complication a little faster.
How about making this PR in a long-term step-by-step manner in terms of
modules? (the rule of `NoRedundantStringInterpolator` Scala style will be added
in the final step)
Spark 2.1(~ Nov 2016): Mesos (2 files), Yarn (4 files), External (4 files),
GraphX (5 files)
Spark 2.2(~ Mar 2017): Hive-Thrift-Server (4 files), Streaming (8 files),
Example (18 files)
Spark 2.3(~ Jul 2017): Hive (28 files), Catalyst(19 files)
Spark 2.4(~ Nov 2017): SQL Core(47 files)
Spark 2.5(~ 2018): Core(54 files), MLLib (69 files)
If it is ok for you, I can make PRs based on the schedule above (Yes, I
need to fix the failures of Spark unit tests). If not, I can just close this PR
and SPARK-18521. ðð Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
