[GitHub] incubator-trafodion pull request: TRAFODION-1501 Packaging trafodi...

[hegdean]

GitHub user hegdean opened a pull request:

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

TRAFODION-1501 Packaging trafodion source is missing in build script



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

$ git pull https://github.com/hegdean/incubator-trafodion wrk-brnch

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

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


commit 444e0409cf4ef76e24507b05b32a5cc8ccc0ece9
Author: Anuradha Hegde 
Date:   2015-09-30T18:30:20Z

Modified top level makefile to package trafodion source




---
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: TRAFODION-1502 Build remaining J...

[zellerh]

GitHub user zellerh opened a pull request:

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

TRAFODION-1502 Build remaining Java files in SQL executor and ustat through 
Maven

Most of the Trafodion Java source files are built through Maven, using
projects DCS, REST, HBase-trx and SQL. A few files remain in the
core/sql/executor and core/sql/ustat directories that are built
through javac commands in the Makefile and helper files. Also, these
files don't conform to common naming standards and therefore are a bit
harder to debug with tools like Eclipse.

This change does the following:

1. Move these remaining Java source files under core/sql/src,
   where the UDR-related files already are.
2. Include the class files in trafodion-sql-1.2.0.jar, the
   deliverable of the core/sql Maven project.
3. Change the package name to org.trafodion.sql for all these files.
4. Remove the trafodion-HBaseAccess-1.2.0.jar file (content
   will be included in trafodion-sql-1.2.0.jar). This jar file
   used to be built with a Makefile.
5. Make some source code changes to reflect the new package names
   and to avoid calling private method of class
   org.apache.hadoop.hive.ql.io.orc.OrcStruct.
6. Remove Makefile rules for compiling Java files.

Also, this change includes a change of the component name for the
event logger. We use SQL.COMP now, to make analyzing log messages
for Trafodion easier. Venkat asked for this change.

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

$ git pull https://github.com/zellerh/incubator-trafodion bug/1129

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

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


commit a44823fe2fd3991bc13434f73692a70b661efd43
Author: Hans Zeller 
Date:   2015-10-01T02:01:45Z

Most of the Trafodion Java source files are built through Maven, using
projects DCS, REST, HBase-trx and SQL. A few files remain in the
core/sql/executor and core/sql/ustat directories that are built
through javac commands in the Makefile and helper files. Also, these
files don't conform to common naming standards and therefore are a bit
harder to debug with tools like Eclipse.

I would like to do the following:

1. Move these remaining Java source files under core/sql/src,
   where the UDR-related files already are.
2. Include the class files in trafodion-sql-1.2.0.jar, the
   deliverable of the core/sql Maven project.
3. Change the package name to org.trafodion.sql for all these files.
4. Remove the trafodion-HBaseAccess-1.2.0.jar file (content
   will be included in trafodion-sql-1.2.0.jar). This jar file
   used to be built with a Makefile.
5. Make some source code changes to reflect the new package names
   and to avoid calling private method of class
   org.apache.hadoop.hive.ql.io.orc.OrcStruct.
6. Remove Makefile rules for compiling Java files.

Also, this change includes a change of the component name for the
event logger. We use SQL.COMP now, to make analyzing log messages
for Trafodion easier. Venkat asked for this change.




---
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: Stop using custom suffix (Nv44) ...

[ryzuo]

GitHub user ryzuo opened a pull request:

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

Stop using custom suffix (Nv44) of ICU libs.

We keep using our customized suffix from Neoview time, Nv4,
for ICU lib files, this causes confusion to developers who wants
to build Linux ODBC driver. Remove this suffix from the name of
lib files, to get rid of the confusion.

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

$ git pull https://github.com/ryzuo/incubator-trafodion remove_icu_suffix

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

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


commit b05adc0133017f5debada060877078ade510d9c1
Author: zuor 
Date:   2015-09-30T03:13:40Z

Stop using custom suffix (Nv44) of ICU libs.

We keep using our customized suffix from Neoview time, Nv4,
for ICU lib files, this causes confusion to developers who wants
to build Linux ODBC driver. Remove this suffix from the name of
lib files, to get rid of the confusion.




---
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.
---