FWD » Core Development » Database

04/17/2025

12:24 PM Feature #9093 (Closed): add protection for multi-tenant imports to optionally only import tables ...

Greg Shah

09:42 AM Bug #9914: Investigate why ListenerSet implementation is not properly working

Given this situation:... Razvan-Nicolae Chichirau

08:39 AM Bug #9914: Investigate why ListenerSet implementation is not properly working

Razvan-Nicolae Chichirau wrote:
> So, what we need to do is to replace the workaround of just combining all listener... Alexandru Lungu

08:29 AM Bug #9914: Investigate why ListenerSet implementation is not properly working

So, what we need to do is to replace the workaround of just combining all listeners sets and instead use the @Listene... Razvan-Nicolae Chichirau

08:18 AM Bug #9914: Investigate why ListenerSet implementation is not properly working

According to Eric, @ListenerSet@ from @ChangeBroker@ should have worked like a linked list which *references* other d... Alexandru Lungu

08:15 AM Bug #9914 (Test): Investigate why ListenerSet implementation is not properly working

Alexandru Lungu

09:40 AM Bug #9683 (Review): CompoundQuery is doing extra hydration attempts

I addressed the review and fixed a few bugs detected during testing, the code is in 9683a/15727. Artur Școlnic

04/16/2025

03:29 PM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...

That is because #9875 was marked as a subtask of this one, which then blocks the direct setting of the %Done. Greg Shah

03:23 PM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...

I wanted to, but for some unknown reasons to me, I cannot do it. I remember that I wanted to do it, but since *the co... Ovidiu Maxiniuc

02:54 PM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...

Can we set Done to 100%? Greg Shah

04:23 AM Bug #9907: LOOKUP seems to evaluate fast in WHERE clauses from the legacy code, but slow in FWD

This is deferred work to understand the following pattern:... Alexandru Lungu

04:18 AM Bug #9907 (New): LOOKUP seems to evaluate fast in WHERE clauses from the legacy code, but slow in...

Alexandru Lungu

04/15/2025

06:29 PM Feature #9831: add admin API to set c3p0 configuration for individual tenant databases

To upgrade the @landlord@ database without re-creating it from scratch use:... Ovidiu Maxiniuc

06:26 PM Feature #9831 (WIP): add admin API to set c3p0 configuration for individual tenant databases

Rebased branch 9831a and added support for @c3p0.breakAfterAcquireFailure@ from #9736. A couple of fixes. Committed r... Ovidiu Maxiniuc

04/14/2025

07:36 AM Bug #9896: Record reference count should account for dirty records

h4. First problem
Dirty records are not accounted for the reference count, so that a dirty record referenced in tw... Alexandru Lungu

07:34 AM Bug #9896 (New): Record reference count should account for dirty records

Alexandru Lungu

04/09/2025

08:14 AM Feature #9093 (Test): add protection for multi-tenant imports to optionally only import tables th...

Branch 9093a was merged to trunk rev 15848 and archived. Ovidiu Maxiniuc

01:13 AM Feature #9093 (Merge Pending): add protection for multi-tenant imports to optionally only import ...

Please merge 9093a to trunk after 6859a. Eric Faulhaber

08:11 AM Bug #9633: bulk delete for permanent tables

Constantin Asofiei wrote:
> Andrei, shouldn't @loopDelete@ obtain an exclusive-lock on the records?
Right. I'll a... Andrei Iacob

07:07 AM Bug #9633: bulk delete for permanent tables

Andrei, shouldn't @loopDelete@ obtain an exclusive-lock on the records?
Also, the buffer scope is ignored - if a p... Constantin Asofiei

06:48 AM Bug #9633: bulk delete for permanent tables

Ovidiu Maxiniuc wrote:
> There is a single commit (15847) in 9633a.
>
> Good job! The conversion part looks OK to... Andrei Iacob

04/08/2025

07:36 PM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...

