You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org> on 2016/07/19 02:28:37 UTC

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Ian Maxon has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1005

Change subject: Update dev-setup to use IntelliJ instead
......................................................................

Update dev-setup to use IntelliJ instead

Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
---
M dev-setup.md
1 file changed, 13 insertions(+), 29 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site refs/changes/05/1005/1

diff --git a/dev-setup.md b/dev-setup.md
index 75642cc..a71d47c 100644
--- a/dev-setup.md
+++ b/dev-setup.md
@@ -45,39 +45,23 @@
 
     You will now have `$HOME/workspace/asterixdb/`.
 
-2. Go to the asterixdb folder and install it's artifacts to the local Maven repository by executing the following commands:
+2. In Intellij, import asterixdb as an existing Maven Project.
+* `File -> New -> Project from existing sources`, or `Import project...' when on a new install
+* Then, select the 'pom.xml' in the root of the checked out git repository. 
+* The default options for import should suffice, so just click `Next`
+* Also click `Next` when selecting for profiles and versions
+* On the JDK Selection screen, be sure to select a JDK > 1.8, OpenJDK or Oracle is fine.
+* Give IntelliJ some time to import the Maven project and its dependencies
+* Once everything is finished, you should see 'asterixdb' and 'hyracks-fullstack' modules in the "Project" view.
 
-        cd asterixdb/
-        mvn install -DskipTests
-
-3. In Eclipse, import asterixdb as an existing Maven Project.
-* `File -> Import -> Maven -> Existing Maven Projects -> Next`
-* Specify the Root directory as `asterixdb` and then click Next until Finish is enabled.
-* If Eclipse tries to install the `m2e` (Maven To Eclipse) connector, let it do so.
-* There might be some plugin errors; however, that is not a big deal. Wait until the job finishes.
-* Then, click Finish. Restart Eclipse if asked to do so.
-
-4. Fix Eclipse's build path to include compile-time generated sources.
-* Right click the project where a red X mark is shown(e.g. `asterix-algebra`. Then resolve by applying the following:
-  * Right click asterix-algebra. Click Build Path and Configure Build Path. Click Add Folder.
-  * Under the `target -> generated sources`, check the parent folder of the `org` folder and click OK.
-  * For example, if the directory structure is "target - generated-sources - javacc - org - apache ...", check the javacc directory and click OK. Then X mark will dissapear.
-* Repeat this step to all projects which show a red X mark except "asterix-fuzzyjoin" and "asterix-transactions".
-* It may be the case that only "asterix-algebra" and "asterix-external-data" will require these steps.
-* If you see "plugin execution not covered by lifecycle configuration" errors such as this one - "Plugin execution not covered by lifecycle configuration: org.apache.asterix:asterix-test-datagenerator-maven-plugin:0.8.9-SNAPSHOT:generate-testdata (execution: replace-template-data, phase: process-test-resources)", try to resolve it using the quick fix feature. You can call this feature by right click on the error and select "quick fix". And then choose "Mark goal generate-testdata as ignored in Eclipse build in Eclipse preferences". 
-
-5. Set up Eclipse code formatting rules
+3. Set up IntelliJ code formatting rules
 * Download files [AsterixCodeFormatProfile.xml](https://cwiki.apache.org/confluence/download/attachments/61322291/AsterixCodeFormatProfile.xml) 
     [AsterixCleanupFormatProfile.xml](https://cwiki.apache.org/confluence/download/attachments/61322291/AsterixCleanupFormatProfile.xml)
-* Import profiles into Eclipse
-  * Preferences -> Java -> Code style -> Formatter -> Import -> Select AsterixCodeFormatProfile.xml
-  * Preferences -> Java -> Code style -> Clean up -> Import -> Select AsterixCleanupFormatProfile.xml
-  * Preferences -> Java -> Editor -> Save actions -> Perform the selected action on save -> Format source code
+* Import profiles into IntelliJ
+    * `File -> Settings -> Editor -> Code Style -> Java`
+    * Then, click the `Manage...` button
+    * Click `Import` and import the `AsterixCodeFormatProfile.xml` as an Eclipse XML profile.
 
-6. Lastly, go to the asterixdb folder and execute following command again. This is required since Eclipse might have cleaned the projects and rebuilt the them without creating all necessary classes. Currently, some of the class files can be only built using mvn command.
-
-        cd asterixdb/
-        mvn install -DskipTests
 
 ---
 

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 1
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has uploaded a new patch set (#4).

Change subject: Update dev-setup to use IntelliJ instead
......................................................................

Update dev-setup to use IntelliJ instead

Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
---
M dev-setup.md
1 file changed, 25 insertions(+), 1 deletion(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site refs/changes/05/1005/4
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 4
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 5:

Build Started https://asterix-jenkins.ics.uci.edu/job/asf-site/79/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 5
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Taewoo Kim (Code Review)" <do...@asterixdb.incubator.apache.org>.
Taewoo Kim has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

(2 comments)

It looks good. I agree with Till regarding his comments.

https://asterix-gerrit.ics.uci.edu/#/c/1005/2/dev-setup.md
File dev-setup.md:

Line 111: For details see the ASF website [here](https://www.apache.org/dev/new-committers-guide.html#cla)
WS


Line 177: 
WS


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 6:

Build Started https://asterix-jenkins.ics.uci.edu/job/asf-site/80/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 6
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Michael Blow (Code Review)" <do...@asterixdb.incubator.apache.org>.
Michael Blow has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1005/2/dev-setup.md
File dev-setup.md:

Line 55: #### IntelliJ IDE Setup
> The product is actually IntelliJ IDEA- should we use this at least once?
Note to self- read Till's comments before commenting.


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Till Westmann (Code Review)" <do...@asterixdb.incubator.apache.org>.
Till Westmann has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 1:

Indeed, I think that it would be good to keep both ...

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 1
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 3:

Build Started https://asterix-jenkins.ics.uci.edu/job/asf-site/77/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 3
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 4:

Build Started https://asterix-jenkins.ics.uci.edu/job/asf-site/78/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 4
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

Alright, fixed. I suppose my bias in which IDE I prefer is apparent ;)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has uploaded a new patch set (#3).

Change subject: Update dev-setup to use IntelliJ instead
......................................................................

Update dev-setup to use IntelliJ instead

Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
---
M dev-setup.md
1 file changed, 25 insertions(+), 1 deletion(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site refs/changes/05/1005/3
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 3
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Taewoo Kim (Code Review)" <do...@asterixdb.incubator.apache.org>.
Taewoo Kim has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 5:

(2 comments)

two whitespaces

https://asterix-gerrit.ics.uci.edu/#/c/1005/5/dev-setup.md
File dev-setup.md:

Line 111: For details see the ASF website [here](https://www.apache.org/dev/new-committers-guide.html#cla)
WS


Line 177: 
WS


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 5
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Taewoo Kim (Code Review)" <do...@asterixdb.incubator.apache.org>.
Taewoo Kim has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 6: Code-Review+2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 6
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has uploaded a new patch set (#2).

Change subject: Update dev-setup to use IntelliJ instead
......................................................................

Update dev-setup to use IntelliJ instead

Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
---
M dev-setup.md
1 file changed, 24 insertions(+), 0 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site refs/changes/05/1005/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has uploaded a new patch set (#5).

Change subject: Update dev-setup to use IntelliJ instead
......................................................................

Update dev-setup to use IntelliJ instead

Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
---
M content/community.html
M content/dev-setup.html
M dev-setup.md
3 files changed, 61 insertions(+), 4 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site refs/changes/05/1005/5
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 5
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Till Westmann (Code Review)" <do...@asterixdb.incubator.apache.org>.
Till Westmann has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1005/2/dev-setup.md
File dev-setup.md:

Line 55: #### IntelliJ IDE Setup
> Note to self- read Till's comments before commenting.
overrated


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

Build Started https://asterix-jenkins.ics.uci.edu/job/asf-site/75/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Michael Blow (Code Review)" <do...@asterixdb.incubator.apache.org>.
Michael Blow has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1005/2/dev-setup.md
File dev-setup.md:

Line 55: #### IntelliJ IDE Setup
The product is actually IntelliJ IDEA- should we use this at least once?


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asf-site/74/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 1
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has submitted this change and it was merged.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Update dev-setup to use IntelliJ instead

Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1005
Reviewed-by: Taewoo Kim <wa...@yahoo.com>
---
M content/community.html
M content/dev-setup.html
M dev-setup.md
3 files changed, 63 insertions(+), 6 deletions(-)

Approvals:
  Taewoo Kim: Looks good to me, approved



diff --git a/content/community.html b/content/community.html
index c95bb47..a9f0987 100644
--- a/content/community.html
+++ b/content/community.html
@@ -261,7 +261,7 @@
     <tr>
       <td><img src="img/team/murtadha_hubail.jpg" width="100" height="100" /></td>
       <td>Murtadha Hubail</td>
-      <td>UC Irvine</td>
+      <td>Saudi Aramco</td>
       <td>mhubail</td>
     </tr>
     <tr>
diff --git a/content/dev-setup.html b/content/dev-setup.html
index a584074..09fe42e 100644
--- a/content/dev-setup.html
+++ b/content/dev-setup.html
@@ -113,7 +113,7 @@
       <li><a href="#yarn-installer" id="markdown-toc-yarn-installer">YARN Installer</a></li>
     </ul>
   </li>
-  <li><a href="#setting-up-an-asterix-development-environment-in-eclipse" id="markdown-toc-setting-up-an-asterix-development-environment-in-eclipse">Setting up an Asterix Development environment in Eclipse</a>    <ul>
+  <li><a href="#setting-up-an-asterix-development-environment" id="markdown-toc-setting-up-an-asterix-development-environment">Setting up an Asterix Development environment</a>    <ul>
       <li><a href="#prerequisites" id="markdown-toc-prerequisites">Prerequisites</a></li>
       <li><a href="#steps" id="markdown-toc-steps">Steps</a></li>
     </ul>
@@ -152,7 +152,7 @@
 
 <hr />
 
-<h2 id="setting-up-an-asterix-development-environment-in-eclipse">Setting up an Asterix Development environment in Eclipse</h2>
+<h2 id="setting-up-an-asterix-development-environment">Setting up an Asterix Development environment</h2>
 
 <h3 id="prerequisites">Prerequisites</h3>
 <ul>
@@ -163,6 +163,8 @@
 </ul>
 
 <h3 id="steps">Steps</h3>
+
+<h4 id="general-setup">General setup</h4>
 
 <ol>
   <li>
@@ -180,6 +182,37 @@
  mvn install -DskipTests
 </code></pre>
   </li>
+</ol>
+
+<h4 id="intellij-idea-ide-setup">IntelliJ IDEA IDE Setup</h4>
+
+<ol>
+  <li>In IntelliJ IDEA, import asterixdb as an existing Maven Project.
+    <ul>
+      <li><code>File -&gt; New -&gt; Project from existing sources</code>, or `Import project\u2026\u2019 when on a new install</li>
+      <li>Then, select the \u2018pom.xml\u2019 in the root of the checked out git repository.</li>
+      <li>The default options for import should suffice, so just click <code>Next</code></li>
+      <li>Also click <code>Next</code> when selecting for profiles and versions</li>
+      <li>On the JDK Selection screen, be sure to select a JDK &gt; 1.8, OpenJDK or Oracle is fine.</li>
+      <li>Give IntelliJ some time to import the Maven project and its dependencies</li>
+      <li>Once everything is finished, you should see \u2018asterixdb\u2019 and \u2018hyracks-fullstack\u2019 modules in the \u201cProject\u201d view.</li>
+    </ul>
+  </li>
+  <li>Set up IntelliJ code formatting rules
+    <ul>
+      <li>Download files <a href="https://cwiki.apache.org/confluence/download/attachments/61322291/AsterixCodeFormatProfile.xml">AsterixCodeFormatProfile.xml</a>
+ <a href="https://cwiki.apache.org/confluence/download/attachments/61322291/AsterixCleanupFormatProfile.xml">AsterixCleanupFormatProfile.xml</a></li>
+      <li>Import profiles into IntelliJ</li>
+      <li><code>File -&gt; Settings -&gt; Editor -&gt; Code Style -&gt; Java</code></li>
+      <li>Then, click the <code>Manage...</code> button</li>
+      <li>Click <code>Import</code> and import the <code>AsterixCodeFormatProfile.xml</code> as an Eclipse XML profile.</li>
+    </ul>
+  </li>
+</ol>
+
+<h4 id="eclipse-ide-setup">Eclipse IDE Setup</h4>
+
+<ol>
   <li>In Eclipse, import asterixdb as an existing Maven Project.
     <ul>
       <li><code>File -&gt; Import -&gt; Maven -&gt; Existing Maven Projects -&gt; Next</code></li>
diff --git a/dev-setup.md b/dev-setup.md
index 75642cc..f1f66ab 100644
--- a/dev-setup.md
+++ b/dev-setup.md
@@ -29,7 +29,7 @@
 
 ---
 
-## Setting up an Asterix Development environment in Eclipse
+## Setting up an Asterix Development environment
 
 ### Prerequisites
   * A suitable \*nix environment (Linux, OSX)
@@ -38,6 +38,8 @@
   * A relatively recent version of Eclipse
 
 ### Steps
+
+#### General setup
 
 1. Check out AsterixDB master in one folder via git in the command line. Assume that the path is `$HOME/workspace`.
 
@@ -49,6 +51,28 @@
 
         cd asterixdb/
         mvn install -DskipTests
+
+#### IntelliJ IDEA IDE Setup
+
+
+3. In IntelliJ IDEA, import asterixdb as an existing Maven Project.
+* `File -> New -> Project from existing sources`, or `Import project...' when on a new install
+* Then, select the 'pom.xml' in the root of the checked out git repository.
+* The default options for import should suffice, so just click `Next`
+* Also click `Next` when selecting for profiles and versions
+* On the JDK Selection screen, be sure to select a JDK > 1.8, OpenJDK or Oracle is fine.
+* Give IntelliJ some time to import the Maven project and its dependencies
+* Once everything is finished, you should see 'asterixdb' and 'hyracks-fullstack' modules in the "Project" view.
+
+4. Set up IntelliJ code formatting rules
+* Download files [AsterixCodeFormatProfile.xml](https://cwiki.apache.org/confluence/download/attachments/61322291/AsterixCodeFormatProfile.xml)
+    [AsterixCleanupFormatProfile.xml](https://cwiki.apache.org/confluence/download/attachments/61322291/AsterixCleanupFormatProfile.xml)
+* Import profiles into IntelliJ
+    * `File -> Settings -> Editor -> Code Style -> Java`
+    * Then, click the `Manage...` button
+    * Click `Import` and import the `AsterixCodeFormatProfile.xml` as an Eclipse XML profile.
+
+#### Eclipse IDE Setup
 
 3. In Eclipse, import asterixdb as an existing Maven Project.
 * `File -> Import -> Maven -> Existing Maven Projects -> Next`
@@ -85,7 +109,7 @@
 
 First, to contribute patches to AsterixDB, you should have an ICLA on file with the Apache Software Foundation.
 For details see the ASF website [here](https://www.apache.org/dev/new-committers-guide.html#cla)
-You should only need to perform the following steps once. 
+You should only need to perform the following steps once.
 
 Our Gerrit server is [here](https://asterix-gerrit.ics.uci.edu/)
 
@@ -151,7 +175,7 @@
 
 ## Making Changes - working method
 
-  1. When you want to start working on a bug, feature, etc, first make a local `git` branch. Never work directly on 
+  1. When you want to start working on a bug, feature, etc, first make a local `git` branch. Never work directly on
     `master`! `master` should always be a pure mirror of `origin/master`, i.e., the GitHub mirror or the ASF repository.
 
          git checkout -b my_branch

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 7
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has uploaded a new patch set (#6).

Change subject: Update dev-setup to use IntelliJ instead
......................................................................

Update dev-setup to use IntelliJ instead

Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
---
M content/community.html
M content/dev-setup.html
M dev-setup.md
3 files changed, 63 insertions(+), 6 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site refs/changes/05/1005/6
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 6
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Till Westmann (Code Review)" <do...@asterixdb.incubator.apache.org>.
Till Westmann has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

(4 comments)

https://asterix-gerrit.ics.uci.edu/#/c/1005/2/dev-setup.md
File dev-setup.md:

Line 38:   * A relatively recent version of Eclipse
Should we mention a version of IntelliJ IDEA?


Line 55: #### IntelliJ IDE Setup
I think that the full name of the IDE is "IntelliJ IDEA". Should we use that?


Line 60: * Then, select the 'pom.xml' in the root of the checked out git repository. 
WS


Line 68: * Download files [AsterixCodeFormatProfile.xml](https://cwiki.apache.org/confluence/download/attachments/61322291/AsterixCodeFormatProfile.xml) 
WS


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

Also I will have to see about the IntelliJ code formatter being a plugin or not, it seems like it isn't for me but I will try it on a clean install later today to be certain.

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Michael Carey (Code Review)" <do...@asterixdb.incubator.apache.org>.
Michael Carey has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 2:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1005/2/dev-setup.md
File dev-setup.md:

Line 32: ## Setting up an Asterix Development environment in Eclipse
We should strike "in Eclipse" from the above (or say "in Eclipse or IntelliJ"


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 5:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1005/5/dev-setup.md
File dev-setup.md:

Line 111: For details see the ASF website [here](https://www.apache.org/dev/new-committers-guide.html#cla)
> WS
Oh, I didn't see this at first since these lines weren't changed.


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 5
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

Posted by "Taewoo Kim (Code Review)" <do...@asterixdb.incubator.apache.org>.
Taewoo Kim has posted comments on this change.

Change subject: Update dev-setup to use IntelliJ instead
......................................................................


Patch Set 1:

Wasn't the original plan adding the IntelliJ in addition to Eclipse?

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1005
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I888ff0eacf5b3cb6ad7ec002c74f113c6ffcd497
Gerrit-PatchSet: 1
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Taewoo Kim <wa...@yahoo.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No