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

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-24786 New component column in datastore is too small

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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.
            Hide
            Permalink
            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.

              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: