Skip to content

Comments

feat: show nameserver job#3293

Merged
dl239 merged 15 commits into4paradigm:mainfrom
dl239:feat/show_nameserver_job
Jun 19, 2023
Merged

feat: show nameserver job#3293
dl239 merged 15 commits into4paradigm:mainfrom
dl239:feat/show_nameserver_job

Conversation

@dl239
Copy link
Collaborator

@dl239 dl239 commented May 24, 2023

add new SQL statement to show the jobs of nameserver as below:
show jobs from nameserver;
show jobs from nameserver like '10';

@github-actions github-actions bot added execute-engine hybridse sql engine storage-engine openmldb storage engine. nameserver & tablet labels May 24, 2023
@codecov
Copy link

codecov bot commented May 24, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (8882a0a) 37.18% compared to head (4f3efdf) 37.18%.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #3293   +/-   ##
=========================================
  Coverage     37.18%   37.18%           
  Complexity      411      411           
=========================================
  Files           177      177           
  Lines         10214    10214           
  Branches       1185     1185           
=========================================
  Hits           3798     3798           
  Misses         6178     6178           
  Partials        238      238           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dl239 dl239 marked this pull request as ready for review May 25, 2023 03:35
@github-actions github-actions bot added the documentation Improvements or additions to documentation label May 25, 2023
@@ -1828,40 +1829,22 @@ std::shared_ptr<hybridse::sdk::ResultSet> SQLClusterRouter::HandleSQLCmd(const h
exit(0);
}
case hybridse::node::kCmdShowJobs: {
Copy link
Collaborator

Choose a reason for hiding this comment

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

just in case. @lqy222 should testing the compatibility between different versions, like old SDK and newer OpenMLDB server. To insure old approach works as expect

@dl239 dl239 merged commit 9e43558 into 4paradigm:main Jun 19, 2023
@dl239 dl239 deleted the feat/show_nameserver_job branch June 19, 2023 08:18
dl239 added a commit that referenced this pull request Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation execute-engine hybridse sql engine storage-engine openmldb storage engine. nameserver & tablet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants