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

Specify string lengths in PosixDatastoreRecords table

    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:

                  Summary Panel