-
Notifications
You must be signed in to change notification settings - Fork 2
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
Issue to get data using php8 client which stored from php7 client #48
Comments
Can you provide a PHP 7 snippet that reproduces the issue? The information you have provided is not enough for me to reproduce the issue unfortunately. What is the error you get from the PHP8 client? |
@khaf / @vmsachin: PFA the sample data, code snippet for both php7 and php8 clients. First run the old-aerospike -client which put and get data from aerospike. I didn't get any error from php8 client recieved the following kind of response. Aerospike Response
Aerospike\Record Object
) |
@khaf / @vmsachin : I've found the issue, The PHP 8 Aerospike client cannot reliably retrieve data written by the PHP 7 Aerospike client when that data is stored in array/list format and contains true/false/null values.
Could you please fix this issue? or suggest some other solution. |
Hi @piyush-bansal1989 , happy new year! Sorry I have been looking into this issue for the last few days but I have not updated this ticket. I need until early next week to give a proper update. ps. @vmsachin is not working on this project anymore, you can ping me instead. |
@khaf: Do you have any update on this issue. |
@khaf : I would request you to share some update on this issue. |
@khaf : I hope you doing well! , will you please share the update on this issue? |
@piyush-bansal1989 Hi Piyush, sorry for my late reply. I am working on this issue still, it seems like a serialization compatibility issue between our old C client and the newer clients. I'll keep you updated on what exactly is the issue and how we will deal with it in a couple of days. |
Hi @khaf Is there any update on it, our product upgrade peding on it. |
Hey @piyush-bansal1989 - I am a new engineer here at Aerospike & I will be helping @khaf on this one. Just to update you, I have been able to reproduce your problem. You were correct in your assessment that there is an incompatibility between the boolean encoding of these 2 clients. Essentially, the old client is doing it wrong. I am looking to see if we can get a possible workaround into the new PHP client. RN it doesn't look like repairing the old client is in the cards. So, say we got a patch into the new client - you would just have to start using the new client for everything & get rid of the old client. |
@piyush-bansal1989 We have addressed the issue and will release the fix before the end of this week. It took much longer than anticipated due to the changes required in the Go client, in which we had to come to an agreement regarding the introduction of the support for the old PHP client. |
@khaf : Thank you for addressing the issue. I understand that the resolution required more time due to necessary changes in the Go client to support the legacy PHP client. I appreciate your efforts and look forward to the fix being released by the end of this week. |
@vmsachin / @khaf : I'm encountering an issue when retrieving data stored by a PHP 7 client using a PHP 8 client. The vice versa operation (retrieving data stored by PHP 8 using PHP 7) works correctly. The data format for storage is consistent.
Could you please assist me in resolving this issue? Please find the sample data attached.
PHP7 client store data which not able retrieving from php8 client
aql> select * from ns_amcat_datastore.amcat_auth_tokens where PK='1d6eed27fdc8b35d46ed7d48'; +----------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------+-------+ | PK | content | {ttl} | {gen} | +----------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------+-------+ | "1d6eed27fdc8b35d46ed7d48" | MAP('{"authDetails":{"apiSecret":"**********", "apiToken":"1d6eed27fdc8b35d46ed7d48", "createdOn":"2024-12-20 10:34:38", "expiresOn":"2024-12-21 10:34:38", "isActive":1, "userName":"******", "userType":"******"}, "configEntities":{"A | 85586 | 5 | +----------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------+-------+ 1 row in set (0.000 secs)
Below data stored using php8 client which able to retrieving in php7 client
aql> select * from ns_amcat_datastore.amcat_auth_tokens where PK='f8aa10dac9111e70b4b09de2'; +----------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------+-------+ | PK | content | {ttl} | {gen} | +----------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------+-------+ | "f8aa10dac9111e70b4b09de2" | MAP('{"authDetails":{"apiSecret":"*********", "apiToken":"f8aa10dac9111e70b4b09de2", "createdOn":"2024-12-19 18:24:32", "expiresOn":"2024-12-20 18:24:32", "isActive":1, "userName":"********", "userType":"********"}, "configEntities":{"A | 27454 | 4 | +----------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------+-------+
We require fix for this issue as we move traffic from old to new one distributed way.
Waiting for your revert thanks
The text was updated successfully, but these errors were encountered: