Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Omid 1.1.1 #553

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Add Omid 1.1.1 #553

wants to merge 7 commits into from

Conversation

soenkeliebau
Copy link
Member

Description

Adds Omid version 1.1.1.
Adds upload script for omid src archives.

Changelog

Release 1.1.1

IMPROVEMENTS:

JIRA Summary Priority Component
OMID-275 Expose backing HBase Table from TTable Major .
OMID-249 Improve default network address logic Major .
OMID-272 Support JDK17 Major .
OMID-258 Bump maven plugins/dependencies to latest Major .
OMID-264 Fix deprecated WARNING in check-license stage Minor .

BUG FIXES:

JIRA Summary Priority Component
OMID-250 Remove duplicate declarations of hadoop-hdfs-client dependency in pom.xml Trivial .
OMID-240 Transactional visibility is broken Critical .
OMID-248 Transactional Phoenix tests fail on Java 17 in getDefaultNetworkInterface Major .
OMID-237 TestHBaseTransactionClient.testReadCommitTimestampFromCommitTable fails Major .
OMID-247 Change TSO default port to be outside the ephemeral range Critical .
OMID-246 Update Surefire plugin to 3.0.0 and switch to TCP forkNode implementation Major .
OMID-236 Upgrade Netty to 4.1.86.Final Major .

SUB-TASKS:

JIRA Summary Priority Component
OMID-254 Upgrade to phoenix-thirdparty 2.1.0 Major .
OMID-256 Bump hbase and other dependencies to latest version Major .
OMID-253 Upgrade Netty to 4.1.100.Final Major .
OMID-255 Upgrade guava to 32.1.3-jre Major .
OMID-257 Upgrade bouncycastle and move from jdk15on to latest jdk18on Major .

OTHER:

JIRA Summary Priority Component
OMID-251 Bump license-maven-plugin to latest version Major .
OMID-245 Add dependency management for Guava to use 32.1.1 Major .
OMID-244 Upgrade SnakeYaml version to 2.0 Major .
OMID-242 Bump guice version to 5.1.0 to support JDK 17 Major .
OMID-241 Add logging to TSO server crash Major .
OMID-239 OMID TLS support Major .
OMID-234 Bump SnakeYaml version to 1.33 Major .

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes

Tasks

Edit tasklist title
Beta Give feedback Tasklist Tasks, more options

Delete tasklist

Delete tasklist block?
Are you sure? All relationships in this tasklist will be removed.
  1. Changes are OpenShift compatible
    Options
  2. All added packages (via microdnf or otherwise) have a comment on why they are added
    Options
  3. Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
    Options
  4. All packages should have (if available) signatures/hashes verified
    Options
  5. Does your change affect an SBOM? Make sure to update all SBOMs
    Options

@lfrancke
Copy link
Member

lfrancke commented Feb 9, 2024

Can this cause problems again: https://github.com/apache/phoenix-omid/pull/154/files#diff-6c4b5f133b92f9439af086250e44f2d31e3c8f67c949b2f0f9f06d40dcd8858cR144-R161

I don't have any evidence that it will but I'd feel better if someone else looks at it again.
Want to avoid another forward/reverse lookup DNS confusion issue.

@soenkeliebau
Copy link
Member Author

It might cause issues ... not sure tbh.

Not because of a reverse lookup snafu or tls stuff I think though. That code as far as I can tell determines the address that tso thinks it should be reachable at, and that address is then written into ZK for Omid clients to use when disovering their TSO server.
If this is resolved to something other cannot reach ..

I've put this down for now .. will try and test it over the weekend.

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.

None yet

2 participants