FWD » Core Development » Base Language

06/03/2024

08:12 AM Feature #6417: cleanup some known items in appserver support

I find it very weird that so many tests were failing in OE, but not fixed, instead directly ported to testcases and s... Galya B

03:13 AM Feature #4347: add runtime support for STOP-AFTER block option

Review for 4347a rev 15206:
* StandardServer
** only change is a 'bad import', revert this file to rev 15203.
* Bl... Constantin Asofiei

05/30/2024

08:46 AM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

Marian Edu wrote:
> Marian Edu wrote:
> > Some of them are just caused by the conversion when unknown (?) is passed... Greg Shah

08:45 AM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

Artur: Please make all of these testcases work properly, including the conversion fixes. Constantin, Marian and I wi... Greg Shah

04:35 AM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

Marian Edu wrote:
> Some of them are just caused by the conversion when unknown (?) is passed directly as parameter ... Marian Edu

04:22 AM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

Marian Edu wrote:
> @undoReturnErrorTopLevel@ doesn't have an override using @ProError@.
This is for the @return ... Marian Edu

03:47 AM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

Greg Shah wrote:
> > I do not think we have tests for default error handling for all blocks but we can maybe add tho... Marian Edu

08:19 AM Feature #4347: add runtime support for STOP-AFTER block option

Constantin Asofiei wrote:
> * I think if we need to have a way to disable @STOP-AFTER@, via a directory configuratio... Eduard Soltan

05/29/2024

02:18 PM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

> I do not think we have tests for default error handling for all blocks but we can maybe add those to somewhere if n... Greg Shah

04:16 AM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

Greg Shah wrote:
> Marian: Where are we with testcases for @ON ERROR UNDO, THROW@?
We have tests for error handling... Marian Edu

05/28/2024

02:15 PM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

Marian: Where are we with testcases for @ON ERROR UNDO, THROW@?
Constantin: Do we have a standalone testcase to recr... Greg Shah

05/24/2024

01:47 AM Feature #4347: add runtime support for STOP-AFTER block option

Greg Shah wrote:
>From my point of view yes. The inner blocks with on stop undo, ... does not respond to StopConditi... Eduard Soltan

05/23/2024

10:58 AM Feature #4347: add runtime support for STOP-AFTER block option

>> Is @STOP-AFTER@ essentially an @ON STOP UNDO, LEAVE@ but only matching a timeout and not other kinds of @STOP@?
>... Greg Shah

10:33 AM Feature #4347: add runtime support for STOP-AFTER block option

