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

Improve user expression handling in pre-flight

    XMLWordPrintable

    Details

      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:

                Jenkins

                No builds found.