Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22250 )

Change subject: IMPALA-12785: Add commands to control event-processor status
......................................................................


Patch Set 5:

(1 comment)

> Patch Set 4:
>
> Wouldn't it be great to know the status of the event processor in the 
> terminal?
> [localhost:21050] default> :event_processor('status');
> Query: :event_processor('status')
> Query submitted at: 2025-01-06 00:24:51 (Coordinator: 
> http://cloudera-22747:25000)
> 2025-01-06 00:24:51 [Exception]  ERROR: Query 
> 0742eff0775d2f46:c0b61a4d00000000 failed:
> Unknown command: STATUS

Good point! Added this command.

http://gerrit.cloudera.org:8080/#/c/22250/4/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java
File fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java:

http://gerrit.cloudera.org:8080/#/c/22250/4/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java@3757
PS4, Line 3757:           
Arrays.stream(MetastoreEventsProcessor.EventProcessorCmdType.values())
> Should we throw the list of acceptable status values here? like "Start, PAU
Yeah, that would be helpful.



--
To view, visit http://gerrit.cloudera.org:8080/22250
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5a19f67264cfe06a1819a22c0c4f0cf174c9b958
Gerrit-Change-Number: 22250
Gerrit-PatchSet: 5
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]>
Gerrit-Comment-Date: Mon, 06 Jan 2025 15:05:26 +0000
Gerrit-HasComments: Yes

Reply via email to