[GitHub] [skywalking-banyandb-java-client] wu-sheng opened a new pull request, #26: Support timestamp lazy initialization for StreamWrite.

[GitBox <gi...@apache.org>]

wu-sheng opened a new pull request, #26:
URL: https://github.com/apache/skywalking-banyandb-java-client/pull/26

   According to https://github.com/apache/skywalking/issues/10008 discussion, we noticed the current timestamp of stream write is from the time bucket, which is not precise enough(only the second level).
   
   So, I asked @wankai123 to add a BanyanDB annotation to indicate which field of the real timestamp should be used for BanyanDB server to improve the precision. If we don't expose this lazy set, then we can't leverage the converter to read the timestamp field, but have to use reflection. I think this would be better.
   
   This is not exposed to measure write, as AFAIK the measure write is correct to use the time bucket for overriding the previous value in the same time bucket.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking-banyandb-java-client] wu-sheng merged pull request #26: Support timestamp lazy initialization for StreamWrite.

[GitBox <gi...@apache.org>]

wu-sheng merged PR #26:
URL: https://github.com/apache/skywalking-banyandb-java-client/pull/26


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org