Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-12-11 Thread via GitHub
jbonofre commented on PR #11669: URL: https://github.com/apache/iceberg/pull/11669#issuecomment-2534627053 Should we reopen this PR or is it superseded by another one ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-12-02 Thread via GitHub
ajantha-bhat commented on PR #11669: URL: https://github.com/apache/iceberg/pull/11669#issuecomment-2511694995 Yes, it is still active. But it is not getting enough reviews. I am facing very hard to get reviews. https://github.com/apache/iceberg/pull/11216 is the last PR that is n

Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-12-02 Thread via GitHub
ajantha-bhat commented on PR #11669: URL: https://github.com/apache/iceberg/pull/11669#issuecomment-2512123092 @deniskuzZ: Could you please comment on my last PR that this feature will be helpful for Hive? and you are looking for it. It might help get more attention for review. -- T

Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-11-28 Thread via GitHub
pvary commented on PR #11669: URL: https://github.com/apache/iceberg/pull/11669#issuecomment-2506498681 And here is the relevant mailing list thread: https://lists.apache.org/thread/knl1ol7s1o2p7rglgl2mm8c5mc2pk0sx @ajantha-bhat: Are you still working on the proposal? -- This is an

Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-11-28 Thread via GitHub
deniskuzZ commented on PR #11669: URL: https://github.com/apache/iceberg/pull/11669#issuecomment-2506025938 Found partition stats tracker issue https://github.com/apache/iceberg/issues/8450 with the following design doc: https://docs.google.com/document/d/1vaufuD47kMijz97LxM67X8OX-W2Wq7nmlz

Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-11-28 Thread via GitHub
deniskuzZ closed pull request #11669: Core: Support aggregated basic stats in partition summary URL: https://github.com/apache/iceberg/pull/11669 -- 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

Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-11-28 Thread via GitHub
deniskuzZ commented on PR #11669: URL: https://github.com/apache/iceberg/pull/11669#issuecomment-2505896721 @pvary, unfortunately, that won't work. I was looking for an easy way to get basic partition stats, however, I missed the part that iceberg only keeps the changed partitions in a Snap

Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-11-27 Thread via GitHub
pvary commented on PR #11669: URL: https://github.com/apache/iceberg/pull/11669#issuecomment-2504732771 This discussion could be relevant here too: https://lists.apache.org/thread/0q1csnkfg8jc11zo1dlssjkr4v8s8zz0 -- This is an automated message from the Apache Git Service. To respond to t

Re: [PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-11-27 Thread via GitHub
pvary commented on PR #11669: URL: https://github.com/apache/iceberg/pull/11669#issuecomment-2504724953 @deniskuzZ: Could you please provide a short description what data is stored in the summary and in what format? I think it is important to understand the cost for keeping this stat

[PR] Core: Support aggregated basic stats in partition summary [iceberg]

2024-11-27 Thread via GitHub
deniskuzZ opened a new pull request, #11669: URL: https://github.com/apache/iceberg/pull/11669 (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,