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

start_datetime argument required #11

Closed
xarasco opened this issue Sep 9, 2020 · 2 comments
Closed

start_datetime argument required #11

xarasco opened this issue Sep 9, 2020 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@xarasco
Copy link

xarasco commented Sep 9, 2020

--start_datetime is a required argument.

Running this

~/.venvs/tap-bigquery/bin/tap-bigquery \
    -c ~/singer.io/tap_bq_systemchecker/tap_config.json \
    --catalog ~/singer.io/tap_bq_systemchecker/catalog.json \
    | ~/.venvs/pipelinewise-target-snowflake/bin/target-snowflake \
    -c ~/singer.io/pipelinewise_target_snowflake/systemchecker_config.json \
    >> ~/singer.io/tap_bq_systemchecker/state.json

returns this

(tap-bigquery) ubuntu@ip-172-23-84-190:~$ ./scripts/systemchecker_bq.sh
CRITICAL start_datetime not specified

It should be an optional argument.

@daigotanaka daigotanaka added the bug Something isn't working label Sep 9, 2020
@daigotanaka daigotanaka self-assigned this Sep 9, 2020
@daigotanaka
Copy link
Collaborator

daigotanaka commented Sep 9, 2020

Oops, this is a bug I introduced 8 day ago. Let me work on this.

@xarasco
Copy link
Author

xarasco commented Sep 9, 2020

I just realized that the example I pasted in didn't correctly include the --state argument. It also did not work with a state argument provided.

~/.venvs/tap-bigquery/bin/tap-bigquery \
    -c ~/singer.io/tap_bq_systemchecker/tap_config.json \
    --catalog ~/singer.io/tap_bq_systemchecker/catalog.json \
    --state ~/singer.io/tap_bq_systemchecker/state.json \
    | ~/.venvs/pipelinewise-target-snowflake/bin/target-snowflake \
    -c ~/singer.io/pipelinewise_target_snowflake/systemchecker_config.json \
    >> ~/singer.io/tap_bq_systemchecker/state.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants