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

Delete the char0 field from Test arrays topics

    Details

    • Story Points:
      0
    • Sprint:
      TSSW Sprint - Nov 11 - Nov 22
    • Team:
      Telescope and Site

      Description

      The char0 field should never have been part of the arrays topics for the Test component because char is a synonym for string and arrays of strings are not allowed.

      I have slowly removed all code that requires char0 to be present. At this point I believe there is just one unit test in ts_salkafka that needs it, and I will fix that on the same ticket.

      There is also some code in ts_salobj that handles char0 if present, but should work if the field is absent.

        Attachments

          Issue Links

            Activity

            Show
            rowen Russell Owen added a comment - Pull requests: https://github.com/lsst-ts/ts_xml/pull/179 https://github.com/lsst-ts/ts_salkafka/pull/5
            Hide
            ecoughlin Eric Coughlin added a comment -

            Reviewed in github.

            Show
            ecoughlin Eric Coughlin added a comment - Reviewed in github.
            Hide
            rowen Russell Owen added a comment - - edited

            Merged ts_xml to develop. This m.ade it into ts_xml 4.6
            Merged ts_salkafka to develop and master and released as v1.0.1.

            Show
            rowen Russell Owen added a comment - - edited Merged ts_xml to develop. This m.ade it into ts_xml 4.6 Merged ts_salkafka to develop and master and released as v1.0.1.

              People

              • Assignee:
                rowen Russell Owen
                Reporter:
                rowen Russell Owen
                Reviewers:
                Eric Coughlin
                Watchers:
                Eric Coughlin, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel