Merge pull request #248 from ropensci/r6-conn

R6 connection done

Author: sckott

.Rbuildignore

@@ -16,4 +16,5 @@ notes-v2.md
 .github
 ^codemeta\.json$
 ^LICENSE\.md$
-
+^_pkgdown\.yml$
+^docs$

.travis.yml

@@ -39,6 +39,9 @@ matrix:
     - os: linux
       r: release
       env: ES_VERSION=6.5.3 R_CODECOV=true
+    - os: linux
+      r: release
+      env: ES_VERSION=7.0.0-alpha2
 
 r_binary_packages:
   - covr 
@@ -91,6 +94,11 @@ before_install:
       export ES_VERSION=6.5.3 ;
       curl -O https://artifacts.elastic.co/downloads/elasticsearch/elasticsearch-$ES_VERSION.deb && sudo dpkg -i --force-confnew elasticsearch-$ES_VERSION.deb && sudo service elasticsearch start
       ;;
+
+    "7.0.0-alpha2")
+      export ES_VERSION=7.0.0-alpha2 ;
+      curl -O https://artifacts.elastic.co/downloads/elasticsearch/elasticsearch-$ES_VERSION.deb && sudo dpkg -i --force-confnew elasticsearch-$ES_VERSION.deb && sudo service elasticsearch start
+      ;;
    esac
 
   - sleep 3

CODE_OF_CONDUCT.md

@@ -21,5 +21,5 @@ Instances of abusive, harassing, or otherwise unacceptable behavior may be repor
 opening an issue or contacting one or more of the project maintainers.
 
 This Code of Conduct is adapted from the Contributor Covenant 
-(http:contributor-covenant.org), version 1.0.0, available at 
-http://contributor-covenant.org/version/1/0/0/
+(https://contributor-covenant.org), version 1.0.0, available at 
+https://contributor-covenant.org/version/1/0/0/

DESCRIPTION

@@ -8,7 +8,7 @@ Description: Connect to 'Elasticsearch', a 'NoSQL' database built on the 'Java'
     requests. In addition, 'elastic' provides functions for interacting with API's
     for 'indices', documents, nodes, clusters, an interface to the cat API, and
     more.
-Version: 0.8.4.9410
+Version: 1.0.0
 License: MIT + file LICENSE
 Authors@R: person("Scott", "Chamberlain", role = c("aut", "cre"),
     email = "[email protected]")
@@ -20,12 +20,12 @@ Encoding: UTF-8
 Language: en-US
 Imports:
     utils,
-    methods,
-    httr (>= 1.2.1),
     curl (>= 2.2),
-    jsonlite (>= 1.1)
+    crul (>= 0.7.0),
+    jsonlite (>= 1.1),
+    R6
 Suggests:
-    roxygen2 (>= 6.1.0),
+    roxygen2 (>= 6.1.1),
     knitr,
     testthat
 RoxygenNote: 6.1.1

LICENSE

@@ -1,2 +1,2 @@
-YEAR: 2018
+YEAR: 2019
 COPYRIGHT HOLDER: Scott Chamberlain

LICENSE.md

@@ -1,4 +1,4 @@
-Copyright (c) 2018, Scott Chamberlain
+Copyright (c) 2019, Scott Chamberlain
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

Makefile

@@ -2,13 +2,13 @@ all: move rmd2md
 
 vignettes:
 		cd inst/vign;\
-		Rscript -e 'library(knitr); knit("search.Rmd"); knit("elastic_intro.Rmd")'
+		Rscript -e 'library(knitr); knit("search.Rmd"); knit("elastic.Rmd")'
 
 move:
 		cp inst/vign/search.md vignettes;\
-		cp inst/vign/elastic_intro.md vignettes
+		cp inst/vign/elastic.md vignettes
 
 rmd2md:
 		cd vignettes;\
 		mv search.md search.Rmd;\
-		mv elastic_intro.md elastic_intro.Rmd
+		mv elastic.md elastic.Rmd

NAMESPACE

@@ -4,6 +4,12 @@ S3method(docs_bulk,character)
 S3method(docs_bulk,data.frame)
 S3method(docs_bulk,default)
 S3method(docs_bulk,list)
+S3method(docs_bulk_create,data.frame)
+S3method(docs_bulk_create,default)
+S3method(docs_bulk_delete,data.frame)
+S3method(docs_bulk_delete,default)
+S3method(docs_bulk_index,data.frame)
+S3method(docs_bulk_index,default)
 S3method(docs_bulk_prep,data.frame)
 S3method(docs_bulk_prep,default)
 S3method(docs_bulk_prep,list)
@@ -21,7 +27,6 @@ S3method(es_parse,elastic_stats)
 S3method(es_parse,elastic_status)
 S3method(es_parse,es_GET)
 S3method(es_parse,index_delete)
-S3method(print,es_conn)
 S3method(scroll,character)
 S3method(scroll,default)
 S3method(scroll,list)
@@ -61,16 +66,20 @@ export(cluster_settings)
 export(cluster_state)
 export(cluster_stats)
 export(connect)
-export(connection)
 export(count)
 export(docs_bulk)
+export(docs_bulk_create)
+export(docs_bulk_delete)
+export(docs_bulk_index)
 export(docs_bulk_prep)
 export(docs_bulk_update)
 export(docs_create)
 export(docs_delete)
+export(docs_delete_by_query)
 export(docs_get)
 export(docs_mget)
 export(docs_update)
+export(docs_update_by_query)
 export(es_parse)
 export(explain)
 export(field_caps)
@@ -116,6 +125,7 @@ export(percolate_list)
 export(percolate_match)
 export(percolate_register)
 export(ping)
+export(pipeline_attachment)
 export(pipeline_create)
 export(pipeline_delete)
 export(pipeline_get)
@@ -129,23 +139,14 @@ export(tasks_cancel)
 export(termvectors)
 export(tokenizer_set)
 export(type_exists)
+export(type_remover)
 export(validate)
+importFrom(R6,R6Class)
+importFrom(crul,HttpClient)
+importFrom(crul,upload)
 importFrom(curl,curl_escape)
-importFrom(httr,DELETE)
-importFrom(httr,GET)
-importFrom(httr,HEAD)
-importFrom(httr,POST)
-importFrom(httr,PUT)
-importFrom(httr,add_headers)
-importFrom(httr,authenticate)
-importFrom(httr,content)
-importFrom(httr,content_type_json)
-importFrom(httr,http_status)
-importFrom(httr,stop_for_status)
-importFrom(httr,upload_file)
 importFrom(jsonlite,fromJSON)
 importFrom(jsonlite,toJSON)
-importFrom(methods,is)
 importFrom(utils,URLdecode)
 importFrom(utils,modifyList)
 importFrom(utils,read.delim)

NEWS.md

@@ -1,3 +1,44 @@
+elastic 1.0
+=============
+
+### BREAKING CHANGE
+
+(#87) The `connect()` function is essentially the same, with some changes, but now you pass the connection object to each function all. This indeed will break code. That's why this is a major version bump. 
+
+There is one very big downside to this: breaks existing code. That's the big one. I do apologize for this, but I believe that is outweighed by the upsides: passing the connection object matches behavior in similar R packages (e.g., all the SQL database clients); you can now manage as many different connection objects as you like in the same R session; having the connection object as an R6 class allows us to have some simple methods on that object to ping the server, etc. In addition, all functions will error with an informative message if you don't pass the connection object as the first thing.
+
+### NEW FEATURES
+
+* gains new ingest functions `pipeline_create`, `pipeline_delete`, `pipeline_get`, `pipeline_simulate`, and `pipeline_attachment()` (#191) (#226) 
+* gains new function `docs_delete_by_query()` and `docs_update_by_query()` to delete or update multiple documents at once, respectively; and new function `reindex()` to reindex all documents from one index to another (#237) (#195)
+* now using `crul` for HTTP requests. this only should matter with respect to passing in curl options  (#168)
+* recent versions of Elasticsearch are starting to include warnings in response headers for deprecations and other things. These can now be turned on or off with `connect()` (#241)
+* gains new functions for the bulk API: `docs_bulk_create()`, `docs_bulk_delete()`, `docs_bulk_index()`. each of which are tailored to doing the operation in the function name: creating docs, deleting docs, or indexing docs (#183)
+* gains new function `type_remover()` as a utility function to help users remove types from their files to use for bulk loading; could be used on example files in this package or user supplied files (#180)
+* gains function `alias_rename()` to rename aliases
+
+### MINOR IMPROVEMENTS
+
+* fixed `scroll()` example that wasn't working (#228)
+* rework `alias_create()` (#230)
+* move initialize Elasticsearch connection section of README higher up to emphasize it in the right place (#231) thanks @mbannert
+* whether you want "simple" or "complete" errors no longer sets env vars internally, but is passed through the internal error checker so that choices about type of errors for different connection objects do not affect one another (#242)
+* `docs_get` gains new parameters `source_includes` and `source_excludes` to include or exclude certain fields in the returned document (#246) thanks @Jensxy
+* added more examples to `index_create()` (#211)
+* add examples to `Search()` and `Search_uri()` docs of how to use profiles (https://www.elastic.co/guide/en/elasticsearch/reference/current/search-profile.html) (#194)
+* additional example added to `docs_bulk_prep()` for doing a mix of actions (i.e., delete, create, etc.)
+* improved examples throughout package docs so that examples are more self-contained
+
+### BUG FIXES
+
+* `docs_bulk_update()` was not handling boolean values correctly. now fixed (#239) (#240) thanks to @dpmccabe
+
+## DEPRECATED AND DEFUNCT
+
+* the `info()` method has been moved inside of the connection object. after calling `x = connect()` you can call `x$info()`
+* the `ping()` method has been marked as deprecated; instead, call `ping()` on the connection object created by a call to `connect()`
+
+
 elastic 0.8.4
 =============