search

IBM-Cloud / get-started-java

Sample and tutorial to help you get started with a Java EE app, REST API and a database.
https://console.bluemix.net/docs/runtimes/liberty/getting-started.html
Apache License 2.0
27 stars 88 forks source link

addNames should be POST #1

Closed rvennam closed 7 years ago

rvennam commented 7 years ago

@opiethehokie Can you review the latest commits? I removed the servlets and wrote an API instead.

opiethehokie commented 7 years ago

I like this better than just having the servlet.

I would suggest renaming CloudantStore to CloudantVisitorStore and I think VisitorStoreFactory needs some work. On every HTTP request isn't VCAP_SERVICES getting parsed again? Seems like the same DB connection could be reused, or am I missing something?

And databaseName in CloudantStore should be final.

rvennam commented 7 years ago

Done.