Uploaded image for project: 'OODT (Retired)'
  1. OODT (Retired)
  2. OODT-727

Rename actions to their prior names in CAS-Crawler

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Won't Fix
    • None
    • 1.1
    • crawler
    • None
    • Don't Know (Unsure) - The default level

    Description

      OODT-325 and this commit http://svn.apache.org/viewvc?view=revision&revision=1294389 introduced new names to CrawlerActionBeans, such as

      • Renaming UpdateWorkflowStatusToIngest to TriggerPostIngestWorkflow
      • Renaming MoveMetadataToBackupDir to MoveMetadataFileToBackupDir
      • Renaming MoveMetadataToFailureDir to MoveMetadataFileToFailureDir

      You can see all the differences by running this command:

      curl "http://svn.apache.org/viewvc/oodt/trunk/crawler/src/main/resources/action-beans.xml?r1=1294389&r2=1294388&pathrev=1294389&view=patch" | grep id | cut -d\" -f2 | cut -d\+ -f2 | grep -v \ | sort<

      Which amount to:

      CheckMetFileExists
      CheckMetFileExists
      DeleteDataFile
      DeleteDataFile
      DeleteMetadataFile
      DeleteMetadataFile
      DeletePushpullMetFile
      DeletePushpullMetFile
      IngestAncillaryFile
      IngestAncillaryFile
      MoveDataFileToFailureDir
      MoveDataFileToFailureDir
      MoveMetadataFileToBackupDir
      MoveMetadataFileToFailureDir
      MoveMetadataToBackupDir
      MoveMetadataToFailureDir
      MovePushpullMetFileToBackupDir
      MovePushpullMetFileToBackupDir
      MovePushpullMetFileToFailureDir
      MovePushpullMetFileToFailureDir
      SendNotification
      SendNotification
      TriggerPostIngestWorkflow
      Unique
      Unique
      UpdateWorkflowStatusToIngest
      ignoreInvalidAddresses
      ignoreInvalidAddresses
      

      I can find no justification for these changes and technically it broke back compatibility with folks using those action names. I am going to change the action names back to what they were called.

      Attachments

        Activity

          People

            chrismattmann Chris A. Mattmann
            chrismattmann Chris A. Mattmann
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: