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

[facets] remove faceted search functionality #171

Open
wants to merge 34 commits into
base: develop
Choose a base branch
from

Conversation

zouzias
Copy link
Owner

@zouzias zouzias commented Apr 9, 2019

Changelog:

  • Remove FacetedLuceneRDD class

Anastasios Zouzias and others added 24 commits March 11, 2019 22:07
* Remove unused code (#141)

* Revert "Setting version to 0.3.4-SNAPSHOT"

This reverts commit 2f1d7be.

* README: update to 0.3.3

* README: fix javadoc badge

* remove unused param

* [sbt] version updates

* [conf] allow not_analyzed string fields (#145)

* [not-analyzed-fields] do not analyzed fields ending with _notanalyzed

* [hotfix] fixes issue 150

* [tests] issue 150

* fix typo

* [blockEntityLinkage] drop queryPartColumns
* [linkage] block linker with => Query

* [linkage] block linker is Row => Query

* remove Query analyzer on methods
* [analyzers] custom analyzer

* test return null

* [travis] travis_wait 1 min

* Revert "[travis] travis_wait 1 min"

This reverts commit c79456e.

* use lucene examples

* custom analyzer return null

* fix java reflection

* add docs
* [lucene] upgrade to version 8.0.0

* [lucene] remove ngram analyzer

* delete ngram analyzer

* minor fix

* add scaladoc
@codecov
Copy link

codecov bot commented Apr 9, 2019

Codecov Report

❗ No coverage uploaded for pull request base (develop@9442bbb). Click here to learn what that means.
The diff coverage is 81.60%.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop     #171   +/-   ##
==========================================
  Coverage           ?   77.86%           
==========================================
  Files              ?       29           
  Lines              ?      926           
  Branches           ?       89           
==========================================
  Hits               ?      721           
  Misses             ?      205           
  Partials           ?        0           
Impacted Files Coverage Δ
...ias/spark/lucenerdd/LuceneRDDKryoRegistrator.scala 100.00% <ø> (ø)
...spark/lucenerdd/config/LuceneRDDConfigurable.scala 61.76% <ø> (ø)
.../spatial/shape/ShapeLuceneRDDKryoRegistrator.scala 100.00% <ø> (ø)
.../zouzias/spark/lucenerdd/store/IndexWritable.scala 100.00% <ø> (ø)
...ark/lucenerdd/analyzers/AnalyzerConfigurable.scala 30.35% <46.66%> (ø)
...tial/shape/partition/ShapeLuceneRDDPartition.scala 90.00% <80.00%> (ø)
.../scala/org/zouzias/spark/lucenerdd/LuceneRDD.scala 86.66% <82.60%> (ø)
...in/scala/org/zouzias/spark/lucenerdd/package.scala 97.22% <90.00%> (ø)
...spark/lucenerdd/partition/LuceneRDDPartition.scala 87.01% <90.47%> (ø)
...uzias/spark/lucenerdd/config/LuceneRDDParams.scala 100.00% <100.00%> (ø)
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9442bbb...094e08d. Read the comment docs.

* [issue_163] per field analysis

* [sbt] update scalatest to 3.0.7

* [issue_163] fix docs; order of arguments

* fixes on ShapeLuceneRDD

* [issue_163] fix test

* issue_163: minor fix

* introduce LuceneRDDParams case class

* fix apply in LuceneRDDParams

* [issue_163] remove duplicate apply defn

* add extra LuceneRDD.apply
[issue_165] throw runtime exception; handle multi-valued fields in DataFrames
* [refactor] configuration loading

* [travis] code hygiene
@zouzias
Copy link
Owner Author

zouzias commented Apr 14, 2019

Remove mentions on readme and reference.conf file about facets.

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