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

IceGrid object adapter registration #19

Open
iivdam opened this issue Jan 16, 2023 · 2 comments
Open

IceGrid object adapter registration #19

iivdam opened this issue Jan 16, 2023 · 2 comments
Labels

Comments

@iivdam
Copy link

iivdam commented Jan 16, 2023

Hi,

I'm experimenting with DataStorm and IceGrid. If I use DataStorm with IceGrid I have to enable IceGrid.Registry.DynamicRegistration because DataStrom creates an object adapter with an UUID and the AdapterId proprerty set (src\DataStorm\Instance.cpp line 99)
// // Create a collocated object adapter with a random name to prevent user configuration // of the adapter. // auto collocated = IceUtil::generateUUID(); properties->setProperty(collocated + ".AdapterId", collocated); _collocatedAdapter = _communicator->createObjectAdapter(collocated);

Ik think this is undesirable behavior, the locator for this object adapter must be nullified.
auto collocated = IceUtil::generateUUID(); properties->setProperty(collocated + ".AdapterId", collocated); _collocatedAdapter = _communicator->createObjectAdapter(collocated); _collocatedAdapter ->setLocator(0);

Cheers,
Emko

@bentoi
Copy link
Member

bentoi commented Jan 17, 2023

Thanks for the report. We'll fix this!

Can you explain how you use DataStorm with IceGrid? Do you use it just to start the process? Or do you also use it to use register DataStorm node endpoints with the IceGrid registry?

If you use it just to start the process, as a workaround you could clear the server locator as follow:

<server id="Reader" exe="./reader" activation="always">
  <option>--Ice.Default.Locator=</option>
</server>

@bentoi bentoi added the bug label Jan 17, 2023
@iivdam
Copy link
Author

iivdam commented Jan 23, 2023

Hi Benoit,

I want to use the DataStorm Node class inside multiple IceBox services. These services do also contain 'normal' RPC functionality with the use of IceGrid. The distributed DataStorm data will include some of these RPC proxies.
In the meantime I develop with a patched DataStorm.

Cheers,
Emko

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants