Skip to content

Commit

Permalink
fix: Use dateutil parser to parse materialization times (feast-dev#2464)
Browse files Browse the repository at this point in the history
Signed-off-by: Achal Shah <[email protected]>
Signed-off-by: joostvan <[email protected]>
  • Loading branch information
achals authored and joostvan committed Mar 30, 2022
1 parent 3789cf3 commit eea7dcf
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions sdk/python/feast/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
import pkg_resources
import yaml
from colorama import Fore, Style
from dateutil import parser

from feast import flags, flags_helper, utils
from feast.constants import DEFAULT_FEATURE_TRANSFORMATION_SERVER_PORT
Expand All @@ -41,7 +42,6 @@
)

_logger = logging.getLogger(__name__)
DATETIME_ISO = "%Y-%m-%dT%H:%M:%s"


class NoOptionDefaultFormat(click.Command):
Expand Down Expand Up @@ -501,8 +501,8 @@ def materialize_command(
store = FeatureStore(repo_path=str(repo))
store.materialize(
feature_views=None if not views else views,
start_date=utils.make_tzaware(datetime.fromisoformat(start_ts)),
end_date=utils.make_tzaware(datetime.fromisoformat(end_ts)),
start_date=utils.make_tzaware(parser.parse(start_ts)),
end_date=utils.make_tzaware(parser.parse(end_ts)),
)


Expand Down

0 comments on commit eea7dcf

Please sign in to comment.