Github user texasmichelle commented on the pull request:
https://github.com/apache/spark/pull/5149#issuecomment-94523955
Great, detailed suggestions @pwendell - thanks! A few of the things you
pointed out were remnants of the old format (no brackets, adding a colon) and
could be removed. I initially chose deque for its performance over lists, but
considering the scale here, those benefits are negligible. For better
maintainability, deque was swapped out. Also, a prompt was added for choosing
between the original and modified titles.
---
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]
