Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ shortcuts, views, color schema and of course features)**

If something does not work, like you have too old version of ClickHouse, consider upgrading.

Later some backward compatiblity will be added as well.
Later some backward compatibility will be added as well.

*Note: the oldest version that had been tested was 21.2*

Expand Down
4 changes: 2 additions & 2 deletions src/interpreter/context.rs
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@ impl Context {
*new_start -= Duration::try_minutes(minutes).unwrap();
*new_end -= Duration::try_minutes(minutes).unwrap();
log::debug!(
"Set time frame to ({}, {}) (seeked to {} minutes backward)",
"Set time frame to ({}, {}) (sought to {} minutes backward)",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was wrong

new_start,
new_end,
minutes
Expand All @@ -173,7 +173,7 @@ impl Context {
*new_start += Duration::try_minutes(minutes).unwrap();
*new_end += Duration::try_minutes(minutes).unwrap();
log::debug!(
"Set time frame to ({}, {}) (seeked to {} minutes backward)",
"Set time frame to ({}, {}) (sought to {} minutes backward)",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So as this

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is, but sounds odd/old fashion - 23b3706

new_start,
new_end,
minutes
Expand Down
2 changes: 1 addition & 1 deletion src/interpreter/flamegraph.rs
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ pub fn show(block: Columns) -> AppResult<()> {
})?;

// FIXME: note, right now I cannot use EventHandle with Tui, since EventHandle is not
// terminated gracefuly
// terminated gracefully
if event::poll(timeout).expect("failed to poll new events") {
match event::read().expect("unable to read event") {
CrosstermEvent::Key(e) => {
Expand Down
4 changes: 2 additions & 2 deletions src/interpreter/options.rs
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ pub enum ChDigViews {
ReplicatedFetches,
/// Show information about replicas (system.replicas)
Replicas,
/// Show all errors that happend in a server since start (system.errors)
/// Show all errors that happened in a server since start (system.errors)
Errors,
/// Show information about backups (system.backups)
Backups,
Expand Down Expand Up @@ -123,7 +123,7 @@ pub fn parse_datetime_or_date(value: &str) -> Result<DateTime<Local>, String> {
Ok(datetime) => return Ok(datetime),
Err(err) => errors.push(err),
}
// Pase as date
// Parse as date
match value.parse::<NaiveDate>() {
Ok(date) => {
return Ok(date
Expand Down
4 changes: 2 additions & 2 deletions src/interpreter/worker.rs
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ impl Worker {
// since standard does not allow to configure backlog (queue max size), while in case of
// very low --delay-interval it may fill queue with i.e. UpdateProcessList, which can be
// quite heavy, especially with the --cluster, and this will lead to UI will not show
// anything else until it will get to the event that is requried for that action.
// anything else until it will get to the event that is required for that action.
//
// Note, by default channel reserves slot for each sender [1].
//
Expand Down Expand Up @@ -194,7 +194,7 @@ async fn start_tokio(context: ContextArc, receiver: ReceiverArc) {
stopwatch.elapsed_ms(),
));

// It should not be reseted, since delay_interval should be set to the maximum service
// It should not be reset, since delay_interval should be set to the maximum service
// query duration time.
if stopwatch.elapsed() > options.view.delay_interval {
slow_processing = true;
Expand Down
2 changes: 1 addition & 1 deletion src/utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ pub fn highlight_sql(text: &String) -> Result<StyledString> {
.context("Cannot load SQL syntax")?,
&ts.themes["base16-ocean.dark"],
);
// NOTE: parse() does not interpret syntect::highlighting::Color::a (alpha/tranparency)
// NOTE: parse() does not interpret syntect::highlighting::Color::a (alpha/transparency)
return cursive_syntect::parse(text, &mut highlighter, &syntax_set)
.context("Cannot highlight query");
}
Expand Down
2 changes: 1 addition & 1 deletion src/view/navigation.rs
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ impl Navigation for Cursive {
let is_paused;
{
let mut context = self.user_data::<ContextArc>().unwrap().lock().unwrap();
// NOTE: though it will be better to stop sending any message completelly, instead of
// NOTE: though it will be better to stop sending any message completely, instead of
// simply ignoring them
context.worker.toggle_pause();
is_paused = context.worker.is_paused();
Expand Down