Eric Faulhaber wrote:
> I didn't see any recent changes in the branch that were not related to the rebase, so can I ... Ovidiu Maxiniuc

08:46 AM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...

I didn't see any recent changes in the branch that were not related to the rebase, so can I assume that since interna... Eric Faulhaber

01:30 PM Bug #9633: bulk delete for permanent tables

There is a single commit (15847) in 9633a.
Good job! The conversion part looks OK to me. The bulk delete runtime s... Ovidiu Maxiniuc

06:05 AM Bug #9633 (Review): bulk delete for permanent tables

Created task branch *9633a* from trunk rev 15846. The changes are in 15847:
* extended @bulk_delete_table@ processin... Andrei Iacob

06:05 AM Bug #9633 (WIP): bulk delete for permanent tables

Andrei Iacob

04/07/2025

04:21 AM Feature #7020: always use "expanded" extent fields

7020c & 7020d were rebased to trunk revision 15842.
7020d is now at revision 15858. Stefanel Pezamosca

04/05/2025

04:42 PM Feature #9831: add admin API to set c3p0 configuration for individual tenant databases

Created branch 9831a and committed the implementation.
Committed revision 15843. Ovidiu Maxiniuc

11:23 AM Bug #9872: Runtime import statement relies on the ImportWorker to read lob data.

The @PropertyMapper@ is one class which was created only during the import, as an utility of @ImportWorker@. The fact... Ovidiu Maxiniuc

04/04/2025

06:10 PM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...

I rebased 9093a. Pushed up to revision 15848.
I also created #9875 to hold customer specific scripts for importing... Ovidiu Maxiniuc

03:45 PM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...

Indeed, I tried to keep it up-to-date.
However, there were a couple of big updates this wee, so it requires rebasi... Ovidiu Maxiniuc

03:40 PM Feature #9093 (Internal Test): add protection for multi-tenant imports to optionally only import ...

Eric Faulhaber wrote:
> Ovidiu, I think you said last standup that this branch was ready for merge, but it is still ... Eric Faulhaber

03:33 PM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...

Ovidiu, I think you said last standup that this branch was ready for merge, but it is still in review status, with no... Eric Faulhaber

06:14 AM Bug #9872: Runtime import statement relies on the ImportWorker to read lob data.

In @PropertyMapper.readLobData@, @Importworker@ is used to retrieve the path to the lob file. @Importworker@ is not s... Artur Școlnic

06:12 AM Bug #9872 (New): Runtime import statement relies on the ImportWorker to read lob data.

Artur Școlnic

05:00 AM Bug #9853 (Test): ProgressiveResults invalidate badly when changing the record on the edge of a b...

Branch 9853a was merged to trunk rev 15841 and archived. Alexandru Lungu

04:50 AM Bug #9853 (Internal Test): ProgressiveResults invalidate badly when changing the record on the ed...

Alexandru Lungu

04/03/2025

05:04 PM Bug #9853: ProgressiveResults invalidate badly when changing the record on the edge of a bracket

Alexandru Lungu wrote:
> Committed 9853a/15839 and 15840, replacing !isPreselect with errorCondition. Please review.... Constantin Asofiei

08:26 AM Bug #9853: ProgressiveResults invalidate badly when changing the record on the edge of a bracket

Committed 9853a/15839 and 15840, replacing !isPreselect with errorCondition. Please review. Alexandru Lungu

06:51 AM Bug #9853: ProgressiveResults invalidate badly when changing the record on the edge of a bracket

Alexandru Lungu wrote:
> Would you prefer @errorCondition@? I guess it is less confusing than seeing @isPreselect@ a... Constantin Asofiei

06:50 AM Bug #9853: ProgressiveResults invalidate badly when changing the record on the edge of a bracket

Yes, this is what is happening in @next@. Would you prefer @errorCondition@? I guess it is less confusing than seeing... Alexandru Lungu

