Uploaded image for project: 'Cayenne'
  1. Cayenne
  2. CAY-2212 cdbimport cleanup and configuration schema refactoring
  3. CAY-2203

cdbimport: Mixing "includeTable" with "includeTables" causes ignored configs

Attach filesAttach ScreenshotVotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • None
    • None
    • None
    • Maven 3.3.9, Java 8

    Description

      I started with the following cdbimport configuration in pom.xml:

      <catalog>
          <name>c1</name>
          <includeTable>
              <pattern>table1</pattern>
              <excludeColumn>.*XYZ</excludeColumn>
          </includeTable>
          <includeTables>
              <pattern>table2</pattern>
              <pattern>table3</pattern>
          </includeTables>
      </catalog>
      

      It mostly worked, except "excludeColumn" spec was ignored. I changed this to this (replacing one "includeTables" with multiple "includeTable" tags) :

      <catalog>
          <name>c1</name>
          <includeTable>
              <pattern>table1</pattern>
              <excludeColumn>.*XYZ</excludeColumn>
          </includeTable>
          <includeTable>table2</includeTable>
          <includeTable>table3</includeTable>
      </catalog>
      

      And now everything started working. So we should either fix this scenario. Or even better - disallow plural tags (such as "includeTables") ...

      Is plural tag support automatically added by Maven, or is this our code?

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ntimofeev Nikita Timofeev
            andrus Andrus Adamchik
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment