[GitHub] [hive] kasakrisz commented on a diff in pull request #3761: HIVE-26737: Subquery returning wrong results when database has materialized views

2022-11-29 Thread GitBox
kasakrisz commented on code in PR #3761: URL: https://github.com/apache/hive/pull/3761#discussion_r1035633744 ## ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveSubQueryRemoveRule.java: ## @@ -104,44 +106,43 @@ private HiveSubQueryRemoveRule(RelOptRuleOperand o

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3802: HIVE-26771: Use DDLTask to created Iceberg table when running ctas statement

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3802: URL: https://github.com/apache/hive/pull/3802#discussion_r1035630068 ## iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergSerDe.java: ## @@ -120,16 +114,11 @@ public void initialize(@Nullable Configuration confi

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3802: HIVE-26771: Use DDLTask to created Iceberg table when running ctas statement

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3802: URL: https://github.com/apache/hive/pull/3802#discussion_r1035628537 ## iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergMetaHook.java: ## @@ -236,11 +236,25 @@ public void commitCreateTable(org.apache.hadoop.h

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3802: HIVE-26771: Use DDLTask to created Iceberg table when running ctas statement

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3802: URL: https://github.com/apache/hive/pull/3802#discussion_r1035625885 ## iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergMetaHook.java: ## @@ -236,11 +236,25 @@ public void commitCreateTable(org.apache.hadoop.h

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3802: HIVE-26771: Use DDLTask to created Iceberg table when running ctas statement

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3802: URL: https://github.com/apache/hive/pull/3802#discussion_r1035623871 ## iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergMetaHook.java: ## @@ -236,11 +236,25 @@ public void commitCreateTable(org.apache.hadoop.h

[GitHub] [hive] xiuzhu9527 commented on pull request #3762: HIVE-26738: Use spotless-maven-plugin to check and constrain unused imports

2022-11-29 Thread GitBox
xiuzhu9527 commented on PR #3762: URL: https://github.com/apache/hive/pull/3762#issuecomment-1331742477 @ayushtkn Have you had a chance to look over the PR I sent you? THX! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [hive] xiuzhu9527 commented on pull request #3764: HIVE-26739: Fix the hiveserver2 error(No valid credentials provided) connecting metastore when HADOOP_USER_NAME environment variable exists

2022-11-29 Thread GitBox
xiuzhu9527 commented on PR #3764: URL: https://github.com/apache/hive/pull/3764#issuecomment-1331742130 @deniskuzZ Have you had a chance to look over the PR I sent you? THX! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [hive] shameersss1 commented on pull request #3807: HIVE-26778: Pushdown Date data type to metastore via directsql/JDO

2022-11-29 Thread GitBox
shameersss1 commented on PR #3807: URL: https://github.com/apache/hive/pull/3807#issuecomment-1331643135 @kasakrisz - I have addressed your comments. Please review the changes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [hive] sonarcloud[bot] commented on pull request #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3813: URL: https://github.com/apache/hive/pull/3813#issuecomment-1331633764 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] sonarcloud[bot] commented on pull request #3781: HIVE-26758: Allow use scratchdir for staging final job

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3781: URL: https://github.com/apache/hive/pull/3781#issuecomment-1331577058 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] sonarcloud[bot] commented on pull request #3816: Update orc

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3816: URL: https://github.com/apache/hive/pull/3816#issuecomment-1331516395 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] github-actions[bot] closed pull request #3618: HIVE-26557: AbstractMethodError when running TestWebHCatE2e

2022-11-29 Thread GitBox
github-actions[bot] closed pull request #3618: HIVE-26557: AbstractMethodError when running TestWebHCatE2e URL: https://github.com/apache/hive/pull/3618 -- 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

[GitHub] [hive] difin opened a new pull request, #3816: Update orc

2022-11-29 Thread GitBox
difin opened a new pull request, #3816: URL: https://github.com/apache/hive/pull/3816 ### What changes were proposed in this pull request? Bumping to 1.8.0. Also includes as part of upgrade to ORC 1.7.2: LLAP EncodedTreeReaderFactory is implementing its own TreeReaderFact

[GitHub] [hive] sonarcloud[bot] commented on pull request #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3813: URL: https://github.com/apache/hive/pull/3813#issuecomment-1331433988 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] cnauroth opened a new pull request, #3815: HIVE-26702: Backport HIVE-17317 (DBCP and HikariCP property configuration support) to 3.2.0.

2022-11-29 Thread GitBox
cnauroth opened a new pull request, #3815: URL: https://github.com/apache/hive/pull/3815 ### What changes were proposed in this pull request? Backport [HIVE-17317](https://issues.apache.org/jira/browse/HIVE-17317). This patch introduced support for DBCP as a connection pool for the me

[GitHub] [hive] nrg4878 merged pull request #3776: HIVE-26753:Upgrade Apache Ivy to 2.5.1 due to CVE-2022-37865, CVE-202…

2022-11-29 Thread GitBox
nrg4878 merged PR #3776: URL: https://github.com/apache/hive/pull/3776 -- 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.or

[GitHub] [hive] ayushtkn commented on pull request #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
ayushtkn commented on PR #3813: URL: https://github.com/apache/hive/pull/3813#issuecomment-1331264062 Thanx @cnauroth for the review, Have addressed the review comments, my initial use case was around the queries only so ignored setting user in the other places, I have now added everywhere

[GitHub] [hive] ayushtkn commented on a diff in pull request #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
ayushtkn commented on code in PR #3813: URL: https://github.com/apache/hive/pull/3813#discussion_r1035251314 ## cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java: ## @@ -250,7 +250,8 @@ CommandProcessorResponse processLocalCmd(String cmd, CommandProcessor proc, CliS

[GitHub] [hive] ayushtkn commented on a diff in pull request #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
ayushtkn commented on code in PR #3813: URL: https://github.com/apache/hive/pull/3813#discussion_r1035250375 ## service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java: ## @@ -328,7 +328,8 @@ public Object run() throws HiveSQLException { if (!embedded

[GitHub] [hive] ayushtkn commented on a diff in pull request #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
ayushtkn commented on code in PR #3813: URL: https://github.com/apache/hive/pull/3813#discussion_r1035247107 ## service/src/java/org/apache/hive/service/cli/operation/Operation.java: ## @@ -237,7 +238,9 @@ protected void createOperationLog() { * Set up some preconditions, or

[GitHub] [hive] yigress commented on pull request #3781: HIVE-26758: Allow use scratchdir for staging final job

2022-11-29 Thread GitBox
yigress commented on PR #3781: URL: https://github.com/apache/hive/pull/3781#issuecomment-1331243809 @pvary can you help review this? Thank you! -- 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 th

[GitHub] [hive] cnauroth commented on a diff in pull request #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
cnauroth commented on code in PR #3813: URL: https://github.com/apache/hive/pull/3813#discussion_r1035232269 ## service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java: ## @@ -328,7 +328,8 @@ public Object run() throws HiveSQLException { if (!embedded

[GitHub] [hive] ayushtkn merged pull request #3810: HIVE-26786: Iceberg: Read queries with copy-on-write failing.

2022-11-29 Thread GitBox
ayushtkn merged PR #3810: URL: https://github.com/apache/hive/pull/3810 -- 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.o

[GitHub] [hive] saihemanth-cloudera commented on a diff in pull request #3599: HIVE-26537: Deprecate older APIs in the HMS thrift interface.

2022-11-29 Thread GitBox
saihemanth-cloudera commented on code in PR #3599: URL: https://github.com/apache/hive/pull/3599#discussion_r1035187132 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java: ## @@ -1363,18 +1363,62 @@ private void create_databas

[GitHub] [hive] sonarcloud[bot] commented on pull request #3814: HIVE-26790: update protobug/grpc versions, use them

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3814: URL: https://github.com/apache/hive/pull/3814#issuecomment-1331130659 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] sonarcloud[bot] commented on pull request #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3813: URL: https://github.com/apache/hive/pull/3813#issuecomment-1331107654 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] henrib opened a new pull request, #3814: HIVE-26790: update protobug/grpc versions, use them

2022-11-29 Thread GitBox
henrib opened a new pull request, #3814: URL: https://github.com/apache/hive/pull/3814 [

[GitHub] [hive] ayushtkn opened a new pull request, #3813: HIVE-26789: Add UserName in CallerContext for queries.

2022-11-29 Thread GitBox
ayushtkn opened a new pull request, #3813: URL: https://github.com/apache/hive/pull/3813 ### What changes were proposed in this pull request? Add UserName in CallerContext for HDFS Audit logs to pick. ### Why are the changes needed? With Impersonation turned off, every ca

[GitHub] [hive] sonarcloud[bot] commented on pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3746: URL: https://github.com/apache/hive/pull/3746#issuecomment-1330925882 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] sonarcloud[bot] commented on pull request #3812: HIVE-26788: Update stats of table/partition after minor compaction us…

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3812: URL: https://github.com/apache/hive/pull/3812#issuecomment-1330870758 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] shameersss1 commented on a diff in pull request #3807: HIVE-26778: Pushdown Date data type to metastore via directsql/JDO

2022-11-29 Thread GitBox
shameersss1 commented on code in PR #3807: URL: https://github.com/apache/hive/pull/3807#discussion_r1034825055 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/parser/Filter.g: ## @@ -141,7 +141,11 @@ binOpExpression ( (k

[GitHub] [hive] sonarcloud[bot] commented on pull request #3807: HIVE-26778: Pushdown Date data type to metastore via directsql/JDO

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3807: URL: https://github.com/apache/hive/pull/3807#issuecomment-1330564077 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] sonarcloud[bot] commented on pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3746: URL: https://github.com/apache/hive/pull/3746#issuecomment-1330515935 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] zabetak commented on a diff in pull request #3448: HIVE-26400: Provide docker images for Hive

2022-11-29 Thread GitBox
zabetak commented on code in PR #3448: URL: https://github.com/apache/hive/pull/3448#discussion_r1034626727 ## dev-support/docker/docker-compose.yml: ## @@ -0,0 +1,71 @@ +version: '3.9' +services: Review Comment: OK I've seen that there is an intention to also launch HMS as

[GitHub] [hive] zabetak commented on a diff in pull request #3448: HIVE-26400: Provide docker images for Hive

2022-11-29 Thread GitBox
zabetak commented on code in PR #3448: URL: https://github.com/apache/hive/pull/3448#discussion_r1034624562 ## dev-support/docker/docker-compose.yml: ## @@ -0,0 +1,71 @@ +version: '3.9' +services: Review Comment: We could have HS2, Metastore, DBMS in a single image; wouldn't

[GitHub] [hive] zabetak commented on a diff in pull request #3448: HIVE-26400: Provide docker images for Hive

2022-11-29 Thread GitBox
zabetak commented on code in PR #3448: URL: https://github.com/apache/hive/pull/3448#discussion_r1034613441 ## dev-support/docker/README.md: ## @@ -0,0 +1,93 @@ +### Introduction + +--- +Run Apache Hive inside docker container in pseudo-distributed mode, with MySQL as its back

[GitHub] [hive] kasakrisz commented on a diff in pull request #3807: HIVE-26778: Pushdown Date data type to metastore via directsql/JDO

2022-11-29 Thread GitBox
kasakrisz commented on code in PR #3807: URL: https://github.com/apache/hive/pull/3807#discussion_r1034612361 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/parser/Filter.g: ## @@ -141,7 +141,11 @@ binOpExpression ( (key

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3803: HIVE-26770: Make end of loop compaction logs appear more selectively and reduce code duplication

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3803: URL: https://github.com/apache/hive/pull/3803#discussion_r1034607698 ## ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorThread.java: ## @@ -61,6 +61,13 @@ public abstract class CompactorThread extends Thread implements Confi

[GitHub] [hive] shameersss1 commented on a diff in pull request #3807: HIVE-26778: Pushdown Date data type to metastore via directsql/JDO

2022-11-29 Thread GitBox
shameersss1 commented on code in PR #3807: URL: https://github.com/apache/hive/pull/3807#discussion_r1034579395 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/parser/Filter.g: ## @@ -141,7 +141,11 @@ binOpExpression ( (k

[GitHub] [hive] shameersss1 commented on a diff in pull request #3807: HIVE-26778: Pushdown Date data type to metastore via directsql/JDO

2022-11-29 Thread GitBox
shameersss1 commented on code in PR #3807: URL: https://github.com/apache/hive/pull/3807#discussion_r1034579395 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/parser/Filter.g: ## @@ -141,7 +141,11 @@ binOpExpression ( (k

[GitHub] [hive] sonarcloud[bot] commented on pull request #3803: HIVE-26770: Make end of loop compaction logs appear more selectively and reduce code duplication

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3803: URL: https://github.com/apache/hive/pull/3803#issuecomment-1330407841 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] shameersss1 commented on a diff in pull request #3807: HIVE-26778: Pushdown Date data type to metastore via directsql/JDO

2022-11-29 Thread GitBox
shameersss1 commented on code in PR #3807: URL: https://github.com/apache/hive/pull/3807#discussion_r1034468264 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/parser/Filter.g: ## @@ -141,7 +141,11 @@ binOpExpression ( (k

[GitHub] [hive] veghlaci05 commented on a diff in pull request #3803: HIVE-26770: Make end of loop compaction logs appear more selectively and reduce code duplication

2022-11-29 Thread GitBox
veghlaci05 commented on code in PR #3803: URL: https://github.com/apache/hive/pull/3803#discussion_r1034502962 ## ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorThread.java: ## @@ -61,6 +61,13 @@ public abstract class CompactorThread extends Thread implements Conf

[GitHub] [hive] akshat0395 commented on a diff in pull request #3803: HIVE-26770: Make end of loop compaction logs appear more selectively and reduce code duplication

2022-11-29 Thread GitBox
akshat0395 commented on code in PR #3803: URL: https://github.com/apache/hive/pull/3803#discussion_r1034499433 ## ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Cleaner.java: ## @@ -104,9 +103,8 @@ public class Cleaner extends MetaStoreCompactorThread { @Override publ

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3803: HIVE-26770: Make end of loop compaction logs appear more selectively and reduce code duplication

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3803: URL: https://github.com/apache/hive/pull/3803#discussion_r1034492451 ## ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Cleaner.java: ## @@ -104,9 +103,8 @@ public class Cleaner extends MetaStoreCompactorThread { @Override publi

[GitHub] [hive] veghlaci05 commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
veghlaci05 commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034485550 ## ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/MRCompactor.java: ## @@ -261,7 +285,7 @@ public void run(HiveConf conf, String jobName, Table t, Partition p, St

[GitHub] [hive] shameersss1 commented on a diff in pull request #3807: HIVE-26778: Pushdown Date data type to metastore via directsql/JDO

2022-11-29 Thread GitBox
shameersss1 commented on code in PR #3807: URL: https://github.com/apache/hive/pull/3807#discussion_r1034468264 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/parser/Filter.g: ## @@ -141,7 +141,11 @@ binOpExpression ( (k

[GitHub] [hive] veghlaci05 commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
veghlaci05 commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034467684 ## ql/src/java/org/apache/hadoop/hive/ql/plan/FileSinkDesc.java: ## @@ -390,6 +394,17 @@ public boolean isCompactionTable() { : AcidUtils.isCompactionTable(get

[GitHub] [hive] veghlaci05 commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
veghlaci05 commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034448421 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnStore.java: ## @@ -115,6 +115,7 @@ enum MUTEX_KEY { // Compactor types

[GitHub] [hive] sonarcloud[bot] commented on pull request #3812: HIVE-26788: Update stats of table/partition after minor compaction us…

2022-11-29 Thread GitBox
sonarcloud[bot] commented on PR #3812: URL: https://github.com/apache/hive/pull/3812#issuecomment-1330272664 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034435640 ## ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactionQueryBuilder.java: ## @@ -200,22 +207,22 @@ CompactionQueryBuilder setIsDeleteDelta(boolean deleteDelta

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034433076 ## common/src/java/org/apache/hadoop/hive/common/FileUtils.java: ## @@ -1347,4 +1352,35 @@ public static RemoteIterator listFiles(FileSystem fs, Path pa return Rem

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034415840 ## ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/RebalanceQueryCompactor.java: ## @@ -0,0 +1,106 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034414269 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/CompactionTxnHandler.java: ## @@ -273,6 +273,7 @@ public CompactionInfo find

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034413527 ## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnStore.java: ## @@ -115,6 +115,7 @@ enum MUTEX_KEY { // Compactor types

[GitHub] [hive] deniskuzZ commented on a diff in pull request #3746: HIVE-26716: Query based Rebalance compaction on full acid tables

2022-11-29 Thread GitBox
deniskuzZ commented on code in PR #3746: URL: https://github.com/apache/hive/pull/3746#discussion_r1034411726 ## ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/MRCompactor.java: ## @@ -261,7 +285,7 @@ public void run(HiveConf conf, String jobName, Table t, Partition p, Sto