[jira] [Commented] (OMID-109) Unable to build phoenix-integration branch through Jenkins job

2018-09-17 Thread James Taylor (JIRA)


[ 
https://issues.apache.org/jira/browse/OMID-109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617684#comment-16617684
 ] 

James Taylor commented on OMID-109:
---

Will do. Please let me know once it’s been committed.

> Unable to build phoenix-integration branch through Jenkins job
> --
>
> Key: OMID-109
> URL: https://issues.apache.org/jira/browse/OMID-109
> Project: Apache Omid
>  Issue Type: Sub-task
>Reporter: James Taylor
>Assignee: Yonatan Gottesman
>Priority: Major
> Attachments: OMID-109.patch, omid109.patch
>
>
> Based on Jenkins job failures 
> (https://builds.apache.org/job/Phoenix-omid2/81/), the repo URL in the pom 
> needs to be updated to 
> https://raw.githubusercontent.com/synergian/wagon-git/releases.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (OMID-109) Unable to build phoenix-integration branch through Jenkins job

2018-09-17 Thread Yonatan Gottesman (JIRA)


 [ 
https://issues.apache.org/jira/browse/OMID-109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yonatan Gottesman updated OMID-109:
---
Attachment: omid109.patch

> Unable to build phoenix-integration branch through Jenkins job
> --
>
> Key: OMID-109
> URL: https://issues.apache.org/jira/browse/OMID-109
> Project: Apache Omid
>  Issue Type: Sub-task
>Reporter: James Taylor
>Assignee: Yonatan Gottesman
>Priority: Major
> Attachments: OMID-109.patch, omid109.patch
>
>
> Based on Jenkins job failures 
> (https://builds.apache.org/job/Phoenix-omid2/81/), the repo URL in the pom 
> needs to be updated to 
> https://raw.githubusercontent.com/synergian/wagon-git/releases.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (OMID-109) Unable to build phoenix-integration branch through Jenkins job

2018-09-17 Thread Yonatan Gottesman (JIRA)


[ 
https://issues.apache.org/jira/browse/OMID-109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617517#comment-16617517
 ] 

Yonatan Gottesman commented on OMID-109:


[~jamestaylor], I uploaded a patch that removes the repositories that were 
problematic.

I couldn't test this because i never could reproduce your errors myself on 
jenkins.

Can you please test you can build after Ohad merges the patch?

Thanks

> Unable to build phoenix-integration branch through Jenkins job
> --
>
> Key: OMID-109
> URL: https://issues.apache.org/jira/browse/OMID-109
> Project: Apache Omid
>  Issue Type: Sub-task
>Reporter: James Taylor
>Assignee: Yonatan Gottesman
>Priority: Major
> Attachments: OMID-109.patch, omid109.patch
>
>
> Based on Jenkins job failures 
> (https://builds.apache.org/job/Phoenix-omid2/81/), the repo URL in the pom 
> needs to be updated to 
> https://raw.githubusercontent.com/synergian/wagon-git/releases.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (OMID-109) Unable to build phoenix-integration branch through Jenkins job

2018-09-17 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/OMID-109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617515#comment-16617515
 ] 

ASF GitHub Bot commented on OMID-109:
-

GitHub user yonigottesman opened a pull request:

https://github.com/apache/incubator-omid/pull/44

[OMID-109] Remove wagon dependencies



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/yonigottesman/incubator-omid 
phoenix-integration

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-omid/pull/44.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #44


commit a2d00dbafa59f792a0cc35e8c6f686c2fc8bbe1a
Author: yonigo 
Date:   2018-09-17T08:57:56Z

remove wagon dependencies




> Unable to build phoenix-integration branch through Jenkins job
> --
>
> Key: OMID-109
> URL: https://issues.apache.org/jira/browse/OMID-109
> Project: Apache Omid
>  Issue Type: Sub-task
>Reporter: James Taylor
>Assignee: Yonatan Gottesman
>Priority: Major
> Attachments: OMID-109.patch
>
>
> Based on Jenkins job failures 
> (https://builds.apache.org/job/Phoenix-omid2/81/), the repo URL in the pom 
> needs to be updated to 
> https://raw.githubusercontent.com/synergian/wagon-git/releases.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (OMID-110) Support HBase 2.0

2018-09-17 Thread Yonatan Gottesman (JIRA)


 [ 
https://issues.apache.org/jira/browse/OMID-110?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yonatan Gottesman updated OMID-110:
---
Attachment: omid110-last_bug_fix.patch

> Support HBase 2.0
> -
>
> Key: OMID-110
> URL: https://issues.apache.org/jira/browse/OMID-110
> Project: Apache Omid
>  Issue Type: Sub-task
>Reporter: James Taylor
>Assignee: Yonatan Gottesman
>Priority: Major
> Attachments: 0001-omid-110.patch, OMID-110_wip1.patch, 
> omid110-last_bug_fix.patch, omid110.patch
>
>
> Once OMID-107 is committed (which replaced all deprecated methods), the only 
> things I've identified that need to change are:
>  * Instantiation of CellComparator should be done through HBaseShims since 
> it's different between HBase 1.x and 2.x. In HBase 2.0, CellComparator was 
> changed to an interface. Another alternative would be to write our own 
> CellComparator.
> {code:java}
> public static SortedMap> 
> mapCellsToShadowCells(List cells) {
> // Move CellComparator to HBaseSims for 2.0 support
> // Need to access through CellComparatorImpl.COMPARATOR
> SortedMap> cellToShadowCellMap
> = new TreeMap>(new CellComparator());{code}
> The HBase 2.0 version would use CellComparatorImpl.COMPARATOR instead of 
> instantiating a CellComparator.
>  * Add createTable method to HBaseShims. Instead of HTableDescriptor, HBase 
> 2.0 uses TableDescriptor and TableDescriptorBuilder. 
> [~chrajeshbab...@gmail.com] and [~an...@apache.org] - do you know why 
> existing admin.createTable(HTableDescriptor) wouldn't work? HTableDescriptor 
> implements TableDescriptor.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] incubator-omid pull request #43: [OMID-110]

2018-09-17 Thread yonigottesman
Github user yonigottesman closed the pull request at:

https://github.com/apache/incubator-omid/pull/43


---


[jira] [Commented] (OMID-110) Support HBase 2.0

2018-09-17 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/OMID-110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617150#comment-16617150
 ] 

ASF GitHub Bot commented on OMID-110:
-

Github user yonigottesman closed the pull request at:

https://github.com/apache/incubator-omid/pull/43


> Support HBase 2.0
> -
>
> Key: OMID-110
> URL: https://issues.apache.org/jira/browse/OMID-110
> Project: Apache Omid
>  Issue Type: Sub-task
>Reporter: James Taylor
>Assignee: Yonatan Gottesman
>Priority: Major
> Attachments: 0001-omid-110.patch, OMID-110_wip1.patch, omid110.patch
>
>
> Once OMID-107 is committed (which replaced all deprecated methods), the only 
> things I've identified that need to change are:
>  * Instantiation of CellComparator should be done through HBaseShims since 
> it's different between HBase 1.x and 2.x. In HBase 2.0, CellComparator was 
> changed to an interface. Another alternative would be to write our own 
> CellComparator.
> {code:java}
> public static SortedMap> 
> mapCellsToShadowCells(List cells) {
> // Move CellComparator to HBaseSims for 2.0 support
> // Need to access through CellComparatorImpl.COMPARATOR
> SortedMap> cellToShadowCellMap
> = new TreeMap>(new CellComparator());{code}
> The HBase 2.0 version would use CellComparatorImpl.COMPARATOR instead of 
> instantiating a CellComparator.
>  * Add createTable method to HBaseShims. Instead of HTableDescriptor, HBase 
> 2.0 uses TableDescriptor and TableDescriptorBuilder. 
> [~chrajeshbab...@gmail.com] and [~an...@apache.org] - do you know why 
> existing admin.createTable(HTableDescriptor) wouldn't work? HTableDescriptor 
> implements TableDescriptor.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (OMID-110) Support HBase 2.0

2018-09-17 Thread Ohad Shacham (JIRA)


 [ 
https://issues.apache.org/jira/browse/OMID-110?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ohad Shacham resolved OMID-110.
---
Resolution: Fixed

> Support HBase 2.0
> -
>
> Key: OMID-110
> URL: https://issues.apache.org/jira/browse/OMID-110
> Project: Apache Omid
>  Issue Type: Sub-task
>Reporter: James Taylor
>Assignee: Yonatan Gottesman
>Priority: Major
> Attachments: 0001-omid-110.patch, OMID-110_wip1.patch, omid110.patch
>
>
> Once OMID-107 is committed (which replaced all deprecated methods), the only 
> things I've identified that need to change are:
>  * Instantiation of CellComparator should be done through HBaseShims since 
> it's different between HBase 1.x and 2.x. In HBase 2.0, CellComparator was 
> changed to an interface. Another alternative would be to write our own 
> CellComparator.
> {code:java}
> public static SortedMap> 
> mapCellsToShadowCells(List cells) {
> // Move CellComparator to HBaseSims for 2.0 support
> // Need to access through CellComparatorImpl.COMPARATOR
> SortedMap> cellToShadowCellMap
> = new TreeMap>(new CellComparator());{code}
> The HBase 2.0 version would use CellComparatorImpl.COMPARATOR instead of 
> instantiating a CellComparator.
>  * Add createTable method to HBaseShims. Instead of HTableDescriptor, HBase 
> 2.0 uses TableDescriptor and TableDescriptorBuilder. 
> [~chrajeshbab...@gmail.com] and [~an...@apache.org] - do you know why 
> existing admin.createTable(HTableDescriptor) wouldn't work? HTableDescriptor 
> implements TableDescriptor.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)