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

Fixes #1600: Enable stream in server #1601

Closed
wants to merge 2 commits into from

Conversation

simsausaurabh
Copy link
Member

Short description

I have:

  • There is a corresponding issue for this pull request.
  • Mentioned the Issue number in the pull request commit message Fixes #<number> commit message
  • There is only strictly only one commit per issue.

For the reviewers

I have:

  • Reviewed this pull request by an authorized contributor.
  • The reviewer is assigned to the pull request.

Fixes: #1600

classification.emotion.disgust=loathing,disgust,boredom,abscheu,abneigung,langeweile,dio,aversi�n,tedio,aversion,d�go�t,ennui
classification.emotion.anger=rage,anger,annoyance,wut,groll,verdruss,furia,ira,enfado,rage,col�re,contrari�t�,Annoyed,Apathetic,Bored,Certain,Cold,Crabby,Cranky,Critical,Cross,Detached,Displeased,Frustrated,Impatient,Indifferent,Irritated,Peeved,Rankled,Affronted,Aggravated,Angry,Antagonized,Arrogant,Bristling,Exasperated,Incensed,Indignant,Inflamed,Mad,Offended,Resentful,Sarcastic,Aggressive,Appalled,Belligerent,Bitter,Contemptuous,Disgusted,Furious,Hateful,Hostile,Irate,Livid,Menacing,Outraged,Ranting,Raving,Seething,Spiteful,Vengeful,Vicious,Vindictive,Violent
classification.emotion.anticipation=vigilance,anticipation,interest,umsicht,erwartung,interesse,vigilancia,anticipaci�n,inter�s,vigilance,anticipation,int�r�t
classification.emotion.joy=ecstacy,joy,serenity,ekstase,freude,gelassenheit,axtasis,alegr�a,serenidad,extase,joie,s�r�nit�,Amused,Calm,Encouraged,Friendly,Hopeful,Inspired,Jovial,Open,Peaceful,Cheerful,Contented,Delighted,Excited,Fulfilled,Glad,Gleeful,Gratified,Happy,Selfesteem,Joyful,Lively,Merry,Optimistic,Playful,Pleased,Proud,Rejuvenated,Satisfied,Awe-filled,Blissful,Ecstatic,Egocentric,Elated,Enthralled,Euphoric,Exhilarated,Giddy,Jubilant,Manic,Overconfident,Overjoyed,Radiant,Rapturous,Self-aggrandized,Thrilled
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this diff.

@@ -299,5 +299,5 @@ classifierservlet.cache.timeout = 300000
dump.write_enabled = true

# Stream
stream.enabled = false
stream.enabled = true
Copy link
Member

Choose a reason for hiding this comment

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

Enabling stream is not this simple. The node running it should install and start Mosquitto broker on the port in order to successfully start the server.

Copy link
Member

@sudheesh001 sudheesh001 left a comment

Choose a reason for hiding this comment

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

Are you running a MQTT server/broker for this to be enabled? We also then need to set up the infrastructure and then work on the streams accordingly. Mosquitto or MQTT are the ideal choices for streaming. This is a much needed feature. Thank you for taking time but it needs more work to make it work properly.

@simsausaurabh
Copy link
Member Author

No I am not running a MQTT server/broker. I am searching for already deployed MQTT servers. I also tried to deploy one but it didn't work, that's why keeping this on hold for a while. Once I get success in deploying or once I get already deployed free of cost MQTT server, I'll update the patch.

@simsausaurabh
Copy link
Member Author

Closing it as for now. Will send a patch once MQTT server/broker is deployed.

@simsausaurabh simsausaurabh deleted the enableStream branch August 22, 2018 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants