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

Improve user expression handling in pre-flight

    Details

    • Story Points:
      2
    • Sprint:
      BG3_S19_02
    • Team:
      Data Access and Database

      Description

      User expression is now passed to pre-flight as a plain string (even though that expression is parsed and checked for syntax in advance). This makes it difficult to do smart things with it like deducing table names from link names or converting it to true sqlalchemy construct.

      I want to try to pass parsed tree to preflight and build sqlalchemy selection from that instead.

        Attachments

          Activity

          Hide
          salnikov Andy Salnikov added a comment -

          Jim Bosch, could you look at this whenever you have time for it? It's not critical, but I think it's a useful improvement. Please ignore ctrl_mpexec PR, I messed up different branch, only daf_butler and pipe_base changes are for this ticket.

          Show
          salnikov Andy Salnikov added a comment - Jim Bosch , could you look at this whenever you have time for it? It's not critical, but I think it's a useful improvement. Please ignore ctrl_mpexec PR, I messed up different branch, only daf_butler and pipe_base changes are for this ticket.
          Hide
          salnikov Andy Salnikov added a comment -

          Thanks for review and suggestions!

          Centos7 Jenkins build passed (centos6 was building OK but then disappeared from a panel). Merged both packages, done.

          Show
          salnikov Andy Salnikov added a comment - Thanks for review and suggestions! Centos7 Jenkins build passed (centos6 was building OK but then disappeared from a panel). Merged both packages, done.

            People

            • Assignee:
              salnikov Andy Salnikov
              Reporter:
              salnikov Andy Salnikov
              Reviewers:
              Jim Bosch
              Watchers:
              Andy Salnikov, Jim Bosch, Vaikunth Thukral
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel