larshp / abapGitServer

Git server implemented in ABAP
MIT License
64 stars 25 forks source link

Dump when pulling ECTD ZAGS_REPO_SNAPSHOT #154

Closed mbtools closed 1 year ago

mbtools commented 3 years ago

pulling latest version on 750 SPS 19 (AG 1.107) causes a dump for this object

image

CL_APL_ECATT_XML_DATA->ADD_ITEM

image

image

Might be AG problem in deserializer though

larshp commented 3 years ago

hmm, I have not tried pulling the code for a while

https://github.com/larshp/abapGitServer/blob/main/src/service/zags_repo_snapshot.ectd.xml

larshp commented 3 years ago

@germanysources any thoughts?

mbtools commented 3 years ago

it's a new object and there are no other ectd in this repo.

I tested with https://github.com/abapGit-tests/ECATT which has 2 ectd and that works fine (some unrelated error with ecvo).

germanysources commented 3 years ago

I could reproduce the error, but didn't find the root cause yet (it throws the exception when importing ZAGS_MERGE_REQ in variant MERGE). As a quick fix, you can remove the container. It is just needed for unit-testing.

germanysources commented 3 years ago

@mbtools Could you check out branch broken_tdc and check, if the dump stil occurs? For me it solved the problem.

mbtools commented 3 years ago

No more dump, but "Error in eCATT Help Function, Import of object ZAGS_REPO_SNAPSHOT failed"

image

PS: I need several tries to pull all objects, some of which I had to activate manually. PPS: Plus a diff for the MIME folder /SAP/PUBLIC/zgit. I will check what should be added to AG issue list

mbtools commented 3 years ago

Same error msg: https://github.com/abapGit/abapGit/issues/2535 (but no solution)

nununo commented 3 years ago

Same problem here. I have a dump when using the main branch, switched to broken-tdc and no more dump. But ended up with the same errors as described above.

The tool seems to be working fine. At least at first sight.

HuprichT commented 2 years ago

I have the same Problems as described. Can you still run AbapGitServer with this Errors?

larshp commented 2 years ago

@seniorFreshmen try pulling the "broken-tdc" branch instead, as noted in https://github.com/larshp/abapGitServer/issues/154#issuecomment-891739241

mbtools commented 1 year ago

There are some diffs but pulling does not dump anymore. Closing

pokrakam commented 9 months ago

It still dumps if you pull from a clean system or wipe AGS first. Tried on a 7.52 and 7.57 system with latest abapGit. Reopen?

larshp commented 9 months ago

@pokrakam open a new issue and refer to this one?