Update the overall db timeout for writing data from a single file during ingestion - This value is used in the processSingleFile function. - Internally that function invokes WriteTraces on sqlTraceStore. WriteTraces has configured the timeout to 15 minutes. - Since the parent context object has a lower timeout than the child context (in WriteTraces), the context was getting cancelled in a minute instead of 15 minutes as expected in WriteTraces. - This caused the WriteTraces function to timeout in a minute. This impacts writing of large files which is expected to take longer. - Updating the parent timeout to 15 minutes so that WriteTraces is not cancelled before the expected deadline. Change-Id: Ia44830d037cfc3afeafece175821b48385633c60 Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/932616 Auto-Submit: Ashwin Verleker <ashwinpv@google.com> Commit-Queue: Ashwin Verleker <ashwinpv@google.com> Reviewed-by: Farid (Mojtaba) Faridzad <faridzad@google.com> Commit-Queue: Farid (Mojtaba) Faridzad <faridzad@google.com>
diff --git a/perf/go/ingest/process/process.go b/perf/go/ingest/process/process.go index b29d3ab..15fa245 100644 --- a/perf/go/ingest/process/process.go +++ b/perf/go/ingest/process/process.go
@@ -31,7 +31,7 @@ // defaultDatabaseTimeout is the context timeout used when making a request that // involves the database. For more complex requests use config.QueryMaxRuntime. -const defaultDatabaseTimeout = time.Minute +const defaultDatabaseTimeout = 15 * time.Minute // sendPubSubEvent sends the unencoded params and paramset found in a single // ingested file to the PubSub topic specified in the selected Perf instances