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

Specify string lengths in PosixDatastoreRecords table

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: daf_butler
    • Labels:
      None

      Description

      We do not currently specify string lengths for DatabaseDict tables. Update the constructor used from PosixDatastore to include lengths for the fields. This is done currently in code because the tables are dynamic and each datastore gets to name the table to use.

        Attachments

          Issue Links

            Activity

            tjenness Tim Jenness created issue -
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Epic Link DM-16576 [ 234914 ]
            Hide
            tjenness Tim Jenness added a comment -

            I have something that works. Specifies lengths as separate dict. It does fix the lengths in the schema but I can't test with sqlite since sqlite seems to ignore the length setting on inserts.

            Show
            tjenness Tim Jenness added a comment - I have something that works. Specifies lengths as separate dict. It does fix the lengths in the schema but I can't test with sqlite since sqlite seems to ignore the length setting on inserts.
            tjenness Tim Jenness made changes -
            Reviewers Nate Lust [ nlust ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            Hide
            nlust Nate Lust added a comment -

            Minor point to think about but I dont care strongly if you change or not, otherwise looks good.

            Show
            nlust Nate Lust added a comment - Minor point to think about but I dont care strongly if you change or not, otherwise looks good.
            nlust Nate Lust made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            ktl Kian-Tat Lim made changes -
            Sprint Arch 2019-03-11 [ 870 ] Arch 2019-03-11, Arch 2019-03-18 [ 870, 871 ]
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to DM-24786 [ DM-24786 ]

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Nate Lust
              Watchers:
              Christopher Stephens, Jim Bosch, Michelle Gower, Nate Lust, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: