zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-491760194
Great, I have no other concerns now, then I could continue on this PR. :)
-
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-491258890
@tillrohrmann thanks for your further great suggestions!
I totally agree the above points
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-490529494
Thanks for the further review @tillrohrmann . Let me try to explain above
two concerns.
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-490377541
Yes, I think we are on the same page now.
I would focus on `b, c` in this PR, and launch
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-490348518
Yes, I think we are on the same page now.
I would focus on `b, c` in this PR, and launch
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-490011255
Yes, the blacklist could solve the hardware problems future.
As for d, you are right that
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-489901309
@tillrohrmann thanks for the further suggestions! I agree with your overall
ideas.
->Wha
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-489389513
Thanks for your professional reviews and valuable suggestions @tillrohrmann .
I think th
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-485687952
@flinkbot attention @tillrohrmann
zhijiangW commented on issue #8242: [FLINK-6227][network] Introduce the
DataConsumptionException for downstream task failure
URL: https://github.com/apache/flink/pull/8242#issuecomment-485687507
Some thoughts in implementation:
1. The `DataConsumptionException` should be general for
10 matches
Mail list logo