Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BIGTOP-3987: Upgrade phoenix version to 5.1.3 #1181

Closed
wants to merge 1 commit into from

Conversation

Akanksha-kedia
Copy link

Apache Phoenix is an add-on for Apache HBase that provides a programmatic ANSI SQL interface. Apache Phoenix implements best-practice optimizations to enable software engineers to develop next-generation data-driven applications based on HBase.

Hbase in Bigtop 3.3 is 2.5 that needs Phoenix 5.1.3 to support.

++ PHOENIX_VERSION=5.1.3
++ HBASE_VERSION=2.5.3
++ JDK_VERSION=1.8

export MAVEN_OPTS=-Xmx1024m
MAVEN_OPTS=-Xmx1024m
MVN_ARGS='-Phadoop-3.0 '
MVN_ARGS+='-Dhadoop-three.version=3.3.5 '
MVN_ARGS+='-Dhadoop.guava.version=27.0-jre '
MVN_ARGS+='-Djetty.version=9.3.29.v20201019 '
MVN_ARGS+='-Dzookeeper.version=3.5.9 '
MVN_ARGS+='-DskipTests '
MVN_ARGS+='-Dcheckstyle.skip=true '

@Akanksha-kedia
Copy link
Author

@smallyao please review

@sekikn
Copy link
Contributor

sekikn commented Oct 15, 2023

The current PR doesn't work as follows. Removing patch0-PHOENIX-6766-branch-5.1.diff (and some additional work on install_phoenix.sh) seem to be required.

$ curl -sL https://github.com/apache/bigtop/pull/1181.diff | git apply
$ git diff
diff --git a/bigtop.bom b/bigtop.bom
index 23e2a07..160666b 100644
--- a/bigtop.bom
+++ b/bigtop.bom
@@ -247,7 +247,7 @@ bigtop {
     'phoenix' {
       name    = 'phoenix'
       relNotes = 'Apache Phoenix: A SQL skin over HBase'
-      version { base = "5.1.2"; pkg = base; release = 1 }
+      version { base = "5.1.3"; pkg = base; release = 1 }
       tarball { destination = "$name-${version.base}-src.tar.gz"
                 source      = destination }
       url     { download_path = "/$name/$name-${version.base}"
$ ./gradlew phoenix-clean phoenix-pkg

...

Patch #0 (patch0-PHOENIX-6766-branch-5.1.diff):
+ /usr/bin/cat /home/vagrant/bigtop/build/phoenix/rpm/SOURCES/patch0-PHOENIX-6766-branch-5.1.diff
+ /usr/bin/patch -p1 --fuzz=0
patching file pom.xml
Reversed (or previously applied) patch detected!  Assume -R? [n] 
Apply anyway? [n] 
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file pom.xml.rej
error: Bad exit status from /var/tmp/rpm-tmp.NiKOV9 (%prep)
    Bad exit status from /var/tmp/rpm-tmp.NiKOV9 (%prep)


RPM build errors:

FAILURE: Build failed with an exception.

@sekikn
Copy link
Contributor

sekikn commented Oct 18, 2023

@Akanksha-kedia Is there any progress on it? If you don't have enough time for now, may I take it over?

@Akanksha-kedia
Copy link
Author

@sekikn i built with phenox 5.1.3 with hbase 2.5.3 and i was able without any error. Will try with this indiviually, just guide i ll try, Thanks for the concern.

@Akanksha-kedia
Copy link
Author

Apache Phoenix is an add-on for Apache HBase that provides a programmatic ANSI SQL interface. Apache Phoenix implements best-practice optimizations to enable software engineers to develop next-generation data-driven applications based on HBase.

Hbase in Bigtop 3.3 is 2.5 that needs Phoenix 5.1.3 to support.

++ PHOENIX_VERSION=5.1.3 ++ HBASE_VERSION=2.5.3 ++ JDK_VERSION=1.8

export MAVEN_OPTS=-Xmx1024m MAVEN_OPTS=-Xmx1024m MVN_ARGS='-Phadoop-3.0 ' MVN_ARGS+='-Dhadoop-three.version=3.3.5 ' MVN_ARGS+='-Dhadoop.guava.version=27.0-jre ' MVN_ARGS+='-Djetty.version=9.3.29.v20201019 ' MVN_ARGS+='-Dzookeeper.version=3.5.9 ' MVN_ARGS+='-DskipTests ' MVN_ARGS+='-Dcheckstyle.skip=true '

@sekikn

@Akanksha-kedia
Copy link
Author

[INFO] Apache HBase - Shaded - Testing Util Tester ........ SUCCESS [ 5.031 s]
[INFO] Apache HBase Shaded Packaging Invariants ........... SUCCESS [ 6.212 s]
[INFO] Apache HBase Shaded Packaging Invariants (with Hadoop bundled) FAILURE [ 4.116 s]
[INFO] Apache HBase - Archetypes .......................... SKIPPED
[INFO] Apache HBase - Exemplar for hbase-client archetype . SKIPPED
[INFO] Apache HBase - Exemplar for hbase-shaded-client archetype SKIPPED
[INFO] Apache HBase - Archetype builder ................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 05:56 min
[INFO] Finished at: 2023-10-18T04:33:32Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.6.0:exec (check-jar-contents-for-stuff-with-hadoop) on project hbase-shaded-with-hadoop-check-invariants: Command execution failed.: Process exited with an error: 1 (Exit value: 1) -> [Help 1]

@sekikn
Copy link
Contributor

sekikn commented Oct 18, 2023

I'm afraid I can't give any meaningful advice from your last two comments, because it only describes your maven options and a part of the build results. Please provide your environment, branch/revision you're working on, code modification and executed commands. See the following examples.

Environment information:

$ facter os
{
  architecture => "amd64",
  distro => {
    codename => "jammy",
    description => "Ubuntu 22.04.3 LTS",
    id => "Ubuntu",
    release => {
      full => "22.04",
      major => "22.04"
    }
  },
  family => "Debian",
  hardware => "x86_64",
  name => "Ubuntu",
  release => {
    full => "22.04",
    major => "22.04"
  },
  selinux => {
    enabled => false
  }
}

Base branch/revision and code modification:

$ git rev-parse HEAD
f4d65ae97b484c8e01efa936948ed01574f149d2
$ git status 
On branch master
Your branch is ahead of 'origin/master' by 59 commits.
  (use "git push" to publish your local commits)

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   bigtop.bom

no changes added to commit (use "git add" and/or "git commit -a")
$ git diff
diff --git a/bigtop.bom b/bigtop.bom
index 70ff2c1d..34ded091 100644
--- a/bigtop.bom
+++ b/bigtop.bom
@@ -247,7 +247,7 @@ bigtop {
     'phoenix' {
       name    = 'phoenix'
       relNotes = 'Apache Phoenix: A SQL skin over HBase'
-      version { base = "5.1.2"; pkg = base; release = 1 }
+      version { base = "5.1.3"; pkg = base; release = 1 }
       tarball { destination = "$name-${version.base}-src.tar.gz"
                 source      = destination }
       url     { download_path = "/$name/$name-${version.base}"

Executed command and its result containing error messages:

$ ./gradlew phoenix-clean phoenix-pkg

...

dpkg-source: error: LC_ALL=C patch -t -F 0 -N -p1 -u -V never -E -b -B .pc/patch0-PHOENIX-6766-branch-5.1.diff/ --reject-file=- < phoenix-5.1.3.orig.abmmQO/debian/patches/patch0-PHOENIX-6766-branch-5.1.diff subprocess returned exit status 1
patching file pom.xml
Reversed (or previously applied) patch detected!  Skipping patch.
1 out of 1 hunk ignored
dpkg-source: info: the patch has fuzz which is not allowed, or is malformed
dpkg-source: info: if patch 'patch0-PHOENIX-6766-branch-5.1.diff' is correctly applied by quilt, use 'quilt refresh' to update it
dpkg-source: info: if the file is present in the unpacked source, make sure it is also present in the orig tarball
dpkg-buildpackage: error: dpkg-source -b . subprocess returned exit status 2

FAILURE: Build failed with an exception.

* Where:
Script '/home/sekikn/repos/bigtop/packages.gradle' line: 456

* What went wrong:
Execution failed for task ':phoenix-sdeb'.
> Process 'command 'dpkg-buildpackage'' finished with non-zero exit value 2

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 7s
5 actionable tasks: 5 executed

@Akanksha-kedia
Copy link
Author

I'm afraid I can't give any meaningful advice from your last two comments, because it only describes your maven options and a part of the build results. Please provide your environment, branch/revision you're working on, code modification and executed commands. See the following examples.

Environment information:

$ facter os
{
  architecture => "amd64",
  distro => {
    codename => "jammy",
    description => "Ubuntu 22.04.3 LTS",
    id => "Ubuntu",
    release => {
      full => "22.04",
      major => "22.04"
    }
  },
  family => "Debian",
  hardware => "x86_64",
  name => "Ubuntu",
  release => {
    full => "22.04",
    major => "22.04"
  },
  selinux => {
    enabled => false
  }
}

Base branch/revision and code modification:

$ git rev-parse HEAD
f4d65ae97b484c8e01efa936948ed01574f149d2
$ git status 
On branch master
Your branch is ahead of 'origin/master' by 59 commits.
  (use "git push" to publish your local commits)

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   bigtop.bom

no changes added to commit (use "git add" and/or "git commit -a")
$ git diff
diff --git a/bigtop.bom b/bigtop.bom
index 70ff2c1d..34ded091 100644
--- a/bigtop.bom
+++ b/bigtop.bom
@@ -247,7 +247,7 @@ bigtop {
     'phoenix' {
       name    = 'phoenix'
       relNotes = 'Apache Phoenix: A SQL skin over HBase'
-      version { base = "5.1.2"; pkg = base; release = 1 }
+      version { base = "5.1.3"; pkg = base; release = 1 }
       tarball { destination = "$name-${version.base}-src.tar.gz"
                 source      = destination }
       url     { download_path = "/$name/$name-${version.base}"

Executed command and its result containing error messages:

$ ./gradlew phoenix-clean phoenix-pkg

...

dpkg-source: error: LC_ALL=C patch -t -F 0 -N -p1 -u -V never -E -b -B .pc/patch0-PHOENIX-6766-branch-5.1.diff/ --reject-file=- < phoenix-5.1.3.orig.abmmQO/debian/patches/patch0-PHOENIX-6766-branch-5.1.diff subprocess returned exit status 1
patching file pom.xml
Reversed (or previously applied) patch detected!  Skipping patch.
1 out of 1 hunk ignored
dpkg-source: info: the patch has fuzz which is not allowed, or is malformed
dpkg-source: info: if patch 'patch0-PHOENIX-6766-branch-5.1.diff' is correctly applied by quilt, use 'quilt refresh' to update it
dpkg-source: info: if the file is present in the unpacked source, make sure it is also present in the orig tarball
dpkg-buildpackage: error: dpkg-source -b . subprocess returned exit status 2

FAILURE: Build failed with an exception.

* Where:
Script '/home/sekikn/repos/bigtop/packages.gradle' line: 456

* What went wrong:
Execution failed for task ':phoenix-sdeb'.
> Process 'command 'dpkg-buildpackage'' finished with non-zero exit value 2

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 7s
5 actionable tasks: 5 executed

@Akanksha-kedia Is there any progress on it? If you don't have enough time for now, may I take it over?

please take over

@sekikn
Copy link
Contributor

sekikn commented Oct 25, 2023

Thanks, I'll close this PR and submit another one once it's prepared.

@sekikn sekikn closed this Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants