Skip to content

Fix Spacing for DagBag.dagbag_report()'s Output#31095

Merged
potiuk merged 2 commits intoapache:mainfrom
mtsadler-branch:patch-2
May 5, 2023
Merged

Fix Spacing for DagBag.dagbag_report()'s Output#31095
potiuk merged 2 commits intoapache:mainfrom
mtsadler-branch:patch-2

Conversation

@mtsadler-branch
Copy link
Copy Markdown
Contributor

The whitespace before {table} is throwing off the headers when printing DagBag.dagbag_report()

image

The whitespace before `{table}` 
is throwing off the headers when printing `DagBag.dagbag_report()`
@mtsadler-branch
Copy link
Copy Markdown
Contributor Author

I dont think these failures are related to the small change I made:

FAILED tests/jobs/test_triggerer_job.py::test_trigger_logging_sensitive_info - AssertionError: assert 'test_dag/test_run/sensitive_arg_task/-1/1 (ID 1) 
    """
    Checks that when a trigger fires, it doesn't log any sensitive
    information from arguments
    """

and


FAILED tests/providers/amazon/aws/operators/test_sqs.py::TestSqsPublishOperator::test_execute_success - botocore.exceptions.ClientError: An error occurred (400) when calling the SendMessage operation: <ErrorResponse xmlns="http://queue.amazonaws.com/doc/2012-11-05/"><Error><Type>Sender</Type><Code>AWS.SimpleQueueService.NonExistentQueue</Code><Message>The specified queue does not exist for this wsdl version.</Message><Detail/></Error><RequestId>86uWM0Jl6aJc4ryo8UcAN5fOLZTgL4oFScWcXOqVpCfP0HUgezA7</RequestId></ErrorResponse>

@potiuk potiuk merged commit 41c8746 into apache:main May 5, 2023
@ephraimbuddy ephraimbuddy added this to the Airflow 2.6.1 milestone May 8, 2023
@ephraimbuddy ephraimbuddy added the type:bug-fix Changelog: Bug Fixes label May 8, 2023
ephraimbuddy pushed a commit that referenced this pull request May 8, 2023
The whitespace before `{table}`
is throwing off the headers when printing `DagBag.dagbag_report()`

(cherry picked from commit 41c8746)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants