Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

rich package version conflict with Airflow2.3.3 #208

Closed
tripathiabhay09 opened this issue Aug 17, 2022 · 6 comments
Closed

rich package version conflict with Airflow2.3.3 #208

tripathiabhay09 opened this issue Aug 17, 2022 · 6 comments

Comments

@tripathiabhay09
Copy link

Describe the bug
I am trying to use data-diff in airflow dags. While installing it on Airflow server, I ran into the following conflict:

apache-airflow 2.3.3 requires rich>=12.4.4, but you have rich 10.16.2 which is incompatible.
So, I tried changing the version with poetry inside pyproject.toml file. This step resulted in the following issue:

Resolving dependencies... (4.3s)

  SolverProblemError

  Because preql (0.2.17) depends on rich (>=10.7.0,<11.0.0)
   and no versions of preql match >0.2.17,<0.3.0, preql (>=0.2.17,<0.3.0) requires rich (>=10.7.0,<11.0.0).
  So, because data-diff depends on both rich (12.5.1) and preql (^0.2.17), version solving failed.**

Make sure to include the following (minus sensitive information):

  • poetry add "rich12.5.1"
  • traceback
-Updating dependencies
Resolving dependencies... (4.3s)

  SolverProblemError

  Because preql (0.2.17) depends on rich (>=10.7.0,<11.0.0)
   and no versions of preql match >0.2.17,<0.3.0, preql (>=0.2.17,<0.3.0) requires rich (>=10.7.0,<11.0.0).
  So, because data-diff depends on both rich (12.5.1) and preql (^0.2.17), version solving failed.

  at /opt/homebrew/lib/python3.9/site-packages/poetry/puzzle/solver.py:241 in _solve
      237│             packages = result.packages
      238│         except OverrideNeeded as e:
      239│             return self.solve_in_compatibility_mode(e.overrides, use_latest=use_latest)
      240│         except SolveFailure as e:
    → 241│             raise SolverProblemError(e)
      242│ 
      243│         results = dict(
      244│             depth_first_search(
      245│                 PackageNode(self._package, packages), aggregate_package_nodes

If possible, please paste these as text, and not a screenshot.

Describe the environment

I am using macos monterey12.3 with latest version of data-diff.
Screen Shot 2022-08-17 at 11 21 17 am
Screen Shot 2022-08-17 at 11 22 17 am

@erezsh
Copy link
Contributor

erezsh commented Aug 17, 2022

Thanks for letting us know. We will fix it soon.

Meanwhile, just FYI, you don't need preql to run data-diff, it's only for initially seeding the databases with data, for manual tests.

@tripathiabhay09
Copy link
Author

Yes, I noticed preql is only for initial setup with data. Can you please advise what is the approx eta for this issues.Currently, we are building by ignoring this conflict completely. So, just wanted to have an idea of time line so that we can wait until we deploy our changes.
Thank you so much!

@erezsh
Copy link
Contributor

erezsh commented Aug 18, 2022

I will probably merge the fix to master today or tomorrow, and we might make a new release on Monday,

@tripathiabhay09
Copy link
Author

Thank you so much!

erezsh added a commit that referenced this issue Aug 18, 2022
erezsh added a commit that referenced this issue Aug 18, 2022
@erezsh
Copy link
Contributor

erezsh commented Aug 18, 2022

The fix has been merged to master with #212 .

@tripathiabhay09
Copy link
Author

Thank you!

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

No branches or pull requests

2 participants