Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-32062

Fix ap_verify with gen2 butler after APDB update

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Sprint:
      DB_F21_06
    • Team:
      Data Access and Database
    • Urgent?:
      No

      Description

      DM-31458 had one unexpected consequence - it broke ap_verify in gen2 mode. The reason is that Apdb is trying to use polymorphic config object and that config is stored in Butler, but gen2 Butler does not support polymorphic pex_config types. One option that I like most for is to add support for polymophism to gen2 Butler similarly to how it is done in gen3 formatters.

       

        Attachments

          Issue Links

          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-31458 Prepare to merge Cassandra branch of dax_apdb

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            salnikov Andy Salnikov created issue -
            salnikov Andy Salnikov made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-31458 [ DM-31458 ]
            salnikov Andy Salnikov made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            Hide
            Permalink
            salnikov Andy Salnikov added a comment -

            Ready for review, Jenkins is still running, will wait until it finishes. This adds a direct dependency on pex_config which was not there before.

            Show
            salnikov Andy Salnikov added a comment - Ready for review, Jenkins is still running, will wait until it finishes. This adds a direct dependency on pex_config which was not there before.
            salnikov Andy Salnikov made changes -
            Reviewers Kian-Tat Lim [ ktl ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            Permalink
            salnikov Andy Salnikov added a comment -

            Jenkins finished OK.

            Show
            salnikov Andy Salnikov added a comment - Jenkins finished OK.
            Hide
            Permalink
            ktl Kian-Tat Lim added a comment -

            Looks OK. The extra dependency shouldn't be a problem in the short remaining lifetime of this code.

            Show
            ktl Kian-Tat Lim added a comment - Looks OK. The extra dependency shouldn't be a problem in the short remaining lifetime of this code.
            ktl Kian-Tat Lim made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            salnikov Andy Salnikov made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              salnikov Andy Salnikov
              Reporter:
              salnikov Andy Salnikov
              Reviewers:
              Kian-Tat Lim
              Watchers:
              Andy Salnikov, Kian-Tat Lim, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.