Github user concretevitamin commented on the pull request:
https://github.com/apache/spark/pull/948#issuecomment-45574240
Hey @liancheng - I think this refactoring will solve this particular ticket
(i.e. queries w/ sorts will not be eagerly executed anymore). However, I don't
see why we don't *need* `processCmd` and `eagerlyProcess` anymore; for
instance, for set commands this PR's changes will eagerly call `toRdd`, but we
still need some logic in the two `toRdd` to handle the set. Are you proposing
to just take that piece of logic and put them into SchemaRDDLike? That could
work. And what about `processCmd` then?
---
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.
---