-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
refactor: introduce ConnectorIntegrationNew and add default implementation for each Connector #4989
refactor: introduce ConnectorIntegrationNew and add default implementation for each Connector #4989
Conversation
…ation for each Connector
…mmyConnector inside a different module with dummy_connector feature flag
|
||
use std::{marker::PhantomData, ops::Deref}; | ||
|
||
pub use flow_common_types::*; |
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.
Nit: Can we avoid the glob imports everywhere (among all new imports added in this PR)?
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.
Sure.
What is the advantage of doing this?
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.
The primary reason I'd like not having glob imports is better code navigation. Sure, Ctrl/Cmd + clicking on the type name does take us to the relevant type when we're within the IDE, but if we're quickly checking code outside the IDE, or without rust-analyzer available, such as within the GitHub web UI, finding out where a type is made available from becomes quite difficult.
…ntegration-new-and-add-default-implementation-for-each-connector
…default-implementation-for-each-connector
…default-implementation-for-each-connector
Type of Change
Description
Introduced ConnectorIntegrationNew and added default implementation for each Connector using macros
Additional Changes
Motivation and Context
How did you test it?
Checked if code compiles successfully
Checklist
cargo +nightly fmt --all
cargo clippy