Both HSQL and Firebird define DECIMAL data types with user supplied precision setting.
HSQL supports a larger precision setting than does Firebird.
Firebird limit is 18 and HSQL supports values up to 100.
The attached example with HSQL decimal(20, 2) [precision 20, 2 decimal places]) and a range of values displayed as Euros.
Steps to Reproduce:
1. download the attached file and make a copy of it
2. open the copy and click on tables, say yes to start migration assistant
3. when it finishes open the data table
4. open the original file you download, open the same data table
the values do not match in every row
check out the attached screen shot for highlighted rows.
the values stored properly in a decimal(18,2) column in the new schema.
User Profile Reset: No
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/66.0.3359.181 Chrome/66.0.3359.181 Safari/537.36
Created attachment 142574 [details]
file to test migration
hsql file with three columns defined as decimal(20,0) and data
Created attachment 142575 [details]
screen shot comparing before and after migration
Table on the left is in HSQL file
Table on the right is in Firebird after migration.
Rows with data mismatch are highlighted.
Build ID: 8b96445766efe237eb47608ade6c147673466e2e
CPU threads: 4; OS: Linux 4.13; UI render: default; VCL: gtk3;
Locale: ca-ES (ca_ES.UTF-8); Calc: group threaded
Thank you very much for investigating this problem and creating a testcase!!
Created attachment 142576 [details]
new test file
This isn't just where the precision in the column definition is greater than 18.
I updated the test file with a second table, decimal(10,2) and get some of the same data changes.
Tamas Bunth committed a patch related to this issue.
It has been pushed to "master":
tdf#118043 dbahsql: Fix putDot funciton
It will be available in 6.2.0.
The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
Affected users are encouraged to test the fix and report feedback.
Checked this with Version:
Build ID: cbd0d1da85a7ae6b35023274c7267872b3d920e7
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk2;
TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time: 2018-06-18_00:24:33
Locale: en-US (en_US.UTF-8); Calc: group threaded
That does include the patch.
The problem is unchanged when trying to migrate the test file.
set the component type to Base vs Basic so this shows up in QA team canned searchs
(In reply to Drew Jensen from comment #6)
> Checked this with Version:
> Build ID: cbd0d1da85a7ae6b35023274c7267872b3d920e7
> CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk2;
> TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time:
> Locale: en-US (en_US.UTF-8); Calc: group threaded
> That does include the patch.
> The problem is unchanged when trying to migrate the test file.
I believe, it's the same root cause as bug 126268
(In reply to Xisco Faulí from comment #8)
> (In reply to Drew Jensen from comment #6)
> > Checked this with Version:
> > 22.214.171.124.alpha0+
> > Build ID: cbd0d1da85a7ae6b35023274c7267872b3d920e7
> > CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk2;
> > TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time:
> > 2018-06-18_00:24:33
> > Locale: en-US (en_US.UTF-8); Calc: group threaded
> > That does include the patch.
> > The problem is unchanged when trying to migrate the test file.
> I believe, it's the same root cause as bug 126268
Yes, it's, I have a patch in gerrit already.
Since this issue has a commit, let's close it as RESOLVED FIXED