Merge pull request #1261 from muzarski/metadata-non-exhaustive

mark metadata public structs as `non_exhaustive`

Author: Lorak-mmk

scylla/src/cluster/metadata.rs

@@ -152,13 +152,12 @@ impl UntranslatedEndpoint {
 /// Data used to issue connections to a node.
 ///
 /// Fetched from the cluster in Metadata.
-#[non_exhaustive] // <- so that we can add more fields in a backwards-compatible way
 #[derive(Clone, Debug)]
-pub struct PeerEndpoint {
-    pub host_id: Uuid,
-    pub address: NodeAddr,
-    pub datacenter: Option<String>,
-    pub rack: Option<String>,
+pub(crate) struct PeerEndpoint {
+    pub(crate) host_id: Uuid,
+    pub(crate) address: NodeAddr,
+    pub(crate) datacenter: Option<String>,
+    pub(crate) rack: Option<String>,
 }
 
 impl Peer {
@@ -184,7 +183,9 @@ impl Peer {
     }
 }
 
+/// Describes a keyspace in the cluster.
 #[derive(Clone, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub struct Keyspace {
     pub strategy: Strategy,
     /// Empty HashMap may as well mean that the client disabled schema fetching in SessionConfig
@@ -195,7 +196,9 @@ pub struct Keyspace {
     pub user_defined_types: HashMap<String, Arc<UserDefinedType<'static>>>,
 }
 
+/// Describes a table in the cluster.
 #[derive(Clone, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub struct Table {
     pub columns: HashMap<String, Column>,
     /// Names of the column of partition key.
@@ -208,13 +211,17 @@ pub struct Table {
     pub(crate) pk_column_specs: Vec<ColumnSpec<'static>>,
 }
 
+/// Describes a materialized view in the cluster.
 #[derive(Clone, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub struct MaterializedView {
     pub view_metadata: Table,
     pub base_table_name: String,
 }
 
+/// Describes a column of the table.
 #[derive(Clone, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub struct Column {
     pub typ: ColumnType<'static>,
     pub kind: ColumnKind,
@@ -316,6 +323,7 @@ impl PreCollectionType {
 }
 
 #[derive(Clone, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 pub enum ColumnKind {
     Regular,
     Static,
@@ -342,6 +350,7 @@ impl std::str::FromStr for ColumnKind {
 }
 
 #[derive(Clone, Debug, PartialEq, Eq)]
+#[non_exhaustive]
 #[allow(clippy::enum_variant_names)]
 pub enum Strategy {
     SimpleStrategy {