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.

Sign up for GitHub

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

Jump to bottom

H2 upgrade to v2.1.214 #3447

Draft
wants to merge 39 commits into
base: master
Choose a base branch
from
Draft

H2 upgrade to v2.1.214 #3447

wants to merge 39 commits into from

Conversation

Paarth002
Copy link
Contributor

H2 upgrade to v2.1.214

Paarth002 and others added 30 commits January 7, 2025 05:56
@Paarth002
delete h2 1.4.2 support
c2638bd
@Paarth002
trim char column for h2 new version
357d24b
@Paarth002
Revert "trim char column for h2 new version"
9d66e7c
@Paarth002
overwrite H2 files - allow comparison of boolean and
c655800 
 varchars/numerics - dont do padding in char columns in legacy mode
@Paarth002
skip checkstyle for h2 hacked files
aabf568
@Paarth002
add over to h2 non keywords
3f00014
@Paarth002
legend-engine version change to PARTH-4.67.12-SNAPSHOT
7724a8b
@Paarth002
H2 upgrade - fix parseDateTime
92e6fdc
@gs-saurabh
Adds float tolerance in json assert
35e0e24
@gs-saurabh
add precision in cast as decimal of h2-v2 extension
4b5ea86
@Paarth002
H2 upgrade - update extension for fixing parseDateTime, boolean vs st…
829cc12 
…ring, alias limit
@Paarth002
legend-engine version change to 4.68.1-SNAPSHOT
bfa9acc
@Paarth002
Merge remote-tracking branch 'origin/finos-master' into h2-local
df5dafe 
# Conflicts:
#	legend-engine-xts-relationalStore/legend-engine-xt-relationalStore-execution/legend-engine-xt-relationalStore-h2-1.4.200-execution/pom.xml
#	pom.xml
@Paarth002
legend-pure version change to 5.31.1-SNAPSHOT
c274333
@Paarth002
Revert "add over to h2 non keywords"
6fb42af 
This reverts commit 3f00014.
@Paarth002
Revert "skip checkstyle for h2 hacked files"
a95117b 
This reverts commit aabf568.
@Paarth002
Revert "overwrite H2 files - allow comparison of boolean and varchars…
eb92e15 
…/numerics - dont do padding in char columns in legacy mode"

This reverts commit c655800
@Paarth002
H2 upgrade - Delete H2 pom
29b2100
@Paarth002
Revert "H2 upgrade - Delete H2 pom"
d3b0759 
This reverts commit 29b2100.
@Paarth002
Revert "Revert "overwrite H2 files - allow comparison of boolean and …
a581c7c 
…varchars/numerics - dont do padding in char columns in legacy mode""

This reverts commit eb92e15.
@Paarth002
Revert "Revert "skip checkstyle for h2 hacked files""
9818d2c 
This reverts commit a95117b.
@Paarth002
Revert "Revert "add over to h2 non keywords""
2c27779 
This reverts commit 6fb42af.
@gs-saurabh
Apply alias limit for H2
60f4da9
@gs-saurabh
Merge branch 'refs/heads/finos-master' into h2Upgrade
274a440 
# Conflicts:
#	legend-engine-xts-relationalStore/legend-engine-xt-relationalStore-execution/legend-engine-xt-relationalStore-h2-1.4.200-execution/pom.xml
#	pom.xml
@gs-saurabh
updates legend pure version
c37d9a9
@gs-saurabh
updates engine version
6b98fce
@gs-saurabh
Merge branch 'refs/heads/finos-master' into h2Upgrade
a7f2397 
# Conflicts:
#	legend-engine-xts-relationalStore/legend-engine-xt-relationalStore-execution/legend-engine-xt-relationalStore-h2-1.4.200-execution/pom.xml
#	pom.xml
@gs-saurabh
updates engine version for h2 v2
aef0783
@gs-saurabh
Merge branch 'refs/heads/finos-master' into h2Upgrade
999bfdf 
# Conflicts:
#	legend-engine-xts-relationalStore/legend-engine-xt-relationalStore-execution/legend-engine-xt-relationalStore-h2-1.4.200-execution/pom.xml
#	pom.xml
@gs-saurabh
updates engine version for h2 v2
476ee23
gs-saurabh and others added 4 commits February 18, 2025 10:47
@gs-saurabh
reset legend pure version
6067337
@Paarth002
Merge remote-tracking branch 'origin/finos-master' into review-h2-upg…
3ffff13 
…rade

# Conflicts:
#	legend-engine-xts-relationalStore/legend-engine-xt-relationalStore-execution/legend-engine-xt-relationalStore-h2-1.4.200-execution/pom.xml
@Paarth002
Merge remote-tracking branch 'origin/finos-master' into review-h2-upg…
4ba710e 
…rade
@Paarth002
Merge pull request #5 from goldmansachs/review-h2-upgrade-new
9995a83 
Upgrade H2 to v2.1.214
@Paarth002 Paarth002 requested a review from a team as a code owner March 4, 2025 22:17
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025 •
edited
Loading

Test Results

0 files   -   2  0 suites   - 2   0s ⏱️ -21s
0 tests  - 38  0 ✔️  - 38  0 💤 ±0  0 ❌ ±0 
0 runs   - 76  0 ✔️  - 76  0 💤 ±0  0 ❌ ±0 

Results for commit 03c15c1. ± Comparison against base commit fc69bb5.

This pull request removes 38 tests. 
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - abs(0/2)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - abs(1/2)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - abs(2/2)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - cbrt(0/2)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - cbrt(1/2)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - cbrt(2/2)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - ceil(0/3)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - ceil(1/3)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - ceil(2/3)
org.finos.legend.engine.relational.test.sdt.SdtTestSuiteBuilder$SDTTestCase ‑ FuncTests - ceil(3/3)
…

♻️ This comment has been updated with latest results.

@aziemchawdhary-gs aziemchawdhary-gs marked this pull request as draft March 7, 2025 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Paarth002 @gs-saurabh