Here is my opinion:
#1) It is best if you can tear down session if you know for sure you don’t need it for a long time. To save power. But if you transfer every 5 minutes it may be better to keep data session up all the time to save bandwidth.
#2) Yes, le_avdata_ReleaseSession() should be called