Re: Review Request 62681: ATLAS-2075: notification enhancement to handle large messages, using compression and multi-part messages

2017-09-28 Thread Abhay Kulkarni
s should never happen! notification/src/main/java/org/apache/atlas/notification/AlasNotificationMessageDeserializer.java Lines 138 (patched) <https://reviews.apache.org/r/62681/#comment263340> Consider adding another validation to ensure that the msgStr's size is equal to or less than

Re: Review Request 62681: ATLAS-2075: notification enhancement to handle large messages, using compression and multi-part messages

2017-09-30 Thread Abhay Kulkarni
apache.org/r/62681/ > --- > > (Updated Sept. 30, 2017, 11:51 a.m.) > > > Review request for atlas, Apoorv Naik, Ashutosh Mestry, Abhay Kulkarni, > Sarath Subramanian, and Suma Shivaprasad. > > > Bugs: ATLAS-2075 > https://issues.apache.org/jira/browse/ATLAS

Re: Review Request 62681: ATLAS-2075: notification enhancement to handle large messages, using compression and multi-part messages

2017-10-01 Thread Abhay Kulkarni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62681/#review186808 --- Ship it! - Abhay Kulkarni On Sept. 30, 2017, 9:58 p.m

Re: Review Request 62681: ATLAS-2075: notification enhancement to handle large messages, using compression and multi-part messages

2017-09-29 Thread Abhay Kulkarni
To reply, visit: > https://reviews.apache.org/r/62681/ > --- > > (Updated Sept. 29, 2017, 6:30 a.m.) > > > Review request for atlas, Apoorv Naik, Ashutosh Mestry, Abhay Kulkarni, > Sarath Subramanian, and Suma Shivaprasad. > > > Bugs: ATLAS-2075 > http

Re: Review Request 62681: ATLAS-2075: notification enhancement to handle large messages, using compression and multi-part messages

2017-09-29 Thread Abhay Kulkarni
Lines 36 (patched) <https://reviews.apache.org/r/62681/#comment263475> Consider if default value of this parameter can be directly referenced from some Kafka class. - Abhay Kulkarni On Sept. 29, 2017, 6:30 a.m., Madhan Neethiraj

Re: Review Request 62681: ATLAS-2075: notification enhancement to handle large messages, using compression and multi-part messages

2017-09-29 Thread Abhay Kulkarni
/AbstractNotification.java Lines 192 (patched) <https://reviews.apache.org/r/62681/#comment263567> Please base64 encode msgBytes if maxLengthExceedsLimit is true. - Abhay Kulkarni On Sept. 29, 2017, 10:32 p.m., Madhan Neethiraj

Re: [VOTE] Release Apache Atlas version 1.0.0 - rc2

2018-05-30 Thread Abhay Kulkarni
+1 -Abhay On 5/30/18, 12:31 AM, "Madhan Neethiraj" wrote: >Atlas team, > >Apache Atlas 1.0.0 release candidate #2 is now available for a vote >within dev community. > >Following commit went into branch-1.0 since the last release candidate, >#1, was sent yesterday. > ATLAS-2726: updated

Re: Review Request 68593: ATLAS-2856: added utility methods to RequestContext, to find number of active requests and earliest active request-time

2018-08-31 Thread Abhay Kulkarni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68593/#review208212 --- Ship it! Ship It! - Abhay Kulkarni On Aug. 31, 2018, 11:31

Re: Review Request 71982: ATLAS-3581:HBase hook converts qualifiedName to lowercase for namespace, table, column-family and column

2020-01-10 Thread Abhay Kulkarni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71982/#review219220 --- Ship it! Ship It! - Abhay Kulkarni On Jan. 10, 2020, 9:56

[jira] [Created] (ATLAS-2794) Ensure that Atlas REST client uses JacksonJsonProvider from com.fasterxml.jackson package

2018-07-26 Thread Abhay Kulkarni (JIRA)
Abhay Kulkarni created ATLAS-2794: - Summary: Ensure that Atlas REST client uses JacksonJsonProvider from com.fasterxml.jackson package Key: ATLAS-2794 URL: https://issues.apache.org/jira/browse/ATLAS-2794