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

changed Relation::filter_columns #74

Closed
wants to merge 2 commits into from
Closed

changed Relation::filter_columns #74

wants to merge 2 commits into from

Conversation

vserret
Copy link
Contributor

@vserret vserret commented Aug 1, 2023

No description provided.

victoria de sainte agathe added 2 commits August 1, 2023 10:43
@@ -2,6 +2,8 @@
//!

use super::{Join, Map, Reduce, Relation, Set, Table, Values, Variant as _};
// use crate::data_type::product::Error;
use crate::data_type::value::Variant;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you explicitly use Variant or do you just need the method of the trait, in this case use Variant as _

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

data_type is already imported, please always factor the crate uses

@vserret vserret marked this pull request as draft September 8, 2023 12:27
@vserret vserret closed this Sep 18, 2023
@vserret vserret deleted the add_utils_for_ds branch October 13, 2023 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants