[GitHub] incubator-trafodion pull request #972: here is the change on pom.xml used to...

2017-02-21 Thread hegdean
Github user hegdean commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/972#discussion_r102244502
  
--- Diff: core/conn/jdbcT4/pom.xml ---
@@ -27,11 +27,34 @@
 -->
 
   4.0.0
-  org.trafodion.jdbc.t4
-  jdbcT4
+  
+org.apache.trafodion
--- End diff --

This may not work as the package names does not contain apache in them. 
First we will need to make the package name change for T4


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-trafodion pull request #972: here is the change on pom.xml used to...

2017-02-21 Thread AlexPeng19
GitHub user AlexPeng19 opened a pull request:

https://github.com/apache/incubator-trafodion/pull/972

here is the change on pom.xml used to publish jdbcT4 to maven staging 
repository

here is the change on pom.xml used to publish jdbcT4 to maven staging 
repository

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

$ git pull https://github.com/AlexPeng19/incubator-trafodion master

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

https://github.com/apache/incubator-trafodion/pull/972.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 #972


commit af3c89aee5dbb00ce195e3b51550f1aeb01b6506
Author: alex 
Date:   2017-02-17T09:51:36Z

added one profile to upload jdbcT4 to maven central

commit 50a2d0c6654ba64cae0a3f5c9352f9f49ce911c3
Author: alex 
Date:   2017-02-17T10:00:42Z

remove the snapshot

commit f9ae1cb87cbe1e04cd2d3e03a6f50d9f0241c850
Author: alex 
Date:   2017-02-20T08:30:26Z

verified the pom change worked to publish to maven staging repository

commit 85b73bcefd8a6db42b518973eeef3439657dac1f
Author: alex 
Date:   2017-02-20T08:47:00Z

removed commented content




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---