Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-12-16 Thread via GitHub
nastra merged PR #11620: URL: https://github.com/apache/iceberg/pull/11620 -- 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.ap

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-12-15 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1886323591 ## core/src/main/java/org/apache/iceberg/SerializableTable.java: ## @@ -158,6 +160,21 @@ public Map properties() { return properties; } + public int formatV

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-12-13 Thread via GitHub
aokolnychyi commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1884369789 ## core/src/main/java/org/apache/iceberg/SerializableTable.java: ## @@ -158,6 +160,21 @@ public Map properties() { return properties; } + public int fo

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-12-13 Thread via GitHub
aokolnychyi commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1884367833 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,40 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contri

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-12-05 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1872737219 ## core/src/main/java/org/apache/iceberg/SerializableTable.java: ## @@ -62,13 +62,14 @@ public class SerializableTable implements Table, HasTableOperations, Serializa

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-12-05 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1872731920 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,40 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-12-05 Thread via GitHub
RussellSpitzer commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1872409911 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-12-05 Thread via GitHub
aokolnychyi commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1872275033 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,40 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contri

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-26 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1860118523 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-26 Thread via GitHub
aokolnychyi commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1859414335 ## core/src/main/java/org/apache/iceberg/SerializableTable.java: ## @@ -143,6 +143,10 @@ protected Table newTable(TableOperations ops, String tableName) { r

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-26 Thread via GitHub
aokolnychyi commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1859414335 ## core/src/main/java/org/apache/iceberg/SerializableTable.java: ## @@ -143,6 +143,10 @@ protected Table newTable(TableOperations ops, String tableName) { r

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-26 Thread via GitHub
aokolnychyi commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1859417454 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contri

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-26 Thread via GitHub
aokolnychyi commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1859414335 ## core/src/main/java/org/apache/iceberg/SerializableTable.java: ## @@ -143,6 +143,10 @@ protected Table newTable(TableOperations ops, String tableName) { r

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-22 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1853654808 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
RussellSpitzer commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852570677 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
RussellSpitzer commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852542188 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852565228 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
jbonofre commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852471394 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contribut

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
jbonofre commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852473169 ## core/src/main/java/org/apache/iceberg/SerializableTable.java: ## @@ -120,7 +120,7 @@ private FileIO fileIO(Table table) { return table.io(); } - privat

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852502401 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852487389 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
RussellSpitzer commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852452253 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852425487 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
RussellSpitzer commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852350086 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
RussellSpitzer commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852339111 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
RussellSpitzer commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852337171 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
nastra commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852176503 ## core/src/main/java/org/apache/iceberg/SerializableTable.java: ## @@ -120,7 +120,7 @@ private FileIO fileIO(Table table) { return table.io(); } - private

Re: [PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
jbonofre commented on code in PR #11620: URL: https://github.com/apache/iceberg/pull/11620#discussion_r1852161094 ## core/src/main/java/org/apache/iceberg/TableUtil.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contribut

[PR] Core: Add TableUtil to provide access to a table's format version [iceberg]

2024-11-21 Thread via GitHub
nastra opened a new pull request, #11620: URL: https://github.com/apache/iceberg/pull/11620 (no comment) -- 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-m