michaelsembwever commented on code in PR #2973:
URL: https://github.com/apache/cassandra/pull/2973#discussion_r1421093030


##########
doc/modules/cassandra/pages/architecture/storage-engine.adoc:
##########
@@ -1,46 +1,42 @@
 = Storage Engine
 
+{cassandra} processes data at several stages on the write path, starting with 
the immediate logging of a write and ending in with a write of data to disk:
+
+* Logging data in the commit log
+* Writing data to the memtable
+* Flushing data from the memtable
+* Storing data on disk in SSTables
+
 [[commit-log]]
-== CommitLog
-
-Commitlogs are an append-only log of all mutations local to a Cassandra
-node. Any data written to Cassandra will first be written to a commit
-log before being written to a memtable. This provides durability in the
-case of unexpected shutdown. On startup, any mutations in the commit log
-will be applied to memtables.
-
-All mutations are write-optimized by storing in commitlog segments, reducing
-the number of seeks needed to write to disk. Commitlog segments are
-limited by the `commitlog_segment_size` option. Once the size is
-reached, a new commitlog segment is created. Commitlog segments can be
-archived, deleted, or recycled once all the data has been flushed to
-SSTables. Commitlog segments are truncated when Cassandra has written
-data older than a certain point to the SSTables. Running "nodetool
-drain" before stopping Cassandra will write everything in the memtables
-to SSTables and remove the need to sync with the commitlogs on startup.
-
-* `commitlog_segment_size`: The default size is 32MiB, which is
-almost always fine, but if you are archiving commitlog segments (see
-commitlog_archiving.properties), then you probably want a finer
-granularity of archiving; 8 or 16 MiB is reasonable.
-`commitlog_segment_size` also determines the default value of
-`max_mutation_size` in `cassandra.yaml`. By default,
-`max_mutation_size` is a half the size of `commitlog_segment_size`.
+== Logging writes to commit logs
+
+When a write occurs, {cassandra} writes the data to a local append-only 
https://cassandra.apache.org/_/glossary.html#commit-log[commit log] on disk.

Review Comment:
   fixed



##########
doc/modules/cassandra/pages/developing/cql/indexing/sai/sai-query.adoc:
##########
@@ -4,6 +4,8 @@
 The xref:cassandra:getting-started/sai-quickstart.adoc[SAI quickstart] focuses 
only on defining multiple indexes based on non-primary key columns (a very 
useful feature).
 Let's explore other options using some examples of how you can run queries on 
tables that have differently defined SAI indexes.
 
+include::developing:partial$sai/sai-only-select.adoc[]

Review Comment:
   later



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to