Skip to content

Draft: Hive iceberg materialized view #6393

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
InvisibleProgrammer wants to merge 30 commits into
base: master
Choose a base branch
from

Attempt to fix upstream compile error

8d6463f
Select commit
Loading
Failed to load commit list.
Draft

Draft: Hive iceberg materialized view #6393

Attempt to fix upstream compile error
8d6463f
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Apr 4, 2026 in 17m 36s

Quality Gate passed

Issues
64 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
1.1% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 1780 in standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/MetaStoreServerUtils.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 120 characters (found 129). 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa4JZ_dZP7ZB6obL&open=AZ02sa4JZ_dZP7ZB6obL&pullRequest=6393

Check warning on line 260 in ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.locationtech.jts.geom.util.GeometryMapper.flatMap'. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa2xZ_dZP7ZB6obC&open=AZ02sa2xZ_dZP7ZB6obC&pullRequest=6393

Check failure on line 427 in iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/HiveIcebergOutputCommitter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa_YZ_dZP7ZB6obe&open=AZ02sa_YZ_dZP7ZB6obe&pullRequest=6393

Check warning on line 2893 in ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'case' child has incorrect indentation level 6, expected level should be 4. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sazDZ_dZP7ZB6oa9&open=AZ02sazDZ_dZP7ZB6oa9&pullRequest=6393

Check warning on line 409 in itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unnecessary null check; "instanceof" returns false for nulls. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa-MZ_dZP7ZB6obZ&open=AZ02sa-MZ_dZP7ZB6obZ&pullRequest=6393

Check warning on line 354 in iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/Catalogs.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this "switch" statement by "if" statements to increase readability. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sbBCZ_dZP7ZB6obk&open=AZ02sbBCZ_dZP7ZB6obk&pullRequest=6393

Check warning on line 162 in standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/handler/DropTableHandler.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

')' is preceded with whitespace. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa3hZ_dZP7ZB6obK&open=AZ02sa3hZ_dZP7ZB6obK&pullRequest=6393

Check failure on line 2130 in common/src/java/org/apache/hadoop/hive/conf/HiveConf.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "metastore" 4 times. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sbFVZ_dZP7ZB6oby&open=AZ02sbFVZ_dZP7ZB6oby&pullRequest=6393

Check warning on line 36 in standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/TableType.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 120 characters (found 124). 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa9JZ_dZP7ZB6obU&open=AZ02sa9JZ_dZP7ZB6obU&pullRequest=6393

Check warning on line 37 in ql/src/java/org/apache/hadoop/hive/ql/ddl/view/create/CreateMaterializedViewDesc.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.apache.hadoop.hive.ql.metadata.DefaultStorageHandler'. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02saq8Z_dZP7ZB6oa0&open=AZ02saq8Z_dZP7ZB6oa0&pullRequest=6393

Check warning on line 36 in standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/TableType.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

The type of "ALL_MATERIALIZED_VIEWS" should be an interface such as "Set" rather than the implementation "EnumSet". 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa9JZ_dZP7ZB6obO&open=AZ02sa9JZ_dZP7ZB6obO&pullRequest=6393

Check failure on line 371 in iceberg/patched-iceberg-core/src/main/java/org/apache/iceberg/view/ViewMetadata.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to not always return the same value. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sbCKZ_dZP7ZB6obt&open=AZ02sbCKZ_dZP7ZB6obt&pullRequest=6393

Check warning on line 38 in standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/TableType.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make this member "protected". 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa9JZ_dZP7ZB6obT&open=AZ02sa9JZ_dZP7ZB6obT&pullRequest=6393

Check warning on line 411 in itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant jump. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa-MZ_dZP7ZB6obY&open=AZ02sa-MZ_dZP7ZB6obY&pullRequest=6393

Check warning on line 398 in itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa-MZ_dZP7ZB6obX&open=AZ02sa-MZ_dZP7ZB6obX&pullRequest=6393

Check warning on line 210 in standalone-metastore/metastore-client/src/main/java/org/apache/hadoop/hive/metastore/client/HookEnabledMetaStoreClient.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 120 characters (found 123). 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa9gZ_dZP7ZB6obW&open=AZ02sa9gZ_dZP7ZB6obW&pullRequest=6393

Check warning on line 38 in standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/TableType.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

The type of "ALL_VIEWS" should be an interface such as "Set" rather than the implementation "EnumSet". 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa9JZ_dZP7ZB6obQ&open=AZ02sa9JZ_dZP7ZB6obQ&pullRequest=6393

Check warning on line 37 in ql/src/java/org/apache/hadoop/hive/ql/ddl/view/create/CreateMaterializedViewDesc.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unused import - org.apache.hadoop.hive.ql.metadata.DefaultStorageHandler. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02saq8Z_dZP7ZB6oa1&open=AZ02saq8Z_dZP7ZB6oa1&pullRequest=6393

Check warning on line 27 in iceberg/patched-iceberg-core/src/main/java/org/apache/iceberg/view/StorageTableParser.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "name" private field. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sbCBZ_dZP7ZB6obr&open=AZ02sbCBZ_dZP7ZB6obr&pullRequest=6393

Check warning on line 229 in iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/IcebergTableUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof MaterializedView materializedview' 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa-pZ_dZP7ZB6obc&open=AZ02sa-pZ_dZP7ZB6obc&pullRequest=6393

Check warning on line 58 in ql/src/java/org/apache/hadoop/hive/ql/parse/StorageFormat.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make this member "protected". 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sau5Z_dZP7ZB6oa5&open=AZ02sau5Z_dZP7ZB6oa5&pullRequest=6393

Check warning on line 561 in iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/hive/BaseHiveIcebergMetaHook.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge the previous cases into this one using comma-separated label. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sbAeZ_dZP7ZB6obh&open=AZ02sbAeZ_dZP7ZB6obh&pullRequest=6393

Check failure on line 283 in iceberg/patched-iceberg-core/src/main/java/org/apache/iceberg/view/ViewMetadata.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to not always return the same value. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sbCKZ_dZP7ZB6obs&open=AZ02sbCKZ_dZP7ZB6obs&pullRequest=6393

Check warning on line 1912 in ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 120 characters (found 168). 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sa2xZ_dZP7ZB6obE&open=AZ02sa2xZ_dZP7ZB6obE&pullRequest=6393

Check failure on line 46 in iceberg/patched-iceberg-core/src/main/java/org/apache/iceberg/view/BaseMetastoreViewCatalog.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. 

See more on https://sonarcloud.io/project/issues?id=apache_hive&issues=AZ02sbB1Z_dZP7ZB6obm&open=AZ02sbB1Z_dZP7ZB6obm&pullRequest=6393
View more details on SonarQubeCloud