[
https://issues.apache.org/jira/browse/HIVE-27013?focusedWorklogId=848291&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-848291
]
ASF GitHub Bot logged work on HIVE-27013:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 01/Mar/23 11:20
Start Date: 01/Mar/23 11:20
Worklog Time Spent: 10m
Work Description: deniskuzZ commented on code in PR #4088:
URL: https://github.com/apache/hive/pull/4088#discussion_r1121552124
##########
iceberg/iceberg-handler/src/main/java/org/apache/iceberg/mr/Catalogs.java:
##########
@@ -243,10 +243,13 @@ static Optional<Catalog> loadCatalog(Configuration conf,
String catalogName) {
* @return complete map of catalog properties
*/
private static Map<String, String> getCatalogProperties(Configuration conf,
String catalogName, String catalogType) {
- String keyPrefix = InputFormatConfig.CATALOG_CONFIG_PREFIX + catalogName;
Map<String, String> catalogProperties = Streams.stream(conf.iterator())
- .filter(e -> e.getKey().startsWith(keyPrefix))
- .collect(Collectors.toMap(e ->
e.getKey().substring(keyPrefix.length() + 1), Map.Entry::getValue));
+ .filter(e ->
e.getKey().startsWith(InputFormatConfig.GLOBAL_CATALOG_CONFIG_PREFIX)).collect(
Review Comment:
could we rewrite this to filter just in 1 iteration?
Issue Time Tracking
-------------------
Worklog Id: (was: 848291)
Time Spent: 0.5h (was: 20m)
> Provide an option to enable iceberg manifest caching for all catalogs
> ---------------------------------------------------------------------
>
> Key: HIVE-27013
> URL: https://issues.apache.org/jira/browse/HIVE-27013
> Project: Hive
> Issue Type: Improvement
> Components: Iceberg integration
> Reporter: Rajesh Balamohan
> Assignee: Ayush Saxena
> Priority: Major
> Labels: pull-request-available
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> {color:#222222}I tried the following thinking that it would work with iceberg
> manifest caching; but it didn't.{color}
> {noformat}
> alter table store_sales set
> tblproperties('io.manifest.cache-enabled'='true');{noformat}
> {color:#222222}Creating this ticket as a placeholder to fix the same.{color}
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)