Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-18410

Put data to metastorage during some components start leads to assertion error

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 3.0
    • None
    • Docs Required, Release Notes Required

    Description

      When we was developing https://issues.apache.org/jira/browse/IGNITE-18087 some tests on restart started to fail on the assertion

      assert rev >= appliedRevision : IgniteStringFormatter.format(
      

      in ConfigurationCatchUpListener.

      It means that we have inconsistency when we compare storage revision from configuration and revision from the event. Needed to investigate.

      UPD:

      We made a fix in DistributedConfigurationStorage#resolveRevision and added else section were we initialised master key with 0 when there was no changes in configuration, but some puts to the metastorage were done, so the DistributedConfigurationStorage#resolveRevision method returned wrong non-null revision for the configuration. But seems that this is not enough and we still get some failed tests.

      We fail we the assertion

      assert data.isEmpty() || cfgRevision > 0;
      

      in DistributedConfigurationStorage#readDataOnRecovery0 (see muted test ItDistributedConfigurationStorageTest#testRestartWithPds)

      or get exception

                          if (revision <= appliedRevision) {
                              throw new MetaStorageException(DEPLOYING_WATCH_ERR, String.format(
                                      "Current revision (%d) must be greater than the revision in the Vault (%d)",
                                      revision, appliedRevision
                              ));
                          }
      

      in

      MetaStorageManager#storeEntries (see muted test ItTablesApiTest#testGetTableFromLaggedNode)

      Attachments

        Issue Links

          Activity

            People

              ibessonov Ivan Bessonov
              maliev Mirza Aliev
              Mirza Aliev Mirza Aliev
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 20m
                  20m