[GitHub] [accumulo] keith-turner opened a new pull request #1654: Added unit test to exercise bulk import range sanity check code #1618

2020-07-09 Thread GitBox


keith-turner opened a new pull request #1654:
URL: https://github.com/apache/accumulo/pull/1654


   



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.

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




[GitHub] [accumulo] cradal commented on pull request #1637: #1618 Investigate TODO in exception message in new Bulk Import

2020-07-09 Thread GitBox


cradal commented on pull request #1637:
URL: https://github.com/apache/accumulo/pull/1637#issuecomment-656218117


   @keith-turner Looks good.



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.

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




Accumulo-Master - Build # 3119 - Still Unstable

2020-07-09 Thread Apache Jenkins Server
The Apache Jenkins build system has built Accumulo-Master (build #3119)

Status: Still Unstable

Check console output at https://builds.apache.org/job/Accumulo-Master/3119/ to 
view the results.

[GitHub] [accumulo] keith-turner commented on pull request #1637: #1618 Investigate TODO in exception message in new Bulk Import

2020-07-09 Thread GitBox


keith-turner commented on pull request #1637:
URL: https://github.com/apache/accumulo/pull/1637#issuecomment-656201740


   @cradal I merged this in 25a55df2309fc1891ef208e35f2268b254ffb4e8.  Please 
take a look at it to make sure I got it correct.



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.

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




[GitHub] [accumulo] keith-turner closed pull request #1637: #1618 Investigate TODO in exception message in new Bulk Import

2020-07-09 Thread GitBox


keith-turner closed pull request #1637:
URL: https://github.com/apache/accumulo/pull/1637


   



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.

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




[GitHub] [accumulo] keith-turner closed issue #1618: Investigate TODO in exception message in new Bulk Import

2020-07-09 Thread GitBox


keith-turner closed issue #1618:
URL: https://github.com/apache/accumulo/issues/1618


   



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.

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




[GitHub] [accumulo] Manno15 opened a new pull request #1653: WIP. Previously Reverted Code with Fix

2020-07-09 Thread GitBox


Manno15 opened a new pull request #1653:
URL: https://github.com/apache/accumulo/pull/1653


   This is the rest of the reverted code from pull request #1453 and includes 
some slight fixes for some of the IT's that it broke. This is still a work in 
progress due to some inconsistencies in my testing of the IT's and I know there 
are currently some fixes in the works for a few of those.  
   
   Most of the code changes are aimed to use TabletMutator in the various state 
store setLocation functions (some of these functions were already doing so but 
not all) and deleting unused code. Parts of these changes coincide with my 
change in #1616 in when we update lastLocation. 



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.

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




[GitHub] [accumulo] cradal commented on pull request #1637: #1618 Investigate TODO in exception message in new Bulk Import

2020-07-09 Thread GitBox


cradal commented on pull request #1637:
URL: https://github.com/apache/accumulo/pull/1637#issuecomment-656056603


   @keith-turner cdsch...@gmail.com Thanks!



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.

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




[GitHub] [accumulo] markxiaotao opened a new pull request #1652: Remove an workaround profile exists in core/pom.xml file since the related issue had been already fixed.

2020-07-09 Thread GitBox


markxiaotao opened a new pull request #1652:
URL: https://github.com/apache/accumulo/pull/1652


   According to the link 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=541772#c30, I observed that such 
issue had already been fixed in 4.12M1 release. The workaround for this profile 
is not needed in my opinion.
   I suggest to remove this profile.
   
   The workaround profile:
   
https://github.com/apache/accumulo/blob/07bc9d329ea0b8da9a577863acfe009910e189cd/core/pom.xml#L386-L401
   



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.

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