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

KylinConfigExt lost base properties when store into file

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • v1.5.2
    • v1.5.2
    • Metadata
    • 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

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

            Dates

              Created:
              Updated:
              Resolved: