Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16548 )

Change subject: IMPALA-10219: Expose DEBUG_ACTION query option to reset metadata
......................................................................


Patch Set 4:

(9 comments)

http://gerrit.cloudera.org:8080/#/c/16548/4/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java
File fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java:

http://gerrit.cloudera.org:8080/#/c/16548/4/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java@2284
PS4, Line 2284:   public TCatalogObject reloadTable(Table tbl, 
TResetMetadataRequest request, String reason)
line too long (92 > 90)


http://gerrit.cloudera.org:8080/#/c/16548/4/fe/src/main/java/org/apache/impala/catalog/HdfsTable.java
File fe/src/main/java/org/apache/impala/catalog/HdfsTable.java:

http://gerrit.cloudera.org:8080/#/c/16548/4/fe/src/main/java/org/apache/impala/catalog/HdfsTable.java@1149
PS4, Line 1149:               storageMetadataLoadTime_ += 
updateUnpartitionedTableFileMd(client, debugAction);
line too long (94 > 90)


http://gerrit.cloudera.org:8080/#/c/16548/4/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
File fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java:

http://gerrit.cloudera.org:8080/#/c/16548/4/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@942
PS4, Line 942:             reloadFileMetadata, reloadTableSchema, false, 
partitionsToUpdate, null, reason);
line too long (92 > 90)


http://gerrit.cloudera.org:8080/#/c/16548/4/fe/src/test/java/org/apache/impala/util/DebugUtilsTest.java
File fe/src/test/java/org/apache/impala/util/DebugUtilsTest.java:

http://gerrit.cloudera.org:8080/#/c/16548/4/fe/src/test/java/org/apache/impala/util/DebugUtilsTest.java@32
PS4, Line 32:     
DebugUtils.executeDebugAction("TEST_SLEEP_ACTION:SLEEP@100|SOME_OTHER_ACTION:SLEEP@10",
line too long (91 > 90)


http://gerrit.cloudera.org:8080/#/c/16548/4/tests/custom_cluster/test_topic_update_frequency.py
File tests/custom_cluster/test_topic_update_frequency.py:

http://gerrit.cloudera.org:8080/#/c/16548/4/tests/custom_cluster/test_topic_update_frequency.py@12
PS4, Line 12: from multiprocessing.pool import ThreadPool
flake8: F401 'multiprocessing.pool.ThreadPool' imported but unused


http://gerrit.cloudera.org:8080/#/c/16548/4/tests/custom_cluster/test_topic_update_frequency.py@14
PS4, Line 14: import pytest
flake8: F401 'pytest' imported but unused


http://gerrit.cloudera.org:8080/#/c/16548/4/tests/custom_cluster/test_topic_update_frequency.py@15
PS4, Line 15: import threading
flake8: F401 'threading' imported but unused


http://gerrit.cloudera.org:8080/#/c/16548/4/tests/custom_cluster/test_topic_update_frequency.py@18
PS4, Line 18: from time import sleep
flake8: F401 'time.sleep' imported but unused


http://gerrit.cloudera.org:8080/#/c/16548/4/tests/custom_cluster/test_topic_update_frequency.py@20
PS4, Line 20: from tests.beeswax.impala_beeswax import ImpalaBeeswaxException
flake8: F401 'tests.beeswax.impala_beeswax.ImpalaBeeswaxException' imported but 
unused



--
To view, visit http://gerrit.cloudera.org:8080/16548
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7196b1ce76415a5faf3fa8575a26d22b2bf50b1
Gerrit-Change-Number: 16548
Gerrit-PatchSet: 4
Gerrit-Owner: Vihang Karajgaonkar <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
Gerrit-Comment-Date: Fri, 09 Oct 2020 23:13:22 +0000
Gerrit-HasComments: Yes

Reply via email to