Hi Jing,
Thanks for letting us know. I think it should be ok
to merge this PR since the issue itself is a severe
bug that would block the releasing, and the fix itself
is not complex and is thus controllable.
Best,
Yun
--Original Mail --
Sender:Jing Ge
Send Da
Hi all,
I'd like to ask if it is fine to merge this PR[1]. It is an extension of
sink metrics that has been covered originally by FLIP-33 [2]. Such
extension has been pointed out there: "*The following metric convention is
not a complete list. More conventional metrics will be added over time.*"
T
Hi Fabian,
Thanks for letting us know. I believe that it should be ok to merge this PR
since it is only affecting the ES connector.
Cheers,
Till
On Wed, Feb 23, 2022 at 10:54 AM Fabian Paul wrote:
> Hi all,
>
> I would like to merge the following PR [1]. It has been approved
> before the featu
Hi all,
I would like to merge the following PR [1]. It has been approved
before the feature freeze but no one had time to merge it,
unfortunately. The feature is very contained and only adds a simple
capability to the Elastic connector when used with Flink SQL. If there
are no concerns until end o
Thanks for letting us know Wenlong. Since the SQL client testing hasn't
really started yet, I think it is ok to merge this PR.
Cheers,
Till
On Thu, Feb 17, 2022 at 2:32 PM wenlong.lwl wrote:
> Hi,all, I want to merge a pr (https://github.com/apache/flink/pull/18363)
> belonging to FLIP-190, whi
Hi,all, I want to merge a pr (https://github.com/apache/flink/pull/18363)
belonging to FLIP-190, which was approved yesterday, but not merged before
code freeze because of CI queueing and failed by other changes.
WDYT?
Best,
Wenlong
On Thu, 17 Feb 2022 at 02:08, Till Rohrmann wrote:
> Hi everyo