[GitHub] [bigtop] evans-ye commented on a change in pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6.

2020-02-14 Thread GitBox
evans-ye commented on a change in pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6. URL: https://github.com/apache/bigtop/pull/596#discussion_r379591761 ## File path: bigtop-packages/src/common/flume/patch3-FLUME-3354.diff ## @@ -0,0 +1,67 @@

[GitHub] [bigtop] evans-ye commented on a change in pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6.

2020-02-14 Thread GitBox
evans-ye commented on a change in pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6. URL: https://github.com/apache/bigtop/pull/596#discussion_r379591761 ## File path: bigtop-packages/src/common/flume/patch3-FLUME-3354.diff ## @@ -0,0 +1,67 @@

[GitHub] [bigtop] sekikn commented on a change in pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6.

2020-02-14 Thread GitBox
sekikn commented on a change in pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6. URL: https://github.com/apache/bigtop/pull/596#discussion_r379680941 ## File path: bigtop-packages/src/common/flume/patch3-FLUME-3354.diff ## @@ -0,0 +1,67 @@

[GitHub] [bigtop] sekikn merged pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6.

2020-02-14 Thread GitBox
sekikn merged pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6. URL: https://github.com/apache/bigtop/pull/596 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [bigtop] evans-ye commented on a change in pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6.

2020-02-14 Thread GitBox
evans-ye commented on a change in pull request #596: BIGTOP-3304. Fix build failure of Flume against hive-2.3.6. URL: https://github.com/apache/bigtop/pull/596#discussion_r379713443 ## File path: bigtop-packages/src/common/flume/patch3-FLUME-3354.diff ## @@ -0,0 +1,67 @@

[GitHub] [bigtop] evans-ye commented on issue #597: BIGTOP-3165. Bump Spark to 2.4.4.

2020-02-14 Thread GitBox
evans-ye commented on issue #597: BIGTOP-3165. Bump Spark to 2.4.4. URL: https://github.com/apache/bigtop/pull/597#issuecomment-586547392 I've no much knowledge here so CMIIW. Why we need to add these LICENSE files? I though it should be able to obtain from spark's code release.

[GitHub] [bigtop] tsudukim commented on issue #597: BIGTOP-3165. Bump Spark to 2.4.4.

2020-02-14 Thread GitBox
tsudukim commented on issue #597: BIGTOP-3165. Bump Spark to 2.4.4. URL: https://github.com/apache/bigtop/pull/597#issuecomment-586564757 @evans-ye Thank you for your review! I agree with you, license files should be able to obtain from spark's source but they aren't. Spark community

[GitHub] [bigtop] tsudukim opened a new pull request #597: BIGTOP-3165. Bump Spark to 2.4.4.

2020-02-14 Thread GitBox
tsudukim opened a new pull request #597: BIGTOP-3165. Bump Spark to 2.4.4. URL: https://github.com/apache/bigtop/pull/597 Bump Spark to 2.4.4. `./docker-hadoop.sh -c 1 -s` passed on my local with the following configuration. ``` docker: memory_limit: "16g"

[GitHub] [bigtop] sekikn commented on issue #597: BIGTOP-3165. Bump Spark to 2.4.4.

2020-02-14 Thread GitBox
sekikn commented on issue #597: BIGTOP-3165. Bump Spark to 2.4.4. URL: https://github.com/apache/bigtop/pull/597#issuecomment-586166811 Basically the PR looks good to me but I'd like to upgrade Spark to 2.4.5, because 2.4.4 doesn't work with the recent k8s versions. Could you address it?

[GitHub] [bigtop] tsudukim commented on issue #597: BIGTOP-3165. Bump Spark to 2.4.4.

2020-02-14 Thread GitBox
tsudukim commented on issue #597: BIGTOP-3165. Bump Spark to 2.4.4. URL: https://github.com/apache/bigtop/pull/597#issuecomment-586178748 @sekikn Thank you for your comment. I will update this PR to 2.4.5 after BOM is updated on BIGTOP-3123.

[GitHub] [bigtop] sekikn commented on issue #569: BIGTOP-3237: Add smoke tests for YCSB

2020-02-14 Thread GitBox
sekikn commented on issue #569: BIGTOP-3237: Add smoke tests for YCSB URL: https://github.com/apache/bigtop/pull/569#issuecomment-586168439 The PR worked in my environment, as follows. @evans-ye Would you kick the CI again? (Sorry that I'm not familar with the CI environment...) ```