You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by "dependabot[bot] (via GitHub)" <gi...@apache.org> on 2023/03/16 05:09:05 UTC

[GitHub] [beam] dependabot[bot] opened a new pull request, #25855: Update sqlalchemy requirement from <2.0,>=1.3 to >=1.3,<3.0 in /sdks/python

dependabot[bot] opened a new pull request, #25855:
URL: https://github.com/apache/beam/pull/25855

   Updates the requirements on [sqlalchemy](https://github.com/sqlalchemy/sqlalchemy) to permit the latest version.
   <details>
   <summary>Release notes</summary>
   <p><em>Sourced from <a href="https://github.com/sqlalchemy/sqlalchemy/releases">sqlalchemy's releases</a>.</em></p>
   <blockquote>
   <h1>2.0.6</h1>
   <p>Released: March 13, 2023</p>
   <h2>orm</h2>
   <ul>
   <li>
   <p><strong>[orm] [bug]</strong> Fixed bug where the &quot;active history&quot; feature was not fully
   implemented for composite attributes, making it impossible to receive
   events that included the &quot;old&quot; value.   This seems to have been the case
   with older SQLAlchemy versions as well, where &quot;active_history&quot; would
   be propagated to the underlying column-based attributes, but an event
   handler listening to the composite attribute itself would not be given
   the &quot;old&quot; value being replaced, even if the composite() were set up
   with active_history=True.</p>
   <p>Additionally, fixed a regression that's local to 2.0 which disallowed
   active_history on composite from being assigned to the impl with
   <code>attr.impl.active_history=True</code>.</p>
   <p>References: <a href="https://www.sqlalchemy.org/trac/ticket/9460">#9460</a></p>
   </li>
   <li>
   <p><strong>[orm] [bug]</strong> Fixed regression involving pickling of Python rows between the cython and
   pure Python implementations of <code>Row</code>, which occurred as part of
   refactoring code for version 2.0 with typing. A particular constant were
   turned into a string based <code>Enum</code> for the pure Python version of
   <code>Row</code> whereas the cython version continued to use an integer
   constant, leading to deserialization failures.</p>
   <p>References: <a href="https://www.sqlalchemy.org/trac/ticket/9418">#9418</a></p>
   </li>
   </ul>
   <h2>sql</h2>
   <ul>
   <li>
   <p><strong>[sql] [bug] [regression]</strong> Fixed regression where the fix for <a href="https://www.sqlalchemy.org/trac/ticket/8098">#8098</a>, which was released in
   the 1.4 series and provided a layer of concurrency-safe checks for the
   lambda SQL API, included additional fixes in the patch that failed to be
   applied to the main branch. These additional fixes have been applied.</p>
   <p>References: <a href="https://www.sqlalchemy.org/trac/ticket/9461">#9461</a></p>
   </li>
   <li>
   <p><strong>[sql] [bug]</strong> Fixed regression where the <code>_sql.select()</code> construct would not be able
   to render if it were given no columns and then used in the context of an
   EXISTS, raising an internal exception instead. While an empty &quot;SELECT&quot; is
   not typically valid SQL, in the context of EXISTS databases such as
   PostgreSQL allow it, and in any case the condition now no longer raises
   an internal exception.</p>
   <p>References: <a href="https://www.sqlalchemy.org/trac/ticket/9440">#9440</a></p>
   </li>
   </ul>
   <!-- raw HTML omitted -->
   </blockquote>
   <p>... (truncated)</p>
   </details>
   <details>
   <summary>Commits</summary>
   <ul>
   <li>See full diff in <a href="https://github.com/sqlalchemy/sqlalchemy/commits">compare view</a></li>
   </ul>
   </details>
   <br />
   
   
   Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   <details>
   <summary>Dependabot commands and options</summary>
   <br />
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
   - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
   
   
   </details>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [beam] jrmccluskey commented on pull request #25855: Update sqlalchemy requirement from <2.0,>=1.3 to >=1.3,<3.0 in /sdks/python

Posted by "jrmccluskey (via GitHub)" <gi...@apache.org>.
jrmccluskey commented on PR #25855:
URL: https://github.com/apache/beam/pull/25855#issuecomment-1472124850

   Run Python PreCommit


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [beam] github-actions[bot] commented on pull request #25855: Update sqlalchemy requirement from <2.0,>=1.3 to >=1.3,<3.0 in /sdks/python

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25855:
URL: https://github.com/apache/beam/pull/25855#issuecomment-1471373716

   Assigning reviewers. If you would like to opt out of this review, comment `assign to next reviewer`:
   
   R: @jrmccluskey for label python.
   
   Available commands:
   - `stop reviewer notifications` - opt out of the automated review tooling
   - `remind me after tests pass` - tag the comment author after tests pass
   - `waiting on author` - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)
   
   The PR bot will only process comments in the main thread (not review comments).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [beam] Abacn commented on pull request #25855: Update sqlalchemy requirement from <2.0,>=1.3 to >=1.3,<3.0 in /sdks/python

Posted by "Abacn (via GitHub)" <gi...@apache.org>.
Abacn commented on PR #25855:
URL: https://github.com/apache/beam/pull/25855#issuecomment-1472889185

   This breaks python postcommit: https://ci-beam.apache.org/view/PostCommit/job/beam_PostCommit_Python39/lastCompletedBuild/testReport/


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [beam] jrmccluskey merged pull request #25855: Update sqlalchemy requirement from <2.0,>=1.3 to >=1.3,<3.0 in /sdks/python

Posted by "jrmccluskey (via GitHub)" <gi...@apache.org>.
jrmccluskey merged PR #25855:
URL: https://github.com/apache/beam/pull/25855


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [beam] codecov[bot] commented on pull request #25855: Update sqlalchemy requirement from <2.0,>=1.3 to >=1.3,<3.0 in /sdks/python

Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #25855:
URL: https://github.com/apache/beam/pull/25855#issuecomment-1471338866

   ## [Codecov](https://codecov.io/gh/apache/beam/pull/25855?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#25855](https://codecov.io/gh/apache/beam/pull/25855?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (9bbdd2a) into [master](https://codecov.io/gh/apache/beam/commit/14ca840f3462aa9ba3ebbe773b498d4a914d50aa?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (14ca840) will **decrease** coverage by `0.01%`.
   > The diff coverage is `n/a`.
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #25855      +/-   ##
   ==========================================
   - Coverage   71.45%   71.44%   -0.01%     
   ==========================================
     Files         777      777              
     Lines      102275   102275              
   ==========================================
   - Hits        73083    73075       -8     
   - Misses      27737    27745       +8     
     Partials     1455     1455              
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | python | `80.01% <ø> (-0.02%)` | :arrow_down: |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   [see 6 files with indirect coverage changes](https://codecov.io/gh/apache/beam/pull/25855/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org