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

fix: corrected bad datetime json encoding when selecting jsonl format #41

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

neuromantik33
Copy link

This is a small patch to apply the correct serializer for jsonl as for json. It encountered it while evaluating this target:
The poetry.lock file was also incorrect. Cheers 🍺

2024-06-19T16:26:04.795208Z [info     ] TypeError: Object of type datetime is not JSON serializable cmd_type=elb consumer=True job_name=dev:tap-postgres-to-target-s3 name=target-s3 producer=False run_id=90af8632-26c9-450c-a2e3-69d587253194 stdio=stderr string_id=target-s3
2024-06-19T16:26:04.917234Z [error    ]                               
╭─────────────────────────────── Traceback (most recent call last) ────────────────────────────────╮
│ /home/drnick/repos/drnick/ingester-playground/.venv/lib/python3.10/site-packages/meltano/core/lo │
│ gging/output_logger.py:208 in redirect_logging                                                   │
│                                                                                                  │
│   205 │   │   │   *ignore_errors,                                                                │
│   206 │   │   )                                                                                  │
│   207 │   │   try:                                                                               │
│ ❱ 208 │   │   │   yield                                                                          │
│   209 │   │   except ignored_errors:  # noqa: WPS329                                             │
│   210 │   │   │   raise                                                                          │
│   211 │   │   except RunnerError as err:                                                         │
│                                                                                                  │

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.

1 participant