Skip to content

Conversation

@hiliev
Copy link
Contributor

@hiliev hiliev commented Aug 20, 2021

This PR adds an additional test case to DynamicLogicExpressionSuite that performs the main test using Long numbers and fixes the decoding of LongEnc values in the filter() method of NaiveEncoder.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 1s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1 💚 mvninstall 1m 35s master passed
+1 💚 compile 0m 40s master passed
+1 💚 scaladoc 0m 46s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 0m 46s the patch passed
+1 💚 compile 0m 37s the patch passed
+1 💚 scalac 0m 37s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 scaladoc 0m 46s the patch passed
_ Other Tests _
+1 💚 unit 7m 5s hbase-spark in the patch passed.
14m 0s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-83/1/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR #83
Optional Tests dupname scalac scaladoc unit compile
uname Linux bc40f860ed12 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 GNU/Linux
Build tool hb_maven
Personality dev-support/jenkins/hbase-personality.sh
git revision master / fddb433
Test Results https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-83/1/testReport/
Max. process+thread count 914 (vs. ulimit of 12500)
modules C: spark/hbase-spark U: spark/hbase-spark
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-83/1/console
versions git=2.20.1
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@meszibalu meszibalu merged commit 6b4d985 into apache:master Aug 23, 2021
@meszibalu
Copy link
Contributor

LGTM, thanks @hiliev!

ianawilson pushed a commit to sproutsocial/hbase-connectors that referenced this pull request Dec 28, 2021
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.

4 participants