Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
aokolnychyi commented on PR #9047: URL: https://github.com/apache/iceberg/pull/9047#issuecomment-1815502707 Thanks, @singhpk234 @RussellSpitzer @flyrain! -- 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

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
aokolnychyi merged PR #9047: URL: https://github.com/apache/iceberg/pull/9047 -- 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: issues-unsubscr...@iceberg

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
aokolnychyi commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396432577 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
aokolnychyi commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396432577 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
RussellSpitzer commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396328736 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
RussellSpitzer commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396325583 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
aokolnychyi commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396260261 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
aokolnychyi commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396153376 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
aokolnychyi commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396151524 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
aokolnychyi commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396151524 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
RussellSpitzer commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396132411 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] API: Add CharSequenceMap [iceberg]

2023-11-16 Thread via GitHub
RussellSpitzer commented on code in PR #9047: URL: https://github.com/apache/iceberg/pull/9047#discussion_r1396130069 ## api/src/main/java/org/apache/iceberg/util/CharSequenceMap.java: ## @@ -0,0 +1,230 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[PR] API: Add CharSequenceMap [iceberg]

2023-11-12 Thread via GitHub
aokolnychyi opened a new pull request, #9047: URL: https://github.com/apache/iceberg/pull/9047 This PR adds `CharSequenceMap`, a map implementation similar to existing `CharSequenceSet`. This will be used in writers, delete file index, Spark executor-level content file cache. -- This is