deniskuzZ commented on PR #5036: URL: https://github.com/apache/hive/pull/5036#issuecomment-2451699235
@KiranVelumuri, @okumin sorry for the late jump-in, I overlooked that PR. Changes look good to me, however, I have 1 concern regarding the location of the proto events - it's fixed in sys tables after the schema initialization. What if the end-user decides to change it? Should we move these configs to the hidden and restricted lists? -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
