-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Deduplicate README.md #79
Conversation
Thanks @msathis ! just an idea, maybe we can point the datafusion crate to the readme in the root too? https://doc.rust-lang.org/cargo/reference/manifest.html#the-readme-field |
Nice idea 💡 @Dandandan. Never knew this. Will do it 👌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, looks good! 😎
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @msathis !
- [x] Constant folding | ||
- [x] Join Reordering | ||
- [x] Limit Pushdown | ||
- [x] Projection push down | ||
- [x] Predicate push down |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these were meant to be sub bullets (they are features of the query optimizer) but we can fix that in a subsequent PR)
Which issue does this PR close?
Closes #71 .
Rationale for this change
Removing duplicated content
What changes are included in this PR?
Remove duplicated content & move Developers.md to root
Are there any user-facing changes?
N/A