[ 
https://issues.apache.org/jira/browse/PHOENIX-5034?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16706742#comment-16706742
 ] 

Karan Mehta commented on PHOENIX-5034:
--------------------------------------

[~xucang] Parsing for specific text may not be a be a good idea.

I suggest you to use {{CompilableStatement stmt = parseStatement(sql);}} and 
use {{instanceOf}} to compare the type of statement with DropTable or 
AlterTable statement. 

PHOENIX-5054 can be solved if we add that here as well. Rather creating a 
dynamic statement from SYSCAT, we audit log everything into the table. This 
helps us with creating a history as well. [~elserj] thoughts?

> Log all critical statements in SYSTEM.LOG table.
> ------------------------------------------------
>
>                 Key: PHOENIX-5034
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-5034
>             Project: Phoenix
>          Issue Type: Improvement
>            Reporter: Xu Cang
>            Assignee: Xu Cang
>            Priority: Minor
>         Attachments: PHOENIX-5034-4.x-HBase-1.3.001.patch, 
> PHOENIX-5034-4.x-HBase-1.3.002.patch, PHOENIX-5034-4.x-HBase-1.3.003.patch, 
> PHOENIX-5034-4.x-HBase-1.3.004.patch
>
>
> In production, sometimes engineers see table got dropped unexpectedly. It's 
> not easy to SCAN raw table from HBase itself to understand what happened and 
> when the table get dropped.
> Since we already have SYSTEM.LOG query log facility in Phoenix that sampling 
> query statement (log 1% statement by default). It's good to always log 
> critical statements such as "DROP" or "ALTER" statements.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to