Activity
From 04/03/2025 to 05/02/2025
04/30/2025
- 02:15 PM Feature #9831: add admin API to set c3p0 configuration for individual tenant databases
- Eric, please review.
I will rebase the branch since it is a bit behind the main trunk. - 02:33 AM Bug #9633: bulk delete for permanent tables
- Rebased *9633a* with trunk revision *15878*, new revision is *15880*.
04/28/2025
- 04:51 PM Bug #9950: Upgrade the metadata schema to a newer version
- One of the latest customer projects uses a newer schema (@standard.df@) of the _meta database than the one we have. I...
- 04:36 PM Bug #9950 (Test): Upgrade the metadata schema to a newer version
- 07:32 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- > Well, if the query is static, the scoping from #9914-14 would be correct! But the issue will still reproduce (?).
...
04/25/2025
- 10:33 AM Bug #9907: LOOKUP seems to evaluate fast in WHERE clauses from the legacy code, but slow in FWD
- I agree with #9907-3. But I am more interested to find out how 4GL is able to make this work fast. INDEXED-REPOSITION...
- 10:22 AM Bug #9907: LOOKUP seems to evaluate fast in WHERE clauses from the legacy code, but slow in FWD
- I've seen this task an I thought of something. This:...
- 05:42 AM Bug #8044: Fix "Failed direct-access due to Error direct accessing for finding record by unique i...
- Tomasz Domin wrote:
> Just to notify - I am still getting a lot of "Failed direct-access due to Error direct accessi...
04/24/2025
- 08:19 AM Bug #9937: buffer argument with different DMO than the parameter
- I've tested @DYNAMIC-INVOKE(.., BUFFER tt1)@ and runtime gets this @BUFFER parameter type not allowed for dynamic par...
- 08:10 AM Bug #9937: buffer argument with different DMO than the parameter
- In FWD, we create different DMO interfaces even if the schema matches, but the resource options, like @XML-NODE-NAME@...
- 07:55 AM Bug #9937 (Test): buffer argument with different DMO than the parameter
- 06:52 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- > No matter how many nested RUN statements we have in this testcase, we always end up trying to register the query at...
- 06:09 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- Experimented with @RUN custom.p@ from a @start.p@. Right before we should to add the query as a global listener, this...
- 03:39 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- > Anyway, it would be defined globally.
I am not sure that "globally" is right. It is within the scope of the exte... - 03:38 AM Bug #7364: Build a full test-suite for fast copy-temp-table
- I extended the tests existent at @testcases/tests/table/TestCopyTempTable.cls@ as part of the work on #9397. These ar...
- 03:15 AM Bug #7364: Build a full test-suite for fast copy-temp-table
- Ioana, did you built a test-suite in your recent work for COPY-TEMP-TABLE construct? If so, is it based on plain proc...
- 03:08 AM Bug #7403 (Closed): Copy-temp-table with replace-mode should replace already inserted records
- 02:48 AM Bug #7403: Copy-temp-table with replace-mode should replace already inserted records
- Yes, thanks!
04/23/2025
- 03:53 PM Feature #9831: add admin API to set c3p0 configuration for individual tenant databases
- The default value for all of these is @null@ which means they will default to the c3p0 value of the default database....
- 03:22 PM Feature #9831: add admin API to set c3p0 configuration for individual tenant databases
- Isn't @c3p0.break_on_fail@ always supposed to be @true@? I don't think we want customers to configure that.
- 03:00 PM Feature #9831 (Review): add admin API to set c3p0 configuration for individual tenant databases
- The implementation is finished. I added the support even for @c3p0.break_on_fail@ which was recently added.
I've jus... - 02:11 PM Feature #9831: add admin API to set c3p0 configuration for individual tenant databases
- Ovidiu, what is the status of this task? I did not do a full review, but I took a look at 9831a and the implementatio...
- 11:04 AM Bug #9397 (Closed): Fix COPY-TEMP-TABLE issues
- This can be closed.
- 11:04 AM Bug #7403: Copy-temp-table with replace-mode should replace already inserted records
- > #7403 is fixed by 9397a - FWD behavior aligns with progress with trunk rev.15627.
Ioana, does this mean that #74... - 11:00 AM Feature #7960 (Closed): Allow query logs to be collected in a H2 database
- This can be closed now as the query logs seem to be accumulated well in H2.
- 07:27 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- After converting @start.p@ from #9914-6:...
- 07:18 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- I can't tell in #9914-5 where the buffer is defined.
- 06:53 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- > It looks like you are registering the query at the first nested scope and the buffer at the second nested scope.
... - 04:16 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- > So, I don't think the problem comes from the unlinking of PreselectQuery after popping the procedure local scope, b...
- 02:19 AM Bug #9914 (Review): Investigate why ListenerSet implementation is not properly working
- 02:19 AM Bug #9914 (WIP): Investigate why ListenerSet implementation is not properly working
- Commited to *9914a/rev. 15868*.
Alex: Please review.
04/22/2025
- 05:51 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- Attached the original testcase.
There are 2 important phases while opening the query in @createWidgets@:
# Initia... - 02:36 AM Bug #9683: CompoundQuery is doing extra hydration attempts
- Addressed a few formatting issues and rebased on trunk rev 15866.
04/17/2025
- 12:24 PM Feature #9093 (Closed): add protection for multi-tenant imports to optionally only import tables ...
- 09:42 AM Bug #9914: Investigate why ListenerSet implementation is not properly working
- Given this situation:...
- 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... - 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...
- 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...
- 08:15 AM Bug #9914 (Test): Investigate why ListenerSet implementation is not properly working
- 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.
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.
- 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...
- 02:54 PM Feature #9093: add protection for multi-tenant imports to optionally only import tables that are ...
- Can we set Done to 100%?
- 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:...
- 04:18 AM Bug #9907 (New): LOOKUP seems to evaluate fast in WHERE clauses from the legacy code, but slow in...
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:...
- 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...
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... - 07:34 AM Bug #9896 (New): Record reference count should account for dirty records
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.
- 01:13 AM Feature #9093 (Merge Pending): add protection for multi-tenant imports to optionally only import ...
- Please merge 9093a to trunk after 6859a.
- 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... - 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... - 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...
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 ... - 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...
- 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... - 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... - 06:05 AM Bug #9633 (WIP): bulk delete for permanent tables
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.
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. - 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...
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... - 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... - 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 ... - 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...
- 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...
- 06:12 AM Bug #9872 (New): Runtime import statement relies on the ImportWorker to read lob data.
- 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.- 04:50 AM Bug #9853 (Internal Test): ProgressiveResults invalidate badly when changing the record on the ed...
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.... - 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.
- 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... - 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...
- 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:...
- 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...
- 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... - 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. - 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... - 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... - 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... - 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...
- 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.
Also available in: Atom