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

Make Registry table names lowercase

    XMLWordPrintable

    Details

      Description

      Oracle works much better with lowercase field names, and case-sensitivity is in general inconsistent in SQL databases.

      We should do this after unifying Dimension names and link names on DM-17023, since that will eliminate the only current use of differently-conventioned names to refer to (essentially) the same concepts, and make a good chunk of the Registry table names lowercase already.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            Replacing the blocker on DM-17023 with just a "related" link - this ticket can be done first (and is looking like it will be); it just leaves a minor mess behind for DM-17023 to clean up (essentially a lot of attributes that now return duplicate things).

            Show
            jbosch Jim Bosch added a comment - Replacing the blocker on DM-17023 with just a "related" link - this ticket can be done first (and is looking like it will be); it just leaves a minor mess behind for DM-17023 to clean up (essentially a lot of attributes that now return duplicate things).
            Hide
            jbosch Jim Bosch added a comment -

            Tim Jenness, this is ready for review, but please don't waste time looking at it by hand: the vast majority of the commits are just regex substitutions (albeit done interactively to avoid renaming e.g. afw.image.Exposure to .image.exposure).  Just some spot-checking really ought to suffice.

            PRs are:

            (as discussed on slack, there's also a meas_algorithms branch, with a commit that will be fixed on another ticket)

            Show
            jbosch Jim Bosch added a comment - Tim Jenness , this is ready for review, but please don't waste time looking at it by hand: the vast majority of the commits are just regex substitutions (albeit done interactively to avoid renaming e.g. afw.image.Exposure to .image.exposure).  Just some spot-checking really ought to suffice. PRs are: https://github.com/lsst/daf_butler/pull/150 https://github.com/lsst/obs_base/pull/144 https://github.com/lsst/meas_base/pull/144 https://github.com/lsst/ip_isr/pull/88 https://github.com/lsst/skymap/pull/36 https://github.com/lsst/pipe_tasks/pull/286 https://github.com/lsst/obs_subaru/pull/194 https://github.com/lsst/ci_hsc/pull/78 (as discussed on slack, there's also a meas_algorithms branch, with a commit that will be fixed on another ticket)
            Hide
            tjenness Tim Jenness added a comment -

            Ok. I actually looked last night but I'll have another scan today. I'm fixing meas_algorithms on DM-19506 (which is now in review).

            Show
            tjenness Tim Jenness added a comment - Ok. I actually looked last night but I'll have another scan today. I'm fixing meas_algorithms on DM-19506 (which is now in review).
            Hide
            tjenness Tim Jenness added a comment -

            Looks good.

            Show
            tjenness Tim Jenness added a comment - Looks good.

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Tim Jenness
              Watchers:
              Jim Bosch, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.