Greg Shah wrote:
> * 4GL syntax
> ** "time limit" value
> *** Test both literals and "complex" expressions (a vari... Eduard Soltan

09:35 AM Feature #4347: add runtime support for STOP-AFTER block option

> > Here a StopCondition is triggered on @outerBlock1@, however because the nested @innerBlock1@ which has a @on stop... Greg Shah

09:34 AM Feature #4347: add runtime support for STOP-AFTER block option

Constantin Asofiei wrote:
> Greg Shah wrote:
> > > I think there is some behavior related to blocking UI statements... Greg Shah

09:26 AM Feature #4347: add runtime support for STOP-AFTER block option

Eduard, Please do the following:
* In the testcase, add elapsed timers around each of the cases where you are expe... Greg Shah

09:20 AM Feature #4347: add runtime support for STOP-AFTER block option

Greg Shah wrote:
> > I think there is some behavior related to blocking UI statements (like UDPATE, or PAUSE): if th... Constantin Asofiei

09:16 AM Feature #4347: add runtime support for STOP-AFTER block option

> I think there is some behavior related to blocking UI statements (like UDPATE, or PAUSE): if these exist, then the ... Greg Shah

02:43 AM Feature #4347: add runtime support for STOP-AFTER block option

I think there is some behavior related to blocking UI statements (like UDPATE, or PAUSE): if these exist, then the ST... Constantin Asofiei

02:39 AM Feature #4347: add runtime support for STOP-AFTER block option

Constantin Asofiei wrote:
> Marian Edu wrote:
> > That one works as expected for me (OE12.2), what you can do so it... Marian Edu

02:33 AM Feature #4347: add runtime support for STOP-AFTER block option

Marian Edu wrote:
> That one works as expected for me (OE12.2), what you can do so it doesn't look like it's mining ... Constantin Asofiei

02:28 AM Feature #4347: add runtime support for STOP-AFTER block option

Eduard Soltan wrote:
> Here a StopCondition is triggered on @outerBlock1@, however because the nested @innerBlock1... Marian Edu

05/22/2024

06:04 PM Feature #4347: add runtime support for STOP-AFTER block option

Eduard Soltan wrote:
> Committed on 4347a, revision 15205.
I understand from today's meeting that this commit add... Eric Faulhaber

02:48 AM Feature #4347: add runtime support for STOP-AFTER block option

Committed on 4347a, revision 15205.
Created a suite of test cases.
Also find some scenarios which behaviours in 4... Eduard Soltan

11:01 AM Feature #6417: cleanup some known items in appserver support

Galya B wrote:
> May I get a confirmation that @webtests_from_logs/results@ are results from Progress?
Yes, this is... Constantin Asofiei

08:44 AM Feature #6417: cleanup some known items in appserver support

May I get a confirmation that @webtests_from_logs/results@ are results from Progress? Galya B

02:26 AM Bug #8799: ON ERROR UNDO, THROW problems leading to infinite loops

There are cases when even if an ERROR condition is supposed to be morphed into a structured exception, and re-thrown ... Constantin Asofiei

02:24 AM Bug #8799 (Test): ON ERROR UNDO, THROW problems leading to infinite loops

Constantin Asofiei

05/21/2024

10:38 AM Feature #6161: add support for DYNAMIC-PROPERTY() built-in function and DYNAMIC-PROPERTY language...

Branch 6161a was merged to trunk rev 15229 and archived. Constantin Asofiei

08:34 AM Feature #6161 (Merge Pending): add support for DYNAMIC-PROPERTY() built-in function and DYNAMIC-P...

You can merge to trunk after 7417b. Greg Shah

08:04 AM Feature #6161: add support for DYNAMIC-PROPERTY() built-in function and DYNAMIC-PROPERTY language...

Greg Shah wrote:
> What testing do you propose?
DYNAMIC-PROPERTY is not being used in other projects than #8433. S... Constantin Asofiei

07:58 AM Feature #6161 (Internal Test): add support for DYNAMIC-PROPERTY() built-in function and DYNAMIC-P...

What testing do you propose? Greg Shah

07:58 AM Feature #6161: add support for DYNAMIC-PROPERTY() built-in function and DYNAMIC-PROPERTY language...

I think we just watch it in case it becomes a hotspot. One solution would be to group all the details of a property ... Greg Shah

07:30 AM Feature #6161: add support for DYNAMIC-PROPERTY() built-in function and DYNAMIC-PROPERTY language...

I've moved the bulk getter/setter to be resolved only if no indexed setter/getter is resolved at all. I'm not sure w... Constantin Asofiei

08:38 AM Feature #4347: add runtime support for STOP-AFTER block option

That explains it. I was looking at the 12.x docs. Greg Shah

07:47 AM Feature #4347: add runtime support for STOP-AFTER block option

Constantin Asofiei wrote:
> According to https://docs.progress.com/bundle/openedge-startup-and-parameter-reference-1... Marian Edu

05/20/2024

12:51 PM Feature #6161: add support for DYNAMIC-PROPERTY() built-in function and DYNAMIC-PROPERTY language...

Code Review Task Branch 6161a Revision 15222
The changes look good.
The only concern I have is about performa... Greg Shah

07:34 AM Feature #6161 (Review): add support for DYNAMIC-PROPERTY() built-in function and DYNAMIC-PROPERTY...

Created task branch 6161a from trunk rev 15222. rev 15223 fixed validation of the DYNAMIC-PROPERTY arguments for stat... Constantin Asofiei

11:59 AM Feature #4347: add runtime support for STOP-AFTER block option

Good.
Eduard: Create a new task for the @-catchStop@ behavior. Focus your implementation for this task on the tr... Greg Shah

11:57 AM Feature #4347: add runtime support for STOP-AFTER block option

Greg Shah wrote:
> > Greg, @-catchStop 1@ startup option is required for @progress.lang.stopafter@ to be thrown.
... Constantin Asofiei

11:40 AM Feature #4347: add runtime support for STOP-AFTER block option

> Greg, @-catchStop 1@ startup option is required for @progress.lang.stopafter@ to be thrown.
Are you saying tha... Greg Shah

11:39 AM Feature #4347: add runtime support for STOP-AFTER block option

> I tried to run this code in Procedure Editor, but it does not work as expected.
Do not run any test code from in... Greg Shah

09:12 AM Feature #4347: add runtime support for STOP-AFTER block option

Marian Edu wrote:
> As Constantin said you have to move the @catch@ block out of the @stop-after@ one and also make ... Eduard Soltan

08:51 AM Feature #4347: add runtime support for STOP-AFTER block option

As Constantin said you have to move the @catch@ block out of the @stop-after@ one and also make sure you remove the @... Marian Edu

08:26 AM Feature #4347: add runtime support for STOP-AFTER block option

Greg, @-catchStop 1@ startup option is required for @progress.lang.stopafter@ to be thrown. Do we want to implement ... Constantin Asofiei

05:24 AM Feature #4347: add runtime support for STOP-AFTER block option

... Eduard Soltan

05/16/2024

09:48 AM Feature #6407: name_map.xml improvements

Greg Shah wrote:
> I thought that in #6407-71, this was changed to a @List@ approach which has proper FIFO ordering.... Galya B

09:31 AM Feature #6407: name_map.xml improvements

Greg Shah wrote:
> You can include it in 8667a.
The fix was merged to trunk as rev. 15216. Galya B

08:30 AM Feature #6407: name_map.xml improvements

You can include it in 8667a. Greg Shah

02:08 AM Feature #6407: name_map.xml improvements

The comparison in the condition is solid... if it wasn't for @extProg.ooname = ooname.isEmpty() ? null : ooname;@. @o... Galya B

05/15/2024

10:38 AM Feature #6407: name_map.xml improvements

> Greg, why would #6667-995 fail with this test (@(!resolvedProgram.pname.equals(pname) || !ooname.equals(resolvedPro... Greg Shah

08:43 AM Feature #6407: name_map.xml improvements

Greg, why would #6667-995 fail with this test (@(!resolvedProgram.pname.equals(pname) || !ooname.equals(resolvedProgr... Constantin Asofiei

08:30 AM Feature #6407: name_map.xml improvements

> Interesting. Most of the discussion in this task is about how map resolvement should work and it's implemented acco... Greg Shah

03:00 AM Feature #6407: name_map.xml improvements

And let me state again, if there is a discrepancy in the content of the maps of two jars in the classpath, this is a ... Galya B

02:44 AM Feature #6407: name_map.xml improvements

I think we should actually be happy to have found this discrepancy in the scripts. Every time such malfunction appear... Galya B

02:40 AM Feature #6407: name_map.xml improvements

Greg Shah wrote:
> In #6667-995, Tomasz found that the server failed to start after picking up trunk rev 15180. The... Galya B

05/14/2024

09:34 AM Feature #4347: add runtime support for STOP-AFTER block option

Code Review (Part 2) Task Branch 4347a Revision 15204
7. Should we (can we?) name the @Timer@ thread to make it cl... Greg Shah

09:21 AM Feature #4347: add runtime support for STOP-AFTER block option

Code Review Task Branch 4347a Revision 15204
1. Please explain why @Control.blockInterrupt()@ adds a remote call a... Greg Shah

08:29 AM Feature #4347: add runtime support for STOP-AFTER block option

I'm reviewing the code now. After reviewing the 4GL docs, I do have some questions. I also added Marian to coordina... Greg Shah

06:20 AM Feature #4347: add runtime support for STOP-AFTER block option

Constantin Asofiei wrote:
> Ah, you are right, there is a single thread where the tasks are posted. So there is no ... Eduard Soltan

03:12 AM Feature #4347: add runtime support for STOP-AFTER block option

Eduard Soltan wrote:
> Constantin Asofiei wrote:
> > Review for 4347a rev 15204:
> > * what happens if there are n... Constantin Asofiei

02:49 AM Feature #4347: add runtime support for STOP-AFTER block option

Constantin Asofiei wrote:
> Review for 4347a rev 15204:
> * what happens if there are nested blocks with @STOP-AFTE... Eduard Soltan

02:16 AM Feature #4347: add runtime support for STOP-AFTER block option

Review for 4347a rev 15204:
* we need a @SessionListener@ to clean up the @StopAfterTimer.stopAfterTimerOuts@ map. ... Constantin Asofiei

05/13/2024

03:04 PM Feature #4347: add runtime support for STOP-AFTER block option

Greg or Constantin: please review. Eric Faulhaber

09:56 AM Feature #4347 (Review): add runtime support for STOP-AFTER block option

Committed on 4347a, revision 15204. Eduard Soltan

03:03 PM Feature #6407: name_map.xml improvements

In #6667-995, Tomasz found that the server failed to start after picking up trunk rev 15180. The issue turned out to... Greg Shah

05:10 AM Feature #6417: cleanup some known items in appserver support

Marian Edu wrote:
> I'm looking back at the responses (in webtests_from_logs folder) and it does look like the asser... Galya B

05:02 AM Feature #6417: cleanup some known items in appserver support

Galya B wrote:
> Marian Edu wrote:
> > Gayla
> It's Galya.
Galya :)
I'm looking back at the responses (in we... Marian Edu

04:50 AM Feature #6417: cleanup some known items in appserver support

Marian Edu wrote:
> Gayla
It's Galya. Galya B

04:49 AM Feature #6417: cleanup some known items in appserver support

Galya B wrote:
> @com.goldencode.p2j.util.decimal@ has:
>
> [...]
>
> But testcases test @pipe_decimal_decimal... Marian Edu

04:48 AM Feature #6417: cleanup some known items in appserver support

Marian Edu wrote:
> Galya B wrote:
> > Marian, do you have any idea why this test executed as batch in 4GL gives th... Galya B

03:58 AM Feature #6417: cleanup some known items in appserver support

Greg, how is this going to work? I have more questions than the number of failing tests (quite a few and all over the... Galya B

03:26 AM Feature #6417: cleanup some known items in appserver support

@com.goldencode.p2j.util.decimal@ has:... Galya B

05/07/2024

03:09 AM Feature #6410: implement additional built-in OO classes/interfaces

Constantin Asofiei wrote:
> This gets tricky if you pass an extent field via @::@. But yes, this would need to get ... Marian Edu