06:45 AM Bug #9853: ProgressiveResults invalidate badly when changing the record on the edge of a bracket

Can we rely on @errorCondition@ instead of @!isPreselect()@? That's set like this:... Constantin Asofiei

06:32 AM Bug #9853: ProgressiveResults invalidate badly when changing the record on the edge of a bracket

> But, the condition to enter this block is if (cacheOnReval || isPreselect()) - so, can the preselect be false, when... Alexandru Lungu

06:26 AM Bug #9853: ProgressiveResults invalidate badly when changing the record on the edge of a bracket

I don't really understand the fix.
The line @Results cached = cacheResults(old, true);@ where @Results old = adapt... Constantin Asofiei

03:58 AM Bug #9853 (Review): ProgressiveResults invalidate badly when changing the record on the edge of a...

Committed 9853a/rev. 15838.
Constantin, please review. Alexandru Lungu

08:03 AM Feature #7020: always use "expanded" extent fields

Ovidiu Maxiniuc wrote:
> Stefanel,
> I think 7020d is broken at this moment, @RecordMeta@ probably fails to compile... Stefanel Pezamosca

07:57 AM Feature #7020: always use "expanded" extent fields

Ovidiu Maxiniuc wrote:
> Stefanel,
> I think 7020d is broken at this moment, @RecordMeta@ probably fails to compile... Stefanel Pezamosca

07:48 AM Feature #7020: always use "expanded" extent fields

Stefanel,
I think 7020d is broken at this moment, @RecordMeta@ probably fails to compile. The reason is the usage of... Ovidiu Maxiniuc

06:55 AM Bug #9852: Portal "C_10" does not exist error

I think making all FOR EACH queries FORWARD-ONLY was the main cause of this problem. I will revisit this after review... Dănuț Filimon

06:51 AM Bug #9852: Portal "C_10" does not exist error

