Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

TAJO-1506: WorkerConnectionInfo::id field may be not unique, but its uniqueness is assumed throughout codes.#502

Closed
ykrips wants to merge 2 commits intoapache:masterfrom
ykrips:TAJO-1506
Closed

TAJO-1506: WorkerConnectionInfo::id field may be not unique, but its uniqueness is assumed throughout codes.#502
ykrips wants to merge 2 commits intoapache:masterfrom
ykrips:TAJO-1506

Conversation

@ykrips
Copy link

@ykrips ykrips commented Apr 3, 2015

No description provided.

@ykrips
Copy link
Author

ykrips commented Apr 3, 2015

@blrunner
Copy link
Contributor

blrunner commented Apr 4, 2015

Thanks for your contribution.

I tested this on four of linux servers. But I found that worker id had been duplicated as follows.

2015-04-05 07:13:05,916 INFO org.apache.tajo.worker.TajoWorker: Tajo Worker is initialized. connection :id:1, host:tajo004.xyz, PeerRpcPort:28091, PullServerPort:53218, ClientPort:28092, QueryMasterPort:28093, HttpInfoPort:28080

2015-04-05 07:13:05,054 INFO org.apache.tajo.worker.TajoWorker: Tajo Worker is initialized. connection :id:1, host:tajo005.xyz, PeerRpcPort:28091, PullServerPort:44162, ClientPort:28092, QueryMasterPort:28093, HttpInfoPort:28080

So, TajoMaster can't show right TajoWorker list on web pages. Could you check this again?

@ykrips
Copy link
Author

ykrips commented Apr 5, 2015

@blrunner ,
Thanks for testing this task. I could not test for inter-node cases. I will check again and re-open this pull request after modification has finished.

@ykrips ykrips closed this Apr 5, 2015
@ykrips ykrips deleted the TAJO-1506 branch April 5, 2015 00:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants