GeoTools
  1. GeoTools
  2. GEOT-3086

Merge AbstractProcessor with DefaultProcessor

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.7-M0
    • Fix Version/s: 2.7-M0
    • Component/s: coverage
    • Labels:
      None

      Description

      We should merge the coverage Processor hierarchy since right now we are using two classes (one is abstract) while one would be enough.

        Activity

        Hide
        Michael Bedward added a comment -
        Good call Simone. Perhaps consider changing the name to CoverageProcessor or similar ? DefaultProcessor and AbstractProcessor could then be deprecated and just point to the new class.
        Show
        Michael Bedward added a comment - Good call Simone. Perhaps consider changing the name to CoverageProcessor or similar ? DefaultProcessor and AbstractProcessor could then be deprecated and just point to the new class.
        Hide
        Simone Giannecchini added a comment -
        Ciao Michael, thanks for the feedback. I have also added the possibility to specify custom Hints when you execute an operation, which was not possible up to now. I have added back the old classes (with deprecation).
        Ping me in case you have more feedback.
        Show
        Simone Giannecchini added a comment - Ciao Michael, thanks for the feedback. I have also added the possibility to specify custom Hints when you execute an operation, which was not possible up to now. I have added back the old classes (with deprecation). Ping me in case you have more feedback.
        Hide
        Andrea Aime added a comment -
        Mass closing all issues that have been in "resolved" state for 2 months or more without any feedback or update
        Show
        Andrea Aime added a comment - Mass closing all issues that have been in "resolved" state for 2 months or more without any feedback or update

          People

          • Assignee:
            Simone Giannecchini
            Reporter:
            Simone Giannecchini
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: