Activity
From 03/31/2024 to 04/29/2024
04/29/2024
- 04:39 PM Feature #6422: implement auditing support including AUDIT-POLICY and AUDIT-CONTROL
- The only thing pending for this ticket is #6422-65
- 04:56 AM Feature #6407: name_map.xml improvements
- 6407a rebased on trunk r15171.
Constantin Asofiei wrote:
> Otherwise, if some testing with customers app works, ... - 02:50 AM Feature #6410: implement additional built-in OO classes/interfaces
- Constantin Asofiei wrote:
> Marian Edu wrote:
> > The code in `p2j.oo` almost exclusively use `returnNormal` and ex...
04/26/2024
- 11:27 AM Feature #6407: name_map.xml improvements
- Greg Shah wrote:
> Constantin: Are we good to go?
My main concern with Windows was that jar file separator which ne... - 11:17 AM Feature #6407 (Internal Test): name_map.xml improvements
- > do I need to make hotel_gui or testcases run on Windows as part of this task
No. The Windows situation will not... - 10:38 AM Feature #6407: name_map.xml improvements
- I don't know what's going on now, but the client doesn't even spawn and I can't stop it in the cmd prompt. No errors ...
- 08:29 AM Feature #6407: name_map.xml improvements
- OK, the java command for the task @create.db.h2@ exceeds @8191@ chars (the max length of commands in windows) due to ...
- 08:00 AM Feature #6407: name_map.xml improvements
- There is an issue with the file separators indeed. It's @\@ on Windows for the jars on the file system, but the resou...
- 06:26 AM Feature #6407: name_map.xml improvements
- Galya B wrote:
> Not sure what else to test in Win.
Try the Swing client for Hotel GUI on Windows. - 06:25 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> Regarding the @Utils.packageToPath@ change - please check runtime on Windows for i.e. Ho... - 02:37 AM Feature #6407: name_map.xml improvements
- Galya B wrote:
> With or without 6407a I get:
> [...]
>
> Not sure what it means.
There is ambiguity between th... - 02:34 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> Galya B wrote:
> > Actually if a dir with name map file is in the classpath the name ma... - 07:12 AM Feature #6410: implement additional built-in OO classes/interfaces
- Marian Edu wrote:
> The code in `p2j.oo` almost exclusively use `returnNormal` and expect the execution to end there... - 07:04 AM Feature #6410: implement additional built-in OO classes/interfaces
- The code in `p2j.oo` almost exclusively use `returnNormal` and expect the execution to end there instead of the `stor...
- 06:49 AM Feature #6410 (WIP): implement additional built-in OO classes/interfaces
- I have a question on how is the proper way to return a value out of a OO method (really a function block), we've used...
- 04:40 AM Feature #8676: ABLUnit assert error messages are different in different OE versions
- A quick search on testcases project shows we have 500 asserts on 'not null' error messages, for some tests of #6410 w...
- 03:57 AM Feature #8676: ABLUnit assert error messages are different in different OE versions
- Greg Shah wrote:
> Marian: Please provide some more details on the specific changes that need to be made. Also, ple...
04/25/2024
- 02:24 PM Feature #6407: name_map.xml improvements
- Galya B wrote:
> Actually if a dir with name map file is in the classpath the name map should have been found in the... - 07:53 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> You are correct, at this point, unless the current folder is in the classpath, @ClassLoa... - 07:51 AM Feature #6407: name_map.xml improvements
- With the @classNameExists@ in mind, these changes don't affect conversion.
- 07:22 AM Feature #6407: name_map.xml improvements
- You are correct, at this point, unless the current folder is in the classpath, @ClassLoader.getSystemResource(sb.toSt...
- 06:16 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> Galya B wrote:
> > > * @SourceNameMapper@ - @convertJavaProg@ is used during conversion... - 05:32 AM Feature #6407: name_map.xml improvements
- Galya B wrote:
> > * @SourceNameMapper@ - @convertJavaProg@ is used during conversion via @NameMappingWorker.classNa... - 05:13 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> Review for 6407a rev 15157:
> * @WebServer@ - please add a history entry
Fixed.
> * @... - 04:01 AM Feature #6407: name_map.xml improvements
- Review for 6407a rev 15157:
* @WebServer@ - please add a history entry
* @Utils.packageToPath@ - this replaces all ...
04/24/2024
- 05:12 PM Feature #8676: ABLUnit assert error messages are different in different OE versions
- Marian's team has written a large number of ABLUnit tests which relied upon OE (12.x?) error messages from the assert...
- 05:03 PM Feature #8676 (New): ABLUnit assert error messages are different in different OE versions
- 08:34 AM Feature #4347: add runtime support for STOP-AFTER block option
- > To replicate this in Java, we need all *converted* statements to check if there was a time-out or not, so it can ra...
- 04:30 AM Feature #4347: add runtime support for STOP-AFTER block option
- Alexandru, the STOP AFTER needs to use a similar mechanism we have for CTRL-C support (which actually raises a STOP c...
- 04:19 AM Feature #4347: add runtime support for STOP-AFTER block option
- Eduard and I had a chat yesterday on this task an I took some "thinking" time on my own on this matter:
#4347-6 is...
04/22/2024
04/18/2024
- 06:44 AM Feature #6424: fill gaps in Java Open Client replacement (appserver client support)
- This is the server log of errors.
04/17/2024
- 09:36 AM Feature #6424: fill gaps in Java Open Client replacement (appserver client support)
- There are some failing tests. Is it the deployment or it's fine?
- 09:00 AM Bug #8643: ErrorManager not handling caught errors and error-status:error attribute properly
- This issue is a continuation of #8614 that involves two problems that need to be solved. I've attached 4 files:
* ra... - 08:54 AM Bug #8643 (Test): ErrorManager not handling caught errors and error-status:error attribute properly
04/15/2024
- 07:03 AM Feature #6407: name_map.xml improvements
- 6407a rebased on trunk r15146.
New r15156 with a minor fix for #8166-151. - 02:19 AM Feature #6407: name_map.xml improvements
- Reminder: The branch needs a review (at r15110). It also has minor changes for #8166-145 and #8166-150.
04/04/2024
- 05:26 PM Bug #8579 (Test): Unit and OEUnit conversion: ignore annotations from other framework
- 05:21 PM Bug #8579: Unit and OEUnit conversion: ignore annotations from other framework
- Greg Shah wrote:
> You can merge 8579a to trunk now.
Done. Rev. 15123. - 04:23 PM Bug #8579 (Merge Pending): Unit and OEUnit conversion: ignore annotations from other framework
- You can merge 8579a to trunk now.
- 01:14 PM Bug #8579: Unit and OEUnit conversion: ignore annotations from other framework
- Greg Shah wrote:
> Code Review Task Branch 8579a Revision 15121
>
> The change is good.
>
> What additional te... - 12:58 PM Bug #8579: Unit and OEUnit conversion: ignore annotations from other framework
- Code Review Task Branch 8579a Revision 15121
The change is good.
What additional testing is needed? - 11:20 AM Bug #8579: Unit and OEUnit conversion: ignore annotations from other framework
- Greg Shah wrote:
> Code Review Task Branch 8579a Revision 15120
>
> I think the code would be simpler and easier ... - 12:12 PM Feature #6407: name_map.xml improvements
- I did replace the @Set@ with @List@ in r15108.
Tomorrow I'm testing again in testcases. A good list of different p... - 11:45 AM Feature #6407: name_map.xml improvements
- @MultiClassLoader@:...
- 11:43 AM Feature #6407: name_map.xml improvements
- So what I'm saying is that there is no classpath order in my humble opinion. Iterating HashMap is not ordered.
- 11:39 AM Feature #6407: name_map.xml improvements
- Greg Shah wrote:
> > > @MultiClassLoader@ finds the class by iterating @HashSet@.
> > Actually it's a collection, ... - 11:32 AM Feature #6407: name_map.xml improvements
- > > @MultiClassLoader@ finds the class by iterating @HashSet@.
> Actually it's a collection, backed by the @HashMap... - 11:20 AM Feature #6407 (Review): name_map.xml improvements
- 6407a r15107 based on trunk r15101 ready for review: adds support for reading mappings from multiple name_map.xml fil...
- 09:57 AM Feature #6407: name_map.xml improvements
- Galya B wrote:
> @MultiClassLoader@ finds the class by iterating @HashSet@.
Actually it's a collection, backed by ... - 09:42 AM Feature #6407: name_map.xml improvements
- We can rework it to iterate jars alphabetically.
- 09:39 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> My understanding is that JVM keeps the same order of jars as defined in classpath (it do... - 09:37 AM Feature #6407: name_map.xml improvements
- Galya B wrote:
> Greg Shah wrote:
> > Do we really need an explicit list of precedence order? Why not implicitly t... - 09:29 AM Feature #6407: name_map.xml improvements
- Greg Shah wrote:
> Do we really need an explicit list of precedence order? Why not implicitly take it from the clas... - 09:28 AM Feature #6407: name_map.xml improvements
- Do we really need an explicit list of precedence order? Why not implicitly take it from the classpath and honor the ...
- 09:21 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> From what you posted in #6407-56, only one concern: the granularity for the precedence o... - 05:45 AM Feature #6407: name_map.xml improvements
- From what you posted in #6407-56, only one concern: the granularity for the precedence order I think needs to be "thi...
- 04:39 AM Feature #6407: name_map.xml improvements
- P.S. The previous comment got updated.
- 04:36 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> And I think I understand now Greg's point about the .r and folder deploy in OpenEdge:
>...
04/03/2024
- 06:05 PM Bug #8579 (Internal Test): Unit and OEUnit conversion: ignore annotations from other framework
- Code Review Task Branch 8579a Revision 15120
I think the code would be simpler and easier to read if you create a ... - 03:11 PM Bug #8579 (Review): Unit and OEUnit conversion: ignore annotations from other framework
- Fixed in 8579a rev. 15120. Please, review.
- 03:10 PM Bug #8579 (WIP): Unit and OEUnit conversion: ignore annotations from other framework
- 02:51 PM Bug #8579: Unit and OEUnit conversion: ignore annotations from other framework
- Currently, if a test 4gl source includes both ABLUnit annotations and OEUnit annotations, all these annotations are c...
- 02:46 PM Bug #8579 (Test): Unit and OEUnit conversion: ignore annotations from other framework
- 03:08 PM Feature #6407: name_map.xml improvements
- And I think I understand now Greg's point about the .r and folder deploy in OpenEdge:
* in FWD, you would convert th... - 02:34 PM Feature #6407: name_map.xml improvements
- Greg Shah wrote:
> > But, in practice, I think .pl archives are used more - and depending on the PROPATH, runtime co... - 02:29 PM Feature #6407: name_map.xml improvements
- > But, in practice, I think .pl archives are used more - and depending on the PROPATH, runtime could use a .p from di...
- 02:26 PM Feature #6407: name_map.xml improvements
- We had cases of the same OE qualified class name used in different modules. The qualified legacy name can't be chang...
- 01:28 PM Feature #6407: name_map.xml improvements
- To solve case 2. we need the precedence order to be fed to the jmv via startup args, so that the classloader can take...
- 01:09 PM Feature #6407: name_map.xml improvements
- Galya B wrote:
> I speak about case 3: different maps for different files with different attributes. Obviously if yo... - 01:08 PM Feature #6407: name_map.xml improvements
- I speak about case 3: different maps for different files with different attributes, but same path. Obviously if you l...
- 01:02 PM Feature #6407: name_map.xml improvements
- > This is not something we can decide even with configs and Constantin was right that it is a major issue.
Case 1 ... - 12:58 PM Feature #6407: name_map.xml improvements
- One pname corresponds to only one jname, because that's how conversion seems to work. The question is if the other at...
- 11:36 AM Feature #6407: name_map.xml improvements
- Galya B wrote:
> Other than that the jar can have a completely different package for the actual classes.
Actually n... - 11:34 AM Feature #6407: name_map.xml improvements
- If we need to load the jars in a specific order, we'll need explicit instructions, probably new config in directory.
- 11:32 AM Feature #6407: name_map.xml improvements
- Case 2 is the same as java and can cause malfunctioning programs like in java, this is what I meant.
Case 1 is fine ... - 11:20 AM Feature #6407: name_map.xml improvements
- > In java this is the same as having multiple versions of the same dependency in the classpath.
Not exactly. In... - 11:14 AM Feature #6407: name_map.xml improvements
- > If you are saying that module M1 has a program foo/bar/F.p and module M2 has a program foo/bar/F.p. which are compl...
- 11:12 AM Feature #6407: name_map.xml improvements
- This is actually more complicated than it looked at first. We're trying to combine different apps coming possibly fro...
- 11:03 AM Feature #6407: name_map.xml improvements
- Greg Shah wrote:
> Let's consider this from the perspective of how it works for this multi-app case in OE. If both ... - 10:55 AM Feature #6407: name_map.xml improvements
- Originally (currently) if multiple paths are found for the same filename when resolving the name map, the paths end u...
- 10:45 AM Feature #6407: name_map.xml improvements
- > The 'pname' in @class-mapping@ is the full path relative to the @abl/@ folder
Yes, this is why I called them rel... - 10:30 AM Feature #6407: name_map.xml improvements
- Greg Shah wrote:
> > > And all of this just for a warning log? I would say it's more reasonable to log the warning a... - 10:26 AM Feature #6407: name_map.xml improvements
- I don't have an actual example though. Not sure if the path is supposed to be the same in two different jars.
- 10:25 AM Feature #6407: name_map.xml improvements
- Greg Shah wrote:
> > > And all of this just for a warning log? I would say it's more reasonable to log the warning a... - 10:23 AM Feature #6407: name_map.xml improvements
- > > And all of this just for a warning log? I would say it's more reasonable to log the warning always when a second ...
- 06:16 AM Feature #6407: name_map.xml improvements
- > REST services written directly in Java can be configured via the 'rest-service' element.
So these nodes are not ... - 06:06 AM Feature #6407: name_map.xml improvements
- OK, then I need one more clarification. Since there is a separate container @rest-service@, where programs get resolv...
- 06:00 AM Feature #6407: name_map.xml improvements
- Yes.
- 05:57 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> Galya B wrote:
> > And all of this just for a warning log? I would say it's more reason... - 05:57 AM Feature #6407: name_map.xml improvements
- Galya B wrote:
> And all of this just for a warning log? I would say it's more reasonable to log the warning always ... - 05:53 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> Galya B wrote:
> > Constantin, maybe you've missed why I'm assigned to the task, that i... - 05:49 AM Feature #6407: name_map.xml improvements
- Galya B wrote:
> Constantin, maybe you've missed why I'm assigned to the task, that is in #6407-4.
OK, thanks for t... - 05:49 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> The point was to move to annotations/other structures inside the .java
Constantin, ... - 05:43 AM Feature #6407: name_map.xml improvements
- The point was to move to annotations/other structures inside the .java, here I mean all info from name_map.xml; ideal...
- 05:33 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> I don't mean the annotations in the .class - I mean the state from name_map.xml must mat... - 05:22 AM Feature #6407: name_map.xml improvements
- I don't mean the annotations in the .class - I mean the state from name_map.xml must match if there is an existing en...
- 03:55 AM Feature #6407: name_map.xml improvements
- Constantin Asofiei wrote:
> * load them into @SourceNameMapper@, one by one, and address conflicts - if there are co... - 05:26 AM Bug #8510: replace all 'new logical' in FWD runtime with logical.of or logical.UNKNOWN
- 8150a was merged to trunk rev 15116 and archived.
- 05:26 AM Bug #8510: replace all 'new logical' in FWD runtime with logical.of or logical.UNKNOWN
- Created task branch 8510a from trunk rev 15115. In rev 15116 I removed the @logical.of@ and @logical.UNKNOWN@ from R...
04/01/2024
- 06:41 PM Bug #8154: refactor scopeable stacks
- Code Review Task Branch 8154a Revision 15109
The changes look promising. There is clearly quite a bit more that c... - 12:51 PM Bug #8154 (WIP): refactor scopeable stacks
- Created task branch 8154a from trunk rev 15108.
Rev 15109 contains first pass at moving scoped stacks into (TopLev... - 04:00 AM Feature #6424: fill gaps in Java Open Client replacement (appserver client support)
- Galya, I've placed the project under @devsrv01:/tmp/open_client_example.zip@. There is a small readme.txt in the pro...
Also available in: Atom