Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Add more debug log for BigQuery Exception #6

Open
devdinu opened this issue Mar 6, 2019 · 1 comment
Open

Add more debug log for BigQuery Exception #6

devdinu opened this issue Mar 6, 2019 · 1 comment
Labels
good first issue Good for newcomers

Comments

@devdinu
Copy link
Contributor

devdinu commented Mar 6, 2019

Failed to push records to sink FailureStatus{cause=com.google.cloud.bigquery.BigQueryException: www.googleapis.com, message='null'}

We've cases where push failed (which's retriable), but we don't have the logs, which needs to fixed.

@devdinu devdinu added the good first issue Good for newcomers label Mar 6, 2019
@mauliksoneji
Copy link
Contributor

@devdinu the message will stay null for most of the cases.
Failurestatus has two constructors:

public FailureStatus(Exception cause)

and

public FailureStatus(Exception cause, String message)

Mostly I see that we have used the constructor without message. In these cases, message will be null.
I think, we should remove this message logic as we are not populating at all. Cause itself is self descriptive

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants