Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-29449

Add --long-log to faro

    XMLWordPrintable

    Details

      Description

      Recent faro runs have been taking a long time. Adding --long-log will enable us to better see where the time is being spent. See linked slack discussion for details

        Attachments

          Issue Links

            Activity

            Hide
            jcarlin Jeffrey Carlin added a comment -

            From the Slack discussion, it doesn't seem like the issues are related to faro (i.e., they are happening in other parts of the "validate_drp_gen3" processing)?

             

            Show
            jcarlin Jeffrey Carlin added a comment - From the Slack discussion, it doesn't seem like the issues are related to faro (i.e., they are happening in other parts of the "validate_drp_gen3" processing)?  
            Hide
            tjenness Tim Jenness added a comment -

            The long-log option controls the formatting of messages, not the messages themselves (it adds extra context). Is that what you mean? Or is this talking about enabling debug logging?

            Show
            tjenness Tim Jenness added a comment - The long-log option controls the formatting of messages, not the messages themselves (it adds extra context). Is that what you mean? Or is this talking about enabling debug logging?
            Hide
            ktl Kian-Tat Lim added a comment -

            Somehow the pipeline execution in faro is triggering this extra-long runtime which is not apparent in other pipeline executions. But it does not appear to be the metric measurements that are causing the problem.

            Show
            ktl Kian-Tat Lim added a comment - Somehow the pipeline execution in faro is triggering this extra-long runtime which is not apparent in other pipeline executions. But it does not appear to be the metric measurements that are causing the problem.
            Hide
            lguy Leanne Guy added a comment -

            Agree that it does not look like faro is the root cause, but adding --long-log will help to find out where the problem is. We should also add debug logging to faro

            Show
            lguy Leanne Guy added a comment - Agree that it does not look like faro is the root cause, but adding --long-log will help to find out where the problem is. We should also add debug logging to faro
            Hide
            krughoff Simon Krughoff added a comment -

            This has been done. Please see PR for review.

            Show
            krughoff Simon Krughoff added a comment - This has been done. Please see PR for review.

              People

              Assignee:
              krughoff Simon Krughoff
              Reporter:
              lguy Leanne Guy
              Watchers:
              Colin Slater, Jeffrey Carlin, Keith Bechtol, Kian-Tat Lim, Leanne Guy, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: