Event.sessionId can't be set to System.currentTimeMillis()
richard-bio opened this issue · 2 comments
richard-bio commented
Expected Behavior
I expect to be able to set event.sessionId
to System.currentTimeMillis()
Current Behavior
event.essionId
is type int
so the above results in a compile error
Possible Solution
Change event.sessionId
to long
? (which appears to be the case in the other Amplitude SDKs)
Steps to Reproduce
- Try to compile
event.sessionId = System.currentTimeMillis();
Environment
- Amplitude Java SDK 1.2.0, Java Version 11.0.6+10 (64 bit)
- Device: Mac Book Pro (13-inch, 2016, Four Thunderbolt 3 Ports)
- OS: macOS 11.2.3
dantetam commented
Good catch, @richard-bio . We're working on a fix very soon. Will close when the new version is released
dantetam commented
This issue is fixed in the newest version of the Java SDK released just now, v1.2.1. Thank you again for the good reporting.