stevenzwu merged PR #9321:
URL: https://github.com/apache/iceberg/pull/9321
--
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, e-mail: issues-unsubscr...@iceberg.a
stevenzwu commented on PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#issuecomment-2023956526
thanks @pvary and @yegangy0718 for the code review
--
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
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1525823973
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Foundation
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1525279239
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1525279239
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1525236052
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1524317705
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1524316147
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1524316147
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1524300998
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1524300363
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1524298729
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1524298729
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,373 @@
+/*
+ * Licensed to the Apache Software Foundation
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1517846647
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apache Software Founda
yegangy0718 commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1516610638
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apache Software Foun
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1515180358
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,368 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1458488742
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1456707875
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1456707875
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1456529072
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1456527386
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1456529072
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1456527386
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1456527386
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434835739
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434835739
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434825620
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434816970
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434808593
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434802745
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434798523
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434342789
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434342789
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434415826
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Fo
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434415826
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Fo
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434415826
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Fo
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434407274
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434407787
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Fo
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434342789
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1434338290
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1433664382
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1433489506
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1433058848
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1433057435
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1433038190
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1433025320
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1432960039
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Fo
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1432682365
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1432641252
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1432620523
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1432613149
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
stevenzwu commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1432236943
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1430070789
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1430070376
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1430068605
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1430046432
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1430041035
##
flink/v1.17/flink/src/test/java/org/apache/iceberg/flink/sink/shuffle/TestMapRangePartitioner.java:
##
@@ -0,0 +1,511 @@
+/*
+ * Licensed to the Apache Software Founda
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1430040455
##
jmh.gradle:
##
@@ -21,10 +21,15 @@ if (jdkVersion != '8' && jdkVersion != '11' && jdkVersion
!= '17') {
throw new GradleException("The JMH benchamrks must be run w
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1430030696
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1429955804
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
pvary commented on code in PR #9321:
URL: https://github.com/apache/iceberg/pull/9321#discussion_r1429933857
##
flink/v1.17/flink/src/main/java/org/apache/iceberg/flink/sink/shuffle/MapRangePartitioner.java:
##
@@ -0,0 +1,288 @@
+/*
+ * Licensed to the Apache Software Foundation
61 matches
Mail list logo