This seems solved with trunk 15839 (after rollback of #7991). Looks it is related to FORWARD-RESULTS fix. Constantin Asofiei

04/02/2025

11:20 AM Bug #9853 (WIP): ProgressiveResults invalidate badly when changing the record on the edge of a br...

I investigated this case. It is about a fail-safe of @ProgressiveResults@ to invalidate if things look not-alright. B... Alexandru Lungu

05:12 AM Feature #7020: always use "expanded" extent fields

Fixed. Please see *7020d revision 15846*.
I also have rebased 7020c to trunk 15830. Same as 7020d. Stefanel Pezamosca

04/01/2025

12:29 PM Feature #7020: always use "expanded" extent fields

First, a quick look at 7020c (r15824):
* @FqlToSqlConverter.java@
** multiple usages of @appendExpandedName()@: the... Ovidiu Maxiniuc

03/31/2025

10:14 AM Feature #7020: always use "expanded" extent fields

*7020d* was rebased to *trunk revision 15830* and is now at revision *15843*.
In revision *15844* I committed a fi... Stefanel Pezamosca

06:43 AM Bug #9852: Portal "C_10" does not exist error

... Dănuț Filimon

04:12 AM Bug #9852: Portal "C_10" does not exist error

The issue is:... Dănuț Filimon

03:43 AM Bug #9852: Portal "C_10" does not exist error

I was able to reproduce the issue. Dănuț Filimon

03:41 AM Bug #9852: Portal "C_10" does not exist error

Constantin Asofiei wrote:
> Please attach the .df file; what postgresql version do you use?
Just realized I have ... Dănuț Filimon

03:39 AM Bug #9852: Portal "C_10" does not exist error

Please attach the .df file; what postgresql version do you use? Constantin Asofiei

03:39 AM Bug #9852: Portal "C_10" does not exist error

This is the server log of the test, takes more than 5 minutes to run the test. Dănuț Filimon

03:28 AM Bug #9852: Portal "C_10" does not exist error

Dănuț Filimon wrote:
> Alex, I can't reproduce this error with latest trunk/15829.
I've tested now and it fails:
<... Constantin Asofiei

02:55 AM Bug #9852: Portal "C_10" does not exist error

Alex, I can't reproduce this error with latest trunk/15829. Dănuț Filimon

02:28 AM Bug #9852 (WIP): Portal "C_10" does not exist error

Dănuț Filimon

06:18 AM Bug #9763: xml-node-name buffer option

Testing performed with *rev. 15835*:
- unit tests of a large customer application ✅
- harness ✅
- Chui regressio... Eduard Soltan

03/29/2025

01:57 AM Bug #9763: xml-node-name buffer option

Ovidiu Maxiniuc wrote:
> > [...]
> I see @READ-json@, I was expecting a @read-xml@ based on the changes in @XmlIm... Eduard Soltan

03/28/2025

06:14 PM Bug #9763: xml-node-name buffer option

Eduard Soltan wrote:
> I have made the modification in @TempTableBuilder@.
The setters use @_prepared()@ and the ge... Ovidiu Maxiniuc

04:44 PM Bug #9763: xml-node-name buffer option

I have made the modification in @TempTableBuilder@.
But also made some changes in @XmlExport@ and @XmlExport@ to m... Eduard Soltan

04:11 PM Bug #9763: xml-node-name buffer option

Good, that's the idea.
I think the code will work as expected. My observations for @TempTableBuilder.java@, the on... Ovidiu Maxiniuc

03:33 PM Bug #9763: xml-node-name buffer option

Committed on 9763b, rev. 15832.
Sure thing, I added the method in @TempTableBuilder@, and the attributes will be set... Eduard Soltan

03:20 PM Bug #9763: xml-node-name buffer option

Indeed, it does. This is specific to dynamic temp-tables, therefore, this is the proper place to do it.
LE: the im... Ovidiu Maxiniuc

03:03 PM Bug #9763: xml-node-name buffer option

Ovidiu Maxiniuc wrote:
> Not in @AbstractTempTable@, just in @TempTableBuilder@.
But this implies to override all... Eduard Soltan

02:17 PM Bug #9763: xml-node-name buffer option

Not in @AbstractTempTable@, just in @TempTableBuilder@.
LE: and the attributes will be set in @TempTableBuilder.cr... Ovidiu Maxiniuc

02:14 PM Bug #9763: xml-node-name buffer option

Ovidiu Maxiniuc wrote:
> Only in this particular case (@AbstractTempTable._dynamic()@ returns @true@), the attribut... Eduard Soltan

02:09 PM Bug #9763: xml-node-name buffer option

Eduard Soltan wrote:
> I am thinking to initialize the @ctx.tt.tempTablePrepare@ and create the @defaultBuffer@, but... Ovidiu Maxiniuc

02:07 PM Bug #9763: xml-node-name buffer option

The @AbstractTempTable@ has two specialised subclasses: @StaticTempTable@ and @TempTableBuilder@ (for dynamic temp-ta... Ovidiu Maxiniuc

02:06 PM Bug #9763: xml-node-name buffer option

Eduard Soltan wrote:
> The problem is that @TempTableBuilder@ does not initialize the @defaultBuffer@. Hence setting... Eduard Soltan

01:45 PM Bug #9763: xml-node-name buffer option

There is a regression in at @dataset ddt1:read-xml@, when has no buffer set.
In @XmlImport.importDataset@ it go on... Eduard Soltan

12:49 PM Bug #9763: xml-node-name buffer option

Eduard Soltan wrote:
> Ovidiu, why do we use @if (buffer.getSerializeName().toStringMessage().equalsIgnoreCase(name)... Ovidiu Maxiniuc

07:19 AM Bug #9763: xml-node-name buffer option

The unit tests and smoke tests of a large GUI application went fine.✅
Also got some news that unit test of a very ... Eduard Soltan

04:29 AM Bug #9763: xml-node-name buffer option

Testing on another customer application was successful. No more errors encountered nor strange behaviors. ✅ Radu Apetrii

03:10 AM Bug #9763: xml-node-name buffer option

Ovidiu, why do we use @if (buffer.getSerializeName().toStringMessage().equalsIgnoreCase(name)@ in XmlImport.importDat... Eduard Soltan

10:59 AM Bug #9853: ProgressiveResults invalidate badly when changing the record on the edge of a bracket

... Alexandru Lungu

10:54 AM Bug #9853 (Test): ProgressiveResults invalidate badly when changing the record on the edge of a b...

Alexandru Lungu

10:54 AM Bug #9852: Portal "C_10" does not exist error

... Alexandru Lungu

10:53 AM Bug #9852 (WIP): Portal "C_10" does not exist error

Alexandru Lungu

03/27/2025

04:59 PM Bug #9763: xml-node-name buffer option

I tested @read-json@ and @read-xml@, and they are fine.
Also made some tests with datasets (@define dataset dtt1 f... Eduard Soltan

04:44 PM Bug #9763 (Internal Test): xml-node-name buffer option

Eduard Soltan wrote:
> Committed on 9763a, rev. 15796.
A couple of minor observations:
* Please revert @RecordBuff... Ovidiu Maxiniuc

03:00 PM Bug #9763: xml-node-name buffer option

Committed on 9763a, rev. 15796.
I modified the @JsonImport@, @JsonExport@, @XmlExport@ and @XmlImport@, to use just ... Eduard Soltan

12:08 PM Bug #9763: xml-node-name buffer option

Eduard, for some reasons I missed some files in #9763-47 (I think I pressed Ctrl+C in wrong console):
* @JsonExpo... Ovidiu Maxiniuc

11:24 AM Bug #9763: xml-node-name buffer option

Also initialize to defaultBuffer in StaticTempTable before setting up its attributes. Eduard Soltan

08:40 AM Bug #9763: xml-node-name buffer option

Ovidiu Maxiniuc wrote:
> ** lines 2660-2692: these method seem to be duplicates of 2695-2725? The file most likely f... Eduard Soltan

07:41 AM Bug #9763: xml-node-name buffer option

+Review of 9763b/15792+
This is pretty good, but there are several things which must be addressed:
* @AbstractTem... Ovidiu Maxiniuc

03/26/2025

04:26 AM Bug #9763 (Review): xml-node-name buffer option

Constantin Asofiei wrote:
> Eduard, is 9763b ready for review?
Yes, it is ready for review. Eduard Soltan

08:00 PM Feature #9093 (Review): add protection for multi-tenant imports to optionally only import tables ...

I created the new tenant import mode. It is called @strict@ and also added an alias @strict-default@. In this case th... Ovidiu Maxiniuc

03/25/2025

10:32 AM Feature #9831: add admin API to set c3p0 configuration for individual tenant databases

The following c3p0 database connection pool settings, normally set in the directory for a primary database, should be... Eric Faulhaber

10:20 AM Feature #9831 (Closed): add admin API to set c3p0 configuration for individual tenant databases

Eric Faulhaber

09:41 AM Support #4060: investigate converting extent fields to array columns

I wanted to investigate this a little bit. 7020d branch makes it easier to implement this. Most places are already us... Stefanel Pezamosca

09:38 AM Bug #9763: xml-node-name buffer option

Eduard, is 9763b ready for review? Constantin Asofiei

06:29 AM Feature #7020: always use "expanded" extent fields

So, to summarize: *7020d* is based on *trunk 15813*. *7020d* has all the changes from 7020c but with some other impro... Stefanel Pezamosca

03/24/2025

11:06 AM Feature #7020: always use "expanded" extent fields

7020d was rebased to trunk revision 15813.
7020d contains now all the changes from 7020c + some improvements. There ... Stefanel Pezamosca