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

Remove timezone on time format as it violated with BigQuery timeformat #46

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

Conversation

phongtattuan
Copy link
Contributor

@phongtattuan phongtattuan commented Apr 10, 2017

Hi @repeatedly
This PR aim to fix the error when streaming to BigQuery, related to this PR #45

<Google::Apis::BigqueryV2::ErrorProto:0x007fd4aa24d8b8 @debug_info=\“generic::invalid_argument: Could not parse ‘2017-03-13 07:21:09.177000+0000’ as a timestamp. Required format is YYYY-MM-DD HH:MM[:SS[.SSSSSS]]\“, @location=\“created_at\“, @message=\“Could not parse ‘2017-03-13 07:21:09.177000+0000’ as a timestamp. Required format is YYYY-MM-DD HH:MM[:SS[.SSSSSS]]\“, @reason=\“invalid\“>], @index=1>]”

@repeatedly
Copy link
Member

Format should be configurable.

@repeatedly repeatedly self-requested a review April 11, 2017 10:45
@phongtattuan
Copy link
Contributor Author

Thanks @repeatedly, I will fix it soon

@dobesv
Copy link

dobesv commented Jun 22, 2020

By the way there is a BigQuery specific fluentd plugin that I have been using for a while now that seems to work very well.

@patrixr
Copy link
Contributor

patrixr commented Sep 30, 2021

Hi, any updates regarding this PR ?

I'm currently facing a timestamp issue which should be fixed by this change:

The comparison fails with the update_column fails on MySQL because of the timezone, as seen here:

ActiveRecord::StatementInvalid error="Mysql2::Error: Incorrect DATETIME value: '2021-09-30 06:43:14.000000+0000'

If I remove the timezone from the state file manually, it works fine (although it re-writes it incorrectly a second later)

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.

4 participants