Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.3.0, 0.4.0
    • Fix Version/s: 0.3.0, 0.4.0
    • Component/s: Data Collection
    • Labels:
      None

      Description

      This enhancement is to allow extend DirTailingAdaptor that can support custom build adaptors. Its assumed that custom adaptors will be built and shipped in a separate jar file. Following changes suggested in DirTailingAdaptor

      • String adaptorName: protected to allow extending adaptors know underlying file level adaptor
      • protected String getAdaptorAddCommand(File dir) throws IOException: New method: Allows to construct add command based on custom adaptor to be used with custom DirTailing adaptor
      • private void scanDirHierarchy(File dir): call getAdaptorAddCommand to get adaptor to add. This allows to pass on custom adaptor specific arguments

        Activity

        Hide
        Ari Rabkin added a comment -

        Jaydeep, did you mean to attach a patch here?

        Show
        Ari Rabkin added a comment - Jaydeep, did you mean to attach a patch here?
        Hide
        jaydeep ayachit added a comment -

        Attached is the patch. Please review and let me know if anything needs to be added.

        Show
        jaydeep ayachit added a comment - Attached is the patch. Please review and let me know if anything needs to be added.
        Hide
        jaydeep ayachit added a comment -

        Ari, could you please review the patch and let me know if its ok to commit.

        Thanks
        Jaydeep

        Show
        jaydeep ayachit added a comment - Ari, could you please review the patch and let me know if its ok to commit. Thanks Jaydeep
        Hide
        Ari Rabkin added a comment -

        Looks good. I just committed this. Thanks, Jaydeep!

        Show
        Ari Rabkin added a comment - Looks good. I just committed this. Thanks, Jaydeep!

          People

          • Assignee:
            Unassigned
            Reporter:
            jaydeep ayachit
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development