-
Notifications
You must be signed in to change notification settings - Fork 994
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: Added Redshift and Spark typecheck to data_source event_timestam…
…p_col inference (#2389) * added Redshift typecheck to data_source event_timestamp_col inference Signed-off-by: Shai Bruhis <shaibruhis@gmail.com> * address comments Signed-off-by: Shai Bruhis <shaibruhis@gmail.com> * moved non file data sources into their own test Signed-off-by: Shai Bruhis <shaibruhis@gmail.com> * addressed comments Signed-off-by: Shai Bruhis <shaibruhis@gmail.com> * fixed texts Signed-off-by: Shai Bruhis <shaibruhis@gmail.com> * remove previously defined event_timestamp_column from data_source to allow for inference Signed-off-by: Shai Bruhis <shaibruhis@gmail.com> * made a deepcopy of data_sources to not affect other tests Signed-off-by: Shai Bruhis <shaibruhis@gmail.com> * linter Signed-off-by: Shai Bruhis <shaibruhis@gmail.com>
- Loading branch information
1 parent
e17028d
commit 04dea73
Showing
3 changed files
with
30 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters