bbeaudreault commented on code in PR #4770:
URL: https://github.com/apache/hbase/pull/4770#discussion_r1081931496
##########
hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/mapreduce/HFileOutputFormat2.java:
##########
@@ -152,6 +153,10 @@ protected static byte[] combineTableNameSuffix(byte[]
tableName, byte[] suffix)
static final String BLOCK_SIZE_FAMILIES_CONF_KEY =
"hbase.mapreduce.hfileoutputformat.blocksize";
static final String DATABLOCK_ENCODING_FAMILIES_CONF_KEY =
"hbase.mapreduce.hfileoutputformat.families.datablock.encoding";
+ public static final String TABLE_NAME_WITH_NAMESPACE_INCLUSIVE_KEY =
Review Comment:
can you just add a quick comment to this:
```
// When MULTI_TABLE_HFILEOUTPUTFORMAT_CONF_KEY is enabled, should table
names be written
// with namespace included. Enabling this means downstream jobs which use
this output will need to account
// for namespace when finding the directory of the job output.
// For example: `/output/default/my-table` (new) vs `/output/my-table` (old)
// This will be the behavior when upgrading to hbase 3.0.
```
--
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: issues-unsubscr...@hbase.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
