zhangbutao commented on PR #4913: URL: https://github.com/apache/hive/pull/4913#issuecomment-1854063223
> We are explicitily adding MASK_TOTAL_SIZE in every file, can we have it enabled always > > ``` > public static final String HDFS_MASK = "### HDFS PATH ###"; > public static final String HDFS_DATE_MASK = "### HDFS DATE ###"; > public static final String HDFS_USER_MASK = "### USER ###"; > public static final String HDFS_GROUP_MASK = "### GROUP ###"; > ``` > > The HDFS Path and all, get masked always, we need not specify for all qFiles, can't we do something similar. Thanks @ayushtkn for suggestion, yes, this is the most direct way to mask `totalSize`. @zratkai I'm sorry I gave you an wrong reference(HIVE-19612) which is not a complete solution. Would you mind to refine this PR again? Thanks. -- 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
