Skip to content

add localhost as advertised hostname - #4689

Merged
gianm merged 2 commits into
apache:masterfrom
pjain1:kit_test
Aug 14, 2017
Merged

gianm merged 2 commits into
apache:masterfrom
pjain1:kit_test

Conversation

@pjain1

@pjain1 pjain1 commented Aug 14, 2017

Copy link
Copy Markdown
Member

Many times KafkaIndexingTaskTests times out while testing locally, setting advertised.host.name to localhost to prevent that

@gianm gianm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@pjain1

pjain1 commented Aug 14, 2017

Copy link
Copy Markdown
Member Author

changed to set the property in TestBroker

@gianm
gianm merged commit 725a144 into apache:master Aug 14, 2017
@pjain1
pjain1 deleted the kit_test branch August 15, 2017 01:10
seoeun25 added a commit to seoeun25/incubator-druid that referenced this pull request Jan 10, 2020
* Refactoring Appendertor Driver (apache#4292)

* Rename FiniteAppenderatorDriver to AppenderatorDriver (apache#4356)

* Add totalRowCount to appenderator

* add localhost as advertised hostname (apache#4689)

* kafkaIndexTask unannounce service in final block (apache#4736)

* warn if topic not found (apache#4834)

* Kafka: Fixes needlessly low interpretation of maxRowsInMemory. (apache#5034)
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.

2 participants