Skip to content

Conversation

@baiyangtx
Copy link

No description provided.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

摘要这里也提一下 Amoro 么?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

现状需要突出用 Iceberg 遇到问题即抛出使用 Amoro 的动机?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

正文过三分之一,终于见着 Amoro 啦 -_-(建议有的放矢)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

建议标明场景一、二,衔接有点脱,或者就某一场景完整叙述,或者在摘要里点一下。

Copy link

@zhanghuidinah zhanghuidinah Dec 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • 场景二内容也是正文过半,才有了 Amoro;

  • 整篇偏向于从虎牙角度讲述自己的完整实践过程,Amoro 为辅;作为社区用户案例最好突出 为什么用 Amoro

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amoro 社区 --> Amoro 开源社区

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

感觉这个代码没啥必要(不是关键代码),哈哈哈,简单说一下思路就好

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

10分钟一次也是因为我们用watermark来进行判定了,感觉watermark可以提到前面来讲

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里其实iceberg自己也有孤儿文件处理的action(spark), 不过之前我们用离线的方式在每次进行合并之后进行调用都会扫描一次,这个成本有点高,用amoro其实是降低了成本的

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants