Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-17783

Move {{Schema.FORCE_LOAD_KEYSPACES}} and {{Schema.FORCE_LOAD_KEYSPACES_PROP}} to {{CassandraRelevantProps}}

    XMLWordPrintableJSON

Details

    • Operability
    • Low Hanging Fruit
    • All
    • None
    • Hide

      Tested in CQLSSTableWriterClientTest as it requires this prop to be set, which is done via the moved property in CQLSSTableWriter

       

      See below for Trunk and 4.1 PRs for review and Circle CI runs.

       

       

      Show
      Tested in CQLSSTableWriterClientTest as it requires this prop to be set, which is done via the moved property in CQLSSTableWriter   See below for Trunk and 4.1 PRs for review and Circle CI runs.    

    Description

      Schema.FORCE_LOAD_LOCAL_KEYSPACES_PROP and Schema.FORCE_LOAD_LOCAL_KEYSPACES should be moved to CassandraRelevantProps for two reasons.

      1. Generally speaking, this is where these kinds of things live.
      2. By having them in Schema, you can't actually use Schema.FORCE_LOAD_LOCAL_KEYSPACES_PROP without running all of the static initializers... and you want to use Schema.FORCE_LOAD_LOCAL_KEYSPACES_PROP to set the system property before the static initializer runs.

      Attachments

        1. CASSANDRA-17783-rebased.patch
          269 kB
          Doug Rohrer

        Issue Links

          Activity

            People

              drohrer Doug Rohrer
              drohrer Doug Rohrer
              Doug Rohrer
              Brandon Williams, Jacek Lewandowski
              Votes:
              0 Vote for this issue
              Watchers:
              4 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 - 0.5h
                  0.5h