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

Fix a problem whereby array data may be corrupted #147

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

r-owen
Copy link
Contributor

@r-owen r-owen commented Jun 25, 2020

No description provided.

@r-owen
Copy link
Contributor Author

r-owen commented Jun 25, 2020

Here is an example showing the problem. It fails if I use the master branch of the community edition of OpenSplice and succeeds if I use the minor modification in this pull request.

I also observe that the unit tests for ts_salobj pass, whereas one test fails due to corrupt array data without this change.

array_test.zip

@ThijsSassen
Copy link
Contributor

@r-owen thanks for this, in our current release OpenSplice 6.10.4 these defects are already resolved. We are looking into merging those fixes back to the open source as replacement for your PR due to the fact that they contain more fixes than only yours for python.

@r-owen
Copy link
Contributor Author

r-owen commented Jul 3, 2020

That sounds great. I hope you will be able to do that soon. I hope you will also fix the problem that causes segfaults and many entries in the error log at cleanup (separate pull request). Is 6.10.4 actually available? https://um1.salesforce.com/home/home.jsp says the current version is 6.10.3p2.

@reza-ebrahimi
Copy link

@ThijsSassen This PR is still remained open about a year, Any news on this?

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

Successfully merging this pull request may close these issues.

None yet

3 participants