morningman opened a new pull request, #23801:
URL: https://github.com/apache/doris/pull/23801

   ## Proposed changes
   
   The `CreateRowPolicyCommand` is implemented with overriding `run()` method.
   
   So when executing `create row policy` in non-master FE, and forward it to 
Master FE,
   it will call `execute(TUniqueId queryId)` method and go through 
`executeByNereids()`.
   And because without `run()` method, it will do nothing and return OK.
   So after `show row policy`, user will get empty result.
   
   This PR fix it by implmenting the `run()` method but throw an Exception, so 
that it will
   fallback to old planner, to do the creating row policy command normally.
   
   The full implement of `run()` method should be implemented later.
   This is just a tmp fix.
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at 
[d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you 
chose the solution you did and what alternatives you considered, etc...
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to