[GitHub] [drill] luocooong commented on pull request #2596: DRILL-8259: Supports advanced HBase persistence storage options

2022-07-26 Thread GitBox
luocooong commented on PR #2596: URL: https://github.com/apache/drill/pull/2596#issuecomment-1195301013 @cgivre @Z0ltrix I added two new options. If `namespace` is used, the `namespace:table` semantics are applied. And we can also use the `table` configuration only. The `family` is also an

[GitHub] [drill] luocooong commented on pull request #2596: DRILL-8259: Supports advanced HBase persistence storage options

2022-07-23 Thread GitBox
luocooong commented on PR #2596: URL: https://github.com/apache/drill/pull/2596#issuecomment-1193070577 > Why compaction_enabled": false? I thought compaction is important for hbase to boost performance? As you know, HBase is a nightmare for operational services due to the

[GitHub] [drill] luocooong commented on pull request #2596: DRILL-8259: Supports advanced HBase persistence storage options

2022-07-22 Thread GitBox
luocooong commented on PR #2596: URL: https://github.com/apache/drill/pull/2596#issuecomment-1192952208 @cgivre Hi, Thank you for the questions. Actually, Drill PStore' variables are split from the storage configurations, because the goal is to define the initial variables before the Drill