castor
  1. castor
  2. CASTOR-2426

Resolve JDOClassDescriptors on the filesystem

    Details

    • Number of attachments :
      1

      Description

      Draw in a mechanism to lookup JDOClassDescriptors on the filesystem. Basically affects JDOClassDescriptorResolver(Impl).

        Activity

        Hide
        Lukas Lang added a comment -

        Patch also includes changes in the MappingLoader interface. Is there a specific place/package to put a testcase (e.g. cpactf)? I would therefor love to add a testcase.

        Show
        Lukas Lang added a comment - Patch also includes changes in the MappingLoader interface. Is there a specific place/package to put a testcase (e.g. cpactf)? I would therefor love to add a testcase.
        Hide
        Lukas Lang added a comment -

        I forgot to add the addPackage functionality to this task. If package added, a cdr file, equivalent to xml should be consulted.

        Show
        Lukas Lang added a comment - I forgot to add the addPackage functionality to this task. If package added, a cdr file, equivalent to xml should be consulted.
        Hide
        Werner Guttmann added a comment -

        I forgot to add the addPackage functionality to this task

        Don't worry, we can always add this at a later time. I'd like to get above patch in a.s.a.p. as I've got all this Spring code ready that has a dependency on this change.

        Show
        Werner Guttmann added a comment - I forgot to add the addPackage functionality to this task Don't worry, we can always add this at a later time. I'd like to get above patch in a.s.a.p. as I've got all this Spring code ready that has a dependency on this change.
        Hide
        Lukas Lang added a comment -

        The current JDOClassDescriptorResolver prioritizes ClassDescriptors on the filesystem. Hence, there is no way, not to use a ClassDescriptor and use a mapping instead. Maybe we should consider, whether an external Resolver with added classes was set in the JDOManager.

        Show
        Lukas Lang added a comment - The current JDOClassDescriptorResolver prioritizes ClassDescriptors on the filesystem. Hence, there is no way, not to use a ClassDescriptor and use a mapping instead. Maybe we should consider, whether an external Resolver with added classes was set in the JDOManager.
        Hide
        Ralf Joachim added a comment -

        Lukas, tests that do not require database access should be placed under cpa/src/test/java.

        Show
        Ralf Joachim added a comment - Lukas, tests that do not require database access should be placed under cpa/src/test/java.
        Hide
        Ralf Joachim added a comment -

        If you first consult mapping before searching for ClassDescriptors on the filesystem you should be fine without the need to set external Resolver on JDOManager.

        Show
        Ralf Joachim added a comment - If you first consult mapping before searching for ClassDescriptors on the filesystem you should be fine without the need to set external Resolver on JDOManager.
        Hide
        Werner Guttmann added a comment -

        Actually, I have seen the very same problem even in a case where there's mapping only, as all of a sudden no mapping can be loaded anymore (NPE on loading a field mapping). I am actually debugging to see what's going on.

        In general, I think that consulting with the mapping first should be the correct approach for the time being.

        Show
        Werner Guttmann added a comment - Actually, I have seen the very same problem even in a case where there's mapping only, as all of a sudden no mapping can be loaded anymore (NPE on loading a field mapping). I am actually debugging to see what's going on. In general, I think that consulting with the mapping first should be the correct approach for the time being.

          People

          • Assignee:
            Werner Guttmann
            Reporter:
            Lukas Lang
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: