-
Notifications
You must be signed in to change notification settings - Fork 25.2k
[CCR] Fetches/indexing/writes/reads/bulks in status fields vs readers in task params #34703
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Pinging @elastic/es-distributed |
@chrisronline FYI. What ever ends up being deciding here will need a quick update on the UI side. |
@jasontedor @martijnvg and me iterated on this and came up with the following namings. I mark this as a blocker for 6.5.0:
|
Looks good. I'm going to list the fields we are using in the UI and my best guess as to the new field name:
The rest seem unchanged. Is this accurate on terms of the renames? |
Uh oh!
There was an error while loading. Please reload this page.
The status reports for the following tasks often refer to fetches (
number_of_successful_fetches
,number_of_failed_fetches
,total_fetch_time_millis
,fetch_exceptions
). We also have a similar term (which is identical in meaning) call reads (number_of_concurrent_reads
). We also have the genericoperations_received
andtotal_transferred_bytes
andlast_requested_seq_no
.On the write sides we have
bulk
(number_of_successful_bulk_operations
,number_of_failed_bulk_operations
) andwrites
(number_of_concurrent_writes
,number_of_queued_writes
<-- note the different semantics here).I think we should align our terminologies across the various places. I personally like the
read
versuswrite
andbulk
vsops
. Howabout:number_of_succesful_bulk_reads
number_of_failed_bulk_reads
total_bulk_read_time
number_of_ongoing_bulk_reads
total_operations_read
total_bytes_read
last_requested_seq_no_for_reads
number_of_successful_bulk_writes
number_of_failed_bulk_writes
number_of_ongoing_bulk_writes
number_of_queued_ops_to_write
(not too happy with this one)/cc @jasontedor @martijnvg
The text was updated successfully, but these errors were encountered: