zhangbutao commented on PR #6157:
URL: https://github.com/apache/hive/pull/6157#issuecomment-3479685680

   > According to my memory, removing this part of the code means the coloring 
output only means that the header has different font type than the result set 
data. 
   
   As I mentioned, this part of the code relates to the primary key and has 
never taken effect in Hive beeline, so I'm not very clear about the color or 
font of its output display. 
   
   > What about removing the coloring functionality at all? With this change 
Beeline still accepts the color parameter. But what is the effect of that 
parameter?
   
   Regarding whether to remove the entire coloring functionality code, I'm not 
entirely sure if there are real-world users who actually use the color option. 
However, through a Google search for the keyword `beeline --color`, I found 
many related articles. Therefore, I assume that the existence of the --color 
option is meaningful, and there is no need to completely remove the coloring 
functionality at this point.
   


-- 
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