Uploaded image for project: 'Kylin'
  1. Kylin
  2. KYLIN-1721

KylinConfigExt lost base properties when store into file

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: v1.5.2
    • Fix Version/s: v1.5.2
    • Component/s: Metadata
    • Labels:
      None

      Description

      KylinConfigExt overrides getAllProperties() to merge the base properties and the override properties into one Properties, which used to get config or write into file.
      In the following code, super properties stored in Properties.defaults, which will NOT written out by store() method, quoted from java document. That will lost super properties when store KylinConfigExt into file.

          protected Properties getAllProperties() {
              Properties result = new Properties(super.getAllProperties());
              result.putAll(overrides);
              return result;
          }
      

        Attachments

          Activity

            People

            • Assignee:
              sunyerui Yerui Sun
              Reporter:
              sunyerui Yerui Sun
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: