Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/83#issuecomment-108160293
Hi,
I'm not really for merging this PR for multiple reasons:
* Auto ordering can sometimes be helpful at the moment (even if being able
to order we want is the goal)
* There is no replacement for ordering outside of the query order
* We are working as part of the Spring Cleaning PR and GSOC into replacing
the pivot and ultimately integrating ordering inside of it. (Pivot refactoring
coming soon at least), so it would add more cleaning/possible conflicts for us
If however others were inclined to merge:
* Need to remove the commented console.log
This could be eventually merged, and it would be squashed in 3-4weeks
maybe. The decision is more about, if people are okay with losing the auto
ordering during that time.
---
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.
---
