Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Iterate once in Frame.ofRowsOrdinal #396

Merged
merged 1 commit into from Jul 27, 2018

Conversation

Projects
None yet
2 participants
@zyzhu
Copy link
Contributor

zyzhu commented Jul 27, 2018

I've followed the issue #356 and discussion #357 to avoid iterating twice in Frame.ofRowsOrdinal

Ping @kflu and @tpetricek for review

@zyzhu zyzhu requested a review from tpetricek Jul 27, 2018

@zyzhu zyzhu changed the title iterate once in Frame.ofRowsOrdinal Iterate once in Frame.ofRowsOrdinal Jul 27, 2018

@dsyme dsyme merged commit 23a8db7 into fslaborg:master Jul 27, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@zyzhu zyzhu deleted the zyzhu:fix_pull_357 branch Jul 27, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.