Skip to content

[Enhancement] (nereids)implement showFrontendsCommand in nereids#43122

Merged
starocean999 merged 1 commit intoapache:masterfrom
Vallishp:nereids_front_ends
Nov 16, 2024
Merged

[Enhancement] (nereids)implement showFrontendsCommand in nereids#43122
starocean999 merged 1 commit intoapache:masterfrom
Vallishp:nereids_front_ends

Conversation

@Vallishp
Copy link
Contributor

@Vallishp Vallishp commented Nov 1, 2024

What problem does this PR solve?

support showFrontendsCommand in nereids

Issue Number: close #42764

Related PR: #xxx

Problem Summary:

Check List (For Committer)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No colde files have been changed.
      • Other reason
        test_frontend.groovy cover this test case.

mysql> show frontends;
+-----------------------------------------+-------------+-------------+----------+-----------+---------+--------------------+----------+----------+-----------+------+-------+-------------------+---------------------+---------------------+----------+--------+-------------------------+------------------+
| Name | Host | EditLogPort | HttpPort | QueryPort | RpcPort | ArrowFlightSqlPort | Role | IsMaster | ClusterId | Join | Alive | ReplayedJournalId | LastStartTime | LastHeartbeat | IsHelper | ErrMsg | Version | CurrentConnected |
+-----------------------------------------+-------------+-------------+----------+-----------+---------+--------------------+----------+----------+-----------+------+-------+-------------------+---------------------+---------------------+----------+--------+-------------------------+------------------+
| fe_73d1d34b_7ec5_444a_9266_0299c34183e7 | 172.20.80.1 | 9010 | 8030 | 9030 | 9020 | -1 | FOLLOWER | true | 453869832 | true | true | 3425 | 2024-11-01 19:16:13 | 2024-11-01 19:31:54 | true | | doris-0.0.0--f56e46721e | Yes |
+-----------------------------------------+-------------+-------------+----------+-----------+---------+--------------------+----------+----------+-----------+------+-------+-------------------+---------------------+---------------------+----------+--------+-------------------------+------------------+
1 row in set (5.21 sec)

mysql> show frontends disks;
+-----------------------------------------+-------------+-----------+-------------------------------------+----------------+----------+------+-----------+---------+-------------+
| Name | Host | DirType | Dir | Filesystem | Capacity | Used | Available | UseRate | MountOn |
+-----------------------------------------+-------------+-----------+-------------------------------------+----------------+----------+------+-----------+---------+-------------+
| fe_73d1d34b_7ec5_444a_9266_0299c34183e7 | 172.20.80.1 | meta | /root/doris/doris_run/fe/doris-meta | /dev/nvme0n1p6 | 295G | 246G | 34G | 88% | /root/doris |
| fe_73d1d34b_7ec5_444a_9266_0299c34183e7 | 172.20.80.1 | log | /root/doris/doris_run/fe/log | /dev/nvme0n1p6 | 295G | 246G | 34G | 88% | /root/doris |
| fe_73d1d34b_7ec5_444a_9266_0299c34183e7 | 172.20.80.1 | audit-log | /test | overlay | 295G | 246G | 34G | 88% | / |
| fe_73d1d34b_7ec5_444a_9266_0299c34183e7 | 172.20.80.1 | temp | /root/doris/doris_run/fe/temp_dir | /dev/nvme0n1p6 | 295G | 246G | 34G | 88% | /root/doris |
| fe_73d1d34b_7ec5_444a_9266_0299c34183e7 | 172.20.80.1 | deploy | /root/doris/doris_run/fe | /dev/nvme0n1p6 | 295G | 246G | 34G | 88% | /root/doris |
+-----------------------------------------+-------------+-----------+-------------------------------------+----------------+----------+------+-----------+---------+-------------+
5 rows in set (0.00 sec)

  • Behavior changed:

    • No.
    • Yes.
  • Does this need documentation?

    • No.
    • Yes.
  • Release note

    None

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@Vallishp
Copy link
Contributor Author

Vallishp commented Nov 1, 2024

run buildall

@Vallishp Vallishp force-pushed the nereids_front_ends branch 3 times, most recently from b452b63 to 85446f7 Compare November 14, 2024 06:42
@starocean999
Copy link
Contributor

please resolve conflict

@Vallishp
Copy link
Contributor Author

run buildall

@Vallishp
Copy link
Contributor Author

run buildall

@Vallishp
Copy link
Contributor Author

run p0

Copy link
Contributor

@starocean999 starocean999 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Nov 15, 2024
@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

Copy link
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

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

LGTM

@starocean999 starocean999 merged commit 6e0fa77 into apache:master Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement] (nereids)implement showFrontendsCommand in nereids

4 participants