Skip to content

Conversation

dankristensen
Copy link

@dankristensen dankristensen commented Sep 25, 2025

DB2zDialect did not support proper DB2 sequences on zOS

This change ensures that it is possible to use sequences even if some of these are aliases.
According to the documentation here: https://www.ibm.com/docs/en/db2-for-zos/12.0.0?topic=spaces-syssequences, the seqtype is important for parsing the sequences correctly for an alias.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


https://hibernate.atlassian.net/browse/HHH-99999
https://hibernate.atlassian.net/browse/HHH-19752

@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Sep 25, 2025

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HHH-\d+
    ↳ Offending commits: [f497638, f5852f5, c01196f, baca5ad, abc9350, ef412c5, f4f67c9, 23e6533, bb9cfb1, 6e13b77, ace7664]

› This message was automatically generated.

@beikov
Copy link
Member

beikov commented Sep 26, 2025

Thanks for your contribution. Once you applied the changes I asked for, please squash to a single commit including the HHH Jira issue key in the commit message and rebase the PR.

@dankristensen dankristensen changed the title Db2z dialect sequences fixed HHH-19824: Db2z dialect sequences fixed Sep 29, 2025
@beikov
Copy link
Member

beikov commented Sep 29, 2025

Nice, now please squash to a single commit and rebase the PR.

Dan Kristensen and others added 10 commits September 29, 2025 17:36
…ke sequence alias' into consideration when reading these
This fixes issues with ScopeTests when running against CockroachDB.
Also remove test skips that were due to locking not working correctly.
See cockroachdb/cockroach#88995 for details
…ystem properties but also configuration properties
…ristensen/hibernate-orm into DB2zDialect_sequences_fixed

HHH-19824: Fixed sequences on DB2 zOS
@dankristensen
Copy link
Author

I think i did it in a bad way. How do i fix this @beikov

@beikov
Copy link
Member

beikov commented Sep 30, 2025

Read an article about Git rebase: https://git-scm.com/book/en/v2/Git-Branching-Rebasing

The steps I usually do are:

git fetch upstream

To fetch the changes from the upstream repository in my local mirror.

git rebase upstream/main

To do the actual rebase. If there are conflicts, you have to resolve them and then get rebase --continue once the conflicts are resolved.

git push -f

Finally, push the changes to your remote repository with -f (force-push), since you rewrite the history of the remote repository.

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

Successfully merging this pull request may close these issues.

5 participants