Comment 1 Eric Gwin 2011-02-25 12:59:49 EST

moving this to JPA since the other relavant bugs were also moved there by Shaun.

Comment 2 Pascal Filion 2011-03-03 18:13:31 EST

Created attachment 190329 [details]
Hermes parser

Since the project does not exist, I had to create a zip file. It contains:
- Latest Hermes parser code
- @version and @since have the correct version number
- The code was split into a public API and internal code

Comment 3 Pascal Filion 2011-03-03 18:14:39 EST

Created attachment 190330 [details]
Hermes parser tests

Same as attachment 190329 [details].

Comment 4 Pascal Filion 2011-03-08 15:25:32 EST

Created attachment 190690 [details]
Moving Hermes parser to new location

This patch contains
- Latest code (more content assist support)
- Deletion of the parser from utils/org.eclipse.persistence.utils.jpa.query
- Addition of the code to jpa/org.eclipse.persistence.jpa.jpql

Comment 5 Pascal Filion 2011-03-08 15:27:43 EST

Created attachment 190692 [details]
Moving Hermes parser tests to new location

same as attachment 190690 [details]

Comment 6 Neil Hauge 2011-03-08 19:05:18 EST

The only issue I have found with the last round of patches is that the test project is now a full plugin project.  I think the plan was to keep this as a regular Java project for now to avoid the split package issue, among other potential issues.

Comment 7 Shaun Smith 2011-03-09 10:40:20 EST

Agreed, we align the naming with the plug-in style in anticipation of the planned evolution but we don't make the projects plug-in projects at this time.

Comment 8 Pascal Filion 2011-03-09 11:18:21 EST

Created attachment 190772 [details]
Made the test project a simple project

This patch is the same as the previous patch except the test project is no longer a plug-in project.

Comment 9 Pascal Filion 2011-03-09 13:17:54 EST

Created attachment 190781 [details]
Moving Hermes parser to new location

This includes changed made between now and the previous patch, which caused unit-tests to fail since the new patch for the test included the latest.

Comment 10 Neil Hauge 2011-03-09 16:36:30 EST

The latest round of patches for this bug have been reviewed and committed.  The parser and test project are now in trunk/jpa.  The util based projects have been left empty. Eric, let me know if I should do anything special with these.  I think the plan is that I would simply delete them.

The new jpa.jpql project is the key to the Dali dependency here, so please integrate this into the build and published content as soon as it is reasonable to do so.  Thanks.

Comment 11 Neil Hauge 2011-03-09 16:38:54 EST

.

Comment 12 Peter Krogh 2011-05-13 15:14:09 EDT

Bug was incorrectly set to iplog.  Patch should have been set instead.  Fixed

Comment 13 Eclipse Webmaster 2022-06-09 10:27:46 EDT

The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink