Re: [DISCUSS] Deprecate Java 8 support

2021-11-22 Thread Matthias Pohl
Thanks for constantly driving these maintenance topics, Chesnay. +1 from my side for deprecating Java 8. I see the point Jingsong is raising. But I agree with what David already said here. Deprecating the Java version is a tool to make users aware of it (same as starting this discussion thread).

Re: [DISCUSS] Deprecate Java 8 support

2021-11-22 Thread David Morávek
Thank you Chesnay for starting the discussion! This will generate bit of a work for some users, but it's a good thing to keep moving the project forward. Big +1 for this. Jingsong: Receiving this signal, the user may be unhappy because his application > may be all on Java 8. Upgrading is a big

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Yun Tang
Hi, For the likelihood of never deleting some SST files by RocksDB. Unfortunately, it could happen as current level compaction strategy in RocksDB is triggered by upper input level size reached to the threshold and the compaction priority cannot guarantee all files would be choosed during

Re: [DISCUSS] Improve the name and structure of job vertex and operator name for job

2021-11-22 Thread Yun Tang
Hi Wenlong, Thanks for the update. And +1 for this proposal. Best Yun Tang On 2021/11/22 07:28:25 "wenlong.lwl" wrote: > Hi, Yun Gao, Yun Tang, and Aitozi, > thanks for the suggestion again, I have added following section in proposed > changes at >

[jira] [Created] (FLINK-25011) Introduce VertexParallelismDecider

2021-11-22 Thread Lijie Wang (Jira)
Lijie Wang created FLINK-25011: -- Summary: Introduce VertexParallelismDecider Key: FLINK-25011 URL: https://issues.apache.org/jira/browse/FLINK-25011 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-25010) Speed up hive's createMRSplits by multi thread

2021-11-22 Thread Liu (Jira)
Liu created FLINK-25010: --- Summary: Speed up hive's createMRSplits by multi thread Key: FLINK-25010 URL: https://issues.apache.org/jira/browse/FLINK-25010 Project: Flink Issue Type: Improvement

Re: [DISCUSS] FLIP-190: Support Version Upgrades for Table API & SQL Programs

2021-11-22 Thread godfrey he
Hi Timo, Thanks for driving this discussion, the sql job's upgrading compatibility always is a big pain point. In the last version we completed some work, this FLIP will make the whole upgrade story possible. I have a few comments: 1) "EXPLAIN PLAN EXECUTE STATEMENT SET BEGIN ... END" is

Re: [DISCUSS] Deprecate Java 8 support

2021-11-22 Thread Jingsong Li
Hi Chesnay, Thanks for bringing this for discussion. We should dig deeper into the current Java version of Flink users. At least make sure Java 8 is not a mainstream version. Receiving this signal, the user may be unhappy because his application may be all on Java 8. Upgrading is a big job,

Re: [DISCUSS] Shall casting functions return null or throw exceptions for invalid input

2021-11-22 Thread Kurt Young
> This is what I don't really understand here: how adding a configuration option causes issues here? This is why: for most Flink production use cases I see, it's not like a couple of people manage ~5 Flink jobs, so they can easily track all the big changes in every minor Flink version. Typically

[jira] [Created] (FLINK-25009) Output slotSharingGroup as part of JsonGraph

2021-11-22 Thread Xinbin Huang (Jira)
Xinbin Huang created FLINK-25009: Summary: Output slotSharingGroup as part of JsonGraph Key: FLINK-25009 URL: https://issues.apache.org/jira/browse/FLINK-25009 Project: Flink Issue Type:

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Dawid Wysakowicz
There is one more fundamental issue with either of your two proposals that've just came to my mind. What happens if you have externalized checkpoints and the job fails before the initial checkpoint can be safely removed? You start the job from the latest created checkpoint and

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Roman Khachatryan
> If you assume the 1st checkpoint needs to be "full" you know you are not > allowed to use any shared files. > It's true you should know about the shared files of the previous checkpoint, > but e.g. RocksDB already tracks that. I mean that the design described by FLIP implies the following

[jira] [Created] (FLINK-25008) Improve behaviour of NotNullEnforcer when dropping records

2021-11-22 Thread Marios Trivyzas (Jira)
Marios Trivyzas created FLINK-25008: --- Summary: Improve behaviour of NotNullEnforcer when dropping records Key: FLINK-25008 URL: https://issues.apache.org/jira/browse/FLINK-25008 Project: Flink

Re: [DISCUSS] Conventions on assertions to use in tests

2021-11-22 Thread Francesco Guardiani
Hi all, Given I see generally consensus around having a convention and using assertj, I propose to merge these 2 PRs: * Add the explanation of this convention in our code quality guide: https://github.com/apache/flink-web/pull/482 * Add assertj to dependency management in the parent pom and link

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Dawid Wysakowicz
There is one more fundamental issue with either of your two proposals that've just came to my mind. What happens if you have externalized checkpoints and the job fails before the initial checkpoint can be safely removed? You have a situation where you have a retained checkpoint that was built on

Re: [DISCUSS] Deprecate Java 8 support

2021-11-22 Thread Ingo Bürk
Hi, also a +1 from me because of everything Chesnay already said. Ingo On Mon, Nov 22, 2021 at 4:41 PM Martijn Visser wrote: > Hi Chesnay, > > Thanks for bringing this up for discussion. Big +1 for dropping Java 8 and > deprecating it in 1.15, given that Java 8 support will end. We already

Re: [DISCUSS] Deprecate Java 8 support

2021-11-22 Thread Martijn Visser
Hi Chesnay, Thanks for bringing this up for discussion. Big +1 for dropping Java 8 and deprecating it in 1.15, given that Java 8 support will end. We already see other dependencies that Flink use either have dropped support for Java 8 (Trino) or are going to drop it (Kafka). Best regards,

Re: [DISCUSS] Shall casting functions return null or throw exceptions for invalid input

2021-11-22 Thread Francesco Guardiani
> NULL in SQL essentially means "UNKNOWN", it's not as scary as a null in java which will easily cause a NPE or some random behavior with a c++ function call. This is true from the user point of view, except our runtime doesn't treat null as some value where you can safely execute operations and

[DISCUSS] Deprecate Java 8 support

2021-11-22 Thread Chesnay Schepler
As with any Java version there comes a time when one needs to ask themselves for how long one intends to stick with it. With Java 17 being released 2 months ago, and the active support for Java 8 ending in 4 months, it is time for us to think about that with regard to Java 8. As such I'd

[jira] [Created] (FLINK-25007) Session window with dynamic gap doesn't work

2021-11-22 Thread Ori Popowski (Jira)
Ori Popowski created FLINK-25007: Summary: Session window with dynamic gap doesn't work Key: FLINK-25007 URL: https://issues.apache.org/jira/browse/FLINK-25007 Project: Flink Issue Type: Bug

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Dawid Wysakowicz
I don't think there is anything wrong with the API. My comment was about what's behind the API. With tracking the shared files on JM you can not say if you can clear the files after couple of checkpoints or 10s, 100s or 1000s, which translates into minutes/hours/days/weeks of processing. I think

[jira] [Created] (FLINK-25006) Investigate memory issues on CI

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-25006: Summary: Investigate memory issues on CI Key: FLINK-25006 URL: https://issues.apache.org/jira/browse/FLINK-25006 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-25005) Add java17-target profile

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-25005: Summary: Add java17-target profile Key: FLINK-25005 URL: https://issues.apache.org/jira/browse/FLINK-25005 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-25004) Disable spotless on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-25004: Summary: Disable spotless on Java 17 Key: FLINK-25004 URL: https://issues.apache.org/jira/browse/FLINK-25004 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-25003) RestClientTest#testConnectionTimeout fails on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-25003: Summary: RestClientTest#testConnectionTimeout fails on Java 17 Key: FLINK-25003 URL: https://issues.apache.org/jira/browse/FLINK-25003 Project: Flink

