Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Aug 6, 2023

Using same session in different steps of the same test has the potential of not flishing/committing the changes between.

Seems that the #33150 traded one flakiness with another. Attempting to make sure that the flash/commit is executed before the second run.

Error:

The test RuntimeError: number of values in row (0) differ from number of column processors (29)

The error is strange however and indicates a bug in sqlite library.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Using same session in different steps of the same test has the
potential of not flishing/committing the changes between.

Seems that the apache#33150 traded one flakiness with another. Attempting to
make sure that the flash/commit is executed before the second run.

Error:

The test RuntimeError: number of values in row (0) differ from number of
column processors (29)

The error is strange however and indicates a bug in sqlite library.
@potiuk
Copy link
Member Author

potiuk commented Aug 6, 2023

@potiuk potiuk merged commit 6b21b79 into apache:main Aug 6, 2023
@potiuk potiuk deleted the commit-changes-between-runs branch August 6, 2023 14:12
@potiuk
Copy link
Member Author

potiuk commented Aug 6, 2023

Let's see.

@ephraimbuddy ephraimbuddy added this to the Airflow 2.7.0 milestone Aug 8, 2023
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Aug 8, 2023
ephraimbuddy pushed a commit that referenced this pull request Aug 8, 2023
)

Using same session in different steps of the same test has the
potential of not flishing/committing the changes between.

Seems that the #33150 traded one flakiness with another. Attempting to
make sure that the flash/commit is executed before the second run.

Error:

The test RuntimeError: number of values in row (0) differ from number of
column processors (29)

The error is strange however and indicates a bug in sqlite library.

(cherry picked from commit 6b21b79)
ephraimbuddy pushed a commit that referenced this pull request Aug 8, 2023
)

Using same session in different steps of the same test has the
potential of not flishing/committing the changes between.

Seems that the #33150 traded one flakiness with another. Attempting to
make sure that the flash/commit is executed before the second run.

Error:

The test RuntimeError: number of values in row (0) differ from number of
column processors (29)

The error is strange however and indicates a bug in sqlite library.

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

Labels

changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants