zabetak opened a new pull request, #6226: URL: https://github.com/apache/hive/pull/6226
### Why are the changes needed? Observe that value of CBOPlan is in fact a JSON object so wrapping it in a string has various drawbacks: * Bigger size with lots of unnecessary whitespace and escaped characters * Poor readability since it cannot be formatted by a JSON processors * Deserialization overhead since consumers need to read the value of "CBOPlan" and retransform it to a JSON object in order to process it further ### Does this PR introduce _any_ user-facing change? Yes, since it changes the type of the CBOPlan attribute from string to object. The change should pass mostly unnoticed since very few clients/applications are actually using the content of the CBOPlan attribute. ### How was this patch tested? ``` mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=plan_json.q,excluded_rule_explain.q,materialized_view_rebuild_2.q,ctas.q,input4.q,join0.q,materialized_view_create.q,explain_ast.q,explain_locks.q,cte_cbo_plan_json.q,cbo_explain.q,authorization_explain.q,concat_op.q,tez_union.q,parallel_join0.q ``` -- 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]