[jira] [Created] (FLINK-25002) Setup required --add-opens/--add-exports

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-25002: Summary: Setup required --add-opens/--add-exports Key: FLINK-25002 URL: https://issues.apache.org/jira/browse/FLINK-25002 Project: Flink Issue Type:

[jira] [Created] (FLINK-25001) Zookeeper 3.4 fails on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-25001: Summary: Zookeeper 3.4 fails on Java 17 Key: FLINK-25001 URL: https://issues.apache.org/jira/browse/FLINK-25001 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-25000) Scala 2.12.7 doesn't compile on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-25000: Summary: Scala 2.12.7 doesn't compile on Java 17 Key: FLINK-25000 URL: https://issues.apache.org/jira/browse/FLINK-25000 Project: Flink Issue Type:

[jira] [Created] (FLINK-24999) flink-python doesn't work on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24999: Summary: flink-python doesn't work on Java 17 Key: FLINK-24999 URL: https://issues.apache.org/jira/browse/FLINK-24999 Project: Flink Issue Type:

[jira] [Created] (FLINK-24998) SIGSEGV in Kryo / C2 CompilerThread on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24998: Summary: SIGSEGV in Kryo / C2 CompilerThread on Java 17 Key: FLINK-24998 URL: https://issues.apache.org/jira/browse/FLINK-24998 Project: Flink Issue

[jira] [Created] (FLINK-24997) count(null) not supported in flink sql query

2021-11-22 Thread zouyunhe (Jira)
zouyunhe created FLINK-24997: Summary: count(null) not supported in flink sql query Key: FLINK-24997 URL: https://issues.apache.org/jira/browse/FLINK-24997 Project: Flink Issue Type: Bug

[jira] [Created] (FLINK-24996) Update CI image to contain Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24996: Summary: Update CI image to contain Java 17 Key: FLINK-24996 URL: https://issues.apache.org/jira/browse/FLINK-24996 Project: Flink Issue Type:

[jira] [Created] (FLINK-24995) EnumValueSerializerCompatibilityTest fails to compile on later Scala versions

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24995: Summary: EnumValueSerializerCompatibilityTest fails to compile on later Scala versions Key: FLINK-24995 URL: https://issues.apache.org/jira/browse/FLINK-24995

[jira] [Created] (FLINK-24994) WindowedStream#reduce(ReduceFunction function) gives useless suggestion when reducefunction is richfunction.

2021-11-22 Thread bx123 (Jira)
bx123 created FLINK-24994: - Summary: WindowedStream#reduce(ReduceFunction function) gives useless suggestion when reducefunction is richfunction. Key: FLINK-24994 URL: https://issues.apache.org/jira/browse/FLINK-24994

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Roman Khachatryan
Thanks Dawid, Regarding clarity, I think that all proposals require waiting for some event: re-upload / checkpoint completion / api response. But with the current one, there is an assumption: "initial checkpoint can be deleted once a new one completes" (instead of just "initial checkpoint can be

[jira] [Created] (FLINK-24993) ExceptionUtils#isDirectOutOfMemoryError doesn't work on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24993: Summary: ExceptionUtils#isDirectOutOfMemoryError doesn't work on Java 17 Key: FLINK-24993 URL: https://issues.apache.org/jira/browse/FLINK-24993 Project:

Re: [DISCUSS] FLIP-190: Support Version Upgrades for Table API & SQL Programs

2021-11-22 Thread wenlong.lwl
Hi, Timo, thanks for driving the discussion and the preparation on the FLIP. This is a pain point of Flink SQL complaining by our users badly. I have seen many cases where our users suffer while trying to upgrade the flink version in order to take advantage of the bug fixes and performance

[jira] [Created] (FLINK-24992) StreamingWithStateTestBase does not compile on later Scala versions

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24992: Summary: StreamingWithStateTestBase does not compile on later Scala versions Key: FLINK-24992 URL: https://issues.apache.org/jira/browse/FLINK-24992 Project:

[jira] [Created] (FLINK-24991) FlinkStatistic does not compile on later Scala versions

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24991: Summary: FlinkStatistic does not compile on later Scala versions Key: FLINK-24991 URL: https://issues.apache.org/jira/browse/FLINK-24991 Project: Flink

[jira] [Created] (FLINK-24990) LookupJoinITCase fails on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24990: Summary: LookupJoinITCase fails on Java 17 Key: FLINK-24990 URL: https://issues.apache.org/jira/browse/FLINK-24990 Project: Flink Issue Type:

[jira] [Created] (FLINK-24989) Upgrade shade-plugin to 3.2.4

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24989: Summary: Upgrade shade-plugin to 3.2.4 Key: FLINK-24989 URL: https://issues.apache.org/jira/browse/FLINK-24989 Project: Flink Issue Type: Sub-task

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Dawid Wysakowicz
@Yun I think it is a good comment with I agree in principal. However, we use --fromSavepoint (cli), savepointPath (REST API), SavepointRestoreSettings for both restoring from a savepoint and an externalized checkpoint already. I wanted to voice that concern. Nevertheless I am fine with changing

[jira] [Created] (FLINK-24988) Upgrade lombok to 1.18.22

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24988: Summary: Upgrade lombok to 1.18.22 Key: FLINK-24988 URL: https://issues.apache.org/jira/browse/FLINK-24988 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-24987) Enhance ExternalizedCheckpointCleanup enum

2021-11-22 Thread Nicolaus Weidner (Jira)
Nicolaus Weidner created FLINK-24987: Summary: Enhance ExternalizedCheckpointCleanup enum Key: FLINK-24987 URL: https://issues.apache.org/jira/browse/FLINK-24987 Project: Flink Issue

[jira] [Created] (FLINK-24986) StateHandleSerializationTest fails on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24986: Summary: StateHandleSerializationTest fails on Java 17 Key: FLINK-24986 URL: https://issues.apache.org/jira/browse/FLINK-24986 Project: Flink Issue

[jira] [Created] (FLINK-24985) LocalBufferPoolDestroyTest#isInBlockingBufferRequest does not work on Java 17

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24985: Summary: LocalBufferPoolDestroyTest#isInBlockingBufferRequest does not work on Java 17 Key: FLINK-24985 URL: https://issues.apache.org/jira/browse/FLINK-24985

[jira] [Created] (FLINK-24984) Rename MAVEN_OPTS variable in azure yaml

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24984: Summary: Rename MAVEN_OPTS variable in azure yaml Key: FLINK-24984 URL: https://issues.apache.org/jira/browse/FLINK-24984 Project: Flink Issue Type:

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Roman Khachatryan
Hi, Thanks for the proposal Dawid, I have some questions and remarks: 1. How will stop-with-savepoint be handled? Shouldn't side effects be enforced in this case? (i.e. send notifyCheckpointComplete) 2. Instead of forcing re-upload, can we "inverse control" in no-claim mode? Anyways, any

[jira] [Created] (FLINK-24983) Upgrade surefire to 3.0.0-M5

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24983: Summary: Upgrade surefire to 3.0.0-M5 Key: FLINK-24983 URL: https://issues.apache.org/jira/browse/FLINK-24983 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-24982) File leak in AbstractRecoverableWriterTest

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24982: Summary: File leak in AbstractRecoverableWriterTest Key: FLINK-24982 URL: https://issues.apache.org/jira/browse/FLINK-24982 Project: Flink Issue

[jira] [Created] (FLINK-24981) Fix JDK12+ profile activations

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24981: Summary: Fix JDK12+ profile activations Key: FLINK-24981 URL: https://issues.apache.org/jira/browse/FLINK-24981 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-24980) Collect sizes of result partitions when tasks finish

2021-11-22 Thread Lijie Wang (Jira)
Lijie Wang created FLINK-24980: -- Summary: Collect sizes of result partitions when tasks finish Key: FLINK-24980 URL: https://issues.apache.org/jira/browse/FLINK-24980 Project: Flink Issue Type:

[jira] [Created] (FLINK-24979) Remove MaxPermSize configuration in HBase surefire config

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24979: Summary: Remove MaxPermSize configuration in HBase surefire config Key: FLINK-24979 URL: https://issues.apache.org/jira/browse/FLINK-24979 Project: Flink

Re: [DISCUSS] Definition of Done for Apache Flink

2021-11-22 Thread Matthias Pohl
I also like the checklist provided by our current PR template. One annoying thing in my opinion, though, is that we do not rely on checkboxes. Ingo already proposed such a change in [1]. Chesnay had some good points on certain items that were meant to be added to the template but are actually

[jira] [Created] (FLINK-24978) Upgrade Asm to 9.2

2021-11-22 Thread Chesnay Schepler (Jira)
Chesnay Schepler created FLINK-24978: Summary: Upgrade Asm to 9.2 Key: FLINK-24978 URL: https://issues.apache.org/jira/browse/FLINK-24978 Project: Flink Issue Type: Sub-task

[jira] [Created] (FLINK-24977) validatePKConstraints in KafkaDynamicTableFactory throws wrong exception message

2021-11-22 Thread Jingsong Lee (Jira)
Jingsong Lee created FLINK-24977: Summary: validatePKConstraints in KafkaDynamicTableFactory throws wrong exception message Key: FLINK-24977 URL: https://issues.apache.org/jira/browse/FLINK-24977

Re: [DISCUSS] FLIP-193: Snapshots ownership

2021-11-22 Thread Yun Gao
Hi, Very thanks Dawid for proposing the FLIP to clarify the ownership for the states. +1 for the overall changes since it makes the behavior clear and provide users a determined method to finally cleanup savepoints / retained checkpoints. Regarding the changes to the public interface, it

Re: [ANNOUNCE] New Apache Flink Committer - Jing Zhang

2021-11-22 Thread 刘建刚
Congratulations! Matthias Pohl 于2021年11月22日周一 下午4:10写道: > Congratulations :-) > > On Thu, Nov 18, 2021 at 3:23 AM Jingsong Li > wrote: > > > Congratulations, Jing! Well deserved! > > > > On Wed, Nov 17, 2021 at 3:00 PM Lincoln Lee > > wrote: > > > > > > Congratulations, Jing! > > > > > >

Re: [ANNOUNCE] New Apache Flink Committer - Jing Zhang

2021-11-22 Thread Matthias Pohl
Congratulations :-) On Thu, Nov 18, 2021 at 3:23 AM Jingsong Li wrote: > Congratulations, Jing! Well deserved! > > On Wed, Nov 17, 2021 at 3:00 PM Lincoln Lee > wrote: > > > > Congratulations, Jing! > > > > Best, > > Lincoln Lee > > > > > > Jing Zhang 于2021年11月17日周三 上午10:24写道: > > > > >

[jira] [Created] (FLINK-24976) sink utils not check the schema info between query and sink table

2021-11-22 Thread xiaodao (Jira)
xiaodao created FLINK-24976: --- Summary: sink utils not check the schema info between query and sink table Key: FLINK-24976 URL: https://issues.apache.org/jira/browse/FLINK-24976 Project: Flink

Re: [ANNOUNCE] New Apache Flink Committer - Fabian Paul

2021-11-22 Thread Matthias Pohl
Congratulations, Fabian. :-) On Tue, Nov 16, 2021 at 5:08 PM Zakelly Lan wrote: > Congratulations, Fabian > > Best, > Zakelly > > On Tue, Nov 16, 2021 at 11:21 PM liwei li wrote: > > > Congratulations Fabian :-) > > > > Piotr Nowojski 于2021年11月16日周二 下午11:16写道: > > > > > Congratulations :) > >

Re: [ANNOUNCE] New Apache Flink Committer - Yangze Guo

2021-11-22 Thread Matthias Pohl
Congratulations :-) On Tue, Nov 16, 2021 at 2:33 PM Benchao Li wrote: > Congratulations Yangze~ > > Dawid Wysakowicz 于2021年11月15日周一 下午10:28写道: > > > Congrats! > > > > On 15/11/2021 15:22, Marios Trivyzas wrote: > > > Congrats Yangze! > > > > > > On Mon, Nov 15, 2021 at 1:34 PM Martijn Visser