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

Added java code snippet for hyperloglog #3674

Merged
merged 6 commits into from
Jan 13, 2024

Conversation

Ranjeet0611
Copy link
Contributor

Added java code snippet for hyperloglog #3629

@Ranjeet0611 Ranjeet0611 changed the base branch from master to emb-examples January 2, 2024 12:22
@Ranjeet0611
Copy link
Contributor Author

Hi @dwdougherty Please review this example
Thank you

Copy link
Contributor

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks again, Ranjeet!

Copy link
Collaborator

@sazzad16 sazzad16 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dwdougherty Weren't the imports supposed to be removed or at least hidden?

@Ranjeet0611 Here is only one object of UnifiedJedis. So use a shorter variable name; like jedis, client, etc.

@codecov-commenter
Copy link

codecov-commenter commented Jan 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (emb-examples@0526f6c). Click here to learn what that means.

❗ Current head 51ed2be differs from pull request most recent head 0f85c0a. Consider uploading reports for the commit 0f85c0a to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@               Coverage Diff               @@
##             emb-examples    #3674   +/-   ##
===============================================
  Coverage                ?   75.36%           
  Complexity              ?     4894           
===============================================
  Files                   ?      297           
  Lines                   ?    14965           
  Branches                ?     1129           
===============================================
  Hits                    ?    11278           
  Misses                  ?     3192           
  Partials                ?      495           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Ranjeet0611
Copy link
Contributor Author

@sazzad16 did the changes
Thank you

@dwdougherty
Copy link
Contributor

@dwdougherty Weren't the imports supposed to be removed or at least hidden?

Only the code between // STEP_START and // STEP_END is shown in the doc code tabs, so hiding imports, especially this small number of them, is optional.

@sazzad16 sazzad16 requested a review from dwdougherty January 3, 2024 05:22
@Ranjeet0611
Copy link
Contributor Author

@dwdougherty please review

Copy link
Contributor

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sazzad16 This PR builds correctly and it's good to go.

@sazzad16 sazzad16 merged commit 783f92e into redis:emb-examples Jan 13, 2024
1 check passed
uglide added a commit that referenced this pull request Jun 18, 2024
* Fix search_quickstart example

* Add data_class step to SearchQuickstartExample

* Update examples with new bikes and align to new tutorial

* Fixes doctests for search tutorial

* Small fixes doctests

* Updates search tutorial doc examples to new format and data

* Fix formatting in doc tests

* Add Hashes and String Example (#3477)

* Add Hashes Example

* Fixes examples for hash tutorial

* Refactor Hash Tutorial to Java 8 (from 17)

* Change indentation to 4 spaces

* Add Strings Example

* Reformat samples to 2-space indentation

---------

Co-authored-by: Elena Kolevska <[email protected]>

* Add Geospatial data type example (#3487)

* Add Geospatial data type example

* Add missing REMOVE_END

* Added example Java code snippets for Streams (#3641)

* Java example code snippets for T-Digest and TopK (#3660)

* Created example for t digest

* Added example for topK

---------

Co-authored-by: Ranjeet Singh <[email protected]>
Co-authored-by: Ranjeet Singh <[email protected]>

* Create Java code snippets for sets #3626 (#3661)

* Create Java code snippets for sets #3626

* Apply suggestions from code review to add // STEP_END lines.

Co-authored-by: David Dougherty <[email protected]>

* format imports

---------

Co-authored-by: David Dougherty <[email protected]>
Co-authored-by: M Sazzadul Hoque <[email protected]>

* Create Java code snippets for count min sketch (#3644)

* added code snippet for CMS in Java

* Update CMSExample.java

* format spaces

* use cmsIncrBy simple variant

* more format spaces

---------

Co-authored-by: M Sazzadul Hoque <[email protected]>

* Added Java code snippets for Bloom #3630  (#3671)

* Added java example for bloom filter

* Removed space

* fix and format imports

* fix compile error

* format spaces

* Add del command and rename obj

* Add del command and fix format

* change to junit assert

* Update BloomFilterExample.java

* Add correct output comment

---------

Co-authored-by: Ranjeet Singh <[email protected]>
Co-authored-by: M Sazzadul Hoque <[email protected]>
Co-authored-by: Ranjeet Singh <[email protected]>

* Added java code snippet for hyperloglog (#3674)

* Added Java code snippet for HyperLogLog

* fix format and imports

* fix spaces

* change to junit assert

* Added correct output comment

* Update HyperLogLogExample.java

---------

Co-authored-by: Ranjeet Singh <[email protected]>
Co-authored-by: M Sazzadul Hoque <[email protected]>

* Added Java code snippets for cuckoo filters (#3679)

* Add CuckooFilterExample

* Add step start comment

* Update CuckooFilterExample.java

---------

Co-authored-by: Ranjeet Singh <[email protected]>
Co-authored-by: M Sazzadul Hoque <[email protected]>

* Create Java code snippet for lists #3625 (#3677)

* Added example Java code snippets for the List#3625

* format spaces

---------

Co-authored-by: M Sazzadul Hoque <[email protected]>

* Create Java code snippet for sorted sets  (#3680)

* added an example for SortedSets in java

* Update SortedSetsExample.java

* made changes as per the review.

* Format spaces in SortedSetsExample.java

---------

Co-authored-by: M Sazzadul Hoque <[email protected]>

* Added Java code snippet for bitmap  (#3687)

* Add bitmap example

* Change class name

* Add assert for res3 and res4

* Update BitMapsExample.java

* Format spaces in BitMapsExample.java

---------

Co-authored-by: Ranjeet Singh <[email protected]>
Co-authored-by: M Sazzadul Hoque <[email protected]>

---------

Co-authored-by: Elena Kolevska <[email protected]>
Co-authored-by: Brian Sam-Bodden <[email protected]>
Co-authored-by: M Sazzadul Hoque <[email protected]>
Co-authored-by: Harshvardhan Parmar <[email protected]>
Co-authored-by: Ranjeet SIngh <[email protected]>
Co-authored-by: Ranjeet Singh <[email protected]>
Co-authored-by: Ranjeet Singh <[email protected]>
Co-authored-by: Suraj <[email protected]>
Co-authored-by: David Dougherty <[email protected]>
Co-authored-by: Ranjeet Singh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants