Reuben

Results 102 comments of Reuben

https://github.com/Matatika/target-snowflake--meltanolabs/commit/101290a3f26d3fa4c0479abc98cbef7a7cf7bc09

> `-` is an informal standard within the singer community to map tables to schemas in SQL targets, so I would that the tap is misbehaving in this case. I'd...

Should this be reopened since #3066 was merged? I think there should be a way to opt-out of this behaviour, or better docs at a minimum.

@edgarrmondragon In our own taps, I don't think there are any. I was thinking more about various other legacy taps we use (not yet adopted/maintained by us) that do contain...

I think https://github.com/microsoft/vscode-python-debugger/issues/316 would have to be addressed to make this possible.

(sorry, not sure if this is a bug report or feature request)

The workaround is fairly simple: ```py if start_date := self.get_starting_timestamp(context) or ( "start_date" in self.config and datetime.fromisoformat(self.config["start_date"]) ): payload["startDateTime"] = start_date.strftime(r"%Y-%m-%d %H:%M:%S") ```

> You mean on the tap implementation side? Yep.

Looks like that would work. Why is merging state not the default behaviour though?

> The next version adds the alternative, i.e. so we have something like `--merge-state/--overwrite-state` and we communicate in the release notes that the default behavior will change. So `--overwrite-state` would...