maruyue commented on issue #175:
URL: 
https://github.com/apache/doris-mcp-server/issues/175#issuecomment-5163721511

    The problems with the current old MCP server are:                           
                                                                                
                                                                                
                                                                                
    - It doesn't use database accounts, making it inconvenient for users to use 
and understand.                                                                 
    - Access control is not based on proper database permission isolation — 
instead, it naively checks whether query strings contain keywords like 
"delete",    which is unprofessional.                                           
                                                                                
       
    - There should be no password-free mode.                                    
                                                                                
    - Semantic models are stored locally with no update mechanism and no hot 
reload — how are users supposed to work with that?                              
   
                                                                                
                                                                                
    All of these need to be addressed together.   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to