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

            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.
            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.

              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